2012/6/30 Roger Leigh <rle...@codelibre.net>:
> On Sat, Jun 30, 2012 at 12:35:28AM +0200, Gonéri Le Bouder wrote:
>> 2012/6/30 Michael Biebl <bi...@debian.org>:
>> > On 30.06.2012 00:18, Gonéri Le Bouder wrote:
>> >> On Thu, Jan 19, 2012 at 10:49:29AM +0000, Roger Leigh wrote:
>> >>> On Tue, Oct 04, 2011 at 11:48:06PM +0200, bi...@debian.org wrote:
>>
>> > /dev/.udev/db has been deprecated a long time ago.
>> > If fusioninventory-agent does not actually need that information, why
>> > keep the code?
>> The agent can still run on older system.
>
> When you're packaging this software for Debian, you're targetting it
> at the versions of the packages in unstable.  This code is useless for
> wheezy, since all systems will be using the udev version provided with
> wheezy, which doesn't include /dev/.udev/db irrespective of whether or
> not udev is using /run/udev or /dev/.udev as its directory.  It's long
> gone, and so is useless to even attempt to use.
/dev/.udev doesn't exist anymore on up to date Debian release but the
directory, will remains here on older Linux installations.

I don't see a good reason to patch that in the Debian package. At best
we will avoid a stat() call on a virtual filesystem and the code is designed
to deal with that.

> It might be fine to conditionally enable it /if /dev/.udev is present/,
> but it should certainly not be using these paths now by default.  I
> would recommend patching it out in Debian if you do choose to retain it
> upstream.
Do you know a similar alternative solution to get the list of the block devices
and the associated information?

Best regards,
-- 
     Gonéri Le Bouder



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to