All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop
@ 2018-06-05 13:47 Florian La Roche
  2018-06-05 16:51 ` Thomas Petazzoni
  2018-06-17 15:50 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Florian La Roche @ 2018-06-05 13:47 UTC (permalink / raw)
  To: buildroot

The PAGER environment variable is including a blank character at the
end. Remove this.
A for loop has been unsetting the variable inside the loop, this is only
needed once at the end of the loop.

Signed-off-by: Florian La Roche <F.LaRoche@pilz.de>
---
 system/skeleton/etc/profile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/system/skeleton/etc/profile b/system/skeleton/etc/profile
index 3a97427fdf..1255d23ff4 100644
--- a/system/skeleton/etc/profile
+++ b/system/skeleton/etc/profile
@@ -8,7 +8,7 @@ if [ "$PS1" ]; then
 	fi
 fi
 
-export PAGER='/bin/more '
+export PAGER='/bin/more'
 export EDITOR='/bin/vi'
 
 # Source configuration files from /etc/profile.d
@@ -16,5 +16,5 @@ for i in /etc/profile.d/*.sh ; do
 	if [ -r "$i" ]; then
 		. $i
 	fi
-	unset i
 done
+unset i
-- 
2.17.0

Gesch?ftsf?hrung: Susanne Kunschert, Thomas Pilz
Pilz GmbH & Co. KG, Sitz: Ostfildern, HRA 210 893, Amtsgericht Stuttgart
Kompl. Ges. Peter Pilz GmbH, Sitz: Ostfildern, HRB 210 612, Amtsgericht Stuttgart
Umsatzsteuer: ID-Nr. DE 145 355 773, WEEE-Reg.-Nr. DE 71636849
This email is intended solely for the use of the named address(es). Any unauthorised disclosure, copying or distribution of these confidential information contained therein, or the taking of any action based on it, is prohibited. The sender disclaims any liability for the integrity of this email. Legally binding declarations must be in written form.
Umweltschutz liegt uns am Herzen! - Bitte denken Sie an unsere Umwelt, bevor Sie diese E-Mail drucken.
We do care about the environment! - Please consider the environment before printing this e-mail.

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

* [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop
  2018-06-05 13:47 [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop Florian La Roche
@ 2018-06-05 16:51 ` Thomas Petazzoni
  2018-06-17 15:50 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-06-05 16:51 UTC (permalink / raw)
  To: buildroot

Hello,

On Tue, 5 Jun 2018 15:47:45 +0200, Florian La Roche wrote:
> The PAGER environment variable is including a blank character at the
> end. Remove this.
> A for loop has been unsetting the variable inside the loop, this is only
> needed once at the end of the loop.
> 
> Signed-off-by: Florian La Roche <F.LaRoche@pilz.de>
> ---
>  system/skeleton/etc/profile | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop
  2018-06-05 13:47 [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop Florian La Roche
  2018-06-05 16:51 ` Thomas Petazzoni
@ 2018-06-17 15:50 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-06-17 15:50 UTC (permalink / raw)
  To: buildroot

>>>>> "Florian" == Florian La Roche <F.LaRoche@pilz.de> writes:

 > The PAGER environment variable is including a blank character at the
 > end. Remove this.
 > A for loop has been unsetting the variable inside the loop, this is only
 > needed once at the end of the loop.

 > Signed-off-by: Florian La Roche <F.LaRoche@pilz.de>

Committed to 2018.02.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2018-06-17 15:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-05 13:47 [Buildroot] [PATCH 1/1] skeleton: PAGER without blank and unset at end of for loop Florian La Roche
2018-06-05 16:51 ` Thomas Petazzoni
2018-06-17 15:50 ` Peter Korsgaard

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.