Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Randy Witt <randy.e.witt@linux.intel.com>
To: Bruno Bottazzini <bruno.bottazzini@intel.com>,
	 openembedded-core@lists.openembedded.org
Cc: Darren Hart <dvhart@linux.intel.com>
Subject: Re: [PATCH] mkefidisk: change filesystem to be writeable on grub
Date: Thu, 26 Feb 2015 13:44:17 -0800	[thread overview]
Message-ID: <54EF93B1.2020902@linux.intel.com> (raw)
In-Reply-To: <1424978954-25411-1-git-send-email-bruno.bottazzini@intel.com>

On 02/26/2015 11:29 AM, Bruno Bottazzini wrote:
> This patch changes grub configuration so the filesystem will be writeable
> instead, just read only when running the script.
> ---
>   scripts/contrib/mkefidisk.sh | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> index b96b7d4..9b84831 100755
> --- a/scripts/contrib/mkefidisk.sh
> +++ b/scripts/contrib/mkefidisk.sh
> @@ -342,7 +342,7 @@ if [ -e "$GRUB_CFG" ]; then
>   	sed -i "s/ LABEL=[^ ]*/ /" $GRUB_CFG
>
>   	sed -i "s@ root=[^ ]*@ @" $GRUB_CFG
> -	sed -i "s@vmlinuz @vmlinuz root=$TARGET_ROOTFS ro rootwait quiet @" $GRUB_CFG
> +	sed -i "s@vmlinuz @vmlinuz root=$TARGET_ROOTFS rw rootwait quiet @" $GRUB_CFG
>   fi
>
>   # Look for a gummiboot installation
>

This change basically reverts 
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=24cd3ddeb70a3d3f2985ec71c87ae0108f3d2777 
by Darren. It also doesn't change the ro option when using gummiboot instead.

What is this change trying to fix?


  reply	other threads:[~2015-02-26 21:44 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bruno.bottazzini@intel.com>
2015-01-21 20:00 ` [PATCH 0/2] kmod new packageconfig and dbus split package Bruno Bottazzini
2015-01-21 20:00   ` [PATCH 1/2] kmod: new PACKAGECONFIG debug and logging to help reduce binary size Bruno Bottazzini
2015-01-21 20:00   ` [PATCH 2/2] dbus: split tools package Bruno Bottazzini
2015-01-28 15:47   ` [PATCH 0/2] kmod new packageconfig and dbus split package Bottazzini, Bruno
2015-01-28 16:00     ` Burton, Ross
2015-01-28 18:55       ` Dan McGregor
2015-01-28 19:57         ` Burton, Ross
2015-02-02 12:39           ` Bottazzini, Bruno
2015-02-03 18:21 ` [PATCH] systemd: update version from 216 to 218 Bruno Bottazzini
2015-02-03 18:21   ` Bruno Bottazzini
2015-02-03 21:23     ` Randy Witt
2015-02-03 21:33       ` Bottazzini, Bruno
2015-02-04 11:48     ` Enrico Scholz
2015-02-04 17:04       ` Bottazzini, Bruno
2015-02-04 17:59         ` Enrico Scholz
2015-02-04 17:04 ` [PATCH v2 0/2] " Bruno Bottazzini
2015-02-04 17:04   ` [PATCH v2 1/2] systemd: update " Bruno Bottazzini
2015-02-04 17:04   ` [PATCH v2 2/2] systemd: split modules into packages Bruno Bottazzini
2015-02-06 18:45     ` Randy Witt
2015-03-02 20:21       ` Bottazzini, Bruno
2015-03-04  9:22         ` Anders Darander
2015-02-26 19:29 ` [PATCH] mkefidisk: change filesystem to be writeable on grub Bruno Bottazzini
2015-02-26 21:44   ` Randy Witt [this message]
2015-02-26 22:21     ` Bottazzini, Bruno
2015-02-27  1:05       ` randy.e.witt
2015-03-03 10:26         ` Patrick Ohly
2015-03-04 21:27 ` [PATCH 0/3] systemd and dbus split into packages Bruno Bottazzini
2015-03-04 21:27   ` [PATCH 1/3] dbus: split tools package Bruno Bottazzini
2015-03-05 11:01     ` Andreas Oberritter
2015-03-05 13:16       ` Bottazzini, Bruno
2015-03-05 14:09         ` Andreas Oberritter
2015-03-05 14:29           ` Bottazzini, Bruno
2015-03-05 14:49             ` Andreas Oberritter
2015-03-05 16:05               ` Bottazzini, Bruno
2015-03-04 21:27   ` [PATCH 2/3] systemd: verify if files test-udev and systemd-journal-flush exists Bruno Bottazzini
2015-03-04 21:27   ` [PATCH 3/3] systemd: split modules into packages Bruno Bottazzini
2015-03-05 14:28     ` Anders Darander
2015-03-05 16:14       ` Bottazzini, Bruno
2015-03-06  8:23         ` Anders Darander
2015-03-06 14:03           ` Bottazzini, Bruno
2015-03-06 22:51             ` Khem Raj
2015-03-09 13:37               ` Bottazzini, Bruno
2015-03-09 15:32                 ` Khem Raj
2015-03-09 16:08                   ` Bottazzini, Bruno
2015-03-25 21:49 ` [PATCH v3 0/4] systemd and dbus split " Bruno Bottazzini
2015-03-25 21:49   ` [PATCH 1/4] dbus: split tools package Bruno Bottazzini
2015-03-25 21:49   ` [PATCH 2/4] systemd: verify if files test-udev and systemd-journal-flush exists Bruno Bottazzini
2015-03-25 21:49   ` [PATCH 3/4] systemd: split modules into packages Bruno Bottazzini
2015-03-26  8:29     ` Anders Darander
2015-03-26 11:56       ` Otavio Salvador
2015-03-26 13:40         ` Bottazzini, Bruno
2015-03-26 13:43           ` Otavio Salvador
2015-03-26 13:53             ` Bottazzini, Bruno
2015-03-26 13:59               ` Otavio Salvador
2015-03-27 14:57           ` Anders Darander
2015-03-27 17:09             ` Bottazzini, Bruno
2015-03-27 17:11               ` Otavio Salvador
2015-03-27 17:13                 ` Bottazzini, Bruno
2015-03-26 17:25         ` Khem Raj
2015-03-26  8:52     ` Andreas Oberritter
2015-03-26 13:33       ` Bottazzini, Bruno
2015-03-26 13:38         ` Otavio Salvador
2015-03-25 21:49   ` [PATCH 4/4] systemd: v219 with stable fixes Bruno Bottazzini
2015-03-30 13:05     ` Bottazzini, Bruno
2015-03-30 13:40       ` Otavio Salvador
2015-03-30 17:42         ` Bottazzini, Bruno
2015-04-01 10:16     ` Jussi Kukkonen
2015-04-01 11:21       ` Anders Darander
2015-04-01 12:52       ` Bottazzini, Bruno
2015-04-15 15:01         ` Burton, Ross
2015-04-15 15:37           ` Bottazzini, Bruno
2015-04-15 15:53             ` Burton, Ross
2015-04-15 16:29               ` Bottazzini, Bruno

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=54EF93B1.2020902@linux.intel.com \
    --to=randy.e.witt@linux.intel.com \
    --cc=bruno.bottazzini@intel.com \
    --cc=dvhart@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.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