[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[Issue 2798] svn_load_dirs.pl: Set "svn:executable" only for newly added (executable) files.

From: Daniel Rall <dlr_at_collab.net>
Date: 2007-06-22 21:33:43 CEST

Mark wrote:

> svn_load_dirs.pl will set "svn:executable" on any executable plain
> file, even those that have not changed (MD5). This triggers the file
> to be processed as an addition to the repository and copied from the
> import directory into the staging working copy. This behavior seem
> unintentional (or at least inconsistent) since all other properties
> (via the -p option) are set only for newly added files. More
> importantly, it will trigger a failure if the target in the staging
> working copy is read-only. (That issue coming in another post.)

Mark, I don't know a lot about this script, but what about the "if"
case corresponding to the "else" block that you moved the "set
executable" code snippet into -- in the following sub-conditional:

                if ($new_digest ne $del_digest)
                  {
                    print "U $source_path\n";
                    $upd_files{$source_path} = $del_info;
                  }

...wouldn't we also want to set the "svn:executable" property?

  • application/pgp-signature attachment: stored
Received on Fri Jun 22 21:33:31 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.