Re: [PATCH] boot parameters: quoting of environment variablesrevisited

From: Andrew Morton
Date: Tue Oct 19 2004 - 20:20:50 EST


Werner Almesberger <werner@xxxxxxxxxxxxxxx> wrote:
>
> When passing boot parameters, they can be quoted as follows:
> param="value"
>
> Unfortunately, when passing environment variables this way, the
> quoting causes confusion: in 2.6.7 (etc.), only the variable name
> was placed in the environment, which caused it to be ignored.

Bummer.

> I've sent a patch that adjusted the name, but this patch was
> dropped. Instead, apparently a different fix was attempted in
> 2.6.9, but this now yields param="value in the environment (note
> the embeded double quote), which isn't much better.

Yes, Len's patch purported to fix the same thing. I should have got you to
review&test that change. Didn't think of it, sorry.

> I've attached a patch for 2.6.9 that fixes this. This time, I'm
> shifting the value. Maybe you like it better this way :-)

hm. The environment string handling and the "command line" string handling
appear to be identical in there. How come only one of them has the
problem? That function makes my eyes bleed.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/