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

Looks like wrong value for SVN_VER_REVISION in SWIG bindings for Python and Ruby?

From: Shane Turner <shane.turner_at_newpace.ca>
Date: Tue, 06 Dec 2011 09:47:20 -0400

I was just taking a look at the differences between Subversion 1.7.1 and
1.7.2 and noticed a couple of diffs that look odd.

In subversion/bindings/swig/python/core.c and
subversion/bindings/swig/ruby/core.c, the value of SVN_VER_REVISION
changed from the numeric revision number (previously 1186859) to 0. I
think it's supposed to be 1207936.

Is this a bug, or intended? Be forewarned that I don't use this aspect
of Subversion. I just happened to notice.

The code I was examining was from
http://apache.sunsite.ualberta.ca/subversion/subversion-1.7.2.tar.bz2

I've included the diff below. I've filtered out all of the path
differences in the comments ("/*@SWIG:...").

Shane Turner

diff -rwB -I '.*@SWIG:.*'
subversion-1.7.4bed3649c532/subversion/bindings/swig/python/core.c
subversion-1.7/subversion/bindings/swig/python/core.c
30985,30986c30985,30986
< SWIG_Python_SetConstant(d, "SVN_VER_PATCH",SWIG_From_long((long)(1)));
< SWIG_Python_SetConstant(d, "SVN_VER_MICRO",SWIG_From_long((long)(1)));

---
 >   SWIG_Python_SetConstant(d, "SVN_VER_PATCH",SWIG_From_long((long)(2)));
 >   SWIG_Python_SetConstant(d, "SVN_VER_MICRO",SWIG_From_long((long)(2)));
30988c30988
<   SWIG_Python_SetConstant(d, "SVN_VER_TAG",SWIG_FromCharPtr(" 
(r1186859)"));
---
 >   SWIG_Python_SetConstant(d, "SVN_VER_TAG",SWIG_FromCharPtr(" 
(r1207936)"));
30990,30993c30990,30993
*<   SWIG_Python_SetConstant(d, 
"SVN_VER_REVISION",SWIG_From_long((long)(1186859)));*
<   SWIG_Python_SetConstant(d, "SVN_VER_NUM",SWIG_FromCharPtr("1.7.1"));
<   SWIG_Python_SetConstant(d, "SVN_VER_NUMBER",SWIG_FromCharPtr("1.7.1"));
<   SWIG_Python_SetConstant(d, "SVN_VERSION",SWIG_FromCharPtr("1.7.1 
(r1186859)"));
---
*>   SWIG_Python_SetConstant(d, 
"SVN_VER_REVISION",SWIG_From_long((long)(0)));*
 >   SWIG_Python_SetConstant(d, "SVN_VER_NUM",SWIG_FromCharPtr("1.7.2"));
 >   SWIG_Python_SetConstant(d, "SVN_VER_NUMBER",SWIG_FromCharPtr("1.7.2"));
 >   SWIG_Python_SetConstant(d, "SVN_VERSION",SWIG_FromCharPtr("1.7.2 
(r1207936)"));
diff -rwB -I '.*@SWIG:.*' 
subversion-1.7.4bed3649c532/subversion/bindings/swig/ruby/core.c 
subversion-1.7/subversion/bindings/swig/ruby/core.c
37529,37530c37529,37530
<   rb_define_const(mCore, "SVN_VER_PATCH", SWIG_From_int((int)(1)));
<   rb_define_const(mCore, "SVN_VER_MICRO", SWIG_From_int((int)(1)));
---
 >   rb_define_const(mCore, "SVN_VER_PATCH", SWIG_From_int((int)(2)));
 >   rb_define_const(mCore, "SVN_VER_MICRO", SWIG_From_int((int)(2)));
37532c37532
<   rb_define_const(mCore, "SVN_VER_TAG", SWIG_FromCharPtr(" (r1186859)"));
---
 >   rb_define_const(mCore, "SVN_VER_TAG", SWIG_FromCharPtr(" (r1207936)"));
37534,37537c37534,37537
*<   rb_define_const(mCore, "SVN_VER_REVISION", 
SWIG_From_int((int)(1186859)));*
<   rb_define_const(mCore, "SVN_VER_NUM", SWIG_FromCharPtr("1.7.1"));
<   rb_define_const(mCore, "SVN_VER_NUMBER", SWIG_FromCharPtr("1.7.1"));
<   rb_define_const(mCore, "SVN_VERSION", SWIG_FromCharPtr("1.7.1 
(r1186859)"));
---
*>   rb_define_const(mCore, "SVN_VER_REVISION", SWIG_From_int((int)(0)));*
 >   rb_define_const(mCore, "SVN_VER_NUM", SWIG_FromCharPtr("1.7.2"));
 >   rb_define_const(mCore, "SVN_VER_NUMBER", SWIG_FromCharPtr("1.7.2"));
 >   rb_define_const(mCore, "SVN_VERSION", SWIG_FromCharPtr("1.7.2 
(r1207936)"));
-- 
	
	
	
	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-06 14:56:12 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.