[PATCH v2] udev: Move udevd back to /sbin

[PATCH v2] udev: Move udevd back to /sbin

[Radu Moisan]

Along with v182 upgrade udevd was moved to ${base_libdir}
making scripts like init-live.sh to fail in finding udevd

Fixes [Yocto #4046]

Signed-off-by: Radu Moisan <radu.moisan@intel.com>
---
 meta/recipes-core/udev/udev.inc    |    3 ++-
 meta/recipes-core/udev/udev_182.bb |    2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index e358d2d..c4d2ce4 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -40,7 +40,7 @@ EXTRA_OECONF = "--disable-introspection \
                 ac_cv_file__usr_share_hwdata_pci_ids=no \
                 ac_cv_file__usr_share_misc_pci_ids=yes \
                 --sbindir=${base_sbindir} \
-                --libexecdir=${base_libdir} \
+                --libexecdir=${base_sbindir} \
                 --with-rootlibdir=${base_libdir} \
                 --with-rootprefix= \
                "
@@ -61,6 +61,7 @@ RRECOMMENDS_${PN} += "udev-utils"
 FILES_${PN}-dbg += "${libexecdir}/.debug"
 FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
 FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
+FILES_${PN}-dbg += "${base_sbindir}/udev/.debug/*"
 FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
 FILES_libudev = "${base_libdir}/libudev.so.*"
 FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
diff --git a/meta/recipes-core/udev/udev_182.bb b/meta/recipes-core/udev/udev_182.bb
index 42b4d08..d66292e 100644
--- a/meta/recipes-core/udev/udev_182.bb
+++ b/meta/recipes-core/udev/udev_182.bb
@@ -1,6 +1,6 @@
 include udev.inc
 
-PR = "r6"
+PR = "r7"
 
 # module-init-tools from kmod_git will provide libkmod runtime
 DEPENDS += "module-init-tools"
-- 
1.7.9.5

Re: [PATCH v2] udev: Move udevd back to /sbin

[Koen Kooi]

Op 8 apr. 2013, om 18:29 heeft Radu Moisan <radu.moisan@intel.com> het volgende geschreven:

> Along with v182 upgrade udevd was moved to ${base_libdir}

By upstream, yes

> making scripts like init-live.sh to fail in finding udevd

So you should fix those scripts, not mess around with udev.

> Fixes [Yocto #4046]
> 
> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> ---
> meta/recipes-core/udev/udev.inc    |    3 ++-
> meta/recipes-core/udev/udev_182.bb |    2 +-
> 2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
> index e358d2d..c4d2ce4 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -40,7 +40,7 @@ EXTRA_OECONF = "--disable-introspection \
>                 ac_cv_file__usr_share_hwdata_pci_ids=no \
>                 ac_cv_file__usr_share_misc_pci_ids=yes \
>                 --sbindir=${base_sbindir} \
> -                --libexecdir=${base_libdir} \
> +                --libexecdir=${base_sbindir} \
>                 --with-rootlibdir=${base_libdir} \
>                 --with-rootprefix= \
>                "
> @@ -61,6 +61,7 @@ RRECOMMENDS_${PN} += "udev-utils"
> FILES_${PN}-dbg += "${libexecdir}/.debug"
> FILES_${PN}-dbg += "${base_libdir}/udev/.debug/"
> FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
> +FILES_${PN}-dbg += "${base_sbindir}/udev/.debug/*"
> FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
> FILES_libudev = "${base_libdir}/libudev.so.*"
> FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
> diff --git a/meta/recipes-core/udev/udev_182.bb b/meta/recipes-core/udev/udev_182.bb
> index 42b4d08..d66292e 100644
> --- a/meta/recipes-core/udev/udev_182.bb
> +++ b/meta/recipes-core/udev/udev_182.bb
> @@ -1,6 +1,6 @@
> include udev.inc
> 
> -PR = "r6"
> +PR = "r7"
> 
> # module-init-tools from kmod_git will provide libkmod runtime
> DEPENDS += "module-init-tools"
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Re: [PATCH v2] udev: Move udevd back to /sbin

[Otavio Salvador]

On Mon, Apr 8, 2013 at 5:32 PM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>
> Op 8 apr. 2013, om 18:29 heeft Radu Moisan <radu.moisan@intel.com> het volgende geschreven:
>
>> Along with v182 upgrade udevd was moved to ${base_libdir}
>
> By upstream, yes
>
>> making scripts like init-live.sh to fail in finding udevd
>
> So you should fix those scripts, not mess around with udev.
>
>> Fixes [Yocto #4046]
>>
>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>

This move has been done long time ago so I agree with Koen here: the
scripts needs to be fixed. Another thing, it should also be tested
with systemd feature enabled as udev will be called systemd-udevd in
this case, IIRC.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [PATCH v2] udev: Move udevd back to /sbin

[Richard Purdie]

On Mon, 2013-04-08 at 17:40 -0300, Otavio Salvador wrote:
> On Mon, Apr 8, 2013 at 5:32 PM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> >
> > Op 8 apr. 2013, om 18:29 heeft Radu Moisan <radu.moisan@intel.com> het volgende geschreven:
> >
> >> Along with v182 upgrade udevd was moved to ${base_libdir}
> >
> > By upstream, yes
> >
> >> making scripts like init-live.sh to fail in finding udevd
> >
> > So you should fix those scripts, not mess around with udev.
> >
> >> Fixes [Yocto #4046]
> >>
> >> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> 
> This move has been done long time ago so I agree with Koen here: the
> scripts needs to be fixed. Another thing, it should also be tested
> with systemd feature enabled as udev will be called systemd-udevd in
> this case, IIRC.

It is not the scripts, see the previous discussion that was sent out and
Koen at least replied to. The scripts are the least of the problems.

Yes, the commit message here could be more accurate but that doesn't
change the problem of the issue reported in #4046 and previously
discussed.

Cheers,

Richard

Re: [PATCH v2] udev: Move udevd back to /sbin

[Richard Purdie]

On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
> Along with v182 upgrade udevd was moved to ${base_libdir}
> making scripts like init-live.sh to fail in finding udevd
> 
> Fixes [Yocto #4046]
> 
> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> ---
>  meta/recipes-core/udev/udev.inc    |    3 ++-
>  meta/recipes-core/udev/udev_182.bb |    2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)

We needed a decision on this. I've rewritten the commit message and
merged it. Most of the feedback was about the commit message, not the
patch itself. There were also no better proposals for how we could
actually fix the bugs we were seeing.

Cheers,

Richard

Re: [PATCH v2] udev: Move udevd back to /sbin

[Otavio Salvador]

On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>> Along with v182 upgrade udevd was moved to ${base_libdir}
>> making scripts like init-live.sh to fail in finding udevd
>>
>> Fixes [Yocto #4046]
>>
>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>> ---
>>  meta/recipes-core/udev/udev.inc    |    3 ++-
>>  meta/recipes-core/udev/udev_182.bb |    2 +-
>>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> We needed a decision on this. I've rewritten the commit message and
> merged it. Most of the feedback was about the commit message, not the
> patch itself. There were also no better proposals for how we could
> actually fix the bugs we were seeing.

If I read the thread right, it had two NACK's. So it wasn't a cosmetic
commit log issue.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [PATCH v2] udev: Move udevd back to /sbin

[Koen Kooi]

Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br> het volgende geschreven:

> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>>> Along with v182 upgrade udevd was moved to ${base_libdir}
>>> making scripts like init-live.sh to fail in finding udevd
>>> 
>>> Fixes [Yocto #4046]
>>> 
>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>>> ---
>>> meta/recipes-core/udev/udev.inc    |    3 ++-
>>> meta/recipes-core/udev/udev_182.bb |    2 +-
>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>> 
>> We needed a decision on this. I've rewritten the commit message and
>> merged it. Most of the feedback was about the commit message, not the
>> patch itself. There were also no better proposals for how we could
>> actually fix the bugs we were seeing.
> 
> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
> commit log issue.

4 replies to the patch:

1) me asking about the commit log
2) me NACK'ing the patch
3) Otavio NACK'ing the ptch
4) RP mentioning other discussions

So 25% of the replies are about cosmetics, how is that "most of the replies"?

Re: [PATCH v2] udev: Move udevd back to /sbin

[Koen Kooi]

Op 9 apr. 2013, om 16:41 heeft Koen Kooi <koen@dominion.thruhere.net> het volgende geschreven:

> 
> Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br> het volgende geschreven:
> 
>> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>>>> Along with v182 upgrade udevd was moved to ${base_libdir}
>>>> making scripts like init-live.sh to fail in finding udevd
>>>> 
>>>> Fixes [Yocto #4046]
>>>> 
>>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>>>> ---
>>>> meta/recipes-core/udev/udev.inc    |    3 ++-
>>>> meta/recipes-core/udev/udev_182.bb |    2 +-
>>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>> 
>>> We needed a decision on this. I've rewritten the commit message and
>>> merged it. Most of the feedback was about the commit message, not the
>>> patch itself. There were also no better proposals for how we could
>>> actually fix the bugs we were seeing.
>> 
>> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
>> commit log issue.
> 
> 4 replies to the patch:
> 
> 1) me asking about the commit log
> 2) me NACK'ing the patch
> 3) Otavio NACK'ing the ptch
> 4) RP mentioning other discussions
> 
> So 25% of the replies are about cosmetics, how is that "most of the replies"?

And by the way, does anyone actually bother testing patches to important infrastructure like udev?

[koen@rrMBP udev]$ git log --oneline -1
a866e1e udev: Move udevd back to /sbin

[koen@rrMBP udev]$ git grep /lib/udev/udevd
udev/init:[ -x /lib/udev/udevd ] || exit 1
udev/init:    /lib/udev/udevd -d
udev/udev-cache:[ -x /lib/udev/udevd ] || exit 1

So the patch broke the sysvinit script, congratulations!

Re: [PATCH v2] udev: Move udevd back to /sbin

[Richard Purdie]

On Tue, 2013-04-09 at 16:44 +0200, Koen Kooi wrote:
> Op 9 apr. 2013, om 16:41 heeft Koen Kooi <koen@dominion.thruhere.net> het volgende geschreven:
> 
> > 
> > Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br> het volgende geschreven:
> > 
> >> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
> >> <richard.purdie@linuxfoundation.org> wrote:
> >>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
> >>>> Along with v182 upgrade udevd was moved to ${base_libdir}
> >>>> making scripts like init-live.sh to fail in finding udevd
> >>>> 
> >>>> Fixes [Yocto #4046]
> >>>> 
> >>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> >>>> ---
> >>>> meta/recipes-core/udev/udev.inc    |    3 ++-
> >>>> meta/recipes-core/udev/udev_182.bb |    2 +-
> >>>> 2 files changed, 3 insertions(+), 2 deletions(-)
> >>> 
> >>> We needed a decision on this. I've rewritten the commit message and
> >>> merged it. Most of the feedback was about the commit message, not the
> >>> patch itself. There were also no better proposals for how we could
> >>> actually fix the bugs we were seeing.
> >> 
> >> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
> >> commit log issue.
> > 
> > 4 replies to the patch:
> > 
> > 1) me asking about the commit log
> > 2) me NACK'ing the patch

Without any constructive way of fixing the issues. As I've said,
"fixing" the other scripts does not work. Nobody has proposed any
reasonable realistic way of unbreaking the multilib support which worked
prior to the udev upgrade. Nobody has actually taken the time to even
understand what is breaking as far as I can tell.

> > 3) Otavio NACK'ing the ptch

Again, not constructively. People saying "no" just because they dislike
it doesn't really work.

> > 4) RP mentioning other discussions
>
> And by the way, does anyone actually bother testing patches to important infrastructure like udev?
> 
> [koen@rrMBP udev]$ git log --oneline -1
> a866e1e udev: Move udevd back to /sbin
> 
> [koen@rrMBP udev]$ git grep /lib/udev/udevd
> udev/init:[ -x /lib/udev/udevd ] || exit 1
> udev/init:    /lib/udev/udevd -d
> udev/udev-cache:[ -x /lib/udev/udevd ] || exit 1
> 
> So the patch broke the sysvinit script, congratulations!

Patch screw up :(. This will get fixed shortly, sorry about that.

Cheers,

Richard

Re: [PATCH v2] udev: Move udevd back to /sbin

[Mark Hatle]

On 4/9/13 10:07 AM, Richard Purdie wrote:
> On Tue, 2013-04-09 at 16:44 +0200, Koen Kooi wrote:
>> Op 9 apr. 2013, om 16:41 heeft Koen Kooi <koen@dominion.thruhere.net> het volgende geschreven:
>>
>>>
>>> Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br> het volgende geschreven:
>>>
>>>> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
>>>> <richard.purdie@linuxfoundation.org> wrote:
>>>>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>>>>>> Along with v182 upgrade udevd was moved to ${base_libdir}
>>>>>> making scripts like init-live.sh to fail in finding udevd
>>>>>>
>>>>>> Fixes [Yocto #4046]
>>>>>>
>>>>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>>>>>> ---
>>>>>> meta/recipes-core/udev/udev.inc    |    3 ++-
>>>>>> meta/recipes-core/udev/udev_182.bb |    2 +-
>>>>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>>>>
>>>>> We needed a decision on this. I've rewritten the commit message and
>>>>> merged it. Most of the feedback was about the commit message, not the
>>>>> patch itself. There were also no better proposals for how we could
>>>>> actually fix the bugs we were seeing.
>>>>
>>>> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
>>>> commit log issue.
>>>
>>> 4 replies to the patch:
>>>
>>> 1) me asking about the commit log
>>> 2) me NACK'ing the patch
>
> Without any constructive way of fixing the issues. As I've said,
> "fixing" the other scripts does not work. Nobody has proposed any
> reasonable realistic way of unbreaking the multilib support which worked
> prior to the udev upgrade. Nobody has actually taken the time to even
> understand what is breaking as far as I can tell.

Let me task a whack at this.  If the binary belongs in /usr/lib/udevd/ (or 
/lib/udevd/) then that tells me it's being treated as a libexecdir and not a 
"libdir".  So the 64-bit versions of udev should NOT be installing into 
/usr/lib64/...  (nor /usr/lib32/...)

If that is the case, and all multilibs end up in /usr/lib/udevd (or /lib/udevd) 
then the multilib system should continue to work as expected.

(Note, prefer /lib/udevd myself... but that is a separate issue.)

If this sounds reasonable, I can try to make the changes and test the multilib 
config.  But I am far from a systemd/udev expert!

--Mark

>>> 3) Otavio NACK'ing the ptch
>
> Again, not constructively. People saying "no" just because they dislike
> it doesn't really work.
>
>>> 4) RP mentioning other discussions
>>
>> And by the way, does anyone actually bother testing patches to important infrastructure like udev?
>>
>> [koen@rrMBP udev]$ git log --oneline -1
>> a866e1e udev: Move udevd back to /sbin
>>
>> [koen@rrMBP udev]$ git grep /lib/udev/udevd
>> udev/init:[ -x /lib/udev/udevd ] || exit 1
>> udev/init:    /lib/udev/udevd -d
>> udev/udev-cache:[ -x /lib/udev/udevd ] || exit 1
>>
>> So the patch broke the sysvinit script, congratulations!
>
> Patch screw up :(. This will get fixed shortly, sorry about that.
>
> Cheers,
>
> Richard
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

Re: [PATCH v2] udev: Move udevd back to /sbin

[Otavio Salvador]

On Tue, Apr 9, 2013 at 12:49 PM, Mark Hatle <mark.hatle@windriver.com> wrote:
> On 4/9/13 10:07 AM, Richard Purdie wrote:
>>
>> On Tue, 2013-04-09 at 16:44 +0200, Koen Kooi wrote:
>>>
>>> Op 9 apr. 2013, om 16:41 heeft Koen Kooi <koen@dominion.thruhere.net> het
>>> volgende geschreven:
>>>
>>>>
>>>> Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br>
>>>> het volgende geschreven:
>>>>
>>>>> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
>>>>> <richard.purdie@linuxfoundation.org> wrote:
>>>>>>
>>>>>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>>>>>>>
>>>>>>> Along with v182 upgrade udevd was moved to ${base_libdir}
>>>>>>> making scripts like init-live.sh to fail in finding udevd
>>>>>>>
>>>>>>> Fixes [Yocto #4046]
>>>>>>>
>>>>>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>>>>>>> ---
>>>>>>> meta/recipes-core/udev/udev.inc    |    3 ++-
>>>>>>> meta/recipes-core/udev/udev_182.bb |    2 +-
>>>>>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>>>>>
>>>>>>
>>>>>> We needed a decision on this. I've rewritten the commit message and
>>>>>> merged it. Most of the feedback was about the commit message, not the
>>>>>> patch itself. There were also no better proposals for how we could
>>>>>> actually fix the bugs we were seeing.
>>>>>
>>>>>
>>>>> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
>>>>> commit log issue.
>>>>
>>>>
>>>> 4 replies to the patch:
>>>>
>>>> 1) me asking about the commit log
>>>> 2) me NACK'ing the patch
>>
>>
>> Without any constructive way of fixing the issues. As I've said,
>> "fixing" the other scripts does not work. Nobody has proposed any
>> reasonable realistic way of unbreaking the multilib support which worked
>> prior to the udev upgrade. Nobody has actually taken the time to even
>> understand what is breaking as far as I can tell.
>
>
> Let me task a whack at this.  If the binary belongs in /usr/lib/udevd/ (or
> /lib/udevd/) then that tells me it's being treated as a libexecdir and not a
> "libdir".  So the 64-bit versions of udev should NOT be installing into
> /usr/lib64/...  (nor /usr/lib32/...)
>
> If that is the case, and all multilibs end up in /usr/lib/udevd (or
> /lib/udevd) then the multilib system should continue to work as expected.
>
> (Note, prefer /lib/udevd myself... but that is a separate issue.)
>
> If this sounds reasonable, I can try to make the changes and test the
> multilib config.  But I am far from a systemd/udev expert!

This was my proposed fix when Ross asked me about it. I also did
comment on this in the bug and it seemed ignored.

Another thing, systemd's udevd will be named differently, as:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/meta/recipes-core/initrdscripts?id=1b99640481882d23dc3ded41d9f2aef906f77e67

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [PATCH v2] udev: Move udevd back to /sbin

[Otavio Salvador]

On Tue, Apr 9, 2013 at 12:07 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2013-04-09 at 16:44 +0200, Koen Kooi wrote:
>> Op 9 apr. 2013, om 16:41 heeft Koen Kooi <koen@dominion.thruhere.net> het volgende geschreven:
>>
>> >
>> > Op 9 apr. 2013, om 16:36 heeft Otavio Salvador <otavio@ossystems.com.br> het volgende geschreven:
>> >
>> >> On Tue, Apr 9, 2013 at 9:35 AM, Richard Purdie
>> >> <richard.purdie@linuxfoundation.org> wrote:
>> >>> On Mon, 2013-04-08 at 19:29 +0300, Radu Moisan wrote:
>> >>>> Along with v182 upgrade udevd was moved to ${base_libdir}
>> >>>> making scripts like init-live.sh to fail in finding udevd
>> >>>>
>> >>>> Fixes [Yocto #4046]
>> >>>>
>> >>>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>> >>>> ---
>> >>>> meta/recipes-core/udev/udev.inc    |    3 ++-
>> >>>> meta/recipes-core/udev/udev_182.bb |    2 +-
>> >>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>> >>>
>> >>> We needed a decision on this. I've rewritten the commit message and
>> >>> merged it. Most of the feedback was about the commit message, not the
>> >>> patch itself. There were also no better proposals for how we could
>> >>> actually fix the bugs we were seeing.
>> >>
>> >> If I read the thread right, it had two NACK's. So it wasn't a cosmetic
>> >> commit log issue.
>> >
>> > 4 replies to the patch:
>> >
>> > 1) me asking about the commit log
>> > 2) me NACK'ing the patch
>
> Without any constructive way of fixing the issues. As I've said,
> "fixing" the other scripts does not work. Nobody has proposed any
> reasonable realistic way of unbreaking the multilib support which worked
> prior to the udev upgrade. Nobody has actually taken the time to even
> understand what is breaking as far as I can tell.
>
>> > 3) Otavio NACK'ing the ptch
>
> Again, not constructively. People saying "no" just because they dislike
> it doesn't really work.

It wasn't the case and I even pointed out the systemd-udevd problem
which has not been fixed so it was not "no" but a justified NACK. If
our NACK'ing is ignored what is the  matter of people using their time
to comment in something?

>> > 4) RP mentioning other discussions
>>
>> And by the way, does anyone actually bother testing patches to important infrastructure like udev?
>>
>> [koen@rrMBP udev]$ git log --oneline -1
>> a866e1e udev: Move udevd back to /sbin
>>
>> [koen@rrMBP udev]$ git grep /lib/udev/udevd
>> udev/init:[ -x /lib/udev/udevd ] || exit 1
>> udev/init:    /lib/udev/udevd -d
>> udev/udev-cache:[ -x /lib/udev/udevd ] || exit 1
>>
>> So the patch broke the sysvinit script, congratulations!
>
> Patch screw up :(. This will get fixed shortly, sorry about that.

It seems the right fix, for now, as I commented in the bug, is to use
/lib hardcoded.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br