[OE-core] [PATCH][resend] runqemu: Add option for custom BIOS directory
Ricardo Neri
ricardo.neri-calderon at linux.intel.com
Tue Mar 11 03:14:18 UTC 2014
On Sun, 2014-03-09 at 00:09 -0800, Saul Wold wrote:
> On 03/07/2014 04:30 PM, Ricardo Neri wrote:
> > Add support to specify a directory for custom BIOS, VGA BIOS and
> > keymaps as supported by qemu (-L option). Even though this can be
> > done through qemuparams, having this option provides better user
> > experience by not having to specify a long and cluttered path along
> > other qemuparams that the user might want to specify.
> >
> > This new options assumes that the path provided is relative to
> > OECORE_NATIVE_SYSROOT and will check whether it exists before proceeding.
> >
> > Signed-off-by: Ricardo Neri <ricardo.neri-calderon at linux.intel.com>
> > ---
> > scripts/runqemu | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/scripts/runqemu b/scripts/runqemu
> > index 619ffb6..9eff90d 100755
> > --- a/scripts/runqemu
> > +++ b/scripts/runqemu
> > @@ -149,6 +149,9 @@ while true; do
> > SCRIPT_KERNEL_OPT="$SCRIPT_KERNEL_OPT console=ttyS0"
> > SERIALSTDIO="1"
> > ;;
> > + "biosdir="*)
> > + CUSTOMBIOSDIR="${arg##biosdir=}"
> > + ;;
> > "qemuparams="*)
> > SCRIPT_QEMU_EXTRA_OPT="${arg##qemuparams=}"
> >
> > @@ -489,5 +492,16 @@ if [ ! -f "$INTERNAL_SCRIPT" -o ! -r "$INTERNAL_SCRIPT" ]; then
> > INTERNAL_SCRIPT=`which runqemu-internal`
> > fi
> >
> > +# Specify directory for BIOS, VGA BIOS and keymaps
> > +if [ ! -z "$CUSTOMBIOSDIR" ]; then
> > + if [ -d "$OECORE_NATIVE_SYSROOT/$CUSTOMBIOSDIR" ]; then
> I don't think we should be assuming anything here, if the direcotry is
> passed then it should be given that it is an accurate path to the
> directory that contains the customer bios, don't make people second gess
> that it needs to be in the native sysroot.
>
> Sau!
Thanks for your comments Saul! Are you saying that providing paths to
items outside the build environment is a supported use case? That is the
reason I made it relative to OECORE_NATIVE_SYSROOT. If absolute paths to
anywhere can be provided, I could first check whether the provided
absolute path exist and, if not, assume that it is relative to
OECORE_NATIVE_SYSROOT. Does that make sense to you?
BR,
Ricardo
>
> > + echo "Assuming $CUSTOMBIOSDIR really means $OECORE_NATIVE_SYSROOT/$CUSTOMBIOSDIR"
> > + SCRIPT_QEMU_OPT="$SCRIPT_QEMU_OPT -L $OECORE_NATIVE_SYSROOT/$CUSTOMBIOSDIR"
> > + else
> > + echo "Custom BIOS directory $OECORE_NATIVE_SYSROOT/$CUSTOMBIOSDIR not found."
> > + exit 1;
> > + fi
> > +fi
> > +
> > . $INTERNAL_SCRIPT
> > exit $?
> >
More information about the Openembedded-core
mailing list