* [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
@ 2012-11-16 20:11 dromede
2012-11-16 21:36 ` Martin Jansa
2012-11-17 18:20 ` Otavio Salvador
0 siblings, 2 replies; 14+ messages in thread
From: dromede @ 2012-11-16 20:11 UTC (permalink / raw)
To: openembedded-core
From: Marko Katic <dromede.gmail.com>
udev provides a udev-utils package that once provided utility binaries
such as udevadm, udevtest and udevinfo. udevtest and udevinfo are
deprecated and their functions are now handled by udevadm.
https://bugzilla.redhat.com/show_bug.cgi?id=597539
remove udevinfo and udevtest from the udev-utils package.
Signed-off-by: Marko Katic <dromede.gmail.com>
---
meta/recipes-core/udev/udev.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 4705b9b..7133cc4 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -74,7 +74,7 @@ FILES_udev-cache = "${sysconfdir}/init.d/udev-cache ${sysconfdir}/default/udev-c
FILES_udev-acl = "${base_libdir}/udev/udev-acl ${base_libdir}/udev/rules.d/70-acl.rules"
-FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest ${base_sbindir}/udevadm"
+FILES_udev-utils = "${bindir}/udevadm"
RDEPENDS_udev-consolekit += "udev-acl"
FILES_udev-consolekit = "${libdir}/ConsoleKit"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-16 20:11 [PATCH] [PATCH] udev: Remove deprecated binaries from packaging dromede
@ 2012-11-16 21:36 ` Martin Jansa
[not found] ` <CAHod+GfAof60ycR5EXosair15O87dvZhnxWSouhdF49x8sZQqQ@mail.gmail.com>
2012-11-17 18:20 ` Otavio Salvador
1 sibling, 1 reply; 14+ messages in thread
From: Martin Jansa @ 2012-11-16 21:36 UTC (permalink / raw)
To: dromede; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 1725 bytes --]
On Fri, Nov 16, 2012 at 09:11:39PM +0100, dromede@gmail.com wrote:
> From: Marko Katic <dromede.gmail.com>
>
> udev provides a udev-utils package that once provided utility binaries
> such as udevadm, udevtest and udevinfo. udevtest and udevinfo are
> deprecated and their functions are now handled by udevadm.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=597539
>
> remove udevinfo and udevtest from the udev-utils package.
Are they still built? If yes then there should be warning about
unpackaged files (you would need to delete them in do_install) or they
are picked by some other package (PN probably).
Cheers,
>
> Signed-off-by: Marko Katic <dromede.gmail.com>
> ---
> meta/recipes-core/udev/udev.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
> index 4705b9b..7133cc4 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -74,7 +74,7 @@ FILES_udev-cache = "${sysconfdir}/init.d/udev-cache ${sysconfdir}/default/udev-c
>
> FILES_udev-acl = "${base_libdir}/udev/udev-acl ${base_libdir}/udev/rules.d/70-acl.rules"
>
> -FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest ${base_sbindir}/udevadm"
> +FILES_udev-utils = "${bindir}/udevadm"
>
> RDEPENDS_udev-consolekit += "udev-acl"
> FILES_udev-consolekit = "${libdir}/ConsoleKit"
> --
> 1.7.9.5
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
[not found] ` <CAHod+GfAof60ycR5EXosair15O87dvZhnxWSouhdF49x8sZQqQ@mail.gmail.com>
@ 2012-11-16 22:28 ` Martin Jansa
2012-11-16 23:28 ` Chris Larson
0 siblings, 1 reply; 14+ messages in thread
From: Martin Jansa @ 2012-11-16 22:28 UTC (permalink / raw)
To: Marko Katić; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 2328 bytes --]
On Fri, Nov 16, 2012 at 10:48:41PM +0100, Marko Katić wrote:
> Udevtest and udevinfo are not being built because they no longer exist in
> udev 182 sources.
Ah, so this change is only cosmetic and does not change package output
(otherwise missing PR bump).
Cheers,
> On Nov 16, 2012 10:36 PM, "Martin Jansa" <martin.jansa@gmail.com> wrote:
> >
> > On Fri, Nov 16, 2012 at 09:11:39PM +0100, dromede@gmail.com wrote:
> > > From: Marko Katic <dromede.gmail.com>
> > >
> > > udev provides a udev-utils package that once provided utility binaries
> > > such as udevadm, udevtest and udevinfo. udevtest and udevinfo are
> > > deprecated and their functions are now handled by udevadm.
> > >
> > > https://bugzilla.redhat.com/show_bug.cgi?id=597539
> > >
> > > remove udevinfo and udevtest from the udev-utils package.
> >
> > Are they still built? If yes then there should be warning about
> > unpackaged files (you would need to delete them in do_install) or they
> > are picked by some other package (PN probably).
> >
> > Cheers,
> >
> > >
> > > Signed-off-by: Marko Katic <dromede.gmail.com>
> > > ---
> > > meta/recipes-core/udev/udev.inc | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta/recipes-core/udev/udev.inc
> b/meta/recipes-core/udev/udev.inc
> > > index 4705b9b..7133cc4 100644
> > > --- a/meta/recipes-core/udev/udev.inc
> > > +++ b/meta/recipes-core/udev/udev.inc
> > > @@ -74,7 +74,7 @@ FILES_udev-cache = "${sysconfdir}/init.d/udev-cache
> ${sysconfdir}/default/udev-c
> > >
> > > FILES_udev-acl = "${base_libdir}/udev/udev-acl
> ${base_libdir}/udev/rules.d/70-acl.rules"
> > >
> > > -FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest
> ${base_sbindir}/udevadm"
> > > +FILES_udev-utils = "${bindir}/udevadm"
> > >
> > > RDEPENDS_udev-consolekit += "udev-acl"
> > > FILES_udev-consolekit = "${libdir}/ConsoleKit"
> > > --
> > > 1.7.9.5
> > >
> > >
> > > _______________________________________________
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> >
> > --
> > Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-16 22:28 ` Martin Jansa
@ 2012-11-16 23:28 ` Chris Larson
2012-11-17 18:02 ` Marko Katić
0 siblings, 1 reply; 14+ messages in thread
From: Chris Larson @ 2012-11-16 23:28 UTC (permalink / raw)
To: Martin Jansa; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 659 bytes --]
On Fri, Nov 16, 2012 at 3:28 PM, Martin Jansa <martin.jansa@gmail.com>wrote:
> On Fri, Nov 16, 2012 at 10:48:41PM +0100, Marko Katić wrote:
> > Udevtest and udevinfo are not being built because they no longer exist in
> > udev 182 sources.
>
> Ah, so this change is only cosmetic and does not change package output
> (otherwise missing PR bump).
>
Changing udev.inc like this when you're specifically addressing one version
is cause for concern.
Exactly which udev version deprecated the utilities? oe-core might not have
many udev versions around, but other layers still do, for machines running
older kernels.
--
Christopher Larson
[-- Attachment #2: Type: text/html, Size: 1033 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-16 23:28 ` Chris Larson
@ 2012-11-17 18:02 ` Marko Katić
2012-11-17 18:38 ` Chris Larson
0 siblings, 1 reply; 14+ messages in thread
From: Marko Katić @ 2012-11-17 18:02 UTC (permalink / raw)
To: Chris Larson
Cc: Martin Jansa, Patches and discussions about the oe-core layer
i definitely should have written a much more elaborate commit message,
sorry about that.
The patch is definitely not cosmetic. This patch also deals with the changed
installation path of udevadm. Older versions of udev installed udevadm in /sbin.
Recent versions of udev install udevadm into /usr/bin. This means that the
udev-utils package will be empty because:
a) udevinfo and udevtest no longer exist
b) udevadm is no longer installed in base_sbindir, so it will end up
in the main udev package.
Therefore, udev-utils package will not be made at all. This breaks
do_rootfs stage since udev.inc RRECOMMENDS udev-utils.
Also, the initscript that comes with udev 182 calls udevadm via
it's absolute path - /usr/bin/udevadm. Initscripts in older versions
of udev expect udevadm in /sbin/.
I don't know exactly which version of udev deprecated udevinfo and
udevtest. There is no mention of it
in udev 182 changelog and i couldn't find a git commit that made this
change. The entire udev 182 source
tree only mentions udevinfo and udevtest in the changelog and the NEWS file.
Chris, you raise a valid point concerning older udev versions. If the
general concensus is that udev.inc in
OE-Core should support older and newer versions of udev, then my patch
is clearly not a proper solution.
A proper solution might be to:
a) Leave udevtest and udevinfo in udev-utils to properly support older
versions of udev that have these packages.
b) Move udevadm into the main udev package. This makes a lot of sense
since udevadm is crucial for proper udev operation.
c) Remove udev-utils from RRECOMMENDS.
Or we could just add ALLOW_EMPTY to udev.inc. A messy but simple solution.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-16 20:11 [PATCH] [PATCH] udev: Remove deprecated binaries from packaging dromede
2012-11-16 21:36 ` Martin Jansa
@ 2012-11-17 18:20 ` Otavio Salvador
1 sibling, 0 replies; 14+ messages in thread
From: Otavio Salvador @ 2012-11-17 18:20 UTC (permalink / raw)
To: dromede; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 809 bytes --]
On Fri, Nov 16, 2012 at 6:11 PM, <dromede@gmail.com> wrote:
> From: Marko Katic <dromede.gmail.com>
>
> udev provides a udev-utils package that once provided utility binaries
> such as udevadm, udevtest and udevinfo. udevtest and udevinfo are
> deprecated and their functions are now handled by udevadm.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=597539
>
> remove udevinfo and udevtest from the udev-utils package.
>
> Signed-off-by: Marko Katic <dromede.gmail.com>
>
Please rm them in do_install_append so if they ever are not build anymore
we'll get a build-error and can handle it.
--
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
[-- Attachment #2: Type: text/html, Size: 1509 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-17 18:02 ` Marko Katić
@ 2012-11-17 18:38 ` Chris Larson
2012-11-17 18:54 ` Otavio Salvador
0 siblings, 1 reply; 14+ messages in thread
From: Chris Larson @ 2012-11-17 18:38 UTC (permalink / raw)
To: Marko Katić
Cc: Martin Jansa, Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 544 bytes --]
On Sat, Nov 17, 2012 at 11:02 AM, Marko Katić <dromede@gmail.com> wrote:
> A proper solution might be to:
>
> a) Leave udevtest and udevinfo in udev-utils to properly support older
> versions of udev that have these packages.
> b) Move udevadm into the main udev package. This makes a lot of sense
> since udevadm is crucial for proper udev operation.
> c) Remove udev-utils from RRECOMMENDS.
>
This sounds like a clean, solid route to me. Hopefully someone else will
chime in with input on it as well.
--
Christopher Larson
[-- Attachment #2: Type: text/html, Size: 847 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-17 18:38 ` Chris Larson
@ 2012-11-17 18:54 ` Otavio Salvador
2012-11-20 19:59 ` Marko Katić
0 siblings, 1 reply; 14+ messages in thread
From: Otavio Salvador @ 2012-11-17 18:54 UTC (permalink / raw)
To: Chris Larson
Cc: Martin Jansa, Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 1035 bytes --]
On Sat, Nov 17, 2012 at 4:38 PM, Chris Larson <clarson@kergoth.com> wrote:
> On Sat, Nov 17, 2012 at 11:02 AM, Marko Katić <dromede@gmail.com> wrote:
>
>> A proper solution might be to:
>>
>> a) Leave udevtest and udevinfo in udev-utils to properly support older
>> versions of udev that have these packages.
>> b) Move udevadm into the main udev package. This makes a lot of sense
>> since udevadm is crucial for proper udev operation.
>> c) Remove udev-utils from RRECOMMENDS.
>>
>
> This sounds like a clean, solid route to me. Hopefully someone else will
> chime in with input on it as well.
>
I see no value in having the others and deprecated tools available. Having
udevadm available in udev-utils allows for it use with mdev for trigger the
events in kernel, for example. I like to have this possibility.
--
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
[-- Attachment #2: Type: text/html, Size: 1818 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-17 18:54 ` Otavio Salvador
@ 2012-11-20 19:59 ` Marko Katić
2012-11-20 20:23 ` Otavio Salvador
0 siblings, 1 reply; 14+ messages in thread
From: Marko Katić @ 2012-11-20 19:59 UTC (permalink / raw)
To: Otavio Salvador
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Sat, Nov 17, 2012 at 7:54 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
>
>
> On Sat, Nov 17, 2012 at 4:38 PM, Chris Larson <clarson@kergoth.com> wrote:
>>
>> On Sat, Nov 17, 2012 at 11:02 AM, Marko Katić <dromede@gmail.com> wrote:
>>>
>>> A proper solution might be to:
>>>
>>> a) Leave udevtest and udevinfo in udev-utils to properly support older
>>> versions of udev that have these packages.
>>> b) Move udevadm into the main udev package. This makes a lot of sense
>>> since udevadm is crucial for proper udev operation.
>>> c) Remove udev-utils from RRECOMMENDS.
>>
>>
>> This sounds like a clean, solid route to me. Hopefully someone else will
>> chime in with input on it as well.
>
>
> I see no value in having the others and deprecated tools available. Having
> udevadm available in udev-utils allows for it use with mdev for trigger the
> events in kernel, for example. I like to have this possibility.
>
> --
> 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
>
Ok, so udevadm should stay in udev-utils. In that case i propose we:
a) Add ${bindir}/udevadm and ${base_sbindir}/udevadm to
FILES_udev-utils. This should
ensure that udevadm will always go in udev-utils regardless of the
udev version being built.
This should also ensure that udev-utils package will always be made.
b) Make udevadm available in both ${bindir} and ${base_sbindir} via
symlinks installed in
do_install_append. This should ensure proper support for all versions of udev.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-20 19:59 ` Marko Katić
@ 2012-11-20 20:23 ` Otavio Salvador
2012-12-13 19:53 ` Marko Katić
0 siblings, 1 reply; 14+ messages in thread
From: Otavio Salvador @ 2012-11-20 20:23 UTC (permalink / raw)
To: Marko Katić
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Tue, Nov 20, 2012 at 5:59 PM, Marko Katić <dromede@gmail.com> wrote:
> On Sat, Nov 17, 2012 at 7:54 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>>
>>
>> On Sat, Nov 17, 2012 at 4:38 PM, Chris Larson <clarson@kergoth.com> wrote:
>>>
>>> On Sat, Nov 17, 2012 at 11:02 AM, Marko Katić <dromede@gmail.com> wrote:
>>>>
>>>> A proper solution might be to:
>>>>
>>>> a) Leave udevtest and udevinfo in udev-utils to properly support older
>>>> versions of udev that have these packages.
>>>> b) Move udevadm into the main udev package. This makes a lot of sense
>>>> since udevadm is crucial for proper udev operation.
>>>> c) Remove udev-utils from RRECOMMENDS.
>>>
>>>
>>> This sounds like a clean, solid route to me. Hopefully someone else will
>>> chime in with input on it as well.
>>
>>
>> I see no value in having the others and deprecated tools available. Having
>> udevadm available in udev-utils allows for it use with mdev for trigger the
>> events in kernel, for example. I like to have this possibility.
>>
>> --
>> 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
>>
>
> Ok, so udevadm should stay in udev-utils. In that case i propose we:
>
> a) Add ${bindir}/udevadm and ${base_sbindir}/udevadm to
> FILES_udev-utils. This should
> ensure that udevadm will always go in udev-utils regardless of the
> udev version being built.
> This should also ensure that udev-utils package will always be made.
>
> b) Make udevadm available in both ${bindir} and ${base_sbindir} via
> symlinks installed in
> do_install_append. This should ensure proper support for all versions of udev.
I like your plan.
+1
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-11-20 20:23 ` Otavio Salvador
@ 2012-12-13 19:53 ` Marko Katić
2012-12-13 23:30 ` Otavio Salvador
0 siblings, 1 reply; 14+ messages in thread
From: Marko Katić @ 2012-12-13 19:53 UTC (permalink / raw)
To: Otavio Salvador
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Tue, 2012-11-20 at 18:23 -0200, Otavio Salvador wrote:
> On Tue, Nov 20, 2012 at 5:59 PM, Marko Katić <dromede@gmail.com> wrote:
> > On Sat, Nov 17, 2012 at 7:54 PM, Otavio Salvador
> > <otavio@ossystems.com.br> wrote:
> >>
> >>
> >> On Sat, Nov 17, 2012 at 4:38 PM, Chris Larson <clarson@kergoth.com> wrote:
> >>>
> >>> On Sat, Nov 17, 2012 at 11:02 AM, Marko Katić <dromede@gmail.com> wrote:
> >>>>
> >>>> A proper solution might be to:
> >>>>
> >>>> a) Leave udevtest and udevinfo in udev-utils to properly support older
> >>>> versions of udev that have these packages.
> >>>> b) Move udevadm into the main udev package. This makes a lot of sense
> >>>> since udevadm is crucial for proper udev operation.
> >>>> c) Remove udev-utils from RRECOMMENDS.
> >>>
> >>>
> >>> This sounds like a clean, solid route to me. Hopefully someone else will
> >>> chime in with input on it as well.
> >>
> >>
> >> I see no value in having the others and deprecated tools available. Having
> >> udevadm available in udev-utils allows for it use with mdev for trigger the
> >> events in kernel, for example. I like to have this possibility.
> >>
> >> --
> >> 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
> >>
> >
> > Ok, so udevadm should stay in udev-utils. In that case i propose we:
> >
> > a) Add ${bindir}/udevadm and ${base_sbindir}/udevadm to
> > FILES_udev-utils. This should
> > ensure that udevadm will always go in udev-utils regardless of the
> > udev version being built.
> > This should also ensure that udev-utils package will always be made.
> >
> > b) Make udevadm available in both ${bindir} and ${base_sbindir} via
> > symlinks installed in
> > do_install_append. This should ensure proper support for all versions of udev.
>
> I like your plan.
>
> +1
>
Oh dear, i completely forgot about this patch. It's been sitting in my
local repo for weeks!
So here's my (hopefully) final solution:
diff --git a/meta/recipes-core/udev/udev.inc
b/meta/recipes-core/udev/udev.inc
index 4705b9b..677c33a 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -74,7 +74,7 @@ FILES_udev-cache = "${sysconfdir}/init.d/udev-cache
${sysconfdir}/default/udev-c
FILES_udev-acl = "${base_libdir}/udev/udev-acl
${base_libdir}/udev/rules.d/70-acl.rules"
-FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest
${base_sbindir}/udevadm"
+FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest
${base_sbindir}/udevadm ${bindir}/udevadm"
RDEPENDS_udev-consolekit += "udev-acl"
FILES_udev-consolekit = "${libdir}/ConsoleKit"
@@ -97,4 +97,19 @@ do_install_append () {
rm -f ${D}${base_libdir}/udev/hid2hci
echo 'udev_run="/var/run/udev"' >>
${D}${sysconfdir}/udev/udev.conf
+
+ #Starting with udev 176, udevadm is by default installed in
${bindir}.
+ #To properly support all versions of udev we make udevadm
available
+ #in ${bindir} and in ${base_sbindir}.
+
+ if [ "${PV}" -gt "175" ]
+ then
+ install -d ${D}${base_sbindir}
+ ln -s ${bindir}/udevadm ${D}${base_sbindir}/udevadm
+
+ else
+ install -d ${D}${bindir}
+ ln -s ${base_sbindir}/udevadm ${D}${bindir}/udevadm
+
+ fi
}
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-12-13 19:53 ` Marko Katić
@ 2012-12-13 23:30 ` Otavio Salvador
2012-12-14 15:17 ` Marko Katić
0 siblings, 1 reply; 14+ messages in thread
From: Otavio Salvador @ 2012-12-13 23:30 UTC (permalink / raw)
To: Marko Katić
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Thu, Dec 13, 2012 at 5:53 PM, Marko Katić <dromede@gmail.com> wrote:
> Oh dear, i completely forgot about this patch. It's been sitting in my
> local repo for weeks!
>
> So here's my (hopefully) final solution:
The patch seems broken for my e-mail reader (dunno if you pasted it here).
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-12-13 23:30 ` Otavio Salvador
@ 2012-12-14 15:17 ` Marko Katić
2012-12-14 15:23 ` Otavio Salvador
0 siblings, 1 reply; 14+ messages in thread
From: Marko Katić @ 2012-12-14 15:17 UTC (permalink / raw)
To: Otavio Salvador
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Thu, 2012-12-13 at 21:30 -0200, Otavio Salvador wrote:
> On Thu, Dec 13, 2012 at 5:53 PM, Marko Katić <dromede@gmail.com> wrote:
> > Oh dear, i completely forgot about this patch. It's been sitting in my
> > local repo for weeks!
> >
> > So here's my (hopefully) final solution:
>
> The patch seems broken for my e-mail reader (dunno if you pasted it here).
>
> --
> 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
Sorry about that. This one should work.
diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 4705b9b..9b1ebcf 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -74,7 +74,7 @@ FILES_udev-cache = "${sysconfdir}/init.d/udev-cache ${sysconfdir}/default/udev-c
FILES_udev-acl = "${base_libdir}/udev/udev-acl ${base_libdir}/udev/rules.d/70-acl.rules"
-FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest ${base_sbindir}/udevadm"
+FILES_udev-utils = "${bindir}/udevinfo ${bindir}/udevtest ${base_sbindir}/udevadm ${bindir}/udevadm"
RDEPENDS_udev-consolekit += "udev-acl"
FILES_udev-consolekit = "${libdir}/ConsoleKit"
@@ -97,4 +97,16 @@ do_install_append () {
rm -f ${D}${base_libdir}/udev/hid2hci
echo 'udev_run="/var/run/udev"' >> ${D}${sysconfdir}/udev/udev.conf
+
+ if [ "${PV}" -gt "175" ]
+ then
+ install -d ${D}${base_sbindir}
+ ln -s ${bindir}/udevadm ${D}${base_sbindir}/udevadm
+
+ elif [ "${PV}" -lt "175" ]
+ then
+ install -d ${D}${bindir}
+ ln -s ${base_sbindir}/udevadm ${D}${bindir}/udevadm
+
+ fi
}
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] [PATCH] udev: Remove deprecated binaries from packaging
2012-12-14 15:17 ` Marko Katić
@ 2012-12-14 15:23 ` Otavio Salvador
0 siblings, 0 replies; 14+ messages in thread
From: Otavio Salvador @ 2012-12-14 15:23 UTC (permalink / raw)
To: Marko Katić
Cc: Chris Larson, Martin Jansa,
Patches and discussions about the oe-core layer
On Fri, Dec 14, 2012 at 1:17 PM, Marko Katić <dromede@gmail.com> wrote:
> On Thu, 2012-12-13 at 21:30 -0200, Otavio Salvador wrote:
>> On Thu, Dec 13, 2012 at 5:53 PM, Marko Katić <dromede@gmail.com> wrote:
>> > Oh dear, i completely forgot about this patch. It's been sitting in my
>> > local repo for weeks!
>> >
>> > So here's my (hopefully) final solution:
>>
>> The patch seems broken for my e-mail reader (dunno if you pasted it here).
>>
>> --
>> 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
>
> Sorry about that. This one should work.
Please send a full patch, with commit log and everything. The patch
itself seems fine but it would be better to have the final and ready
to apply patch for test and review.
Regards,
--
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
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2012-12-14 15:38 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-16 20:11 [PATCH] [PATCH] udev: Remove deprecated binaries from packaging dromede
2012-11-16 21:36 ` Martin Jansa
[not found] ` <CAHod+GfAof60ycR5EXosair15O87dvZhnxWSouhdF49x8sZQqQ@mail.gmail.com>
2012-11-16 22:28 ` Martin Jansa
2012-11-16 23:28 ` Chris Larson
2012-11-17 18:02 ` Marko Katić
2012-11-17 18:38 ` Chris Larson
2012-11-17 18:54 ` Otavio Salvador
2012-11-20 19:59 ` Marko Katić
2012-11-20 20:23 ` Otavio Salvador
2012-12-13 19:53 ` Marko Katić
2012-12-13 23:30 ` Otavio Salvador
2012-12-14 15:17 ` Marko Katić
2012-12-14 15:23 ` Otavio Salvador
2012-11-17 18:20 ` Otavio Salvador
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox