Re: ANN: NetworkManager (0.9.10-beta1) released

On Tue, 2014-06-17 at 20:31 +0200, Michael Biebl wrote:
Am 17.06.2014 18:31, schrieb Dan Williams:
Separate directories were used for pre-up/pre-down because these events
block state changes internally.  NM will not advance a device from
until all the scripts execute.

Unfortunately, many scripts don't bother checking the action and just
run stuff, and the goal was to ensure that couldn't adversely impact the
user experience on upgrade.

I checked the complete Debian archive and there is only one package
installing a dispatcher script which doesn't check for the $action
parameter, so is potentially affected. Easy to fix.

You were talking about many scripts doing it wrong? Do you have a list
of such packages/scripts?

(without having any numbers at hand) I would suspect that many scripts
are just cooked-up by users and not part of other packages. Further I
would guess that these scripts are often of lower quality.

I get where you're coming from, but there will always be ways for people
to shoot themselves in the foot and I'd rather prefer consistency here
then convenience for lazy script writers.

I still think, that these separate directories are a good idea.

Also note that all the scripts that exists to date don't expect these
events. They existed without them for years. So probably most script
will continue not to do anything useful with then (if not harmful).

By having separate directories we actually save such invocations
(because NM will not even make the call if the directory contains no
scripts). I am trying to make a performance argument here, although
arguably it might not matter to call a few scripts needlessly.


Attachment: signature.asc
Description: This is a digitally signed message part

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]