apatch/create: don't call cpfile on directories. Seems to create a valid patch, but I wasn't able to test any of the other apatch/ commands. This patch should be a good test case: it was created by running the new apatch/create on /rc/bin/apatch. Will the `apatch' entry in `files', with no corresponding directory entry, break things? Notes: good idea, but i'm going to have to sorry this patch because 0-size files need to be legal. i think you mean -f? - erik Reference: /n/atom/patch/sorry/apatch-recursdir Date: Fri Feb 21 05:46:19 CET 2014 Signed-off-by: root@davidrhoskin.com Reviewed-by: quanstro --- /rc/bin/apatch/create Fri Feb 21 05:32:18 2014 +++ /rc/bin/apatch/create Fri Feb 21 05:32:19 2014 @@ -158,8 +158,8 @@ file=$uniqlist(1) base=$uniqlist(2) uniqlist = $uniqlist(3-) - test -e $SD/$file && cpfile $SD/$file $d/$base.orig - cpfile $file $d/$base || fail cpfile # we need to back out + test -s $SD/$file && cpfile $SD/$file $d/$base.orig + test -s $file && { cpfile $file $d/$base || fail cpfile } # we need to back out }