mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Fabio Porcedda <fabio.porcedda@gmail.com>
To: barebox@lists.infradead.org
Subject: Re: [PATCH v2 next 1/2] defenv2: comment setting default values in /env/config
Date: Fri, 22 Feb 2013 11:54:04 +0100	[thread overview]
Message-ID: <CAHkwnC_a0-C0oAwP33u8xjKpORB=JC4zDhqUDw+OMmdRd8S4Mw@mail.gmail.com> (raw)
In-Reply-To: <1361527842-25815-2-git-send-email-fabio.porcedda@gmail.com>

On Fri, Feb 22, 2013 at 11:10 AM, Fabio Porcedda
<fabio.porcedda@gmail.com> wrote:
> There is no need to set againg deafult values in /env/config
> because they are already setted in /env/bin/init,
> that allow /env/config-board to change those default values
> without being overwritten by /env/config.
>
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> ---
>  defaultenv-2/base/config | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/defaultenv-2/base/config b/defaultenv-2/base/config
> index 189e5a6..dec0595 100644
> --- a/defaultenv-2/base/config
> +++ b/defaultenv-2/base/config
> @@ -6,16 +6,16 @@
>  #global.hostname=
>
>  # set to false if you do not want to have colors
> -global.allow_color=true
> +#global.allow_color=true
>
>  # user (used for network filenames)
> -global.user=none
> +#global.user=none
>
>  # timeout in seconds before the default boot entry is started
> -global.autoboot_timeout=3
> +#global.autoboot_timeout=3
>
>  # default boot entry (one of /env/boot/*)
> -global.boot.default=net
> +#global.boot.default=net
>
>  # base bootargs
>  #global.linux.bootargs.base="console=ttyS0,115200"
> --
> 1.8.1.1
>

This patch works fine on master too.

Best regards
-- 
Fabio Porcedda

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2013-02-22 10:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-22 10:10 [PATCH v2 next 0/2] defenv2: complete move config-board out of /env/init Fabio Porcedda
2013-02-22 10:10 ` [PATCH v2 next 1/2] defenv2: comment setting default values in /env/config Fabio Porcedda
2013-02-22 10:54   ` Fabio Porcedda [this message]
2013-02-23 13:26   ` Alexander Aring
2013-02-23 14:20     ` Fabio Porcedda
2013-02-23 14:37     ` Sascha Hauer
2013-02-23 16:47       ` Alexander Aring
2013-02-25 11:22       ` Fabio Porcedda
2013-02-22 10:10 ` [PATCH v2 next 2/2] ARM at91 telit-evk-pro3: move config-board out of /env/init Fabio Porcedda
2013-02-23 13:01 ` [PATCH v2 next 0/2] defenv2: complete " Sascha Hauer
2013-02-23 14:25   ` Fabio Porcedda

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAHkwnC_a0-C0oAwP33u8xjKpORB=JC4zDhqUDw+OMmdRd8S4Mw@mail.gmail.com' \
    --to=fabio.porcedda@gmail.com \
    --cc=barebox@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox