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

Re: svn_load_dirs.pl: "(in cleanup) Can't return to /tmp/svn_load_dirs_v9leeQUQpQ/my_import_wc"

From: Ryan Schmidt <subversion-2007a_at_ryandesign.com>
Date: 2007-01-12 19:02:28 CET

On Jan 12, 2007, at 03:50, Jonathan Perret wrote:

> On 1/10/07, Jeff Smith wrote:
>
>> My first time trying this `svn_load_dirs.pl`, which did not seem to
>> complete. Could the script be having trouble because some of
>> the .c, .h, etc. have mixed line-end style? That is sure what ruined
>> my attempt earlier to import with auto-properties.
>
> FWIW, when I used svn_load_dirs.pl I ripped out all the code that was
> mucking with line-endings. I'd rather have my files not mucked with at
> all, thankyouverymuch.
>
>> Running /usr/local/bin/svn update
>> Cleaning up /tmp/svn_load_dirs_v9leeQUQpQ
>> (in cleanup) Can't return
>> to /tmp/svn_load_dirs_v9leeQUQpQ/my_import_wc
>> from /tmp/svn_load_dirs_v9leeQUQpQ (No such file or directory)
>> at /usr/local/bin/svn_load_dirs.pl line 1958
>
>
> The "Can't return..." error in cleanup is benign. I fixed it here by
> adding the following line :
> chdir($orig_cwd);
> just after the "print "Cleaning up $temp_dir\n";" line around the end
> of the file. Basically the script tries to delete the directory it is
> currently in.
>
> The error you should be concerned about is the one that led to cleanup
> just after
> the update. Looking again at my local patches, another thing I changed
> was the line
> open(SAFE_READ, "@commandline |")
> to
> open(SAFE_READ, "@commandline 2>&1 |")
> in sub safe_read_from_pipe (around line 1481). I seem to remember I
> had problems with error reporting before this change but it may be
> unrelated to your issue.
>
> Actually, looking through the source, it looks like it is normal that
> the script should end just after an svn update, when you did not ask
> for the creation of a tag.
>
> Try perhaps requesting a tag with the -t option, because it
> additionally enables a consistency check.

Why do you have all these local patches to svn_load_dirs.pl? Might
your changes not be of benefit to everyone? Especially if the script
produces errors in some circumstances, and you've fixed those
problems, please contribute your patches back to the community.

-- 
To reply to the mailing list, please use your mailer's Reply To All  
function
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Fri Jan 12 19:03:28 2007

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