All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
@ 2019-10-18  3:30 sam.voss at gmail.com
  2019-10-18 14:05 ` Carlos Santos
  2019-10-19 19:00 ` Arnout Vandecappelle
  0 siblings, 2 replies; 6+ messages in thread
From: sam.voss at gmail.com @ 2019-10-18  3:30 UTC (permalink / raw)
  To: buildroot

From: Sam Voss <sam.voss@gmail.com>

Add option to enable wipefs as a standalone app, so that it may be
enabled when the flag --disable-all-packages is set.

Signed-off-by: Sam Voss <sam.voss@gmail.com>
---
 ...w-explicit-wipefs-enable-disablement.patch | 33 +++++++++++++++++++
 package/util-linux/Config.in                  | 11 +++++++
 package/util-linux/util-linux.mk              |  1 +
 3 files changed, 45 insertions(+)
 create mode 100644 package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch

diff --git a/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch b/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch
new file mode 100644
index 0000000000..85021e2978
--- /dev/null
+++ b/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch
@@ -0,0 +1,33 @@
+From a76cd5f6abace269b60ef6f1b95526a2d09fd331 Mon Sep 17 00:00:00 2001
+From: Sam Voss <sam.voss@gmail.com>
+Date: Mon, 14 Oct 2019 21:19:16 -0500
+Subject: [PATCH] wipefs: Allow explicitly enable/disablement
+
+Currently if `--disable-all-packages` is set, there is no configure
+option for re-enabling `wipefs`.
+
+As the current default for `wipefs` is "enabled", add `--disable-` flag
+to maintain backward compatibility.
+
+Signed-off-by: Sam Voss <sam.voss@gmail.com>
+---
+ configure.ac | 6 +++++-
+ 1 file changed, 5 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index fc8a9507f..f70f8d1ee 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -1286,7 +1286,11 @@ UL_BUILD_INIT([findfs], [check])
+ UL_REQUIRES_BUILD([findfs], [libblkid])
+ AM_CONDITIONAL([BUILD_FINDFS], [test "x$build_findfs" = xyes])
+ 
+-UL_BUILD_INIT([wipefs], [check])
++AC_ARG_ENABLE([wipefs],
++  AS_HELP_STRING([--disable-wipefs], [do not build wipefs]),
++  [], [UL_DEFAULT_ENABLE([wipefs], [check])]
++)
++UL_BUILD_INIT([wipefs])
+ UL_REQUIRES_BUILD([wipefs], [libblkid])
+ UL_REQUIRES_BUILD([wipefs], [libsmartcols])
+ AM_CONDITIONAL([BUILD_WIPEFS], [test "x$build_wipefs" = xyes])
diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
index 39eb2c03a0..3c64a19d84 100644
--- a/package/util-linux/Config.in
+++ b/package/util-linux/Config.in
@@ -407,6 +407,17 @@ config BR2_PACKAGE_UTIL_LINUX_WALL
 	help
 	  Send a message to everybody's terminal
 
+config BR2_PACKAGE_UTIL_LINUX_WIPEFS
+	bool "wipefs"
+	select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
+	select BR2_PACKAGE_UTIL_LINUX_LIBSMARTCOLS
+	help
+	  wipefs can erase filesystem, raid or partition-table
+	  signatures (magic strings) from the specified device
+	  to make the signatures invisible for libblkid. wipefs
+	  does not erase the filesystem itself nor any other data
+	  from the device.
+
 config BR2_PACKAGE_UTIL_LINUX_WDCTL
 	bool "wdctl"
 	depends on BR2_USE_MMU # libsmartcols
diff --git a/package/util-linux/util-linux.mk b/package/util-linux/util-linux.mk
index ecc3ab62de..0374420b58 100644
--- a/package/util-linux/util-linux.mk
+++ b/package/util-linux/util-linux.mk
@@ -153,6 +153,7 @@ UTIL_LINUX_CONF_OPTS += \
 	$(if $(BR2_PACKAGE_UTIL_LINUX_VIPW),--enable-vipw,--disable-vipw) \
 	$(if $(BR2_PACKAGE_UTIL_LINUX_WALL),--enable-wall,--disable-wall) \
 	$(if $(BR2_PACKAGE_UTIL_LINUX_WDCTL),--enable-wdctl,--disable-wdctl) \
+	$(if $(BR2_PACKAGE_UTIL_LINUX_WIPEFS),--enable-wipefs,--disable-wipefs) \
 	$(if $(BR2_PACKAGE_UTIL_LINUX_WRITE),--enable-write,--disable-write) \
 	$(if $(BR2_PACKAGE_UTIL_LINUX_ZRAMCTL),--enable-zramctl,--disable-zramctl)
 
-- 
2.23.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
  2019-10-18  3:30 [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs sam.voss at gmail.com
@ 2019-10-18 14:05 ` Carlos Santos
  2019-10-19 19:00 ` Arnout Vandecappelle
  1 sibling, 0 replies; 6+ messages in thread
From: Carlos Santos @ 2019-10-18 14:05 UTC (permalink / raw)
  To: buildroot

On Fri, Oct 18, 2019 at 12:31 AM <sam.voss@gmail.com> wrote:
>
> From: Sam Voss <sam.voss@gmail.com>
>
> Add option to enable wipefs as a standalone app, so that it may be
> enabled when the flag --disable-all-packages is set.
>
> Signed-off-by: Sam Voss <sam.voss@gmail.com>
> ---
>  ...w-explicit-wipefs-enable-disablement.patch | 33 +++++++++++++++++++
>  package/util-linux/Config.in                  | 11 +++++++
>  package/util-linux/util-linux.mk              |  1 +
>  3 files changed, 45 insertions(+)
>  create mode 100644 package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch
>
> diff --git a/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch b/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch
> new file mode 100644
> index 0000000000..85021e2978
> --- /dev/null
> +++ b/package/util-linux/0001-Allow-explicit-wipefs-enable-disablement.patch
> @@ -0,0 +1,33 @@
> +From a76cd5f6abace269b60ef6f1b95526a2d09fd331 Mon Sep 17 00:00:00 2001
> +From: Sam Voss <sam.voss@gmail.com>
> +Date: Mon, 14 Oct 2019 21:19:16 -0500
> +Subject: [PATCH] wipefs: Allow explicitly enable/disablement
> +
> +Currently if `--disable-all-packages` is set, there is no configure
> +option for re-enabling `wipefs`.
> +
> +As the current default for `wipefs` is "enabled", add `--disable-` flag
> +to maintain backward compatibility.
> +
> +Signed-off-by: Sam Voss <sam.voss@gmail.com>
> +---
> + configure.ac | 6 +++++-
> + 1 file changed, 5 insertions(+), 1 deletion(-)
> +
> +diff --git a/configure.ac b/configure.ac
> +index fc8a9507f..f70f8d1ee 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -1286,7 +1286,11 @@ UL_BUILD_INIT([findfs], [check])
> + UL_REQUIRES_BUILD([findfs], [libblkid])
> + AM_CONDITIONAL([BUILD_FINDFS], [test "x$build_findfs" = xyes])
> +
> +-UL_BUILD_INIT([wipefs], [check])
> ++AC_ARG_ENABLE([wipefs],
> ++  AS_HELP_STRING([--disable-wipefs], [do not build wipefs]),
> ++  [], [UL_DEFAULT_ENABLE([wipefs], [check])]
> ++)
> ++UL_BUILD_INIT([wipefs])
> + UL_REQUIRES_BUILD([wipefs], [libblkid])
> + UL_REQUIRES_BUILD([wipefs], [libsmartcols])
> + AM_CONDITIONAL([BUILD_WIPEFS], [test "x$build_wipefs" = xyes])
> diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
> index 39eb2c03a0..3c64a19d84 100644
> --- a/package/util-linux/Config.in
> +++ b/package/util-linux/Config.in
> @@ -407,6 +407,17 @@ config BR2_PACKAGE_UTIL_LINUX_WALL
>         help
>           Send a message to everybody's terminal
>
> +config BR2_PACKAGE_UTIL_LINUX_WIPEFS
> +       bool "wipefs"
> +       select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
> +       select BR2_PACKAGE_UTIL_LINUX_LIBSMARTCOLS
> +       help
> +         wipefs can erase filesystem, raid or partition-table
> +         signatures (magic strings) from the specified device
> +         to make the signatures invisible for libblkid. wipefs
> +         does not erase the filesystem itself nor any other data
> +         from the device.
> +
>  config BR2_PACKAGE_UTIL_LINUX_WDCTL
>         bool "wdctl"
>         depends on BR2_USE_MMU # libsmartcols
> diff --git a/package/util-linux/util-linux.mk b/package/util-linux/util-linux.mk
> index ecc3ab62de..0374420b58 100644
> --- a/package/util-linux/util-linux.mk
> +++ b/package/util-linux/util-linux.mk
> @@ -153,6 +153,7 @@ UTIL_LINUX_CONF_OPTS += \
>         $(if $(BR2_PACKAGE_UTIL_LINUX_VIPW),--enable-vipw,--disable-vipw) \
>         $(if $(BR2_PACKAGE_UTIL_LINUX_WALL),--enable-wall,--disable-wall) \
>         $(if $(BR2_PACKAGE_UTIL_LINUX_WDCTL),--enable-wdctl,--disable-wdctl) \
> +       $(if $(BR2_PACKAGE_UTIL_LINUX_WIPEFS),--enable-wipefs,--disable-wipefs) \
>         $(if $(BR2_PACKAGE_UTIL_LINUX_WRITE),--enable-write,--disable-write) \
>         $(if $(BR2_PACKAGE_UTIL_LINUX_ZRAMCTL),--enable-zramctl,--disable-zramctl)
>
> --
> 2.23.0
>

Tested-by: Carlos Santos <unixmania@gmail.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
  2019-10-18  3:30 [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs sam.voss at gmail.com
  2019-10-18 14:05 ` Carlos Santos
@ 2019-10-19 19:00 ` Arnout Vandecappelle
  2019-10-19 19:19   ` Sam Voss
  2019-10-19 21:32   ` Bernd Kuhls
  1 sibling, 2 replies; 6+ messages in thread
From: Arnout Vandecappelle @ 2019-10-19 19:00 UTC (permalink / raw)
  To: buildroot



On 18/10/2019 05:30, sam.voss at gmail.com wrote:
> From: Sam Voss <sam.voss@gmail.com>
> 
> Add option to enable wipefs as a standalone app, so that it may be
> enabled when the flag --disable-all-packages is set.
> 
> Signed-off-by: Sam Voss <sam.voss@gmail.com>

 Applied to master, thanks.

 Did you send the patch upstream? If not, please do so.

 Regards,
 Arnout

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
  2019-10-19 19:00 ` Arnout Vandecappelle
@ 2019-10-19 19:19   ` Sam Voss
  2019-10-19 21:32   ` Bernd Kuhls
  1 sibling, 0 replies; 6+ messages in thread
From: Sam Voss @ 2019-10-19 19:19 UTC (permalink / raw)
  To: buildroot

On Sat, Oct 19, 2019 at 2:00 PM Arnout Vandecappelle <arnout@mind.be> wrote:
>
> On 18/10/2019 05:30, sam.voss at gmail.com wrote:
> > From: Sam Voss <sam.voss@gmail.com>
> >
> > Add option to enable wipefs as a standalone app, so that it may be
> > enabled when the flag --disable-all-packages is set.
> >
> > Signed-off-by: Sam Voss <sam.voss@gmail.com>
>
>  Applied to master, thanks.
>
>  Did you send the patch upstream? If not, please do so.

Sent and accepted

Sam

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
  2019-10-19 19:00 ` Arnout Vandecappelle
  2019-10-19 19:19   ` Sam Voss
@ 2019-10-19 21:32   ` Bernd Kuhls
  2019-10-19 21:59     ` Sam Voss
  1 sibling, 1 reply; 6+ messages in thread
From: Bernd Kuhls @ 2019-10-19 21:32 UTC (permalink / raw)
  To: buildroot

Arnout Vandecappelle <arnout@mind.be> wrote in
news:98c40882-8f1d-61d6-41fa-74d86a8bf5a8 at mind.be: 

> On 18/10/2019 05:30, sam.voss at gmail.com
> wrote: 
>> From: Sam Voss <sam.voss@gmail.com>
>> 
>> Add option to enable wipefs as a standalone app, so that it may be
>> enabled when the flag --disable-all-packages is set.
>> 
>> Signed-off-by: Sam Voss
>> <sam.voss@gmail.com> 
> 
>  Applied to master, thanks.

Hi,

this patch lacks

    	UTIL_LINUX_AUTORECONF = yes

causing build errors:
http://autobuild.buildroot.net/?reason=util-linux-2.34

Regards, Bernd

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs
  2019-10-19 21:32   ` Bernd Kuhls
@ 2019-10-19 21:59     ` Sam Voss
  0 siblings, 0 replies; 6+ messages in thread
From: Sam Voss @ 2019-10-19 21:59 UTC (permalink / raw)
  To: buildroot

On Sat, Oct 19, 2019 at 4:33 PM Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
>
> Arnout Vandecappelle <arnout@mind.be> wrote in
> news:98c40882-8f1d-61d6-41fa-74d86a8bf5a8 at mind.be:
>
> > On 18/10/2019 05:30, sam.voss at gmail.com
> > wrote:
> >> From: Sam Voss <sam.voss@gmail.com>
> >>
> >> Add option to enable wipefs as a standalone app, so that it may be
> >> enabled when the flag --disable-all-packages is set.
> >>
> >> Signed-off-by: Sam Voss
> >> <sam.voss@gmail.com>
> >
> >  Applied to master, thanks.
>
> Hi,
>
> this patch lacks
>
>         UTIL_LINUX_AUTORECONF = yes
>S
> causing build errors:
> http://autobuild.buildroot.net/?reason=util-linux-2.34

I just re-tested this after removing automake from my host, and sure
enough it failed. Sending follow-up patch to clean this up.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-10-19 21:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-18  3:30 [Buildroot] [PATCH] package/util-linux: Add option for enabling wipefs sam.voss at gmail.com
2019-10-18 14:05 ` Carlos Santos
2019-10-19 19:00 ` Arnout Vandecappelle
2019-10-19 19:19   ` Sam Voss
2019-10-19 21:32   ` Bernd Kuhls
2019-10-19 21:59     ` Sam Voss

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.