Am 06.05.2010 12:52, schrieb Stefan Sperling:
[...]
thank you for your comments.
Unfortunately, I've found another problem in my patch:
# cut -c 42-: svn status lists different information. The filename
starts at column 42
is not even vaild for all Linux systems.
As you recommended in your first email, the new patch just checks for
errors and skip the remaining calls.
Your are right, when you point out, that there are still a lot of cases
where this small wrapper script fails. But at least, this patch reduces
the number of problems by one.
New patch desciption:
[[[
reduces error messages by checking svn return code
* contrib/client-side/asvn
(recordpermissions): skip files where "svn propget" returns an error.
It is assumed that these files do not belong to the svn repository
patch by joerg.steffens < at > dass-it.de
]]]
Index: contrib/client-side/asvn
===================================================================
--- contrib/client-side/asvn (Revision 941610)
+++ contrib/client-side/asvn (Arbeitskopie)
@@ -335,7 +335,11 @@
fi
# see if the properties have changed.
- if [ "`$SVN propget $FILE_PROP \"$dir/$file\"`" != "$info" ]
+ FILE_PROP_VALUE=`$SVN propget $FILE_PROP "$dir/$file" 2>/dev/null`
+ if [ $? -ne 0 ]; then
+ # $file is not handled by svn, skipping
+ echo "skipping property $FILE_PROP for $dir/$file"
+ elif [ "$FILE_PROP_VALUE" != "$info" ]
then
if [ "$CHECKIN" = "true" ]
then
