Bug#852594: buzztrax: Update udevadm path

2017-09-13 Thread Sebastian Dröge
On Wed, 2017-09-13 at 15:15 +0200, Michael Biebl wrote:
> Control: tags -1 fixed-upstream
> On Tue, 12 Sep 2017 11:49:40 +0300 Sebastian =?ISO-8859-1?Q?Dr=F6ge?=
>  wrote:
> > severity 852594 minor
> > thanks
> > 
> > Hi,
> > 
> > as this only affects a source code comment and not actual code, I'm
> > downgrading the severity to minor. This is in no way serious.
> 
> Indeed. Thanks for your explanation.
> I did not check each bug report individually before raising the severity
> so I missed Stefan's comment. Sorry for that.

Sure, not a problem at all :)

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


Bug#852594: buzztrax: Update udevadm path

2017-09-13 Thread Michael Biebl
Control: tags -1 fixed-upstream
On Tue, 12 Sep 2017 11:49:40 +0300 Sebastian =?ISO-8859-1?Q?Dr=F6ge?=
 wrote:
> severity 852594 minor
> thanks
> 
> Hi,
> 
> as this only affects a source code comment and not actual code, I'm
> downgrading the severity to minor. This is in no way serious.

Indeed. Thanks for your explanation.
I did not check each bug report individually before raising the severity
so I missed Stefan's comment. Sorry for that.

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



signature.asc
Description: OpenPGP digital signature


Bug#852594: buzztrax: Update udevadm path

2017-09-12 Thread Sebastian Dröge
severity 852594 minor
thanks

Hi,

as this only affects a source code comment and not actual code, I'm
downgrading the severity to minor. This is in no way serious.


Sebastian

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


Bug#852594: buzztrax: Update udevadm path

2017-06-20 Thread Michael Biebl
On Wed, 25 Jan 2017 14:42:37 +0100 bi...@debian.org wrote:
> Package: buzztrax
> Version: 0.10.2-4
> Severity: normal
> User: pkg-systemd-maintain...@lists.alioth.debian.org
> Usertags: udevadm
> 
> Hi,
> 
> since Jessie, the udevadm binary is available as /bin/udevadm.
> To not break existing scripts, which use the old /sbin/udevadm path,
> the udev package currently ships a compat symlink which we would like
> to drop eventually (in buster or buster+1).
> 
> According to codesearch [1] your package buzztrax does hard-code the
> udevadm path as /sbin/udevadm.
> 
> Please change that to /bin/udevadm.

With stretch being released and buster open for development, it would be
a great opportunity to get this fixed now. So please consider adding
this change in your next upload.

If you are worried about backports: /bin/udevadm is already available in
Debian jessie (oldstable) or Ubuntu trusty (14.04LTS). So it's safe to
use the /bin/udevadm path.

You might also consider not hard-coding the path altogether, and simply
rely on PATH being set properly. This should be a safe assumption,
especially since the binary is now in /bin

Regards,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?