* [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename @ 2023-10-19 20:33 Brandon Maier via buildroot 2023-10-22 18:47 ` Yann E. MORIN 2023-10-24 18:48 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Brandon Maier via buildroot 0 siblings, 2 replies; 7+ messages in thread From: Brandon Maier via buildroot @ 2023-10-19 20:33 UTC (permalink / raw) To: buildroot; +Cc: Bernd Kuhls, Brandon Maier, Neal J. Buchmeyer From: "Neal J. Buchmeyer" <neal.buchmeyer@collins.com> Add option to select ifrename tool which is part of the wireless_tools suite. Also add an option to disable iwconfig for targets that only want the library or ifrename. Signed-off-by: Neal J. Buchmeyer <neal.buchmeyer@collins.com> Signed-off-by: Brandon Maier <brandon.maier@collins.com> --- package/wireless_tools/Config.in | 12 ++++++++++++ package/wireless_tools/wireless_tools.mk | 22 ++++++++++++++++------ 2 files changed, 28 insertions(+), 6 deletions(-) diff --git a/package/wireless_tools/Config.in b/package/wireless_tools/Config.in index 349908dbde..ad1c2e8e82 100644 --- a/package/wireless_tools/Config.in +++ b/package/wireless_tools/Config.in @@ -7,6 +7,18 @@ config BR2_PACKAGE_WIRELESS_TOOLS if BR2_PACKAGE_WIRELESS_TOOLS +config BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME + bool "Add ifrename" + help + Build and install the ifrename tool + +config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG + bool "Add iwconfig" + default y + help + Build and install the iwconfig, iwlist, iwspy, iwpriv, and + iwgetid tools. + config BR2_PACKAGE_WIRELESS_TOOLS_LIB bool "Install shared library" depends on !BR2_STATIC_LIBS diff --git a/package/wireless_tools/wireless_tools.mk b/package/wireless_tools/wireless_tools.mk index 1d29c3b102..210d8aa51a 100644 --- a/package/wireless_tools/wireless_tools.mk +++ b/package/wireless_tools/wireless_tools.mk @@ -14,8 +14,14 @@ WIRELESS_TOOLS_CPE_ID_VERSION = $(WIRELESS_TOOLS_VERSION_MAJOR) WIRELESS_TOOLS_CPE_ID_UPDATE = pre9 WIRELESS_TOOLS_INSTALL_STAGING = YES -WIRELESS_TOOLS_BUILD_TARGETS = iwmulticall -WIRELESS_TOOLS_INSTALL_TARGETS = install-iwmulticall +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME),y) +WIRELESS_TOOLS_BUILD_TARGETS += ifrename +endif + +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG),y) +WIRELESS_TOOLS_BUILD_TARGETS += iwmulticall +WIRELESS_TOOLS_INSTALL_TARGETS += install-iwmulticall +endif ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_LIB),y) WIRELESS_TOOLS_BUILD_TARGETS += libiw.so.$(WIRELESS_TOOLS_VERSION_MAJOR) @@ -30,13 +36,17 @@ endef endif define WIRELESS_TOOLS_BUILD_CMDS - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \ - $(WIRELESS_TOOLS_BUILD_TARGETS) + $(if $(WIRELESS_TOOLS_BUILD_TARGETS), + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) CC="$(TARGET_CC)" \ + CFLAGS="$(TARGET_CFLAGS)" $(WIRELESS_TOOLS_BUILD_TARGETS)) endef define WIRELESS_TOOLS_INSTALL_TARGET_CMDS - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" LDCONFIG=/bin/true \ - $(WIRELESS_TOOLS_INSTALL_TARGETS) + $(if $(WIRELESS_TOOLS_INSTALL_TARGETS), + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" \ + LDCONFIG=/bin/true $(WIRELESS_TOOLS_INSTALL_TARGETS)) + $(if $(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME), + $(INSTALL) -D -m 755 $(@D)/ifrename $(TARGET_DIR)/sbin/ifrename) endef $(eval $(generic-package)) -- 2.41.0 _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename 2023-10-19 20:33 [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot @ 2023-10-22 18:47 ` Yann E. MORIN 2023-10-24 18:52 ` Maier, Brandon L Collins via buildroot 2023-10-24 18:48 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Brandon Maier via buildroot 1 sibling, 1 reply; 7+ messages in thread From: Yann E. MORIN @ 2023-10-22 18:47 UTC (permalink / raw) To: Brandon Maier; +Cc: Bernd Kuhls, Neal J. Buchmeyer, buildroot Brandon, Neal, All, On 2023-10-19 20:33 +0000, Brandon Maier via buildroot spake thusly: > From: "Neal J. Buchmeyer" <neal.buchmeyer@collins.com> > > Add option to select ifrename tool which is part of the wireless_tools > suite. Also add an option to disable iwconfig for targets that only want > the library or ifrename. Those are two semantically disjoint changes, so they should be two patches, preferably in this order: 1. one to allow only building the library (and make iwconfig optional) 2. one to add ifrename Plus, see below... > Signed-off-by: Neal J. Buchmeyer <neal.buchmeyer@collins.com> > Signed-off-by: Brandon Maier <brandon.maier@collins.com> > --- > package/wireless_tools/Config.in | 12 ++++++++++++ > package/wireless_tools/wireless_tools.mk | 22 ++++++++++++++++------ > 2 files changed, 28 insertions(+), 6 deletions(-) > > diff --git a/package/wireless_tools/Config.in b/package/wireless_tools/Config.in > index 349908dbde..ad1c2e8e82 100644 > --- a/package/wireless_tools/Config.in > +++ b/package/wireless_tools/Config.in > @@ -7,6 +7,18 @@ config BR2_PACKAGE_WIRELESS_TOOLS > > if BR2_PACKAGE_WIRELESS_TOOLS > > +config BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME > + bool "Add ifrename" > + help > + Build and install the ifrename tool > + > +config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG > + bool "Add iwconfig" > + default y > + help > + Build and install the iwconfig, iwlist, iwspy, iwpriv, and > + iwgetid tools. > + > config BR2_PACKAGE_WIRELESS_TOOLS_LIB > bool "Install shared library" > depends on !BR2_STATIC_LIBS > diff --git a/package/wireless_tools/wireless_tools.mk b/package/wireless_tools/wireless_tools.mk > index 1d29c3b102..210d8aa51a 100644 > --- a/package/wireless_tools/wireless_tools.mk > +++ b/package/wireless_tools/wireless_tools.mk > @@ -14,8 +14,14 @@ WIRELESS_TOOLS_CPE_ID_VERSION = $(WIRELESS_TOOLS_VERSION_MAJOR) > WIRELESS_TOOLS_CPE_ID_UPDATE = pre9 > WIRELESS_TOOLS_INSTALL_STAGING = YES > > -WIRELESS_TOOLS_BUILD_TARGETS = iwmulticall > -WIRELESS_TOOLS_INSTALL_TARGETS = install-iwmulticall > +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME),y) > +WIRELESS_TOOLS_BUILD_TARGETS += ifrename > +endif > + > +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG),y) > +WIRELESS_TOOLS_BUILD_TARGETS += iwmulticall > +WIRELESS_TOOLS_INSTALL_TARGETS += install-iwmulticall > +endif > > ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_LIB),y) > WIRELESS_TOOLS_BUILD_TARGETS += libiw.so.$(WIRELESS_TOOLS_VERSION_MAJOR) > @@ -30,13 +36,17 @@ endef > endif > > define WIRELESS_TOOLS_BUILD_CMDS > - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) CC="$(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)" \ > - $(WIRELESS_TOOLS_BUILD_TARGETS) > + $(if $(WIRELESS_TOOLS_BUILD_TARGETS), > + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) CC="$(TARGET_CC)" \ > + CFLAGS="$(TARGET_CFLAGS)" $(WIRELESS_TOOLS_BUILD_TARGETS)) > endef So, if none of ifrename, iwconfig, or the library is enabled, then there is nothjing to build (or to install, below). In this case, it does not make sense to enable the package altogether. I think we must ensure that at least one of those are enabled, and iwconfig looks like the obvious candidate. So (elided for the sake of example): config BR2_PACKAGE_WIRELESS_TOOLS bool "wireless tools" select BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG if \ !BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME && \ !BR2_PACKAGE_WIRELESS_TOOLS_LIB Of course, in the first patch, only BR2_PACKAGE_WIRELESS_TOOLS_LIB would be needed in the condition, with BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME added in patch 2. Regards, Yann E. MORIN. > define WIRELESS_TOOLS_INSTALL_TARGET_CMDS > - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" LDCONFIG=/bin/true \ > - $(WIRELESS_TOOLS_INSTALL_TARGETS) > + $(if $(WIRELESS_TOOLS_INSTALL_TARGETS), > + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" \ > + LDCONFIG=/bin/true $(WIRELESS_TOOLS_INSTALL_TARGETS)) > + $(if $(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME), > + $(INSTALL) -D -m 755 $(@D)/ifrename $(TARGET_DIR)/sbin/ifrename) > endef > > $(eval $(generic-package)) > -- > 2.41.0 > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename 2023-10-22 18:47 ` Yann E. MORIN @ 2023-10-24 18:52 ` Maier, Brandon L Collins via buildroot 0 siblings, 0 replies; 7+ messages in thread From: Maier, Brandon L Collins via buildroot @ 2023-10-24 18:52 UTC (permalink / raw) To: Yann E. MORIN Cc: Bernd Kuhls, Buchmeyer, Neal J Collins, buildroot@buildroot.org Hi Yann, > -----Original Message----- > From: Yann E. MORIN <yann.morin.1998@free.fr> > Sent: Sunday, October 22, 2023 1:48 PM > > Those are two semantically disjoint changes, so they should be two > patches, preferably in this order: > 1. one to allow only building the library (and make iwconfig optional) > 2. one to add ifrename Agreed, sent a v2 series with the changes split apart > So, if none of ifrename, iwconfig, or the library is enabled, then there > is nothjing to build (or to install, below). In this case, it does not > make sense to enable the package altogether. > > I think we must ensure that at least one of those are enabled, and > iwconfig looks like the obvious candidate. So (elided for the sake of > example): > > config BR2_PACKAGE_WIRELESS_TOOLS > bool "wireless tools" > select BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG if \ > !BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME && \ > !BR2_PACKAGE_WIRELESS_TOOLS_LIB > > Of course, in the first patch, only BR2_PACKAGE_WIRELESS_TOOLS_LIB would > be needed in the condition, with BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME > added in patch 2. Agreed, it makes sense to require one of the tools to be built. Didn't know about this KConfig trick. Fixed as suggested in v2. Thanks, Brandon Maier _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional 2023-10-19 20:33 [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot 2023-10-22 18:47 ` Yann E. MORIN @ 2023-10-24 18:48 ` Brandon Maier via buildroot 2023-10-24 18:48 ` [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot 2023-11-04 18:17 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Thomas Petazzoni via buildroot 1 sibling, 2 replies; 7+ messages in thread From: Brandon Maier via buildroot @ 2023-10-24 18:48 UTC (permalink / raw) To: buildroot; +Cc: Bernd Kuhls, Yann E . MORIN, Brandon Maier, Neal J. Buchmeyer From: "Neal J. Buchmeyer" <neal.buchmeyer@collins.com> Allow disabling iwconfig for targets that only want the library. Signed-off-by: Neal J. Buchmeyer <neal.buchmeyer@collins.com> Signed-off-by: Brandon Maier <brandon.maier@collins.com> --- package/wireless_tools/Config.in | 8 ++++++++ package/wireless_tools/wireless_tools.mk | 2 ++ 2 files changed, 10 insertions(+) diff --git a/package/wireless_tools/Config.in b/package/wireless_tools/Config.in index 349908dbde..f676e33b57 100644 --- a/package/wireless_tools/Config.in +++ b/package/wireless_tools/Config.in @@ -1,5 +1,7 @@ config BR2_PACKAGE_WIRELESS_TOOLS bool "wireless tools" + select BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG if \ + !BR2_PACKAGE_WIRELESS_TOOLS_LIB help A collection of tools to configure wireless lan cards. @@ -7,6 +9,12 @@ config BR2_PACKAGE_WIRELESS_TOOLS if BR2_PACKAGE_WIRELESS_TOOLS +config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG + bool "Add iwconfig" + help + Build and install the iwconfig, iwlist, iwspy, iwpriv, and + iwgetid tools. + config BR2_PACKAGE_WIRELESS_TOOLS_LIB bool "Install shared library" depends on !BR2_STATIC_LIBS diff --git a/package/wireless_tools/wireless_tools.mk b/package/wireless_tools/wireless_tools.mk index 1d29c3b102..3e55eda600 100644 --- a/package/wireless_tools/wireless_tools.mk +++ b/package/wireless_tools/wireless_tools.mk @@ -14,8 +14,10 @@ WIRELESS_TOOLS_CPE_ID_VERSION = $(WIRELESS_TOOLS_VERSION_MAJOR) WIRELESS_TOOLS_CPE_ID_UPDATE = pre9 WIRELESS_TOOLS_INSTALL_STAGING = YES +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG),y) WIRELESS_TOOLS_BUILD_TARGETS = iwmulticall WIRELESS_TOOLS_INSTALL_TARGETS = install-iwmulticall +endif ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_LIB),y) WIRELESS_TOOLS_BUILD_TARGETS += libiw.so.$(WIRELESS_TOOLS_VERSION_MAJOR) -- 2.41.0 _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename 2023-10-24 18:48 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Brandon Maier via buildroot @ 2023-10-24 18:48 ` Brandon Maier via buildroot 2023-11-04 18:18 ` Thomas Petazzoni via buildroot 2023-11-04 18:17 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Thomas Petazzoni via buildroot 1 sibling, 1 reply; 7+ messages in thread From: Brandon Maier via buildroot @ 2023-10-24 18:48 UTC (permalink / raw) To: buildroot; +Cc: Bernd Kuhls, Yann E . MORIN, Brandon Maier, Neal J. Buchmeyer From: "Neal J. Buchmeyer" <neal.buchmeyer@collins.com> The ifrename tool is separate from iwconfig and is not being built. The upstream does not have an install target for ifrename either, so we must manually install it. Signed-off-by: Neal J. Buchmeyer <neal.buchmeyer@collins.com> Signed-off-by: Brandon Maier <brandon.maier@collins.com> --- package/wireless_tools/Config.in | 6 ++++++ package/wireless_tools/wireless_tools.mk | 11 +++++++++-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/package/wireless_tools/Config.in b/package/wireless_tools/Config.in index f676e33b57..d7f2ed2381 100644 --- a/package/wireless_tools/Config.in +++ b/package/wireless_tools/Config.in @@ -1,6 +1,7 @@ config BR2_PACKAGE_WIRELESS_TOOLS bool "wireless tools" select BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG if \ + !BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME && \ !BR2_PACKAGE_WIRELESS_TOOLS_LIB help A collection of tools to configure wireless lan cards. @@ -15,6 +16,11 @@ config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG Build and install the iwconfig, iwlist, iwspy, iwpriv, and iwgetid tools. +config BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME + bool "Add ifrename" + help + Build and install the ifrename tool + config BR2_PACKAGE_WIRELESS_TOOLS_LIB bool "Install shared library" depends on !BR2_STATIC_LIBS diff --git a/package/wireless_tools/wireless_tools.mk b/package/wireless_tools/wireless_tools.mk index 3e55eda600..619df3deaf 100644 --- a/package/wireless_tools/wireless_tools.mk +++ b/package/wireless_tools/wireless_tools.mk @@ -19,6 +19,10 @@ WIRELESS_TOOLS_BUILD_TARGETS = iwmulticall WIRELESS_TOOLS_INSTALL_TARGETS = install-iwmulticall endif +ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME),y) +WIRELESS_TOOLS_BUILD_TARGETS += ifrename +endif + ifeq ($(BR2_PACKAGE_WIRELESS_TOOLS_LIB),y) WIRELESS_TOOLS_BUILD_TARGETS += libiw.so.$(WIRELESS_TOOLS_VERSION_MAJOR) WIRELESS_TOOLS_INSTALL_TARGETS += install-dynamic @@ -37,8 +41,11 @@ define WIRELESS_TOOLS_BUILD_CMDS endef define WIRELESS_TOOLS_INSTALL_TARGET_CMDS - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" LDCONFIG=/bin/true \ - $(WIRELESS_TOOLS_INSTALL_TARGETS) + $(if $(WIRELESS_TOOLS_INSTALL_TARGETS), + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" \ + LDCONFIG=/bin/true $(WIRELESS_TOOLS_INSTALL_TARGETS)) + $(if $(BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME), + $(INSTALL) -D -m 755 $(@D)/ifrename $(TARGET_DIR)/sbin/ifrename) endef $(eval $(generic-package)) -- 2.41.0 _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename 2023-10-24 18:48 ` [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot @ 2023-11-04 18:18 ` Thomas Petazzoni via buildroot 0 siblings, 0 replies; 7+ messages in thread From: Thomas Petazzoni via buildroot @ 2023-11-04 18:18 UTC (permalink / raw) To: Brandon Maier via buildroot Cc: Bernd Kuhls, Yann E . MORIN, Brandon Maier, Neal J. Buchmeyer On Tue, 24 Oct 2023 18:48:35 +0000 Brandon Maier via buildroot <buildroot@buildroot.org> wrote: > diff --git a/package/wireless_tools/Config.in b/package/wireless_tools/Config.in > index f676e33b57..d7f2ed2381 100644 > --- a/package/wireless_tools/Config.in > +++ b/package/wireless_tools/Config.in > @@ -1,6 +1,7 @@ > config BR2_PACKAGE_WIRELESS_TOOLS > bool "wireless tools" > select BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG if \ > + !BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME && \ > !BR2_PACKAGE_WIRELESS_TOOLS_LIB > help > A collection of tools to configure wireless lan cards. > @@ -15,6 +16,11 @@ config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG > Build and install the iwconfig, iwlist, iwspy, iwpriv, and > iwgetid tools. > > +config BR2_PACKAGE_WIRELESS_TOOLS_IFRENAME > + bool "Add ifrename" Changed to: bool "Install ifrename" > define WIRELESS_TOOLS_INSTALL_TARGET_CMDS > - $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" LDCONFIG=/bin/true \ > - $(WIRELESS_TOOLS_INSTALL_TARGETS) > + $(if $(WIRELESS_TOOLS_INSTALL_TARGETS), > + $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) PREFIX="$(TARGET_DIR)" \ > + LDCONFIG=/bin/true $(WIRELESS_TOOLS_INSTALL_TARGETS)) Indented this last line with one more tab. And applied. Thanks! Thomas -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering and training https://bootlin.com _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional 2023-10-24 18:48 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Brandon Maier via buildroot 2023-10-24 18:48 ` [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot @ 2023-11-04 18:17 ` Thomas Petazzoni via buildroot 1 sibling, 0 replies; 7+ messages in thread From: Thomas Petazzoni via buildroot @ 2023-11-04 18:17 UTC (permalink / raw) To: Brandon Maier via buildroot Cc: Bernd Kuhls, Yann E . MORIN, Brandon Maier, Neal J. Buchmeyer On Tue, 24 Oct 2023 18:48:34 +0000 Brandon Maier via buildroot <buildroot@buildroot.org> wrote: > @@ -7,6 +9,12 @@ config BR2_PACKAGE_WIRELESS_TOOLS > > if BR2_PACKAGE_WIRELESS_TOOLS > > +config BR2_PACKAGE_WIRELESS_TOOLS_IWCONFIG > + bool "Add iwconfig" Change to: bool "Install tools" because it's really about "installing", and it is more consistent with the option for the shared library, which is "Install shared library". I've also added: default y to preserve the existing behavior. Applied with those changes. Thanks! Thomas -- Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering and training https://bootlin.com _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-11-04 18:18 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-10-19 20:33 [Buildroot] [PATCH 1/1] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot 2023-10-22 18:47 ` Yann E. MORIN 2023-10-24 18:52 ` Maier, Brandon L Collins via buildroot 2023-10-24 18:48 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Brandon Maier via buildroot 2023-10-24 18:48 ` [Buildroot] [PATCH v2 2/2] package/wireless_tools: add option to build ifrename Brandon Maier via buildroot 2023-11-04 18:18 ` Thomas Petazzoni via buildroot 2023-11-04 18:17 ` [Buildroot] [PATCH v2 1/2] package/wireless_tools: make 'iwconfig' tool optional Thomas Petazzoni via buildroot
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox