Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to argparser, add --with-hidden-ownership option #44

Closed
wants to merge 3 commits into from

Conversation

mweinelt
Copy link
Contributor

It is recommended to reread ./backend.py --help after this change.

Multiple --mesh, --vpn parameters are now unsupported. Instead pass all interfaces / mac addresses separated by spaces.

* --mesh (-m) now accepts the interface:alfred_sock syntax to add multiple batman/alfred instances. Also multiple instances can be added at once now. Only one interface can be added without alfred socket support (available since 2014.4.0) though.
* --alfred-sock (-s) was dropped in favor of the new --mesh syntax, which adds the interface to socket relationship
* --vpn (-V) now accepts multiple mac addresses, ATTENTION: update your calls accordingly
* --prune defaults to int now
* --with-rrd was renamed from --rrd, to better reflect its boolean/toggle like state
@mweinelt mweinelt closed this Mar 26, 2015
@indigo423
Copy link

Was issn der Grund warum das nicht gemerged und abgelehnt ist?

@jplitza
Copy link
Member

jplitza commented Sep 29, 2015

Das weiß wohl nur der Autor, @mweinelt, der den Pull Request selbst noch am selben Tag wieder geschlossen und damit quasi zurückgezogen hat.

@mweinelt
Copy link
Contributor Author

Das geht auf tcatm's Entscheidung zurück, der wollte das nicht da drinnen haben.

Wenn du das anders siehst kann ich das ja nochmal angehen.

Gruß hexa

On 29 September 2015 19:21:36 CEST, Jan-Philipp Litza notifications@github.com wrote:

Das weiß wohl nur der Autor, @mweinelt, der den Pull Request selbst
noch am selben Tag wieder geschlossen und damit quasi zurückgezogen
hat.


Reply to this email directly or view it on GitHub:
#44 (comment)

Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

@mweinelt
Copy link
Contributor Author

Alternativ einfach cherry-picken, falls das passt.

@jplitza
Copy link
Member

jplitza commented Oct 7, 2015

Mir fehlt noch etwas der Grund für die Änderungen: Warum sollte man mehrere Meshes in einem JSON zusammenfassen wollen? Und warum reicht der in der Readme erwähnte jq-Aufruf nicht, um die Kontaktinfos zu entfernen? Und unabhängig davon (ginge auch mit jq): Wäre eine Verschleierung von E-Mailadressen nicht ausreichend? Oder anders formuliert: Zu welchem Zweck möchte man die Kontaktinfos aus der JSON entfernen? [Nur Spamverhinderung oder um tatsächlich den Besitzer zu verbergen?]

@mweinelt
Copy link
Contributor Author

mweinelt commented Oct 7, 2015

73a87f6 ist der einzige Commit der nicht bereit bereits im master branch ist.

Der Nachteil durch jq ist, dass man darauf aufpassen muss die Dateien erneut nur atomar anzufassen, da es sonst zu hickups in der Abfrage kommen kann.

Kontaktdaten verbergen begründet sich auf jeden Fall durch Privacy und Anti-Spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants