./autogen.sh generates unsuported version.h

[kab]

Since ./autogen.sh can generate the version.h file now, I have tested the functionality a little bit. After an svn export, autogen.sh generates a version.h file with the following content:

#define PEERID_PREFIX         "-TR161Z-'"
#define USERAGENT_PREFIX      "1.61+'"
#define SVN_REVISION          "exported"
#define SVN_REVISION_NUM      exported
#define SHORT_VERSION_STRING  "1.61+"
#define LONG_VERSION_STRING   "1.61+ (exported)"

The make will abort with:

rite-strings -Wextra -Wdeclaration-after-statement -Winit-self -MT tr-dht.o -MD -MP -MF .deps/tr-dht.Tpo -c -o tr-dht.o tr-dht.c
tr-dht.c: In function ‘tr_dhtInit’:
tr-dht.c:156: error: ‘exported’ undeclared (first use in this function)
tr-dht.c:156: error: (Each undeclared identifier is reported only once
tr-dht.c:156: error: for each function it appears in.)
make[1]: *** [tr-dht.o] Error 1
make[1]: Leaving directory `/media/hd2/debian/transmission-nightly/nightly-intrepid/libtransmission'
make: *** [all-recursive] Error 1

telling that "exported" is not declared. My new auto build script goes arround the problem, but it would be nice if someone could fix that.

[charles]

kab: I completely agree with what you're saying, but how would you suggest I "fix that" s.t. we get the svn revision info in an svn export'ed directory?

[kab]

I think autogen.sh should abort and complain about the missing .svn folder inside libtransmission and don't generate the version.h.

The second possibility is to adjust the code to support the version "exported" or create an other working dummy.

[charles]

kab: does r8572 work any better?

[charles]

Actually it looks like this ticket is a continuance of #2125. Please let's continue this thread in that ticket.