* [Buildroot] [PATCH 1/9] skeleton/S40network: tweak for debian ifupdown
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 19:24 ` Arnout Vandecappelle
2014-11-18 17:01 ` [Buildroot] [PATCH 2/9] debianutils: new package Gustavo Zacarias
` (7 subsequent siblings)
8 siblings, 1 reply; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Make the S40network script create the /run/network directory for the
debian variant of ifupdown which uses it as a lock directory.
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
system/skeleton/etc/init.d/S40network | 3 +++
1 file changed, 3 insertions(+)
diff --git a/system/skeleton/etc/init.d/S40network b/system/skeleton/etc/init.d/S40network
index e69e613..bfdd491 100755
--- a/system/skeleton/etc/init.d/S40network
+++ b/system/skeleton/etc/init.d/S40network
@@ -3,6 +3,9 @@
# Start the network....
#
+# Debian ifupdown needs the /run/network lock directory
+mkdir -p /run/network
+
case "$1" in
start)
echo "Starting network..."
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 1/9] skeleton/S40network: tweak for debian ifupdown
2014-11-18 17:01 ` [Buildroot] [PATCH 1/9] skeleton/S40network: tweak for debian ifupdown Gustavo Zacarias
@ 2014-11-18 19:24 ` Arnout Vandecappelle
0 siblings, 0 replies; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 19:24 UTC (permalink / raw)
To: buildroot
On 18/11/14 18:01, Gustavo Zacarias wrote:
> Make the S40network script create the /run/network directory for the
> debian variant of ifupdown which uses it as a lock directory.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> system/skeleton/etc/init.d/S40network | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/system/skeleton/etc/init.d/S40network b/system/skeleton/etc/init.d/S40network
> index e69e613..bfdd491 100755
> --- a/system/skeleton/etc/init.d/S40network
> +++ b/system/skeleton/etc/init.d/S40network
> @@ -3,6 +3,9 @@
> # Start the network....
> #
>
> +# Debian ifupdown needs the /run/network lock directory
> +mkdir -p /run/network
> +
> case "$1" in
> start)
> echo "Starting network..."
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 2/9] debianutils: new package
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
2014-11-18 17:01 ` [Buildroot] [PATCH 1/9] skeleton/S40network: tweak for debian ifupdown Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 19:37 ` Arnout Vandecappelle
2014-11-18 22:31 ` Thomas Petazzoni
2014-11-18 17:01 ` [Buildroot] [PATCH 3/9] ifupdown: " Gustavo Zacarias
` (6 subsequent siblings)
8 siblings, 2 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/Config.in | 1 +
package/debianutils/Config.in | 7 +++++++
package/debianutils/debianutils.hash | 2 ++
package/debianutils/debianutils.mk | 14 ++++++++++++++
4 files changed, 24 insertions(+)
create mode 100644 package/debianutils/Config.in
create mode 100644 package/debianutils/debianutils.hash
create mode 100644 package/debianutils/debianutils.mk
diff --git a/package/Config.in b/package/Config.in
index 0dbb788..278134d 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1199,6 +1199,7 @@ endif
source "package/cpuload/Config.in"
if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
source "package/dcron/Config.in"
+ source "package/debianutils/Config.in"
endif
source "package/dsp-tools/Config.in"
source "package/ftop/Config.in"
diff --git a/package/debianutils/Config.in b/package/debianutils/Config.in
new file mode 100644
index 0000000..f865868
--- /dev/null
+++ b/package/debianutils/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_DEBIANUTILS
+ bool "debianutils"
+ depends on BR2_USE_MMU # fork()
+ help
+ Miscellaneous utilities specific to Debian.
+
+ https://tracker.debian.org/pkg/debianutils
diff --git a/package/debianutils/debianutils.hash b/package/debianutils/debianutils.hash
new file mode 100644
index 0000000..089dd42
--- /dev/null
+++ b/package/debianutils/debianutils.hash
@@ -0,0 +1,2 @@
+# From http://ftp.de.debian.org/debian/pool/main/d/debianutils/debianutils_4.4.dsc
+sha256 190850cdd6b5302e0a1ba1aaed1bc7074d67d3bd8d04c613f242f7145afa53a6 debianutils_4.4.tar.gz
diff --git a/package/debianutils/debianutils.mk b/package/debianutils/debianutils.mk
new file mode 100644
index 0000000..43d7cf3
--- /dev/null
+++ b/package/debianutils/debianutils.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# debianutils
+#
+################################################################################
+
+DEBIANUTILS_VERSION = 4.4
+DEBIANUTILS_SOURCE = debianutils_$(DEBIANUTILS_VERSION).tar.gz
+DEBIANUTILS_SITE = http://snapshot.debian.org/archive/debian/20130728T034252Z/pool/main/d/debianutils
+DEBIANUTILS_CONF_OPTS = --exec-prefix=/
+DEBIANUTILS_LICENSE = GPLv2+, SMAIL (savelog)
+DEBIANUTILS_LICENSE_FILES = debian/copyright
+
+$(eval $(autotools-package))
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 2/9] debianutils: new package
2014-11-18 17:01 ` [Buildroot] [PATCH 2/9] debianutils: new package Gustavo Zacarias
@ 2014-11-18 19:37 ` Arnout Vandecappelle
2014-11-18 22:31 ` Thomas Petazzoni
1 sibling, 0 replies; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 19:37 UTC (permalink / raw)
To: buildroot
On 18/11/14 18:01, Gustavo Zacarias wrote:
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
You didn't carry my
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
but anyway here's an
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
> ---
> package/Config.in | 1 +
> package/debianutils/Config.in | 7 +++++++
> package/debianutils/debianutils.hash | 2 ++
> package/debianutils/debianutils.mk | 14 ++++++++++++++
> 4 files changed, 24 insertions(+)
> create mode 100644 package/debianutils/Config.in
> create mode 100644 package/debianutils/debianutils.hash
> create mode 100644 package/debianutils/debianutils.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 0dbb788..278134d 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1199,6 +1199,7 @@ endif
> source "package/cpuload/Config.in"
> if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> source "package/dcron/Config.in"
> + source "package/debianutils/Config.in"
> endif
> source "package/dsp-tools/Config.in"
> source "package/ftop/Config.in"
> diff --git a/package/debianutils/Config.in b/package/debianutils/Config.in
> new file mode 100644
> index 0000000..f865868
> --- /dev/null
> +++ b/package/debianutils/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_DEBIANUTILS
> + bool "debianutils"
> + depends on BR2_USE_MMU # fork()
> + help
> + Miscellaneous utilities specific to Debian.
> +
> + https://tracker.debian.org/pkg/debianutils
> diff --git a/package/debianutils/debianutils.hash b/package/debianutils/debianutils.hash
> new file mode 100644
> index 0000000..089dd42
> --- /dev/null
> +++ b/package/debianutils/debianutils.hash
> @@ -0,0 +1,2 @@
> +# From http://ftp.de.debian.org/debian/pool/main/d/debianutils/debianutils_4.4.dsc
> +sha256 190850cdd6b5302e0a1ba1aaed1bc7074d67d3bd8d04c613f242f7145afa53a6 debianutils_4.4.tar.gz
> diff --git a/package/debianutils/debianutils.mk b/package/debianutils/debianutils.mk
> new file mode 100644
> index 0000000..43d7cf3
> --- /dev/null
> +++ b/package/debianutils/debianutils.mk
> @@ -0,0 +1,14 @@
> +################################################################################
> +#
> +# debianutils
> +#
> +################################################################################
> +
> +DEBIANUTILS_VERSION = 4.4
> +DEBIANUTILS_SOURCE = debianutils_$(DEBIANUTILS_VERSION).tar.gz
> +DEBIANUTILS_SITE = http://snapshot.debian.org/archive/debian/20130728T034252Z/pool/main/d/debianutils
> +DEBIANUTILS_CONF_OPTS = --exec-prefix=/
> +DEBIANUTILS_LICENSE = GPLv2+, SMAIL (savelog)
> +DEBIANUTILS_LICENSE_FILES = debian/copyright
> +
> +$(eval $(autotools-package))
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 2/9] debianutils: new package
2014-11-18 17:01 ` [Buildroot] [PATCH 2/9] debianutils: new package Gustavo Zacarias
2014-11-18 19:37 ` Arnout Vandecappelle
@ 2014-11-18 22:31 ` Thomas Petazzoni
1 sibling, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2014-11-18 22:31 UTC (permalink / raw)
To: buildroot
Dear Gustavo Zacarias,
On Tue, 18 Nov 2014 14:01:50 -0300, Gustavo Zacarias wrote:
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
> package/Config.in | 1 +
> package/debianutils/Config.in | 7 +++++++
> package/debianutils/debianutils.hash | 2 ++
> package/debianutils/debianutils.mk | 14 ++++++++++++++
> 4 files changed, 24 insertions(+)
> create mode 100644 package/debianutils/Config.in
> create mode 100644 package/debianutils/debianutils.hash
> create mode 100644 package/debianutils/debianutils.mk
Thanks, patch applied to next. However, I've added an optional
dependency on Busybox, since debianutils installs 'which', which is
also installed by Busybox.
Thanks to Arnout for the review and testing.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
2014-11-18 17:01 ` [Buildroot] [PATCH 1/9] skeleton/S40network: tweak for debian ifupdown Gustavo Zacarias
2014-11-18 17:01 ` [Buildroot] [PATCH 2/9] debianutils: new package Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 19:30 ` Károly Kasza
2014-11-18 19:56 ` Arnout Vandecappelle
2014-11-18 17:01 ` [Buildroot] [PATCH 4/9] start-stop-daemon: " Gustavo Zacarias
` (5 subsequent siblings)
8 siblings, 2 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/Config.in | 3 +++
package/ifupdown/Config.in | 17 +++++++++++++++++
package/ifupdown/ifupdown.hash | 2 ++
package/ifupdown/ifupdown.mk | 24 ++++++++++++++++++++++++
4 files changed, 46 insertions(+)
create mode 100644 package/ifupdown/Config.in
create mode 100644 package/ifupdown/ifupdown.hash
create mode 100644 package/ifupdown/ifupdown.mk
diff --git a/package/Config.in b/package/Config.in
index 278134d..8f3cc3c 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1014,6 +1014,9 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
source "package/ifplugd/Config.in"
endif
source "package/iftop/Config.in"
+if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
+ source "package/ifupdown/Config.in"
+endif
source "package/igh-ethercat/Config.in"
source "package/igmpproxy/Config.in"
source "package/inadyn/Config.in"
diff --git a/package/ifupdown/Config.in b/package/ifupdown/Config.in
new file mode 100644
index 0000000..10cf2be
--- /dev/null
+++ b/package/ifupdown/Config.in
@@ -0,0 +1,17 @@
+config BR2_PACKAGE_IFUPDOWN
+ bool "ifupdown"
+ depends on BR2_USE_MMU # fork()
+ # Default/our uclibc lacks wordexp()
+ depends on !BR2_TOOLCHAIN_USES_UCLIBC
+ # runtime for ip binary
+ select BR2_PACKAGE_IPROUTE2 if !BR2_PACKAGE_BUSYBOX
+ # runtime for run-parts
+ select BR2_PACKAGE_DEBIANUTILS if !BR2_PACKAGE_BUSYBOX
+ help
+ High level tools to configure network interfaces.
+
+ https://tracker.debian.org/pkg/ifupdown
+
+comment "ifupdown needs an (e)glibc or musl toolchain"
+ depends on BR2_USE_MMU
+ depends on BR2_TOOLCHAIN_USES_UCLIBC
diff --git a/package/ifupdown/ifupdown.hash b/package/ifupdown/ifupdown.hash
new file mode 100644
index 0000000..f0c314b
--- /dev/null
+++ b/package/ifupdown/ifupdown.hash
@@ -0,0 +1,2 @@
+# From http://ftp.de.debian.org/debian/pool/main/i/ifupdown/ifupdown_0.7.49.dsc
+sha256 5d07c73610ff90393c7273e2af9003eeb1825fc88144ae5ed11a5517547d0f58 ifupdown_0.7.49.tar.xz
diff --git a/package/ifupdown/ifupdown.mk b/package/ifupdown/ifupdown.mk
new file mode 100644
index 0000000..f4ad7d3
--- /dev/null
+++ b/package/ifupdown/ifupdown.mk
@@ -0,0 +1,24 @@
+################################################################################
+#
+# ifupdown
+#
+################################################################################
+
+IFUPDOWN_VERSION = 0.7.49
+IFUPDOWN_SOURCE = ifupdown_$(IFUPDOWN_VERSION).tar.xz
+IFUPDOWN_SITE = http://snapshot.debian.org/archive/debian/20140923T221921Z/pool/main/i/ifupdown
+IFUPDOWN_DEPENDENCIES = $(if $(BR2_PACKAGE_BUSYBOX),busybox)
+IFUPDOWN_LICENSE = GPLv2+
+IFUPDOWN_LICENSE_FILES = COPYING
+
+define IFUPDOWN_BUILD_CMDS
+ $(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) \
+ CFLAGS="$(TARGET_CFLAGS) -D'IFUPDOWN_VERSION=\"$(IFUPDOWN_VERSION)\"'" \
+ -C $(@D)
+endef
+
+define IFUPDOWN_INSTALL_TARGET_CMDS
+ $(TARGET_MAKE_ENV) $(MAKE) BASEDIR=$(TARGET_DIR) -C $(@D) install
+endef
+
+$(eval $(generic-package))
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 17:01 ` [Buildroot] [PATCH 3/9] ifupdown: " Gustavo Zacarias
@ 2014-11-18 19:30 ` Károly Kasza
2014-11-18 19:42 ` Károly Kasza
2014-11-18 19:56 ` Arnout Vandecappelle
1 sibling, 1 reply; 25+ messages in thread
From: Károly Kasza @ 2014-11-18 19:30 UTC (permalink / raw)
To: buildroot
Hello Gustavo, Arnout, all,
Unfortunately this fails for me while building:
First it complains about a non-existent "dpkg-architecture" program (I'm
building on Debian x64 and it is not installed on my build machine by
default) - then it fails arch.h no such file or directory - it is created
with the $DEB_HOST_ARCH_OS variable, which is set by the above utility.
This is the same perl script found in "dpkg" source 1.16.15 in the scripts
sub directory (which is already used by start-stop-daemon). It is part of
the dpkg-dev debian package in stable.
I think, that this program is not needed by us, as we only target Linux,
and dpkg-architecture is used to query 2 variables we are compiling for:
- DEB_HOST_ARCH_OS which is always "linux" (I guess this could be hurd or
netbsd, as Debian supports those kernels, not Buildroot though)
- DEB_BUILD_GNU_TYPE which is <architecture>-linux-gnu
Maybe add a small patch, that replaces all dpkg-architecture
DEB_HOST_ARCH_OS queries to a "linux" constant, and the DEB_BUILD_GNU_TYPE
query to a Buildroot specific macro which adds the target architecture?
This call appears in:
defn2c.pl - 5
defn2man.pl - 5
debian/rules - 41,43
debian/testbuild - 3
---
>>> ifupdown 0.7.49 Building
...BLAH...
Can't exec "dpkg-architecture": No such file or directory at ./defn2c.pl
line 5.
Use of uninitialized value $DEB_HOST_ARCH_OS in substitution (s///) at ./
defn2c.pl line 7.
Use of uninitialized value $DEB_HOST_ARCH_OS in string eq at ./defn2c.pl
line 33, <> line 2.
...BLAH...
/home/scythe/buildroot/output/host/usr/bin/x86_64-linux-gcc
-D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -pipe -Os
-D'IFUPDOWN_VERSION="0.7.49"' -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
-D_FILE_OFFSET_BITS=64 -c -o link.o link.c
sed -e '/^##ADDRESSFAM##$/r inet.man' -e '/^##ADDRESSFAM##$/r ipx.man' -e
'/^##ADDRESSFAM##$/r inet6.man' -e '/^##ADDRESSFAM##$/r can.man' \
-e '/^##ADDRESSFAM##$/d' < interfaces.5.pre > interfaces.5
meta.c:5:18: fatal error: arch.h: No such file or directory
#include "arch.h"
^
compilation terminated.
make[1]: *** [meta.o] Error 1
---
Best regards,
Karoly
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141118/6a5cda13/attachment.html>
^ permalink raw reply [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 19:30 ` Károly Kasza
@ 2014-11-18 19:42 ` Károly Kasza
2014-11-18 19:46 ` Gustavo Zacarias
2014-11-18 19:53 ` Arnout Vandecappelle
0 siblings, 2 replies; 25+ messages in thread
From: Károly Kasza @ 2014-11-18 19:42 UTC (permalink / raw)
To: buildroot
I don't know how should I propose a patch to a pending patch :)
So here's attached a patch file, to be put into the package/ifupdown
directory.
BR
Karoly
On Tue, Nov 18, 2014 at 8:30 PM, K?roly Kasza <kaszak@gmail.com> wrote:
> Hello Gustavo, Arnout, all,
>
> Unfortunately this fails for me while building:
> First it complains about a non-existent "dpkg-architecture" program (I'm
> building on Debian x64 and it is not installed on my build machine by
> default) - then it fails arch.h no such file or directory - it is created
> with the $DEB_HOST_ARCH_OS variable, which is set by the above utility.
>
> This is the same perl script found in "dpkg" source 1.16.15 in the scripts
> sub directory (which is already used by start-stop-daemon). It is part of
> the dpkg-dev debian package in stable.
>
> I think, that this program is not needed by us, as we only target Linux,
> and dpkg-architecture is used to query 2 variables we are compiling for:
> - DEB_HOST_ARCH_OS which is always "linux" (I guess this could be hurd or
> netbsd, as Debian supports those kernels, not Buildroot though)
> - DEB_BUILD_GNU_TYPE which is <architecture>-linux-gnu
>
> Maybe add a small patch, that replaces all dpkg-architecture
> DEB_HOST_ARCH_OS queries to a "linux" constant, and the DEB_BUILD_GNU_TYPE
> query to a Buildroot specific macro which adds the target architecture?
>
> This call appears in:
> defn2c.pl - 5
> defn2man.pl - 5
> debian/rules - 41,43
> debian/testbuild - 3
>
> ---
>
> >>> ifupdown 0.7.49 Building
> ...BLAH...
> Can't exec "dpkg-architecture": No such file or directory at ./defn2c.pl
> line 5.
> Use of uninitialized value $DEB_HOST_ARCH_OS in substitution (s///) at ./
> defn2c.pl line 7.
> Use of uninitialized value $DEB_HOST_ARCH_OS in string eq at ./defn2c.pl
> line 33, <> line 2.
> ...BLAH...
> /home/scythe/buildroot/output/host/usr/bin/x86_64-linux-gcc
> -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -pipe -Os
> -D'IFUPDOWN_VERSION="0.7.49"' -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
> -D_FILE_OFFSET_BITS=64 -c -o link.o link.c
> sed -e '/^##ADDRESSFAM##$/r inet.man' -e '/^##ADDRESSFAM##$/r ipx.man' -e
> '/^##ADDRESSFAM##$/r inet6.man' -e '/^##ADDRESSFAM##$/r can.man' \
> -e '/^##ADDRESSFAM##$/d' < interfaces.5.pre > interfaces.5
> meta.c:5:18: fatal error: arch.h: No such file or directory
> #include "arch.h"
> ^
> compilation terminated.
> make[1]: *** [meta.o] Error 1
>
> ---
>
> Best regards,
> Karoly
>
--
?dv,
KK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141118/a2c4292a/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dpkg-architecture.patch
Type: application/octet-stream
Size: 1716 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141118/a2c4292a/attachment.obj>
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 19:42 ` Károly Kasza
@ 2014-11-18 19:46 ` Gustavo Zacarias
2014-11-18 19:53 ` Arnout Vandecappelle
1 sibling, 0 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 19:46 UTC (permalink / raw)
To: buildroot
On 11/18/2014 04:42 PM, K?roly Kasza wrote:
> I don't know how should I propose a patch to a pending patch :)
> So here's attached a patch file, to be put into the package/ifupdown
> directory.
Hi Karoly.
Thanks for spotting this one, i tested on Gentoo thinking "there's
nothing more different than Debian" and i had debianutils installed hah.
Actually just doing:
my $DEB_HOST_ARCH_OS = "linux";
...in defn2c.pl is enough.
I'll add the minor patch.
Regards.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 19:42 ` Károly Kasza
2014-11-18 19:46 ` Gustavo Zacarias
@ 2014-11-18 19:53 ` Arnout Vandecappelle
1 sibling, 0 replies; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 19:53 UTC (permalink / raw)
To: buildroot
On 18/11/14 20:42, K?roly Kasza wrote:
> I don't know how should I propose a patch to a pending patch :)
Like you did, except:
- never attach, always send in-line;
- don't top-post.
> So here's attached a patch file, to be put into the package/ifupdown directory.
Patching the debian/* files isn't necessary, we don't use them.
I agree with the solution of this patch. The alternative would be to create our
own dpkg-architecture in our HOST_DIR, but I think that's a bit too much to ask.
Regards,
Arnout
>
> BR
> Karoly
>
[snip]
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 17:01 ` [Buildroot] [PATCH 3/9] ifupdown: " Gustavo Zacarias
2014-11-18 19:30 ` Károly Kasza
@ 2014-11-18 19:56 ` Arnout Vandecappelle
2014-11-18 19:59 ` Gustavo Zacarias
1 sibling, 1 reply; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 19:56 UTC (permalink / raw)
To: buildroot
On 18/11/14 18:01, Gustavo Zacarias wrote:
> +define IFUPDOWN_INSTALL_TARGET_CMDS
> + $(TARGET_MAKE_ENV) $(MAKE) BASEDIR=$(TARGET_DIR) -C $(@D) install
> +endef
The bundled install tries to symlink ifdown->ifup, without removing ifup first.
So when it's installed a second time, or when busybox is selected, it errors out.
Adding an $(RM) $(TARGET_DIR)/sbin/ifdown solves it.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/9] ifupdown: new package
2014-11-18 19:56 ` Arnout Vandecappelle
@ 2014-11-18 19:59 ` Gustavo Zacarias
0 siblings, 0 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 19:59 UTC (permalink / raw)
To: buildroot
On 11/18/2014 04:56 PM, Arnout Vandecappelle wrote:
> The bundled install tries to symlink ifdown->ifup, without removing ifup first.
> So when it's installed a second time, or when busybox is selected, it errors out.
>
> Adding an $(RM) $(TARGET_DIR)/sbin/ifdown solves it.
Aha! That's the reason i used a manual install before :)
Just found it again when testing without dpkg-architecture (and your mail).
I'll amend for v4.
Regards.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 4/9] start-stop-daemon: new package
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (2 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 3/9] ifupdown: " Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 20:23 ` Arnout Vandecappelle
2014-11-18 17:01 ` [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin Gustavo Zacarias
` (4 subsequent siblings)
8 siblings, 1 reply; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/Config.in | 3 ++
.../0001-add-uclibc-alias-and-musl.patch | 42 ++++++++++++++++++++++
package/start-stop-daemon/Config.in | 8 +++++
package/start-stop-daemon/start-stop-daemon.hash | 2 ++
package/start-stop-daemon/start-stop-daemon.mk | 27 ++++++++++++++
5 files changed, 82 insertions(+)
create mode 100644 package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
create mode 100644 package/start-stop-daemon/Config.in
create mode 100644 package/start-stop-daemon/start-stop-daemon.hash
create mode 100644 package/start-stop-daemon/start-stop-daemon.mk
diff --git a/package/Config.in b/package/Config.in
index 8f3cc3c..ce161cf 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1229,6 +1229,9 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
source "package/rsyslog/Config.in"
endif
source "package/smack/Config.in"
+if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
+ source "package/start-stop-daemon/Config.in"
+endif
source "package/supervisor/Config.in"
if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
source "package/sysklogd/Config.in"
diff --git a/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch b/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
new file mode 100644
index 0000000..0b726d4
--- /dev/null
+++ b/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
@@ -0,0 +1,42 @@
+Make linux-uclibcgnueabi an alias for linux-uclibceabi since it's
+what we use in buildroot toolchains.
+Also add support for musl libc which is a backport from 1.17.x with
+a slight tweak for musleabi (non-hf).
+
+Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
+
+diff -Nura dpkg-1.16.15.orig/ostable dpkg-1.16.15/ostable
+--- dpkg-1.16.15.orig/ostable 2014-11-07 08:09:07.358304926 -0300
++++ dpkg-1.16.15/ostable 2014-11-07 09:35:06.158580079 -0300
+@@ -14,8 +14,10 @@
+ # system part of the output of the GNU config.guess script.
+ #
+ # <Debian name> <GNU name> <config.guess regex>
+-uclibceabi-linux linux-uclibceabi linux[^-]*-uclibceabi
++uclibceabi-linux linux-uclibceabi linux[^-]*-uclibc.*eabi
+ uclibc-linux linux-uclibc linux[^-]*-uclibc
++musleabihf-linux linux-musleabihf linux[^-]*-musleabihf
++musl-linux linux-musl linux[^-]*-musl[^-]*
+ gnueabihf-linux linux-gnueabihf linux[^-]*-gnueabihf
+ gnueabi-linux linux-gnueabi linux[^-]*-gnueabi
+ gnuabin32-linux linux-gnuabin32 linux[^-]*-gnuabin32
+@@ -33,6 +35,6 @@
+ bsd-netbsd netbsd netbsd[^-]*
+ bsd-openbsd openbsd openbsd[^-]*
+ sysv-solaris solaris solaris[^-]*
+-uclibceabi-uclinux uclinux-uclibceabi uclinux[^-]*-uclibceabi
++uclibceabi-uclinux uclinux-uclibceabi uclinux[^-]*-uclibc.*eabi
+ uclibc-uclinux uclinux-uclibc uclinux[^-]*(-uclibc.*)?
+ tos-mint mint mint[^-]*
+diff -Nura dpkg-1.16.15.orig/triplettable dpkg-1.16.15/triplettable
+--- dpkg-1.16.15.orig/triplettable 2014-11-07 08:09:07.345304482 -0300
++++ dpkg-1.16.15/triplettable 2014-11-07 09:35:29.098364036 -0300
+@@ -5,6 +5,8 @@
+ # <Debian triplet> <Debian arch>
+ uclibceabi-linux-arm uclibc-linux-armel
+ uclibc-linux-<cpu> uclibc-linux-<cpu>
++musleabihf-linux-arm musl-linux-armhf
++musl-linux-<cpu> musl-linux-<cpu>
+ gnueabihf-linux-arm armhf
+ gnueabi-linux-arm armel
+ gnuabin32-linux-mips64el mipsn32el
diff --git a/package/start-stop-daemon/Config.in b/package/start-stop-daemon/Config.in
new file mode 100644
index 0000000..7d8eb34
--- /dev/null
+++ b/package/start-stop-daemon/Config.in
@@ -0,0 +1,8 @@
+config BR2_PACKAGE_START_STOP_DAEMON
+ bool "start-stop-daemon"
+ depends on BR2_USE_MMU # fork()
+ help
+ start-stop-daemon is used to control the creation and termination
+ of system-level processes.
+
+ https://tracker.debian.org/pkg/dpkg
diff --git a/package/start-stop-daemon/start-stop-daemon.hash b/package/start-stop-daemon/start-stop-daemon.hash
new file mode 100644
index 0000000..e3ab29f
--- /dev/null
+++ b/package/start-stop-daemon/start-stop-daemon.hash
@@ -0,0 +1,2 @@
+# From http://ftp.de.debian.org/debian/pool/main/d/dpkg/dpkg_1.16.15.dsc
+sha256 92bca9901ba2d9300be42f6de8dbea59b8367a918a2abeeb47d2176c9cf86b55 dpkg_1.16.15.tar.xz
diff --git a/package/start-stop-daemon/start-stop-daemon.mk b/package/start-stop-daemon/start-stop-daemon.mk
new file mode 100644
index 0000000..4790859
--- /dev/null
+++ b/package/start-stop-daemon/start-stop-daemon.mk
@@ -0,0 +1,27 @@
+################################################################################
+#
+# start-stop-daemon
+#
+################################################################################
+
+START_STOP_DAEMON_VERSION = 1.16.15
+START_STOP_DAEMON_SOURCE = dpkg_$(START_STOP_DAEMON_VERSION).tar.xz
+START_STOP_DAEMON_SITE = http://snapshot.debian.org/archive/debian/20140616T044945Z/pool/main/d/dpkg
+START_STOP_DAEMON_CONF_OPTS = --disable-dselect --disable-update-alternatives \
+ --disable-install-info --exec-prefix=/
+START_STOP_DAEMON_DEPENDENCIES = host-pkgconf \
+ $(if $(BR2_PACKAGE_BUSYBOX),busybox)
+START_STOP_DAEMON_LICENSE = Public Domain
+START_STOP_DAEMON_LICENSE_FILES = utils/start-stop-daemon.c
+
+define START_STOP_DAEMON_BUILD_CMDS
+ $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/lib/compat
+ $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/utils
+endef
+
+define START_STOP_DAEMON_INSTALL_TARGET_CMDS
+ $(INSTALL) -m 0755 -D $(@D)/utils/start-stop-daemon \
+ $(TARGET_DIR)/sbin/start-stop-daemon
+endef
+
+$(eval $(autotools-package))
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 4/9] start-stop-daemon: new package
2014-11-18 17:01 ` [Buildroot] [PATCH 4/9] start-stop-daemon: " Gustavo Zacarias
@ 2014-11-18 20:23 ` Arnout Vandecappelle
0 siblings, 0 replies; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 20:23 UTC (permalink / raw)
To: buildroot
On 18/11/14 18:01, Gustavo Zacarias wrote:
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Maybe mention in the commit log why 1.17.21 doesn't work.
> ---
> package/Config.in | 3 ++
> .../0001-add-uclibc-alias-and-musl.patch | 42 ++++++++++++++++++++++
> package/start-stop-daemon/Config.in | 8 +++++
> package/start-stop-daemon/start-stop-daemon.hash | 2 ++
> package/start-stop-daemon/start-stop-daemon.mk | 27 ++++++++++++++
> 5 files changed, 82 insertions(+)
> create mode 100644 package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
> create mode 100644 package/start-stop-daemon/Config.in
> create mode 100644 package/start-stop-daemon/start-stop-daemon.hash
> create mode 100644 package/start-stop-daemon/start-stop-daemon.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 8f3cc3c..ce161cf 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1229,6 +1229,9 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> source "package/rsyslog/Config.in"
> endif
> source "package/smack/Config.in"
> +if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> + source "package/start-stop-daemon/Config.in"
> +endif
> source "package/supervisor/Config.in"
> if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> source "package/sysklogd/Config.in"
> diff --git a/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch b/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
> new file mode 100644
> index 0000000..0b726d4
> --- /dev/null
> +++ b/package/start-stop-daemon/0001-add-uclibc-alias-and-musl.patch
> @@ -0,0 +1,42 @@
> +Make linux-uclibcgnueabi an alias for linux-uclibceabi since it's
> +what we use in buildroot toolchains.
> +Also add support for musl libc which is a backport from 1.17.x with
> +a slight tweak for musleabi (non-hf).
> +
> +Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> +
> +diff -Nura dpkg-1.16.15.orig/ostable dpkg-1.16.15/ostable
> +--- dpkg-1.16.15.orig/ostable 2014-11-07 08:09:07.358304926 -0300
> ++++ dpkg-1.16.15/ostable 2014-11-07 09:35:06.158580079 -0300
> +@@ -14,8 +14,10 @@
> + # system part of the output of the GNU config.guess script.
> + #
> + # <Debian name> <GNU name> <config.guess regex>
> +-uclibceabi-linux linux-uclibceabi linux[^-]*-uclibceabi
> ++uclibceabi-linux linux-uclibceabi linux[^-]*-uclibc.*eabi
> + uclibc-linux linux-uclibc linux[^-]*-uclibc
> ++musleabihf-linux linux-musleabihf linux[^-]*-musleabihf
> ++musl-linux linux-musl linux[^-]*-musl[^-]*
> + gnueabihf-linux linux-gnueabihf linux[^-]*-gnueabihf
> + gnueabi-linux linux-gnueabi linux[^-]*-gnueabi
> + gnuabin32-linux linux-gnuabin32 linux[^-]*-gnuabin32
> +@@ -33,6 +35,6 @@
> + bsd-netbsd netbsd netbsd[^-]*
> + bsd-openbsd openbsd openbsd[^-]*
> + sysv-solaris solaris solaris[^-]*
> +-uclibceabi-uclinux uclinux-uclibceabi uclinux[^-]*-uclibceabi
> ++uclibceabi-uclinux uclinux-uclibceabi uclinux[^-]*-uclibc.*eabi
> + uclibc-uclinux uclinux-uclibc uclinux[^-]*(-uclibc.*)?
> + tos-mint mint mint[^-]*
> +diff -Nura dpkg-1.16.15.orig/triplettable dpkg-1.16.15/triplettable
> +--- dpkg-1.16.15.orig/triplettable 2014-11-07 08:09:07.345304482 -0300
> ++++ dpkg-1.16.15/triplettable 2014-11-07 09:35:29.098364036 -0300
> +@@ -5,6 +5,8 @@
> + # <Debian triplet> <Debian arch>
> + uclibceabi-linux-arm uclibc-linux-armel
> + uclibc-linux-<cpu> uclibc-linux-<cpu>
> ++musleabihf-linux-arm musl-linux-armhf
> ++musl-linux-<cpu> musl-linux-<cpu>
> + gnueabihf-linux-arm armhf
> + gnueabi-linux-arm armel
> + gnuabin32-linux-mips64el mipsn32el
> diff --git a/package/start-stop-daemon/Config.in b/package/start-stop-daemon/Config.in
> new file mode 100644
> index 0000000..7d8eb34
> --- /dev/null
> +++ b/package/start-stop-daemon/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_START_STOP_DAEMON
> + bool "start-stop-daemon"
> + depends on BR2_USE_MMU # fork()
> + help
> + start-stop-daemon is used to control the creation and termination
> + of system-level processes.
> +
> + https://tracker.debian.org/pkg/dpkg
> diff --git a/package/start-stop-daemon/start-stop-daemon.hash b/package/start-stop-daemon/start-stop-daemon.hash
> new file mode 100644
> index 0000000..e3ab29f
> --- /dev/null
> +++ b/package/start-stop-daemon/start-stop-daemon.hash
> @@ -0,0 +1,2 @@
> +# From http://ftp.de.debian.org/debian/pool/main/d/dpkg/dpkg_1.16.15.dsc
> +sha256 92bca9901ba2d9300be42f6de8dbea59b8367a918a2abeeb47d2176c9cf86b55 dpkg_1.16.15.tar.xz
> diff --git a/package/start-stop-daemon/start-stop-daemon.mk b/package/start-stop-daemon/start-stop-daemon.mk
> new file mode 100644
> index 0000000..4790859
> --- /dev/null
> +++ b/package/start-stop-daemon/start-stop-daemon.mk
> @@ -0,0 +1,27 @@
> +################################################################################
> +#
> +# start-stop-daemon
> +#
> +################################################################################
> +
I would like a comment here explaining why the package is not called dpkg. E.g.
# Debian start-stop-daemon is part of dpkg. Since start-stop-daemon is the only
# interesting tool in it, we call the buildroot package start-stop-daemon.
> +START_STOP_DAEMON_VERSION = 1.16.15
> +START_STOP_DAEMON_SOURCE = dpkg_$(START_STOP_DAEMON_VERSION).tar.xz
> +START_STOP_DAEMON_SITE = http://snapshot.debian.org/archive/debian/20140616T044945Z/pool/main/d/dpkg
> +START_STOP_DAEMON_CONF_OPTS = --disable-dselect --disable-update-alternatives \
> + --disable-install-info --exec-prefix=/
If the line is split anyway, I personally prefer
START_STOP_DAEMON_CONF_OPTS = \
--disable-dselect \
--disable-update-alternatives \
--disable-install-info \
--exec-prefix=/
(this gives smaller/clearer diffs when something changes; and if the last line
change, git diff will still show the variable name as the 'function').
> +START_STOP_DAEMON_DEPENDENCIES = host-pkgconf \
> + $(if $(BR2_PACKAGE_BUSYBOX),busybox)
> +START_STOP_DAEMON_LICENSE = Public Domain
> +START_STOP_DAEMON_LICENSE_FILES = utils/start-stop-daemon.c
For static building, I need
START_STOP_DAEMON_CONF_ENV = \
dpkg_cv_va_copy=yes \
dpkg_cv_c99_snprintf=yes
With that added:
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
> +
> +define START_STOP_DAEMON_BUILD_CMDS
> + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/lib/compat
> + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D)/utils
> +endef
> +
> +define START_STOP_DAEMON_INSTALL_TARGET_CMDS
> + $(INSTALL) -m 0755 -D $(@D)/utils/start-stop-daemon \
> + $(TARGET_DIR)/sbin/start-stop-daemon
> +endef
> +
> +$(eval $(autotools-package))
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (3 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 4/9] start-stop-daemon: " Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 20:24 ` Arnout Vandecappelle
2014-11-18 22:57 ` Thomas Petazzoni
2014-11-18 17:01 ` [Buildroot] [PATCH 6/9] dhcp: update dhclient-script Gustavo Zacarias
` (3 subsequent siblings)
8 siblings, 2 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Debian ifupdown uses an explicit call to /sbin/dhclient.
Busybox ifupdown uses the search path.
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/dhcp/dhcp.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/package/dhcp/dhcp.mk b/package/dhcp/dhcp.mk
index 27fb60b..1eb6b32 100644
--- a/package/dhcp/dhcp.mk
+++ b/package/dhcp/dhcp.mk
@@ -59,7 +59,7 @@ define DHCP_INSTALL_CLIENT
mkdir -p $(TARGET_DIR)/var/lib
(cd $(TARGET_DIR)/var/lib; ln -snf /tmp dhcp)
$(INSTALL) -m 0755 -D $(DHCP_DIR)/client/dhclient \
- $(TARGET_DIR)/usr/sbin/dhclient
+ $(TARGET_DIR)/sbin/dhclient
$(INSTALL) -m 0644 -D package/dhcp/dhclient.conf \
$(TARGET_DIR)/etc/dhcp/dhclient.conf
$(INSTALL) -m 0755 -D package/dhcp/dhclient-script \
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin
2014-11-18 17:01 ` [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin Gustavo Zacarias
@ 2014-11-18 20:24 ` Arnout Vandecappelle
2014-11-18 22:57 ` Thomas Petazzoni
1 sibling, 0 replies; 25+ messages in thread
From: Arnout Vandecappelle @ 2014-11-18 20:24 UTC (permalink / raw)
To: buildroot
On 18/11/14 18:01, Gustavo Zacarias wrote:
> Debian ifupdown uses an explicit call to /sbin/dhclient.
> Busybox ifupdown uses the search path.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
> ---
> package/dhcp/dhcp.mk | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/dhcp/dhcp.mk b/package/dhcp/dhcp.mk
> index 27fb60b..1eb6b32 100644
> --- a/package/dhcp/dhcp.mk
> +++ b/package/dhcp/dhcp.mk
> @@ -59,7 +59,7 @@ define DHCP_INSTALL_CLIENT
> mkdir -p $(TARGET_DIR)/var/lib
> (cd $(TARGET_DIR)/var/lib; ln -snf /tmp dhcp)
> $(INSTALL) -m 0755 -D $(DHCP_DIR)/client/dhclient \
> - $(TARGET_DIR)/usr/sbin/dhclient
> + $(TARGET_DIR)/sbin/dhclient
> $(INSTALL) -m 0644 -D package/dhcp/dhclient.conf \
> $(TARGET_DIR)/etc/dhcp/dhclient.conf
> $(INSTALL) -m 0755 -D package/dhcp/dhclient-script \
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin
2014-11-18 17:01 ` [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin Gustavo Zacarias
2014-11-18 20:24 ` Arnout Vandecappelle
@ 2014-11-18 22:57 ` Thomas Petazzoni
1 sibling, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2014-11-18 22:57 UTC (permalink / raw)
To: buildroot
Dear Gustavo Zacarias,
On Tue, 18 Nov 2014 14:01:53 -0300, Gustavo Zacarias wrote:
> Debian ifupdown uses an explicit call to /sbin/dhclient.
> Busybox ifupdown uses the search path.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
> package/dhcp/dhcp.mk | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to next, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 6/9] dhcp: update dhclient-script
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (4 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 5/9] dhcp: install dhclient to /sbin Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-18 17:01 ` [Buildroot] [PATCH 7/9] dhcpcd: bump to version 6.6.2 Gustavo Zacarias
` (2 subsequent siblings)
8 siblings, 0 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
The previous no-op dhclient-script is useless and in fact can lead
users to believe things will work.
dhclient-script is supposed to, among many other things, bring the
interface up for dhclient to pump an address, set the interface
address, set resolv.conf appropiately and set the default gateway.
It's a complex script since it has to deal with a lot of details so
let's just use the best fit which is the OpenWRT one since it uses the
"legacy" tools (ifconfig, route, ...) which can be easily provided by
busybox and/or net-tools.
There are newer and more feature-complete versions around but they
require full iproute2 ip and wouldn't fit as well with the busybox-only
approach.
License: likely GPLv2+
Source:
http://git.openwrt.org/?p=packages.git;a=blob;f=net/isc-dhcp/files/dhclient-script;h=4afebc0ad20ebac51c5baae5ed01c6713e3a0fd0;hb=HEAD
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/dhcp/dhclient-script | 284 ++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 283 insertions(+), 1 deletion(-)
diff --git a/package/dhcp/dhclient-script b/package/dhcp/dhclient-script
index 5566aca..cb292b6 100755
--- a/package/dhcp/dhclient-script
+++ b/package/dhcp/dhclient-script
@@ -1,2 +1,284 @@
#!/bin/sh
-# empty dhclient-script to keep dhclient from complaining
+
+# dhclient-script from OpenWRT project
+# http://git.openwrt.org/?p=packages.git;a=blob;f=net/isc-dhcp/files/dhclient-script;h=4afebc0ad20ebac51c5baae5ed01c6713e3a0fd0;hb=HEAD
+
+make_resolv_conf() {
+ if [ x"$new_domain_name_servers" != x ]; then
+ cat /dev/null > /etc/resolv.conf.dhclient
+ chmod 644 /etc/resolv.conf.dhclient
+ if [ x"$new_domain_search" != x ]; then
+ echo search $new_domain_search >> /etc/resolv.conf.dhclient
+ elif [ x"$new_domain_name" != x ]; then
+ # Note that the DHCP 'Domain Name Option' is really just a domain
+ # name, and that this practice of using the domain name option as
+ # a search path is both nonstandard and deprecated.
+ echo search $new_domain_name >> /etc/resolv.conf.dhclient
+ fi
+ for nameserver in $new_domain_name_servers; do
+ echo nameserver $nameserver >>/etc/resolv.conf.dhclient
+ done
+
+ elif [ "x${new_dhcp6_name_servers}" != x ] ; then
+ cat /dev/null > /etc/resolv.conf.dhclient6
+ chmod 644 /etc/resolv.conf.dhclient6
+
+ if [ "x${new_dhcp6_domain_search}" != x ] ; then
+ echo search ${new_dhcp6_domain_search} >> /etc/resolv.conf.dhclient6
+ fi
+ for nameserver in ${new_dhcp6_name_servers} ; do
+ echo nameserver ${nameserver} >> /etc/resolv.conf.dhclient6
+ done
+ fi
+
+ # if both v4 and v6 clients are running, concatenate results
+ cat /etc/resolv.conf.* > /etc/resolv.conf
+}
+
+# Must be used on exit. Invokes the local dhcp client exit hooks, if any.
+exit_with_hooks() {
+ exit_status=$1
+ if [ -f /etc/dhclient-exit-hooks ]; then
+ . /etc/dhclient-exit-hooks
+ fi
+# probably should do something with exit status of the local script
+ exit $exit_status
+}
+
+# Invoke the local dhcp client enter hooks, if they exist.
+if [ -f /etc/dhclient-enter-hooks ]; then
+ exit_status=0
+ . /etc/dhclient-enter-hooks
+ # allow the local script to abort processing of this state
+ # local script must set exit_status variable to nonzero.
+ if [ $exit_status -ne 0 ]; then
+ exit $exit_status
+ fi
+fi
+
+###
+### DHCPv4 Handlers
+###
+
+if [ x$new_broadcast_address != x ]; then
+ new_broadcast_arg="broadcast $new_broadcast_address"
+fi
+if [ x$new_subnet_mask != x ]; then
+ new_subnet_arg="netmask $new_subnet_mask"
+fi
+if [ x$alias_subnet_mask != x ]; then
+ alias_subnet_arg="netmask $alias_subnet_mask"
+fi
+
+if [ x$reason = xMEDIUM ]; then
+ # Linux doesn't do mediums (ok, ok, media).
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xPREINIT ]; then
+ if [ x$alias_ip_address != x ]; then
+ # Bring down alias interface. Its routes will disappear too.
+ ifconfig $interface:0- 0.0.0.0
+ fi
+ ifconfig $interface 0.0.0.0 up
+
+ # We need to give the kernel some time to get the interface up.
+ sleep 1
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xARPCHECK ] || [ x$reason = xARPSEND ]; then
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xBOUND ] || [ x$reason = xRENEW ] || \
+ [ x$reason = xREBIND ] || [ x$reason = xREBOOT ]; then
+ current_hostname=`hostname`
+ if [ x$current_hostname = x ] || \
+ [ x$current_hostname = x$old_host_name ]; then
+ if [ x$current_hostname = x ] || \
+ [ x$new_host_name != x$old_host_name ]; then
+ hostname $new_host_name
+ fi
+ fi
+
+ if [ x$old_ip_address != x ] && [ x$alias_ip_address != x ] && \
+ [ x$alias_ip_address != x$old_ip_address ]; then
+ # Possible new alias. Remove old alias.
+ ifconfig $interface:0- 0.0.0.0
+ fi
+ if [ x$old_ip_address != x ] && [ x$old_ip_address != x$new_ip_address ]; then
+ # IP address changed. Bringing down the interface will delete all routes,
+ # and clear the ARP cache.
+ ifconfig $interface 0.0.0.0 down
+
+ fi
+ if [ x$old_ip_address = x ] || [ x$old_ip_address != x$new_ip_address ] || \
+ [ x$reason = xBOUND ] || [ x$reason = xREBOOT ]; then
+
+ ifconfig $interface $new_ip_address $new_subnet_arg \
+ $new_broadcast_arg
+ for router in $new_routers; do
+ if [ "x$new_subnet_mask" = "x255.255.255.255" ] ; then
+ route add -host $router dev $interface
+ fi
+ route add default gw $router
+ done
+ fi
+ if [ x$new_ip_address != x$alias_ip_address ] && [ x$alias_ip_address != x ];
+ then
+ ifconfig $interface:0- 0.0.0.0
+ ifconfig $interface:0 $alias_ip_address $alias_subnet_arg
+ route add -host $alias_ip_address $interface:0
+ fi
+ make_resolv_conf
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xEXPIRE ] || [ x$reason = xFAIL ] || [ x$reason = xRELEASE ] \
+ || [ x$reason = xSTOP ]; then
+ if [ x$alias_ip_address != x ]; then
+ # Turn off alias interface.
+ ifconfig $interface:0- 0.0.0.0
+ fi
+ if [ x$old_ip_address != x ]; then
+ # Shut down interface, which will delete routes and clear arp cache.
+ ifconfig $interface 0.0.0.0 down
+ fi
+ if [ x$alias_ip_address != x ]; then
+ ifconfig $interface:0 $alias_ip_address $alias_subnet_arg
+ route add -host $alias_ip_address $interface:0
+ fi
+
+ # remove v4 dns configuration for this interface
+ rm /etc/resolv.conf.dhclient
+ cat /etc/resolv.conf.* > /etc/resolv.conf
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xTIMEOUT ]; then
+ if [ x$alias_ip_address != x ]; then
+ ifconfig $interface:0- 0.0.0.0
+ fi
+ ifconfig $interface $new_ip_address $new_subnet_arg \
+ $new_broadcast_arg
+ set $new_routers
+ if ping -q -c 1 $1; then
+ if [ x$new_ip_address != x$alias_ip_address ] && \
+ [ x$alias_ip_address != x ]; then
+ ifconfig $interface:0 $alias_ip_address $alias_subnet_arg
+ route add -host $alias_ip_address dev $interface:0
+ fi
+ for router in $new_routers; do
+ if [ "x$new_subnet_mask" = "x255.255.255.255" ] ; then
+ route add -host $router dev $interface
+ fi
+ route add default gw $router
+ done
+ make_resolv_conf
+ exit_with_hooks 0
+ fi
+ ifconfig $interface 0.0.0.0 down
+ exit_with_hooks 1
+fi
+
+###
+### DHCPv6 Handlers
+###
+
+if [ x$reason = xPREINIT6 ]; then
+ # Ensure interface is up.
+ ifconfig ${interface} up
+
+ # Remove any stale addresses from aborted clients.
+ ip -f inet6 addr flush dev ${interface} scope global
+
+ exit_with_hooks 0
+fi
+
+if [ x${old_ip6_prefix} != x ] || [ x${new_ip6_prefix} != x ] ; then
+ echo Prefix ${reason} old=${old_ip6_prefix} new=${new_ip6_prefix}
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xBOUND6 ]; then
+ if [ x${new_ip6_address} = x ] || [ x${new_ip6_prefixlen} = x ] ; then
+ exit_with_hooks 2;
+ fi
+
+ ifconfig ${interface} add ${new_ip6_address}/${new_ip6_prefixlen}
+
+ # Check for nameserver options.
+ make_resolv_conf
+
+### <<
+ # Set up softwire tunnel
+ if [ x${new_dhcp6_softwire} != x ] ; then
+ /etc/init.d/dhclient stop
+ ifconfig ${interface} 0.0.0.0
+ ip -6 tunnel add tun0 mode ipip6 \
+ remote ${new_dhcp6_softwire} \
+ local ${new_ip6_address} \
+ dev ${interface} encaplimit none
+ ip link set tun0 up
+ ip route add default dev tun0
+ fi
+### >>
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xRENEW6 ] || [ x$reason = xREBIND6 ]; then
+ if [ x${new_ip6_address} = x ] || [ x${new_ip6_prefixlen} = x ] ; then
+ exit_with_hooks 2;
+ fi
+
+ ifconfig ${interface} add ${new_ip6_address}/${new_ip6_prefixlen}
+
+ # Make sure nothing has moved around on us.
+
+ # Nameservers/domains/etc.
+ if [ "x${new_dhcp6_name_servers}" != "x${old_dhcp6_name_servers}" ] ||
+ [ "x${new_dhcp6_domain_search}" != "x${old_dhcp6_domain_search}" ] ; then
+ make_resolv_conf
+ fi
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xDEPREF6 ]; then
+ if [ x${new_ip6_address} = x ] ; then
+ exit_with_hooks 2;
+ fi
+
+ # Busybox ifconfig has no way to communicate this to the kernel, so ignore it
+
+ exit_with_hooks 0
+fi
+
+if [ x$reason = xEXPIRE6 -o x$reason = xRELEASE6 -o x$reason = xSTOP6 ]; then
+ if [ x${old_ip6_address} = x ] || [ x${old_ip6_prefixlen} = x ] ; then
+ exit_with_hooks 2;
+ fi
+
+ ifconfig ${interface} del ${old_ip6_address}/${old_ip6_prefixlen}
+
+ # remove v6 dns configuration for this interface
+ rm /etc/resolv.conf.dhclient6
+ cat /etc/resolv.conf.* > /etc/resolv.conf
+
+### <<
+ # Tear down softwire tunnel
+ if [ x${old_dhcp6_softwire} != x ] ; then
+ ip link set tun0 down
+ ip tunnel del tun0
+ fi
+### >>
+
+ exit_with_hooks 0
+fi
+
+exit_with_hooks 0
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 7/9] dhcpcd: bump to version 6.6.2
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (5 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 6/9] dhcp: update dhclient-script Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-19 22:23 ` Thomas Petazzoni
2014-11-18 17:01 ` [Buildroot] [PATCH 8/9] dhcpcd: use standard installation Gustavo Zacarias
2014-11-18 17:01 ` [Buildroot] [PATCH 9/9] network-manager: adjust dhclient and dhcpcd directories Gustavo Zacarias
8 siblings, 1 reply; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Also add hash file.
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/dhcpcd/dhcpcd.hash | 2 ++
package/dhcpcd/dhcpcd.mk | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
create mode 100644 package/dhcpcd/dhcpcd.hash
diff --git a/package/dhcpcd/dhcpcd.hash b/package/dhcpcd/dhcpcd.hash
new file mode 100644
index 0000000..c8ae064
--- /dev/null
+++ b/package/dhcpcd/dhcpcd.hash
@@ -0,0 +1,2 @@
+# Locally calculated from download (no sig, hash)
+sha256 879c81febafc95ed9353909b719af765e9925c6cb06226b79da6c0dd176fe89f dhcpcd-6.6.2.tar.bz2
diff --git a/package/dhcpcd/dhcpcd.mk b/package/dhcpcd/dhcpcd.mk
index 3bb7a13..825a345 100644
--- a/package/dhcpcd/dhcpcd.mk
+++ b/package/dhcpcd/dhcpcd.mk
@@ -4,7 +4,7 @@
#
################################################################################
-DHCPCD_VERSION = 6.4.7
+DHCPCD_VERSION = 6.6.2
DHCPCD_SOURCE = dhcpcd-$(DHCPCD_VERSION).tar.bz2
DHCPCD_SITE = http://roy.marples.name/downloads/dhcpcd
DHCPCD_DEPENDENCIES = host-pkgconf
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 7/9] dhcpcd: bump to version 6.6.2
2014-11-18 17:01 ` [Buildroot] [PATCH 7/9] dhcpcd: bump to version 6.6.2 Gustavo Zacarias
@ 2014-11-19 22:23 ` Thomas Petazzoni
0 siblings, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2014-11-19 22:23 UTC (permalink / raw)
To: buildroot
Dear Gustavo Zacarias,
On Tue, 18 Nov 2014 14:01:55 -0300, Gustavo Zacarias wrote:
> Also add hash file.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
> package/dhcpcd/dhcpcd.hash | 2 ++
> package/dhcpcd/dhcpcd.mk | 2 +-
> 2 files changed, 3 insertions(+), 1 deletion(-)
> create mode 100644 package/dhcpcd/dhcpcd.hash
Applied to next, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 8/9] dhcpcd: use standard installation
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (6 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 7/9] dhcpcd: bump to version 6.6.2 Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-19 22:20 ` Thomas Petazzoni
2014-11-18 17:01 ` [Buildroot] [PATCH 9/9] network-manager: adjust dhclient and dhcpcd directories Gustavo Zacarias
8 siblings, 1 reply; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
Drop the custom/manual install cmds and do a normal make install.
This gets dhcpcd-hooks installed which are useful for many accesory
setup functions like dns, ntp server, and so on.
Also install dhcpcd to /sbin since it's expected by debian ifupdown,
For busybox ifupdown when external dhcp clients are enabled (not in the
default config we ship) will search the path.
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/dhcpcd/dhcpcd.mk | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/package/dhcpcd/dhcpcd.mk b/package/dhcpcd/dhcpcd.mk
index 825a345..c06002f 100644
--- a/package/dhcpcd/dhcpcd.mk
+++ b/package/dhcpcd/dhcpcd.mk
@@ -35,12 +35,7 @@ define DHCPCD_BUILD_CMDS
endef
define DHCPCD_INSTALL_TARGET_CMDS
- $(INSTALL) -D -m 0755 $(@D)/dhcpcd \
- $(TARGET_DIR)/usr/sbin/dhcpcd
- $(INSTALL) -D -m 0644 $(@D)/dhcpcd.conf \
- $(TARGET_DIR)/etc/dhcpcd.conf
- $(INSTALL) -D -m 0755 $(@D)/dhcpcd-run-hooks \
- $(TARGET_DIR)/libexec/dhcpcd-run-hooks
+ $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) install DESTDIR=$(TARGET_DIR)
endef
# NOTE: Even though this package has a configure script, it is not generated
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 8/9] dhcpcd: use standard installation
2014-11-18 17:01 ` [Buildroot] [PATCH 8/9] dhcpcd: use standard installation Gustavo Zacarias
@ 2014-11-19 22:20 ` Thomas Petazzoni
0 siblings, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2014-11-19 22:20 UTC (permalink / raw)
To: buildroot
Dear Gustavo Zacarias,
On Tue, 18 Nov 2014 14:01:56 -0300, Gustavo Zacarias wrote:
> Drop the custom/manual install cmds and do a normal make install.
> This gets dhcpcd-hooks installed which are useful for many accesory
> setup functions like dns, ntp server, and so on.
> Also install dhcpcd to /sbin since it's expected by debian ifupdown,
> For busybox ifupdown when external dhcp clients are enabled (not in the
> default config we ship) will search the path.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
> package/dhcpcd/dhcpcd.mk | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
Applied to next, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 9/9] network-manager: adjust dhclient and dhcpcd directories
2014-11-18 17:01 [Buildroot] [PATCHv3 0/9] SystemV init without busybox series Gustavo Zacarias
` (7 preceding siblings ...)
2014-11-18 17:01 ` [Buildroot] [PATCH 8/9] dhcpcd: use standard installation Gustavo Zacarias
@ 2014-11-18 17:01 ` Gustavo Zacarias
2014-11-19 22:21 ` Thomas Petazzoni
8 siblings, 1 reply; 25+ messages in thread
From: Gustavo Zacarias @ 2014-11-18 17:01 UTC (permalink / raw)
To: buildroot
dhclient and dhcpcd were moved from /usr/sbin to /sbin for busybox-less
SysV-style init with Debian utilities, so adjust accordingly.
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
package/network-manager/network-manager.mk | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/package/network-manager/network-manager.mk b/package/network-manager/network-manager.mk
index 7944254..9770cae 100644
--- a/package/network-manager/network-manager.mk
+++ b/package/network-manager/network-manager.mk
@@ -56,11 +56,11 @@ else
endif
ifeq ($(BR2_PACKAGE_DHCP_CLIENT),y)
-NETWORK_MANAGER_CONF_OPTS += --with-dhclient=/usr/sbin/dhclient
+NETWORK_MANAGER_CONF_OPTS += --with-dhclient=/sbin/dhclient
endif
ifeq ($(BR2_PACKAGE_DHCPCD),y)
-NETWORK_MANAGER_CONF_OPTS += --with-dhcpcd=/usr/sbin/dhcpcd
+NETWORK_MANAGER_CONF_OPTS += --with-dhcpcd=/sbin/dhcpcd
endif
# uClibc by default doesn't have backtrace support, so don't use it
--
2.0.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 9/9] network-manager: adjust dhclient and dhcpcd directories
2014-11-18 17:01 ` [Buildroot] [PATCH 9/9] network-manager: adjust dhclient and dhcpcd directories Gustavo Zacarias
@ 2014-11-19 22:21 ` Thomas Petazzoni
0 siblings, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2014-11-19 22:21 UTC (permalink / raw)
To: buildroot
Dear Gustavo Zacarias,
On Tue, 18 Nov 2014 14:01:57 -0300, Gustavo Zacarias wrote:
> dhclient and dhcpcd were moved from /usr/sbin to /sbin for busybox-less
> SysV-style init with Debian utilities, so adjust accordingly.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
> package/network-manager/network-manager.mk | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Applied to next, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread