diff mbox

skeleton: remove unused /etc/issue

Message ID 1447776985-20087-1-git-send-email-jezz@sysmic.org
State Accepted
Commit 29d1f8d1b616fefeaf6a90590f64ea762eb09b41
Headers show

Commit Message

Jérôme Pouiller Nov. 17, 2015, 4:16 p.m. UTC
/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

Comments

Maxime Hadjinlian Nov. 17, 2015, 7:45 p.m. UTC | #1
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@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Nov. 17, 2015, 8:41 p.m. UTC | #2
>>>>> "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.
diff mbox

Patch

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
-