Hi Chris, hi all,
Important note to all Windows users: buildtool.cpp and build.xml have changed. Please rebuild btool (g++ buildtool.cpp -o btool).
De : Chris Morgan <chris.morganiser@...400...> On Sun, Feb 28, 2010 at 11:51 PM, Nicolas Dufour <nicoduf@...48...> wrote:
Since "bzr revno" gives the rev number directly, it may be easier to reuse the commented part (callled "safe way") instead of the currently uncommented one ("universal way"). But I'm not a C++ expert, and there may be a good reason why we used the "universal way" with SVN. Does someone remember?
Most users on Windows will have TortoiseBzr instead of bzr. While TortoiseBzr includes bzr.exe (as I recall it), it's unlikely to be in the PATH.
In the Inkscape installer I do this: !searchparse /noerrors /file .....bzr\branch\last-revision "" BZR_REVISION " "Viz, it takes the bit before the space in .bzr/branch/last-revision. Nice and simple to parse.
It doesn't work if you have a checkout (there's no last-revision at all in this case!). I've committed a new btool version which uses "bzr revno" in rev.9166. It's only a temporary fix, and it needs some cleanup (all the old svn stuff is kept), but at least it works for bzr checkout users (and don't prevent inkscape from compiling if bzr is not in the path).
Fell free to improve it. Maybe by checking .bzr\branch\last-revision first and use "bzr revno" if it fails?
Regards. -- Nicolas