All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] skeleton: remove unused /etc/issue
@ 2015-11-17 16:16 Jérôme Pouiller
  2015-11-17 19:45 ` Maxime Hadjinlian
  2015-11-17 20:41 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Jérôme Pouiller @ 2015-11-17 16:16 UTC (permalink / raw)
  To: buildroot

/etc/issue is managed with BR2_TARGET_GENERIC_ISSUE. In case
BR2_TARGET_GENERIC_ISSUE is set (which is default), /etc/issue
is overwritten. In case BR2_TARGET_GENERIC_ISSUE is not set,
we don't want /etc/issue exist.

Finaly, remove /etc/issue from skeleton.

Signed-off-by: J?r?me Pouiller <jezz@sysmic.org>
---
 system/skeleton/etc/issue | 3 ---
 1 file changed, 3 deletions(-)
 delete mode 100644 system/skeleton/etc/issue

diff --git a/system/skeleton/etc/issue b/system/skeleton/etc/issue
deleted file mode 100644
index 90f1779..0000000
--- a/system/skeleton/etc/issue
+++ /dev/null
@@ -1,3 +0,0 @@
-
-Welcome to Buildroot
-
-- 
2.1.4

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

* [Buildroot] [PATCH] skeleton: remove unused /etc/issue
  2015-11-17 16:16 [Buildroot] [PATCH] skeleton: remove unused /etc/issue Jérôme Pouiller
@ 2015-11-17 19:45 ` Maxime Hadjinlian
  2015-11-17 20:41 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Maxime Hadjinlian @ 2015-11-17 19:45 UTC (permalink / raw)
  To: buildroot

Hi J?r?me, all

On Tue, Nov 17, 2015 at 5:16 PM, J?r?me Pouiller <jezz@sysmic.org> wrote:
> /etc/issue is managed with BR2_TARGET_GENERIC_ISSUE. In case
> BR2_TARGET_GENERIC_ISSUE is set (which is default), /etc/issue
> is overwritten. In case BR2_TARGET_GENERIC_ISSUE is not set,
> we don't want /etc/issue exist.
>
> Finaly, remove /etc/issue from skeleton.
>
> Signed-off-by: J?r?me Pouiller <jezz@sysmic.org>
> ---
>  system/skeleton/etc/issue | 3 ---
>  1 file changed, 3 deletions(-)
>  delete mode 100644 system/skeleton/etc/issue
>
> diff --git a/system/skeleton/etc/issue b/system/skeleton/etc/issue
> deleted file mode 100644
> index 90f1779..0000000
> --- a/system/skeleton/etc/issue
> +++ /dev/null
> @@ -1,3 +0,0 @@
> -
> -Welcome to Buildroot
> -
> --
> 2.1.4
Just to clarify the commit log (maybe it's just me that did not get
it), but without this patch, if you empty BR2_TARGET_GENERIC_ISSUE and
you use the default skeleton, you still get an etc/issue with the
"Welcome to Buildroot" message. I think it must be a long standing
bug.
Thanks for fixing this !

Reviewed-by: "Maxime Hadjinlian" <maxime.hadjinlian@gmail.com>

>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

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

* [Buildroot] [PATCH] skeleton: remove unused /etc/issue
  2015-11-17 16:16 [Buildroot] [PATCH] skeleton: remove unused /etc/issue Jérôme Pouiller
  2015-11-17 19:45 ` Maxime Hadjinlian
@ 2015-11-17 20:41 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2015-11-17 20:41 UTC (permalink / raw)
  To: buildroot

>>>>> "J?r?me" == J?r?me Pouiller <jezz@sysmic.org> writes:

 > /etc/issue is managed with BR2_TARGET_GENERIC_ISSUE. In case
 > BR2_TARGET_GENERIC_ISSUE is set (which is default), /etc/issue
 > is overwritten. In case BR2_TARGET_GENERIC_ISSUE is not set,
 > we don't want /etc/issue exist.

 > Finaly, remove /etc/issue from skeleton.

 > Signed-off-by: J?r?me Pouiller <jezz@sysmic.org>

Committed to next, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2015-11-17 20:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-17 16:16 [Buildroot] [PATCH] skeleton: remove unused /etc/issue Jérôme Pouiller
2015-11-17 19:45 ` Maxime Hadjinlian
2015-11-17 20:41 ` 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.