Thanks for confirming the fix.
For the record, r1211582 did fix the issue you identified, but there was
a separate issue in the rolling of the nightly tarballs --- which
r1212350 fixes.
Shane Turner wrote on Fri, Dec 09, 2011 at 10:22:20 -0400:
> I didn't look carefully enough at the revision. I expected it to be
> corrected based on your comment that you expected it to be fixed in
> the next nightly build.
>
> In any case, it does appear to be fixed now in the nightly tar files
> found in
> http://ci.apache.org/projects/subversion/nightlies/dist/r1212357/
>
>
> Thanks for the fix, Daniel.
>
> $ grep SVN_VER_REVISION
> subversion-{1.7.1,1.7.2,nightly}/subversion/include/svn_version.h
> subversion-1.7.1/subversion/include/svn_version.h:#define
> SVN_VER_REVISION 1186859
> subversion-1.7.2/subversion/include/svn_version.h:#define
> SVN_VER_REVISION 0
> subversion-nightly/subversion/include/svn_version.h:#define
> SVN_VER_REVISION 1212357
>
> $ echo "1.7.1"; find subversion-1.7.1 -type f | xargs egrep 1186859
> ; echo; echo "NIGHTLY"; find subversion-nightly -type f | xargs
> egrep 1212357
> 1.7.1
> subversion-1.7.1/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_FromCharPtr(" (r1186859)"));
> subversion-1.7.1/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_From_int SWIG_PERL_CALL_ARGS_1((int)(1186859)));
> subversion-1.7.1/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_FromCharPtr("1.7.1 (r1186859)"));
> subversion-1.7.1/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VER_TAG", SWIG_FromCharPtr("
> (r1186859)"));
> subversion-1.7.1/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VER_REVISION",
> SWIG_From_int((int)(1186859)));
> subversion-1.7.1/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VERSION", SWIG_FromCharPtr("1.7.1
> (r1186859)"));
> subversion-1.7.1/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d, "SVN_VER_TAG",SWIG_FromCharPtr("
> (r1186859)"));
> subversion-1.7.1/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d,
> "SVN_VER_REVISION",SWIG_From_long((long)(1186859)));
> subversion-1.7.1/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d, "SVN_VERSION",SWIG_FromCharPtr("1.7.1
> (r1186859)"));
> subversion-1.7.1/subversion/include/svn_version.h:#define
> SVN_VER_TAG " (r1186859)"
> subversion-1.7.1/subversion/include/svn_version.h:#define
> SVN_VER_REVISION 1186859
>
> NIGHTLY
> subversion-nightly/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_FromCharPtr(" (r1212357)"));
> subversion-nightly/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_From_int SWIG_PERL_CALL_ARGS_1((int)(1212357)));
> subversion-nightly/subversion/bindings/swig/perl/native/core.c:
> sv_setsv(sv, SWIG_FromCharPtr("1.8.0 (r1212357)"));
> subversion-nightly/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VER_TAG", SWIG_FromCharPtr("
> (r1212357)"));
> subversion-nightly/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VER_REVISION",
> SWIG_From_int((int)(1212357)));
> subversion-nightly/subversion/bindings/swig/ruby/core.c:
> rb_define_const(mCore, "SVN_VERSION", SWIG_FromCharPtr("1.8.0
> (r1212357)"));
> subversion-nightly/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d, "SVN_VER_TAG",SWIG_FromCharPtr("
> (r1212357)"));
> subversion-nightly/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d,
> "SVN_VER_REVISION",SWIG_From_long((long)(1212357)));
> subversion-nightly/subversion/bindings/swig/python/core.c:
> SWIG_Python_SetConstant(d, "SVN_VERSION",SWIG_FromCharPtr("1.8.0
> (r1212357)"));
> subversion-nightly/subversion/include/svn_version.h:#define
> SVN_VER_TAG " (r1212357)"
> subversion-nightly/subversion/include/svn_version.h:#define
> SVN_VER_REVISION 1212357
>
> Shane
>
> On December-08-11 2:52:31 PM, Daniel Shahaf wrote:
> >Thanks for checking, Shane. I note that you checked r1211287, which is
> >prior to my yesterday's fix (r1211582), so the bogus values are expected.
> >
> >Shane Turner wrote on Thu, Dec 08, 2011 at 14:39:03 -0400:
> >>Looks like the problem is still there:
> >>
> >>$ grep SVN_VER_REVISION
> >>subversion-{1.7.1,1.7.2,nightly}/subversion/include/svn_version.h
> >>subversion-1.7.1/subversion/include/svn_version.h:#define
> >>SVN_VER_REVISION 1186859
> >>subversion-1.7.2/subversion/include/svn_version.h:#define
> >>SVN_VER_REVISION 0
> >>subversion-nightly/subversion/include/svn_version.h:#define
> >>SVN_VER_REVISION 0
> >>
> >>The nightly is http://ci.apache.org/projects/subversion/nightlies/dist/r1211287/subversion-nightly.tar.bz2
> >>
> >>Shane
> >>
> >>On 07/12/2011 2:35 PM, Daniel Shahaf wrote:
> >>>r1211582 should make things better.
> >>>
> >>>I think the nightly releases were also affected by this bug, in which
> >>>case tonight's nightly roll should confirm the fix.
> >>>
> >>>http://subversion.apache.org/source-code.html#nightlies
> >>>
> >>>On Wed, Dec 7, 2011, at 19:56, Daniel Shahaf wrote:
> >>>>On Wed, Dec 7, 2011, at 19:53, Daniel Shahaf wrote:
> >>>>>On Wed, Dec 7, 2011, at 11:43, Hyrum K Wright wrote:
> >>>>>>On Wed, Dec 7, 2011 at 11:29 AM, Stefan Sperling<stsp_at_elego.de> wrote:
> >>>>>>>On Wed, Dec 07, 2011 at 11:23:30AM -0400, Shane Turner wrote:
> >>>>>>>>Should I open a bug report to have the packages regenerated,
> >>>>>>>No. Releases are never regenerated. That would invalidate signatures
> >>>>>>>developers sent for the release.[*]
> >>>>>>>
> >>>>>>>We'll have to figure out the source of the problem and then try
> >>>>>>>to avoid it in future releases.
> >>>>>>It may be related to the sed problem we were having in the 1.7 RC
> >>>>>>series. Apparently one of scripts relied upon GNU sed which wasn't
> >>>>>>installed on people.apache.org. I had been using a custom install of
> >>>>>>it, but I thought Daniel had fixed the offending script to not require
> >>>>>>GNU sed. 1.7.2 represents the first release in which I relied upon
> >>>>>>the system sed, and not my custom one.
> >>>>>>
> >>>>>r1159741
> >>>>>
> >>>>>However, SVN_VER_REVISION on the 1.7.2 tag is wrong.
> >>>>... but SVN_VER_TAG, which incorporates the revnum, is right.
> >>>>
> >>>>Hmm.
> >>>>
> >>
> >>--
> >>
> >>
> >>
> >> Shane Turner
> >>
> >> Senior Software Developer
> >>phone +1 (902) 406–8375 x1008
> >>email Shane.Turner_at_NewPace.ca<mailto:Shane.Turner_at_newpace.ca>
> >>aim<aim:GoIm?screenname=Shane.Turner_at_newpace.ca>/msn
> >><msnim:chat?contact=Shane.Turner_at_newpace.ca>
> >> Shane.Turner_at_NewPace.ca
> >><aim:GoIm?screenname=Shane.Turner_at_newpace.ca>
> >>skype saturnjct<skype:saturnjct>
> >>
>
> --
> --
> Shane Turner
> Senior Software Developer
> phone +1 (902) 406–8375 x1008
> email Shane.Turner_at_NewPace.ca <mailto:Shane.Turner_at_newpace.ca>
> aim <aim:GoIm?screenname=Shane.Turner_at_newpace.ca>/msn
> <msnim:chat?contact=Shane.Turner_at_newpace.ca>
> Shane.Turner_at_NewPace.ca
> <aim:GoIm?screenname=Shane.Turner_at_newpace.ca>
> skype saturnjct <skype:saturnjct>
>
>
Received on 2011-12-09 15:39:31 CET