RE: Re: svncopy.pl: option --file

Understand it!

Then, I can say I found a bug.

First, whenever you execute:

"svncopy.pl -F comment.txt --pin-externals url/to/source url/to/destination"

It raises the error:

Subcommand 'checkout' doesn't accept '-F [--file] PAR' option.

As I stated before. Because it is parsed (line 95) and included like any other svn option, the problem is that the first svn subcommand to execute is 'checkout' that it does not admit -F[--file] PAR option.

Secondly, as you told me, --file PAR option is internally used by 'svn propset'. But it does nothing to do with the -F[--file] PAR option that receives svncopy.pl script.

Therefore, the code should be changed according to one of the following options:

  a) Just ignore this option (-F[--file] PAR). And thefore update the documentation indicating this. Because, at first sight and without changing anything, I understand that I can use -F[--file] PAR option as a comment like in any other svn subcommand that admits it. For example, creating a tag, this file could be the followed roadmap (that contains newlines) to get that version/tag.

Following your instructions I understand that it shouldn't be used to add a comment. I should use -m MSG option instead.

  b) Just using it like -m MSG option (but not both at the same time, like you told me). And therefore, change the code as I described previously (line 95).

I hope you understand my explanation.

Thanks for your feedback.

Darío.

> dario.borreguero_at_gmail.com wrote on Fri, 12 Dec 2008 at 03:47 -0800:
> > Hi,
> >
> > I'm not sure if this should be a bug. I'll explain it below:
> >
> > I've started to use 'svncopy.pl' script and I found that if I try to
> > add comments from a file, as I did with 'svn copy' subcommmand. It
> > doesn't work.
> >
> > I returns the error message: Subcommand 'checkout' doesn't accept '-F
> > [--file] PAR' option.
> >
>
> Then don't pass -F to 'svn checkout'. (And "PAR" isn't part of our
> output, where did it come from?)
>
> > I wonder why it tries to insert that file as an option for all the svn
> > subcommand that are used within svncopy.pl script.
> >
> > I've looked into the code and I finally decided to change the line 95
> > after checking that 'svn copy -m "Message" --file messages.txt'
> > ignores the -m option and uses the file for comments.
> >
>
> No. If you specify the log message using both -m and -F, you get
> *undefined behaviour*. Don't do this.
>
> > Therefore,
> >
> > [95] "file|F=s" => sub { push(@svn_options, "--file", $_[1]) },
> >
> > to
> >
> > [95] "file|F=s" => sub { open FILE, $_[1]; my @content = <FILE>;
> > $message = join("\n", @content); close FILE; },
> >
> > Before assuming that my change is correct or it shouldn't be done.
> > I wonder if the option --file PAR is used with another intention than
> > to use it as commit comment??
> >
>
> Yes, by 'svn propset' at least.
>
> > I hope this helps,
> >
> > Dario.
> >
> > ------------------------------------------------------
> > http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=983363
> >

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=993092
Received on 2008-12-26 20:48:56 CET