Re: [PATCH] Parse commandline in grub-xen

[Vladimir 'phcoder' Serbinenko]

On May 11, 2015 1:52 PM, "Olaf Hering" <address@hidden> wrote:
>
> On Mon, May 11, Vladimir 'phcoder' Serbinenko wrote:
>
> >
> > On May 11, 2015 1:23 PM, "Olaf Hering" <address@hidden> wrote:
> > >
> > > On Mon, May 11, Vladimir 'phcoder' Serbinenko wrote:
> > >
> > > > Do you really own all those installs to be able to speak on behalf of all
> > of
> > > > them?
> > >
> > > Since there is zero upstream support for anything regarding grub xen
> > > distros are forced to provide their own grub-xen binary for dom0. This
> > > includes at least some script to do something useful within the very
> > > first grub-xen. So in this sense its up to whoever creates such dom0
> > > binary to handle variables as needed, and document the implemented
> > > features.
> > >
> > And your point is?
>
> That this patch gets in as is?
>
Well your argument doesn't say at all why no prefix is better than separating namespaces
> How do you envison a way to select a boot device, or set debug=all or
> whatever a script may interpret?
>
Setting debug is a good example of abuse that would be sneaking in. Variables set from command line need to have a prefix to indicate where they come from.
> Olaf
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/grub-devel