diff options
author | Euan Harris <euan.harris@docker.com> | 2018-05-14 11:56:34 +0100 |
---|---|---|
committer | Natanael Copa <ncopa@alpinelinux.org> | 2018-05-14 19:07:04 +0000 |
commit | 1723be1db54b844925bdbf6aeac476b00e20fe6c (patch) | |
tree | 936888a491be41e7401cc0f1e15c4a0825dc908a /main/busybox/0001-nsenter-Rename-network-option-to-net.patch | |
parent | 6d2dc8194bf78753b8be2432549387630a7d5759 (diff) | |
download | aports-1723be1db54b844925bdbf6aeac476b00e20fe6c.tar.bz2 aports-1723be1db54b844925bdbf6aeac476b00e20fe6c.tar.xz |
main/busybox: Backport nsenter option parser fixes
* Rename --network to --net, to match util-linux and the help text
* -t, -S and -G should take mandatory integer options, but are defined
as simple flags, causing problems such as:
https://github.com/linuxkit/linuxkit/issues/567
https://github.com/gliderlabs/docker-alpine/issues/359
https://github.com/kontena/pharos-cluster/pull/81
These patches have been accepted upstream by Busybox.
Signed-off-by: Euan Harris <euan.harris@docker.com>
Diffstat (limited to 'main/busybox/0001-nsenter-Rename-network-option-to-net.patch')
-rw-r--r-- | main/busybox/0001-nsenter-Rename-network-option-to-net.patch | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/main/busybox/0001-nsenter-Rename-network-option-to-net.patch b/main/busybox/0001-nsenter-Rename-network-option-to-net.patch new file mode 100644 index 0000000000..ab3cff5e7b --- /dev/null +++ b/main/busybox/0001-nsenter-Rename-network-option-to-net.patch @@ -0,0 +1,36 @@ +From 40394cb1c963c35d2daa9570ba968afa981cf1fc Mon Sep 17 00:00:00 2001 +From: Euan Harris <euan.harris@docker.com> +Date: Thu, 3 May 2018 13:34:46 +0100 +Subject: [PATCH 1/2] nsenter: Rename --network option to --net + +In nsenter from util-linux, the long version of the -n option is +--net=<path>. BusyBox's version expects --network=<path>, so scripts +and examples written for util-linux's version cause BusyBox's version +to exit with the usage message. + +Confusingly, until commit 036585a911a5f, the usage message erroneously +claimed that the long option was indeed called --net; after that commit +long options are not listed at all. + +Signed-off-by: Euan Harris <euan.harris@docker.com> +Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com> +--- + util-linux/nsenter.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/util-linux/nsenter.c b/util-linux/nsenter.c +index c6933c8..35439a2 100644 +--- a/util-linux/nsenter.c ++++ b/util-linux/nsenter.c +@@ -112,7 +112,7 @@ static const char nsenter_longopts[] ALIGN1 = + "user\0" Optional_argument "U" + "ipc\0" Optional_argument "i" + "uts\0" Optional_argument "u" +- "network\0" Optional_argument "n" ++ "net\0" Optional_argument "n" + "pid\0" Optional_argument "p" + "mount\0" Optional_argument "m" + "target\0" Required_argument "t" +-- +2.7.4 + |