-
Bug
-
Resolution: Unresolved
-
Minor
-
None
Ok, so you can see my attempt to fix this issue here:
https://github.com/jenkinsci/cvs-plugin/commit/d0c8df0fa8c0f35c92b43e46d7ac59d21ef926db
I have since backed it out:
https://github.com/jenkinsci/cvs-plugin/commit/7c5f8e917274ae09159190d8294a572d5904659d
But I actually believe that my original change should be OK, except that
Assumes you never want to issue a cvs command like:
cvs update -C -d -P -r BRANCH_NAME -D "14 Oct 2014 07:32:34 -0400" ...
Which seems nuts as I can think of loads of cases where I would want to check out a specific revision of a named branch... (but then I am no expert in CVS so perhaps that is not something you can do!!! )
CheckoutCommand does not seem to have the same logic:
Anyway, in any case I am throwing up my hands and saying " I give up! " over to the plugin maintainer to see about fixing this one!
- links to
I cannot now remember whether you can mix -D and -r. It is possible one supersedes the other.
http://stackoverflow.com/questions/3300471/cvs-get-the-state-of-some-branch-in-some-date is confusing since the querier states in the question that mixing them does not work, but then accepts an answer stating that it does work, with essentially the same options.
I could not find a public CVS repository that uses branches at all.