[v2,1/1] package/arp-scan: allow to not install package data

Message ID 20230804205401.3975960-1-dario.binacchi@amarulasolutions.com
State New
Headers show
Series
  • [v2,1/1] package/arp-scan: allow to not install package data
Related show

Commit Message

Dario Binacchi Aug. 4, 2023, 8:54 p.m. UTC
The patch adds an option that allows you to not install the data along
with the binaries (less than 100kb), saving 1.4Mb of rootfs data.

By default, the data is installed for backward compatibility.

Cc: Angelo Compagnucci <angelo@amarulasolutions.com>
Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>

---

Changes in v2:
- add the option to ensure backward compatibility.
- update the commit message

package/arp-scan/Config.in   | 11 +++++++++++
 package/arp-scan/arp-scan.mk |  4 ++++
 2 files changed, 15 insertions(+)

Comments

'Jan Kiszka' via Amarula Linux Aug. 6, 2023, 11:45 a.m. UTC | #1
On Fri,  4 Aug 2023 22:54:01 +0200
Dario Binacchi <dario.binacchi@amarulasolutions.com> wrote:

> The patch adds an option that allows you to not install the data along
> with the binaries (less than 100kb), saving 1.4Mb of rootfs data.
> 
> By default, the data is installed for backward compatibility.
> 
> Cc: Angelo Compagnucci <angelo@amarulasolutions.com>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> 
> ---

Applied to next, thanks.

Thomas

Patch

diff --git a/package/arp-scan/Config.in b/package/arp-scan/Config.in
index ed70b3958610..cf7377d2f9b6 100644
--- a/package/arp-scan/Config.in
+++ b/package/arp-scan/Config.in
@@ -8,3 +8,14 @@  config BR2_PACKAGE_ARP_SCAN
 	  discover and fingerprint IP hosts on the local network.
 
 	  https://github.com/royhills/arp-scan
+
+if BR2_PACKAGE_ARP_SCAN
+
+config BR2_PACKAGE_ARP_SCAN_DATA
+	bool "install data (OUI)"
+	default y  # legacy
+	help
+	  Say 'y' here (the default) to install the data along the
+	  binaries.
+
+endif # BR2_PACKAGE_ARP_SCAN
diff --git a/package/arp-scan/arp-scan.mk b/package/arp-scan/arp-scan.mk
index 89644be9d3ef..67eb1881dba2 100644
--- a/package/arp-scan/arp-scan.mk
+++ b/package/arp-scan/arp-scan.mk
@@ -23,4 +23,8 @@  else
 ARP_SCAN_CONF_OPTS += --without-libcap
 endif
 
+ifeq ($(BR2_PACKAGE_ARP_SCAN_DATA),)
+ARP_SCAN_INSTALL_TARGET_OPTS = DESTDIR=$(TARGET_DIR) install-exec
+endif
+
 $(eval $(autotools-package))