[Buildroot] [PATCH v2] less: install the lessecho command

[Buildroot] [PATCH v2] less: install the lessecho command

[carrier.nicolas0]

From: Nicolas Carrier <nicolas.carrier@nav-timing.safrangroup.com>

it is required by, for example the s command, internal to less, which
allows to write the less input to a file

Signed-off-by: Nicolas Carrier <carrier.nicolas0@gmail.com>

---

Changes v1 -> v2:
  - add missing Signed-off-by
  - install all of less by defaulting to its 'install' make target
  - attempt to clarify the commit message
---
 package/less/less.mk | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/package/less/less.mk b/package/less/less.mk
index 91ce08f4f0..f7211ef112 100644
--- a/package/less/less.mk
+++ b/package/less/less.mk
@@ -11,8 +11,4 @@ LESS_LICENSE_FILES = COPYING
 LESS_CPE_ID_VENDOR = gnu
 LESS_DEPENDENCIES = ncurses
 
-define LESS_INSTALL_TARGET_CMDS
-	$(INSTALL) -m 0755 $(@D)/less $(TARGET_DIR)/usr/bin/less
-endef
-
 $(eval $(autotools-package))
-- 
2.30.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2] less: install the lessecho command

[Thomas Petazzoni via buildroot]

Hello Nicolas,

On Wed,  2 Aug 2023 08:22:39 +0200
carrier.nicolas0@gmail.com wrote:

> From: Nicolas Carrier <nicolas.carrier@nav-timing.safrangroup.com>
> 
> it is required by, for example the s command, internal to less, which
> allows to write the less input to a file
> 
> Signed-off-by: Nicolas Carrier <carrier.nicolas0@gmail.com>
> 
> ---

I've improved the commit log and applied to master:

  https://gitlab.com/buildroot.org/buildroot/-/commit/8b1bc64d924e386466d715637a4a09228322beef

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2] less: install the lessecho command

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni via buildroot <buildroot@buildroot.org> writes:

 > Hello Nicolas,
 > On Wed,  2 Aug 2023 08:22:39 +0200
 > carrier.nicolas0@gmail.com wrote:

 >> From: Nicolas Carrier <nicolas.carrier@nav-timing.safrangroup.com>
 >> 
 >> it is required by, for example the s command, internal to less, which
 >> allows to write the less input to a file
 >> 
 >> Signed-off-by: Nicolas Carrier <carrier.nicolas0@gmail.com>
 >> 
 >> ---

 > I've improved the commit log and applied to master:

 >   https://gitlab.com/buildroot.org/buildroot/-/commit/8b1bc64d924e386466d715637a4a09228322beef

Committed to 2023.02.x and 2023.05.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot