[PATCH] xfstests: fix specifying configuration value with equality sign
Dave Chinner
david at fromorbit.com
Thu Apr 10 19:14:10 CDT 2014
On Thu, Apr 10, 2014 at 11:55:37AM +0200, Lukas Czerner wrote:
> Currently there is a problem with parse_config_section() when the
> configuration value contains equality sign like this for example:
>
> MOUNT_OPTIONS="-o data=journal"
>
> the result will be
>
> export MOUNT_OPTIONS="-o data="journal"
>
> which is not going to work. The reason is that the expression used to
> parse the configuration options uses greedy matching '.*'. Fix this by
> using non greedy expression to match the first equality sign '[^=]'.
>
> Signed-off-by: Lukas Czerner <lczerner at redhat.com>
Looks fine.
Reviewed-by: Dave Chinner <dchinner at redhat.com>
It would probably be a good idea to comment what the sed line noise
is actually doing, so it's easier to understand in future...
Cheers,
Dave.
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list