possible bug in svn:externals with fixed revision

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

possible bug in svn:externals with fixed revision

Martin Ankerl
I am using hudson 1.100, and there is a project that is constantly
rebuilding. I believe the cause for this is an svn:external, we have
defined it like this:

svn:externals extern/utils -r 15934 https://somewhere/lib/javautils/trunk/src

The subversion poll log is this:

Started on 13.04.2007 11:55:16
Revision:20395
Revision:19818
Done. Took 0 seconds
Changes found

Interestingly, the HEAD of the external is at exactly 19818. I am
guessing that hudson sees that it has used revision 15934, but the
current revision is 19818, so it decides that changes have happend and
starts to rebuild even though we have explicitly defined that we want
revision 15934.

--
Martin Ankerl | http://martin.ankerl.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: possible bug in svn:externals with fixed revision

Renaud Bruyeron-2



Martin Ankerl wrote:
> I am using hudson 1.100, and there is a project that is constantly
> rebuilding. I believe the cause for this is an svn:external, we have
> defined it like this:
>
> svn:externals extern/utils -r 15934
> https://somewhere/lib/javautils/trunk/src
Yes indeed, this type of externals is not supported in the code as it is.
I think it would be tricky to support this, unless svnkit reports the
revision the externals is pegged on. If svnkit does not, it would
require hudson to go find the svn:externals property and parse...

Can't you tag this revision 15934 and then point the external at the
tag? This is how I would work around this.

 - Renaud

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: possible bug in svn:externals with fixed revision

Martin Ankerl
> > svn:externals extern/utils -r 15934
> > https://somewhere/lib/javautils/trunk/src
> Yes indeed, this type of externals is not supported in the code as it is.

Ah, using a tag is an excellent idea; thanks for that.

--
Martin Ankerl | http://martin.ankerl.com

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: possible bug in svn:externals with fixed revision

Kohsuke Kawaguchi
Administrator
In reply to this post by Renaud Bruyeron-2
Renaud Bruyeron wrote:
> Yes indeed, this type of externals is not supported in the code as it is.
> I think it would be tricky to support this, unless svnkit reports the
> revision the externals is pegged on. If svnkit does not, it would
> require hudson to go find the svn:externals property and parse...

So far the SVNKit guys have been fairly responsive to the issues that I
filed. Would it be possible for you to report the problem to them so
that we can get this fixed eventually?

--
Kohsuke Kawaguchi
Sun Microsystems                   [hidden email]

smime.p7s (4K) Download Attachment