* [PATCH] busybox: move watch to /usr/bin for compatibility with procps
@ 2012-03-24 19:46 Andreas Oberritter
2012-03-26 9:26 ` Martin Jansa
2012-04-17 9:40 ` Koen Kooi
0 siblings, 2 replies; 8+ messages in thread
From: Andreas Oberritter @ 2012-03-24 19:46 UTC (permalink / raw)
To: openembedded-core; +Cc: martin.jansa
Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
| update-alternatives: Error: cannot register alternative watch to
| /usr/bin/watch since it is already registered to /bin/watch
Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
* Martin, can you please verify whether this really helps? I've built a
rootfs with both busybox watch and procps installed. I didn't actually
do an opkg upgrade.
.../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
2 files changed, 31 insertions(+), 1 deletions(-)
create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
new file mode 100644
index 0000000..e8e1edf
--- /dev/null
+++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
@@ -0,0 +1,29 @@
+Upstream-Status: Inappropriate [configuration]
+
+From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
+From: Andreas Oberritter <obi@opendreambox.org>
+Date: Sat, 24 Mar 2012 17:36:48 +0100
+Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
+ procps
+
+Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
+---
+ include/applets.src.h | 2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/include/applets.src.h b/include/applets.src.h
+index 87d9cbb..c148a8c 100644
+--- a/include/applets.src.h
++++ b/include/applets.src.h
+@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
+ IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
+ /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
+ IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
+-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
+ IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
+ IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
+ IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
+--
+1.7.5.4
+
diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
index dff616d..3f22669 100644
--- a/meta/recipes-core/busybox/busybox_1.19.4.bb
+++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
@@ -1,5 +1,5 @@
require busybox.inc
-PR = "r0"
+PR = "r1"
SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
file://udhcpscript.patch \
@@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
file://get_header_tar.patch \
file://busybox-appletlib-dependency.patch \
file://run-parts.in.usr-bin.patch \
+ file://watch.in.usr-bin.patch \
file://busybox-udhcpc-no_deconfig.patch \
file://find-touchscreen.sh \
file://busybox-cron \
--
1.7.5.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-03-24 19:46 [PATCH] busybox: move watch to /usr/bin for compatibility with procps Andreas Oberritter
@ 2012-03-26 9:26 ` Martin Jansa
2012-03-26 11:02 ` Richard Purdie
2012-04-17 9:40 ` Koen Kooi
1 sibling, 1 reply; 8+ messages in thread
From: Martin Jansa @ 2012-03-26 9:26 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 6010 bytes --]
On Sat, Mar 24, 2012 at 08:46:47PM +0100, Andreas Oberritter wrote:
> Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
>
> | update-alternatives: Error: cannot register alternative watch to
> | /usr/bin/watch since it is already registered to /bin/watch
>
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> ---
> * Martin, can you please verify whether this really helps? I've built a
> rootfs with both busybox watch and procps installed. I didn't actually
> do an opkg upgrade.
Maybe it does for 2nd and later upgrades, but for some reason I now get
the same error from busybox upgrade too, probably because old u-a for
watch is removed only in busybox's prerm and not during upgrade.
SHR root@Nokia-N900 / $ ll /usr/bin/watch*
lrwxrwxrwx 1 root root 17 Mar 26 09:13 /usr/bin/watch ->
../../bin/busybox
-r-xr-xr-x 1 root root 9508 Mar 26 08:48 /usr/bin/watch.procps
SHR root@Nokia-N900 / $ ls -l /bin/watch
lrwxrwxrwx 1 root root 7 Mar 23 16:06 /bin/watch -> busybox
SHR root@Nokia-N900 / $ cat /etc/busybox.links | grep watch
/usr/bin/watch
SHR root@Nokia-N900 / $ cat /var/lib/opkg/alternatives/watch
/bin/watch
busybox 50
And also busybox is upgraded after procps so even with old alternative
removed it would take 2 opkg upgrades to get it sorted.. but that's
quite common issue :/. e.g. u-a for runlevel causes many postinst issues
because update-rc.d assumes that runlevel is available during upgrade,
but if sysvinit (as runlevel provider) is configured after e.g. dbus you
get:
Configuring dbus-1.
Cannot create link over existing -/etc/resolv.conf-.
/usr/sbin/update-rc.d: line 173: runlevel: not found
Unable to determine current runlevel
...
Configuring busybox-syslog.
update-alternatives: Linking //etc/init.d/syslog to syslog.busybox
update-alternatives: Linking //etc/syslog-startup.conf to syslog-startup.conf.busybox
/usr/sbin/update-rc.d: line 173: runlevel: not found
Unable to determine current runlevel
...
Configuring busybox-udhcpc.
/usr/sbin/update-rc.d: line 173: runlevel: not found
Unable to determine current runlevel
...
Configuring sysvinit.
update-alternatives: Linking //bin/mountpoint to mountpoint.sysvinit
update-alternatives: Linking //sbin/halt to halt.sysvinit
update-alternatives: Linking //sbin/reboot to reboot.sysvinit
update-alternatives: Linking //sbin/runlevel to runlevel.sysvinit
update-alternatives: Linking //sbin/shutdown to shutdown.sysvinit
update-alternatives: Linking //sbin/poweroff to poweroff.sysvinit
update-alternatives: Linking //usr/bin/last to last.sysvinit
update-alternatives: Linking //usr/bin/mesg to mesg.sysvinit
update-alternatives: Linking //usr/bin/wall to wall.sysvinit
update-alternatives: Linking //sbin/init to /sbin/init.sysvinit
And dbus-1. busybox-syslog. busybox-udhcpc. postinsts are fine on 2nd run.
So this patch looks good, but there are still issues with u-a calls in postinst.
Cheers,
>
> .../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
> meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
> 2 files changed, 31 insertions(+), 1 deletions(-)
> create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
>
> diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> new file mode 100644
> index 0000000..e8e1edf
> --- /dev/null
> +++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> @@ -0,0 +1,29 @@
> +Upstream-Status: Inappropriate [configuration]
> +
> +From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
> +From: Andreas Oberritter <obi@opendreambox.org>
> +Date: Sat, 24 Mar 2012 17:36:48 +0100
> +Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
> + procps
> +
> +Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> +---
> + include/applets.src.h | 2 +-
> + 1 files changed, 1 insertions(+), 1 deletions(-)
> +
> +diff --git a/include/applets.src.h b/include/applets.src.h
> +index 87d9cbb..c148a8c 100644
> +--- a/include/applets.src.h
> ++++ b/include/applets.src.h
> +@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> + IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
> + /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
> + IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> +-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
> ++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
> + IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
> + IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
> + IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
> +--
> +1.7.5.4
> +
> diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
> index dff616d..3f22669 100644
> --- a/meta/recipes-core/busybox/busybox_1.19.4.bb
> +++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
> @@ -1,5 +1,5 @@
> require busybox.inc
> -PR = "r0"
> +PR = "r1"
>
> SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> file://udhcpscript.patch \
> @@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> file://get_header_tar.patch \
> file://busybox-appletlib-dependency.patch \
> file://run-parts.in.usr-bin.patch \
> + file://watch.in.usr-bin.patch \
> file://busybox-udhcpc-no_deconfig.patch \
> file://find-touchscreen.sh \
> file://busybox-cron \
> --
> 1.7.5.4
>
>
> _______________________________________________
> 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] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-03-26 9:26 ` Martin Jansa
@ 2012-03-26 11:02 ` Richard Purdie
2012-03-26 11:11 ` Martin Jansa
0 siblings, 1 reply; 8+ messages in thread
From: Richard Purdie @ 2012-03-26 11:02 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Mon, 2012-03-26 at 11:26 +0200, Martin Jansa wrote:
> On Sat, Mar 24, 2012 at 08:46:47PM +0100, Andreas Oberritter wrote:
> > Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
> >
> > | update-alternatives: Error: cannot register alternative watch to
> > | /usr/bin/watch since it is already registered to /bin/watch
> >
> > Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> > ---
> > * Martin, can you please verify whether this really helps? I've built a
> > rootfs with both busybox watch and procps installed. I didn't actually
> > do an opkg upgrade.
>
> Maybe it does for 2nd and later upgrades, but for some reason I now get
> the same error from busybox upgrade too, probably because old u-a for
> watch is removed only in busybox's prerm and not during upgrade.
>
> SHR root@Nokia-N900 / $ ll /usr/bin/watch*
> lrwxrwxrwx 1 root root 17 Mar 26 09:13 /usr/bin/watch ->
> ../../bin/busybox
> -r-xr-xr-x 1 root root 9508 Mar 26 08:48 /usr/bin/watch.procps
> SHR root@Nokia-N900 / $ ls -l /bin/watch
> lrwxrwxrwx 1 root root 7 Mar 23 16:06 /bin/watch -> busybox
>
> SHR root@Nokia-N900 / $ cat /etc/busybox.links | grep watch
> /usr/bin/watch
> SHR root@Nokia-N900 / $ cat /var/lib/opkg/alternatives/watch
> /bin/watch
> busybox 50
>
> And also busybox is upgraded after procps so even with old alternative
> removed it would take 2 opkg upgrades to get it sorted.. but that's
> quite common issue :/. e.g. u-a for runlevel causes many postinst issues
> because update-rc.d assumes that runlevel is available during upgrade,
> but if sysvinit (as runlevel provider) is configured after e.g. dbus you
> get:
>
> Configuring dbus-1.
> Cannot create link over existing -/etc/resolv.conf-.
> /usr/sbin/update-rc.d: line 173: runlevel: not found
> Unable to determine current runlevel
> ...
> Configuring busybox-syslog.
> update-alternatives: Linking //etc/init.d/syslog to syslog.busybox
> update-alternatives: Linking //etc/syslog-startup.conf to syslog-startup.conf.busybox
> /usr/sbin/update-rc.d: line 173: runlevel: not found
> Unable to determine current runlevel
> ...
> Configuring busybox-udhcpc.
> /usr/sbin/update-rc.d: line 173: runlevel: not found
> Unable to determine current runlevel
> ...
> Configuring sysvinit.
> update-alternatives: Linking //bin/mountpoint to mountpoint.sysvinit
> update-alternatives: Linking //sbin/halt to halt.sysvinit
> update-alternatives: Linking //sbin/reboot to reboot.sysvinit
> update-alternatives: Linking //sbin/runlevel to runlevel.sysvinit
> update-alternatives: Linking //sbin/shutdown to shutdown.sysvinit
> update-alternatives: Linking //sbin/poweroff to poweroff.sysvinit
> update-alternatives: Linking //usr/bin/last to last.sysvinit
> update-alternatives: Linking //usr/bin/mesg to mesg.sysvinit
> update-alternatives: Linking //usr/bin/wall to wall.sysvinit
> update-alternatives: Linking //sbin/init to /sbin/init.sysvinit
>
> And dbus-1. busybox-syslog. busybox-udhcpc. postinsts are fine on 2nd run.
>
> So this patch looks good, but there are still issues with u-a calls in postinst.
I'll merge the patch but we should probably open a bug about these u-a
issues so we can properly fix it at some point.
Cheers,
Richard
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-03-26 11:02 ` Richard Purdie
@ 2012-03-26 11:11 ` Martin Jansa
0 siblings, 0 replies; 8+ messages in thread
From: Martin Jansa @ 2012-03-26 11:11 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 4204 bytes --]
On Mon, Mar 26, 2012 at 12:02:28PM +0100, Richard Purdie wrote:
> On Mon, 2012-03-26 at 11:26 +0200, Martin Jansa wrote:
> > On Sat, Mar 24, 2012 at 08:46:47PM +0100, Andreas Oberritter wrote:
> > > Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
> > >
> > > | update-alternatives: Error: cannot register alternative watch to
> > > | /usr/bin/watch since it is already registered to /bin/watch
> > >
> > > Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> > > ---
> > > * Martin, can you please verify whether this really helps? I've built a
> > > rootfs with both busybox watch and procps installed. I didn't actually
> > > do an opkg upgrade.
> >
> > Maybe it does for 2nd and later upgrades, but for some reason I now get
> > the same error from busybox upgrade too, probably because old u-a for
> > watch is removed only in busybox's prerm and not during upgrade.
> >
> > SHR root@Nokia-N900 / $ ll /usr/bin/watch*
> > lrwxrwxrwx 1 root root 17 Mar 26 09:13 /usr/bin/watch ->
> > ../../bin/busybox
> > -r-xr-xr-x 1 root root 9508 Mar 26 08:48 /usr/bin/watch.procps
> > SHR root@Nokia-N900 / $ ls -l /bin/watch
> > lrwxrwxrwx 1 root root 7 Mar 23 16:06 /bin/watch -> busybox
> >
> > SHR root@Nokia-N900 / $ cat /etc/busybox.links | grep watch
> > /usr/bin/watch
> > SHR root@Nokia-N900 / $ cat /var/lib/opkg/alternatives/watch
> > /bin/watch
> > busybox 50
> >
> > And also busybox is upgraded after procps so even with old alternative
> > removed it would take 2 opkg upgrades to get it sorted.. but that's
> > quite common issue :/. e.g. u-a for runlevel causes many postinst issues
> > because update-rc.d assumes that runlevel is available during upgrade,
> > but if sysvinit (as runlevel provider) is configured after e.g. dbus you
> > get:
> >
> > Configuring dbus-1.
> > Cannot create link over existing -/etc/resolv.conf-.
> > /usr/sbin/update-rc.d: line 173: runlevel: not found
> > Unable to determine current runlevel
> > ...
> > Configuring busybox-syslog.
> > update-alternatives: Linking //etc/init.d/syslog to syslog.busybox
> > update-alternatives: Linking //etc/syslog-startup.conf to syslog-startup.conf.busybox
> > /usr/sbin/update-rc.d: line 173: runlevel: not found
> > Unable to determine current runlevel
> > ...
> > Configuring busybox-udhcpc.
> > /usr/sbin/update-rc.d: line 173: runlevel: not found
> > Unable to determine current runlevel
> > ...
> > Configuring sysvinit.
> > update-alternatives: Linking //bin/mountpoint to mountpoint.sysvinit
> > update-alternatives: Linking //sbin/halt to halt.sysvinit
> > update-alternatives: Linking //sbin/reboot to reboot.sysvinit
> > update-alternatives: Linking //sbin/runlevel to runlevel.sysvinit
> > update-alternatives: Linking //sbin/shutdown to shutdown.sysvinit
> > update-alternatives: Linking //sbin/poweroff to poweroff.sysvinit
> > update-alternatives: Linking //usr/bin/last to last.sysvinit
> > update-alternatives: Linking //usr/bin/mesg to mesg.sysvinit
> > update-alternatives: Linking //usr/bin/wall to wall.sysvinit
> > update-alternatives: Linking //sbin/init to /sbin/init.sysvinit
> >
> > And dbus-1. busybox-syslog. busybox-udhcpc. postinsts are fine on 2nd run.
> >
> > So this patch looks good, but there are still issues with u-a calls in postinst.
>
> I'll merge the patch but we should probably open a bug about these u-a
yes please it improves situation for people with reflashed image.. other
need just rm /var/lib/opkg/alternatives/watch to fix upgrade path
> issues so we can properly fix it at some point.
Fixing this issue will be a bit problematic, because for busybox when
you use -force-reinstall it can remove some required commands you need
to install it after it's removed.
So e.g. you need standalone wget and sed to be able to use
opkg install -force-reinstall, but -force-reinstall also doesn't resolve
this watch issue completely, because after first upgrade there is
new /usr/bin/watch path in /etc/busybox.links so prerm also doesn't
remove stalled /bin/watch alternative :/.
Cheers,
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-03-24 19:46 [PATCH] busybox: move watch to /usr/bin for compatibility with procps Andreas Oberritter
2012-03-26 9:26 ` Martin Jansa
@ 2012-04-17 9:40 ` Koen Kooi
2012-04-17 10:44 ` Richard Purdie
1 sibling, 1 reply; 8+ messages in thread
From: Koen Kooi @ 2012-04-17 9:40 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
Ping
Op 24 mrt. 2012, om 20:46 heeft Andreas Oberritter het volgende geschreven:
> Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
>
> | update-alternatives: Error: cannot register alternative watch to
> | /usr/bin/watch since it is already registered to /bin/watch
>
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> ---
> * Martin, can you please verify whether this really helps? I've built a
> rootfs with both busybox watch and procps installed. I didn't actually
> do an opkg upgrade.
>
> .../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
> meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
> 2 files changed, 31 insertions(+), 1 deletions(-)
> create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
>
> diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> new file mode 100644
> index 0000000..e8e1edf
> --- /dev/null
> +++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> @@ -0,0 +1,29 @@
> +Upstream-Status: Inappropriate [configuration]
> +
> +From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
> +From: Andreas Oberritter <obi@opendreambox.org>
> +Date: Sat, 24 Mar 2012 17:36:48 +0100
> +Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
> + procps
> +
> +Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> +---
> + include/applets.src.h | 2 +-
> + 1 files changed, 1 insertions(+), 1 deletions(-)
> +
> +diff --git a/include/applets.src.h b/include/applets.src.h
> +index 87d9cbb..c148a8c 100644
> +--- a/include/applets.src.h
> ++++ b/include/applets.src.h
> +@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> + IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
> + /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
> + IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> +-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
> ++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
> + IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
> + IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
> + IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
> +--
> +1.7.5.4
> +
> diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
> index dff616d..3f22669 100644
> --- a/meta/recipes-core/busybox/busybox_1.19.4.bb
> +++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
> @@ -1,5 +1,5 @@
> require busybox.inc
> -PR = "r0"
> +PR = "r1"
>
> SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> file://udhcpscript.patch \
> @@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> file://get_header_tar.patch \
> file://busybox-appletlib-dependency.patch \
> file://run-parts.in.usr-bin.patch \
> + file://watch.in.usr-bin.patch \
> file://busybox-udhcpc-no_deconfig.patch \
> file://find-touchscreen.sh \
> file://busybox-cron \
> --
> 1.7.5.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-04-17 9:40 ` Koen Kooi
@ 2012-04-17 10:44 ` Richard Purdie
2012-04-17 10:46 ` Koen Kooi
0 siblings, 1 reply; 8+ messages in thread
From: Richard Purdie @ 2012-04-17 10:44 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Tue, 2012-04-17 at 11:40 +0200, Koen Kooi wrote:
> Ping
It merged three weeks ago:
http://git.openembedded.org/openembedded-core/commit/?id=fa8b2d2bed63fa4fc2af45f094998b0f9e69d272
?
Cheers,
Richard
> Op 24 mrt. 2012, om 20:46 heeft Andreas Oberritter het volgende geschreven:
>
> > Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
> >
> > | update-alternatives: Error: cannot register alternative watch to
> > | /usr/bin/watch since it is already registered to /bin/watch
> >
> > Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> > ---
> > * Martin, can you please verify whether this really helps? I've built a
> > rootfs with both busybox watch and procps installed. I didn't actually
> > do an opkg upgrade.
> >
> > .../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
> > meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
> > 2 files changed, 31 insertions(+), 1 deletions(-)
> > create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> >
> > diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> > new file mode 100644
> > index 0000000..e8e1edf
> > --- /dev/null
> > +++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> > @@ -0,0 +1,29 @@
> > +Upstream-Status: Inappropriate [configuration]
> > +
> > +From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
> > +From: Andreas Oberritter <obi@opendreambox.org>
> > +Date: Sat, 24 Mar 2012 17:36:48 +0100
> > +Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
> > + procps
> > +
> > +Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> > +---
> > + include/applets.src.h | 2 +-
> > + 1 files changed, 1 insertions(+), 1 deletions(-)
> > +
> > +diff --git a/include/applets.src.h b/include/applets.src.h
> > +index 87d9cbb..c148a8c 100644
> > +--- a/include/applets.src.h
> > ++++ b/include/applets.src.h
> > +@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> > + IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
> > + /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
> > + IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> > +-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
> > ++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
> > + IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
> > + IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
> > + IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
> > +--
> > +1.7.5.4
> > +
> > diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
> > index dff616d..3f22669 100644
> > --- a/meta/recipes-core/busybox/busybox_1.19.4.bb
> > +++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
> > @@ -1,5 +1,5 @@
> > require busybox.inc
> > -PR = "r0"
> > +PR = "r1"
> >
> > SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> > file://udhcpscript.patch \
> > @@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> > file://get_header_tar.patch \
> > file://busybox-appletlib-dependency.patch \
> > file://run-parts.in.usr-bin.patch \
> > + file://watch.in.usr-bin.patch \
> > file://busybox-udhcpc-no_deconfig.patch \
> > file://find-touchscreen.sh \
> > file://busybox-cron \
> > --
> > 1.7.5.4
> >
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-04-17 10:44 ` Richard Purdie
@ 2012-04-17 10:46 ` Koen Kooi
2012-04-17 10:51 ` Martin Jansa
0 siblings, 1 reply; 8+ messages in thread
From: Koen Kooi @ 2012-04-17 10:46 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
Op 17 apr. 2012, om 12:44 heeft Richard Purdie het volgende geschreven:
> On Tue, 2012-04-17 at 11:40 +0200, Koen Kooi wrote:
>> Ping
>
> It merged three weeks ago:
>
> http://git.openembedded.org/openembedded-core/commit/?id=fa8b2d2bed63fa4fc2af45f094998b0f9e69d272
Weird, I still get the opkg errors about /bin/watch, but that's probably a bug in opkg :(
regards,
Koen
>
> ?
>
> Cheers,
>
> Richard
>
>
>> Op 24 mrt. 2012, om 20:46 heeft Andreas Oberritter het volgende geschreven:
>>
>>> Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
>>>
>>> | update-alternatives: Error: cannot register alternative watch to
>>> | /usr/bin/watch since it is already registered to /bin/watch
>>>
>>> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
>>> ---
>>> * Martin, can you please verify whether this really helps? I've built a
>>> rootfs with both busybox watch and procps installed. I didn't actually
>>> do an opkg upgrade.
>>>
>>> .../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
>>> meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
>>> 2 files changed, 31 insertions(+), 1 deletions(-)
>>> create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
>>>
>>> diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
>>> new file mode 100644
>>> index 0000000..e8e1edf
>>> --- /dev/null
>>> +++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
>>> @@ -0,0 +1,29 @@
>>> +Upstream-Status: Inappropriate [configuration]
>>> +
>>> +From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
>>> +From: Andreas Oberritter <obi@opendreambox.org>
>>> +Date: Sat, 24 Mar 2012 17:36:48 +0100
>>> +Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
>>> + procps
>>> +
>>> +Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
>>> +---
>>> + include/applets.src.h | 2 +-
>>> + 1 files changed, 1 insertions(+), 1 deletions(-)
>>> +
>>> +diff --git a/include/applets.src.h b/include/applets.src.h
>>> +index 87d9cbb..c148a8c 100644
>>> +--- a/include/applets.src.h
>>> ++++ b/include/applets.src.h
>>> +@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
>>> + IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
>>> + /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
>>> + IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
>>> +-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
>>> ++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
>>> + IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
>>> + IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
>>> + IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
>>> +--
>>> +1.7.5.4
>>> +
>>> diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
>>> index dff616d..3f22669 100644
>>> --- a/meta/recipes-core/busybox/busybox_1.19.4.bb
>>> +++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
>>> @@ -1,5 +1,5 @@
>>> require busybox.inc
>>> -PR = "r0"
>>> +PR = "r1"
>>>
>>> SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
>>> file://udhcpscript.patch \
>>> @@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
>>> file://get_header_tar.patch \
>>> file://busybox-appletlib-dependency.patch \
>>> file://run-parts.in.usr-bin.patch \
>>> + file://watch.in.usr-bin.patch \
>>> file://busybox-udhcpc-no_deconfig.patch \
>>> file://find-touchscreen.sh \
>>> file://busybox-cron \
>>> --
>>> 1.7.5.4
>>>
>>>
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] busybox: move watch to /usr/bin for compatibility with procps
2012-04-17 10:46 ` Koen Kooi
@ 2012-04-17 10:51 ` Martin Jansa
0 siblings, 0 replies; 8+ messages in thread
From: Martin Jansa @ 2012-04-17 10:51 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 5166 bytes --]
On Tue, Apr 17, 2012 at 12:46:08PM +0200, Koen Kooi wrote:
>
> Op 17 apr. 2012, om 12:44 heeft Richard Purdie het volgende geschreven:
>
> > On Tue, 2012-04-17 at 11:40 +0200, Koen Kooi wrote:
> >> Ping
> >
> > It merged three weeks ago:
> >
> > http://git.openembedded.org/openembedded-core/commit/?id=fa8b2d2bed63fa4fc2af45f094998b0f9e69d272
>
>
> Weird, I still get the opkg errors about /bin/watch, but that's probably a bug in opkg :(
see
http://permalink.gmane.org/gmane.comp.handhelds.openembedded.core/17867
>
> regards,
>
> Koen
>
> >
> > ?
> >
> > Cheers,
> >
> > Richard
> >
> >
> >> Op 24 mrt. 2012, om 20:46 heeft Andreas Oberritter het volgende geschreven:
> >>
> >>> Commit 68dc2a3fa30c03a196d650de34d0c657a7b85454 broke opkg upgrade:
> >>>
> >>> | update-alternatives: Error: cannot register alternative watch to
> >>> | /usr/bin/watch since it is already registered to /bin/watch
> >>>
> >>> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> >>> ---
> >>> * Martin, can you please verify whether this really helps? I've built a
> >>> rootfs with both busybox watch and procps installed. I didn't actually
> >>> do an opkg upgrade.
> >>>
> >>> .../busybox/busybox-1.19.4/watch.in.usr-bin.patch | 29 ++++++++++++++++++++
> >>> meta/recipes-core/busybox/busybox_1.19.4.bb | 3 +-
> >>> 2 files changed, 31 insertions(+), 1 deletions(-)
> >>> create mode 100644 meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> >>>
> >>> diff --git a/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> >>> new file mode 100644
> >>> index 0000000..e8e1edf
> >>> --- /dev/null
> >>> +++ b/meta/recipes-core/busybox/busybox-1.19.4/watch.in.usr-bin.patch
> >>> @@ -0,0 +1,29 @@
> >>> +Upstream-Status: Inappropriate [configuration]
> >>> +
> >>> +From f2372ab626a395d067704f4c318180fb6039de15 Mon Sep 17 00:00:00 2001
> >>> +From: Andreas Oberritter <obi@opendreambox.org>
> >>> +Date: Sat, 24 Mar 2012 17:36:48 +0100
> >>> +Subject: [PATCH] watch: move from /bin to /usr/bin for compatibility with
> >>> + procps
> >>> +
> >>> +Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> >>> +---
> >>> + include/applets.src.h | 2 +-
> >>> + 1 files changed, 1 insertions(+), 1 deletions(-)
> >>> +
> >>> +diff --git a/include/applets.src.h b/include/applets.src.h
> >>> +index 87d9cbb..c148a8c 100644
> >>> +--- a/include/applets.src.h
> >>> ++++ b/include/applets.src.h
> >>> +@@ -404,7 +404,7 @@ IF_VLOCK(APPLET(vlock, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> >>> + IF_VOLNAME(APPLET(volname, BB_DIR_USR_BIN, BB_SUID_DROP))
> >>> + /* Needs to be run by root or be suid root - needs to write to /dev/TTY: */
> >>> + IF_WALL(APPLET(wall, BB_DIR_USR_BIN, BB_SUID_REQUIRE))
> >>> +-IF_WATCH(APPLET(watch, BB_DIR_BIN, BB_SUID_DROP))
> >>> ++IF_WATCH(APPLET(watch, BB_DIR_USR_BIN, BB_SUID_DROP))
> >>> + IF_WATCHDOG(APPLET(watchdog, BB_DIR_SBIN, BB_SUID_DROP))
> >>> + IF_WC(APPLET(wc, BB_DIR_USR_BIN, BB_SUID_DROP))
> >>> + IF_WGET(APPLET(wget, BB_DIR_USR_BIN, BB_SUID_DROP))
> >>> +--
> >>> +1.7.5.4
> >>> +
> >>> diff --git a/meta/recipes-core/busybox/busybox_1.19.4.bb b/meta/recipes-core/busybox/busybox_1.19.4.bb
> >>> index dff616d..3f22669 100644
> >>> --- a/meta/recipes-core/busybox/busybox_1.19.4.bb
> >>> +++ b/meta/recipes-core/busybox/busybox_1.19.4.bb
> >>> @@ -1,5 +1,5 @@
> >>> require busybox.inc
> >>> -PR = "r0"
> >>> +PR = "r1"
> >>>
> >>> SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> >>> file://udhcpscript.patch \
> >>> @@ -8,6 +8,7 @@ SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> >>> file://get_header_tar.patch \
> >>> file://busybox-appletlib-dependency.patch \
> >>> file://run-parts.in.usr-bin.patch \
> >>> + file://watch.in.usr-bin.patch \
> >>> file://busybox-udhcpc-no_deconfig.patch \
> >>> file://find-touchscreen.sh \
> >>> file://busybox-cron \
> >>> --
> >>> 1.7.5.4
> >>>
> >>>
> >>> _______________________________________________
> >>> Openembedded-core mailing list
> >>> Openembedded-core@lists.openembedded.org
> >>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> >>
> >>
> >> _______________________________________________
> >> Openembedded-core mailing list
> >> Openembedded-core@lists.openembedded.org
> >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> >>
> >
> >
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>
> _______________________________________________
> 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] 8+ messages in thread
end of thread, other threads:[~2012-04-17 11:01 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-03-24 19:46 [PATCH] busybox: move watch to /usr/bin for compatibility with procps Andreas Oberritter
2012-03-26 9:26 ` Martin Jansa
2012-03-26 11:02 ` Richard Purdie
2012-03-26 11:11 ` Martin Jansa
2012-04-17 9:40 ` Koen Kooi
2012-04-17 10:44 ` Richard Purdie
2012-04-17 10:46 ` Koen Kooi
2012-04-17 10:51 ` Martin Jansa
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.