Mason [off-list ref] writes:
Thanks! At least now, I see the light at the end of the tunnel.
I fetched linux-stable.git inside our repo.
I created ~300 patches using git format-patch -1 in a loop.
I can now run 'git am --3way $IGNORE *.patch'
IGNORE is used to --exclude the directories I'm not interested in.
Note: it seems --exclude=arch/mips and --exclude=arch/mips/ are
not sufficient, I need to write --exclude=arch/mips/* for git-apply
to ignore changes to files inside arch/mips.
Is that expected behavior?
I have no idea; at least to me, "--exclude" option to "git apply"
was invented to name individual paths, not patterns, and I wouldn't
be surprised if glob working were merely by accident not by design.
Another nit: if a patch contains only changes to files inside arch/mips
then git-apply will create an "empty commit" (one with no diff). Is there
an option to say "skip empty patches"?
"git am --skip" perhaps?
"git am" may pass the "--exclude/--include" options to "git apply"
but I wouldn't be surprised if that support was added without
thinking. Perhaps the reason why you discovered that it needed a
lot more thinking to properly integrate these options to "git am"
only now is because hardly anybody uses it ;-). Not just passing
these options, the code in "git am" to react to the result of patch
application to avoid the issue you observed when these options are
passed need to be adjusted by changes that started passing them, but
I do not think they did, cf. 77e9e496 (am: pass exclude down to
apply, 2011-08-03).
One more thing: "regular" diff -q returns 0 when the files are identical,
and 1 when they differ. It seems git diff -s does not have that behavior.
Is that by design?
"diff -s" may be accepted but it is an idiotic thing for a user to
say. The "-s" option is to squelch output from "log" and friends,
and it is exposed to "diff" only because these two families of
commands share the command line parser.
Junio C Hamano wrote:
Mason wrote:
quoted
I fetched linux-stable.git inside our repo.
I created ~300 patches using git format-patch -1 in a loop.
I can now run 'git am --3way $IGNORE *.patch'
IGNORE is used to --exclude the directories I'm not interested in.
Note: it seems --exclude=arch/mips and --exclude=arch/mips/ are
not sufficient, I need to write --exclude=arch/mips/* for git-apply
to ignore changes to files inside arch/mips.
Is that expected behavior?
I have no idea; at least to me, "--exclude" option to "git apply"
was invented to name individual paths, not patterns, and I wouldn't
be surprised if glob working were merely by accident not by design.
According to the docs, IIUC globbing is by design:
http://git-scm.com/docs/git-apply
--exclude=<path-pattern>
Don't apply changes to files matching the given path pattern.
This can be useful when importing patchsets, where you want to
exclude certain files or directories.
This lead me to believe that --exclude=arch/mips should work.
quoted
Another nit: if a patch contains only changes to files inside arch/mips
then git-apply will create an "empty commit" (one with no diff). Is there
an option to say "skip empty patches"?
"git am --skip" perhaps?
Nah, as the doc states, --skip "is only meaningful when restarting
an aborted patch."
"git am" may pass the "--exclude/--include" options to "git apply"
but I wouldn't be surprised if that support was added without
thinking. Perhaps the reason why you discovered that it needed a
lot more thinking to properly integrate these options to "git am"
only now is because hardly anybody uses it ;-).
I do have a rather obscure use-case (and if I do it right, I will
never have to do it again).
Not just passing
these options, the code in "git am" to react to the result of patch
application to avoid the issue you observed when these options are
passed need to be adjusted by changes that started passing them, but
I do not think they did, cf. 77e9e496 (am: pass exclude down to
apply, 2011-08-03).
Sorry, I could not parse that paragraph :-)
quoted
One more thing: "regular" diff -q returns 0 when the files are identical,
and 1 when they differ. It seems git diff -s does not have that behavior.
Is that by design?
"diff -s" may be accepted but it is an idiotic thing for a user to
say. The "-s" option is to squelch output from "log" and friends,
and it is exposed to "diff" only because these two families of
commands share the command line parser.
Here is the use-case:
if diff -q A B; then do_X; else do_Y; fi
It makes sense to prevent diff from writing to stdout.
I was planning to write 'git diff -q commit^ commit'
to test for empty commits. Looks like I'll be needing
'git diff commit^ commit | wc -l' instead?
Regards.
Mason wrote:
I was planning to write 'git diff -q commit^ commit'
to test for empty commits. Looks like I'll be needing
'git diff commit^ commit | wc -l' instead?
I wrote a script to generate the list of empty commits.
git log --format="%h" --reverse 413cb08.. | while read H
do
if git diff --quiet $H^ $H; then echo $H; fi
done >empty_commits
But it turns out all this is unnecessary, as git-rebase
will automatically filter empty commits! :-)
Unless given the --keep-empty option, I presume.
Thanks to everyone involved in making this great tool.
Regards.