[Quilt-dev] quilt push and globbing

[Jean Delvare]

Hi all,

I think I hit a minor bug in quilt push. When I try to push a patch with
matching square brackets (like "[PATCH]_something.patch"), quilt push
complains that the patch seems to be empty (but works otherwise).

I have narrowed the problem down to these lines in push:

        if [ "$(shopt -s nullglob ; echo $QUILT_PC/$patch/*)" = "" ]
        then
                printf $"Patch %s appears to be empty, applied\n" \
                       "$(print_patch $patch)"
        elif...

I think that the nullglob applies to $patch here, while it was really
only intended for *, and this is the problem. Thus we probably should
change this line to:

        if [ "$(shopt -s nullglob ; echo "$QUILT_PC/$patch"/*)" = "" ]

i.e. protect the variables with double quotes. I'm not exactly sure it's
syntaxically correct, as we are already inside double quotes, and my
editor won't colorize it properly. However it seems to work just fine
for me. Comments?

Patch against CVS attached, if it happens to be the correct fix.

Thanks,
-- 
Jean Delvare

quilt-CVS-push-nullglob.diff
Description: Text document