All of lore.kernel.org
 help / color / mirror / Atom feed
* [Resend: PATCH] ncurses-terminfo: Remove bashism from basic terminfo installation
@ 2013-10-30 12:16 Seth Bollinger
  2013-10-30 15:50   ` Saul Wold
  0 siblings, 1 reply; 3+ messages in thread
From: Seth Bollinger @ 2013-10-30 12:16 UTC (permalink / raw)
  To: yocto

The vtX terminfo files aren't being copied on systems where bash isn't
the default shell (debian, etc.).  I removed the bash specific syntax
so the files are properly copied on these systems.

Signed-off-by: Seth Bollinger <seth.boll@gmail.com>
---
 meta/recipes-core/ncurses/ncurses.inc |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
index 671daf8..01cdf13 100644
--- a/meta/recipes-core/ncurses/ncurses.inc
+++ b/meta/recipes-core/ncurses/ncurses.inc
@@ -174,7 +174,7 @@ shell_do_install() {
 
         # include some basic terminfo files
         # stolen ;) from gentoo and modified a bit
-        for x in ansi console dumb linux rxvt screen sun vt{52,100,102,200,220} xterm-color xterm-xfree86 xterm-256color
+        for x in ansi console dumb linux rxvt screen sun vt52 vt100 vt102 vt200 vt220 xterm-color xterm-xfree86 xterm-256color
         do
                 local termfile="$(find "${D}${datadir}/terminfo/" -name "${x}" 2>/dev/null)"
                 local basedir="$(basename $(dirname "${termfile}"))"
-- 
1.7.10.4



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

* Re: [yocto] [Resend: PATCH] ncurses-terminfo: Remove bashism from basic terminfo installation
  2013-10-30 12:16 [Resend: PATCH] ncurses-terminfo: Remove bashism from basic terminfo installation Seth Bollinger
@ 2013-10-30 15:50   ` Saul Wold
  0 siblings, 0 replies; 3+ messages in thread
From: Saul Wold @ 2013-10-30 15:50 UTC (permalink / raw)
  To: Seth Bollinger, yocto
  Cc: 'Patches and discussions about the oe-core layer'


I have it pending in the Master under test branch, normally patches to 
the core (meta) go to the openembedded-core@lists.openembedded.org

Thanks
	Sau!


On 10/30/2013 05:16 AM, Seth Bollinger wrote:
> The vtX terminfo files aren't being copied on systems where bash isn't
> the default shell (debian, etc.).  I removed the bash specific syntax
> so the files are properly copied on these systems.
>
> Signed-off-by: Seth Bollinger <seth.boll@gmail.com>
> ---
>   meta/recipes-core/ncurses/ncurses.inc |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
> index 671daf8..01cdf13 100644
> --- a/meta/recipes-core/ncurses/ncurses.inc
> +++ b/meta/recipes-core/ncurses/ncurses.inc
> @@ -174,7 +174,7 @@ shell_do_install() {
>
>           # include some basic terminfo files
>           # stolen ;) from gentoo and modified a bit
> -        for x in ansi console dumb linux rxvt screen sun vt{52,100,102,200,220} xterm-color xterm-xfree86 xterm-256color
> +        for x in ansi console dumb linux rxvt screen sun vt52 vt100 vt102 vt200 vt220 xterm-color xterm-xfree86 xterm-256color
>           do
>                   local termfile="$(find "${D}${datadir}/terminfo/" -name "${x}" 2>/dev/null)"
>                   local basedir="$(basename $(dirname "${termfile}"))"
>


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

* Re: [Resend: PATCH] ncurses-terminfo: Remove bashism from basic terminfo installation
@ 2013-10-30 15:50   ` Saul Wold
  0 siblings, 0 replies; 3+ messages in thread
From: Saul Wold @ 2013-10-30 15:50 UTC (permalink / raw)
  To: Seth Bollinger, yocto
  Cc: 'Patches and discussions about the oe-core layer'


I have it pending in the Master under test branch, normally patches to 
the core (meta) go to the openembedded-core@lists.openembedded.org

Thanks
	Sau!


On 10/30/2013 05:16 AM, Seth Bollinger wrote:
> The vtX terminfo files aren't being copied on systems where bash isn't
> the default shell (debian, etc.).  I removed the bash specific syntax
> so the files are properly copied on these systems.
>
> Signed-off-by: Seth Bollinger <seth.boll@gmail.com>
> ---
>   meta/recipes-core/ncurses/ncurses.inc |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
> index 671daf8..01cdf13 100644
> --- a/meta/recipes-core/ncurses/ncurses.inc
> +++ b/meta/recipes-core/ncurses/ncurses.inc
> @@ -174,7 +174,7 @@ shell_do_install() {
>
>           # include some basic terminfo files
>           # stolen ;) from gentoo and modified a bit
> -        for x in ansi console dumb linux rxvt screen sun vt{52,100,102,200,220} xterm-color xterm-xfree86 xterm-256color
> +        for x in ansi console dumb linux rxvt screen sun vt52 vt100 vt102 vt200 vt220 xterm-color xterm-xfree86 xterm-256color
>           do
>                   local termfile="$(find "${D}${datadir}/terminfo/" -name "${x}" 2>/dev/null)"
>                   local basedir="$(basename $(dirname "${termfile}"))"
>


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

end of thread, other threads:[~2013-10-30 15:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-30 12:16 [Resend: PATCH] ncurses-terminfo: Remove bashism from basic terminfo installation Seth Bollinger
2013-10-30 15:50 ` [yocto] " Saul Wold
2013-10-30 15:50   ` Saul Wold

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.