From: Peter Korsgaard <peter@korsgaard.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] ifenslave: new package
Date: Sun, 08 Jan 2017 18:23:00 +0100 [thread overview]
Message-ID: <87a8b1sbyj.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <1483371929-13907-1-git-send-email-david.bachelart@bbright.com> (David Bachelart's message of "Mon, 2 Jan 2017 16:45:22 +0100")
>>>>> "David" == David Bachelart <david.bachelart@bbright.com> writes:
> Signed-off-by: David Bachelart <david.bachelart@bbright.com>
> ---
> Changes v1 -> v2:
> - download tarball from snapshot.debian.org
> - use upstream SHA256 sum in hash file
> ---
> package/Config.in | 1 +
> package/ifenslave/Config.in | 6 ++++++
> package/ifenslave/ifenslave.hash | 3 +++
> package/ifenslave/ifenslave.mk | 17 +++++++++++++++++
> 4 files changed, 27 insertions(+)
> create mode 100644 package/ifenslave/Config.in
> create mode 100644 package/ifenslave/ifenslave.hash
> create mode 100644 package/ifenslave/ifenslave.mk
> diff --git a/package/Config.in b/package/Config.in
> index 59fa851..5c4e969 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1497,6 +1497,7 @@ menu "Networking applications"
> source "package/httping/Config.in"
> source "package/ibrdtn-tools/Config.in"
> source "package/ibrdtnd/Config.in"
> + source "package/ifenslave/Config.in"
> source "package/ifplugd/Config.in"
> source "package/iftop/Config.in"
> source "package/ifupdown/Config.in"
> diff --git a/package/ifenslave/Config.in b/package/ifenslave/Config.in
> new file mode 100644
> index 0000000..357e870
> --- /dev/null
> +++ b/package/ifenslave/Config.in
> @@ -0,0 +1,6 @@
> +config BR2_PACKAGE_IFENSLAVE
> + bool "ifenslave"
> + help
> + Configure network interfaces for parallel routing (bonding)
> +
> + http://anonscm.debian.org/cgit/collab-maint/ifenslave.git
Out of interest, any reason to use this over the busybox applet? As it
is GPLv3+, it cannot be licensing. Does it have any features the busybox
applet doesn't have?
As it is also available in busybox you need to add a 'depends on
BR2_PACKAGE_BUSYBOX_SHOW_OTHERS'.
> diff --git a/package/ifenslave/ifenslave.hash b/package/ifenslave/ifenslave.hash
> new file mode 100644
> index 0000000..dcf31d0
> --- /dev/null
> +++ b/package/ifenslave/ifenslave.hash
> @@ -0,0 +1,3 @@
> +# From http://snapshot.debian.org/archive/debian/20170102T091407Z/pool/main/i/ifenslave/ifenslave_2.9.dsc
> +sha1 a17e88fa298ef337e872c8aa5d7a390921239d1a ifenslave_2.9.tar.xz
> +sha256 e5666c0b61347bd06f2394c16eca6895fcb5e5cc83d0a1c596a975274de5a3c9 ifenslave_2.9.tar.xz
> diff --git a/package/ifenslave/ifenslave.mk b/package/ifenslave/ifenslave.mk
> new file mode 100644
> index 0000000..f808406
> --- /dev/null
> +++ b/package/ifenslave/ifenslave.mk
> @@ -0,0 +1,17 @@
> +################################################################################
> +#
> +# ifenslave
> +#
> +################################################################################
> +
> +IFENSLAVE_VERSION = 2.9
> +IFENSLAVE_SOURCE = ifenslave_$(IFENSLAVE_VERSION).tar.xz
> +IFENSLAVE_SITE = http://snapshot.debian.org/archive/debian/20170102T091407Z/pool/main/i/ifenslave
> +IFENSLAVE_LICENSE = GPLv3+
> +IFENSLAVE_LICENSE_FILES = debian/copyright
To ensure this 'wins' over the busybox applet if enabled, we need to
ensure this gets installed after busybox by:
IFENSLAVE_DEPENDENCIES = $(if $(BR2_PACKAGE_BUSYBOX),busybox)
I was a bit confused about the missing build command until I realized
that this is just a shell script, so I added a comment to make this
clear.
Committed with these fixes, thanks.
--
Bye, Peter Korsgaard
prev parent reply other threads:[~2017-01-08 17:23 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-02 15:45 [Buildroot] [PATCH v2] ifenslave: new package David Bachelart
2017-01-02 15:45 ` [Buildroot] [PATCH v2] nmon: " David Bachelart
2017-01-08 20:16 ` Peter Korsgaard
2017-01-02 15:45 ` [Buildroot] [PATCH v2] python-arrow: " David Bachelart
2017-01-02 19:58 ` Yegor Yefremov
2017-01-03 8:40 ` David Bachelart
2017-01-03 8:49 ` Yegor Yefremov
2017-01-02 15:45 ` [Buildroot] [PATCH v2] python-chardet: " David Bachelart
2017-01-02 20:00 ` Yegor Yefremov
2017-01-03 8:27 ` David Bachelart
2017-01-02 15:45 ` [Buildroot] [PATCH v2] python-whoosh: " David Bachelart
2017-01-02 20:02 ` Yegor Yefremov
2017-01-08 20:30 ` Peter Korsgaard
2017-01-02 15:45 ` [Buildroot] [PATCH v2] sslh: " David Bachelart
2017-01-25 10:35 ` Thomas Petazzoni
2017-01-25 11:20 ` David Bachelart
2017-01-02 15:45 ` [Buildroot] [PATCH v2] throttle: " David Bachelart
2017-01-25 10:39 ` Thomas Petazzoni
2017-01-26 9:43 ` David Bachelart
2017-01-02 15:45 ` [Buildroot] [PATCH v2] udpxy: " David Bachelart
2017-01-25 10:46 ` Thomas Petazzoni
2017-01-08 17:23 ` Peter Korsgaard [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87a8b1sbyj.fsf@dell.be.48ers.dk \
--to=peter@korsgaard.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox