Falsche Versionsnummern auf der Nightly Seite

[wernieman]

Also bei der 4646 kann ich es nicht mehr ermitteln, aber bei der aktuellen 4653 ...

Nightly-Side: 4653

Der verwendete Code:

Code:

simutrans@hedwig ~/code $ svn info
Path: .
URL: svn://tron.homeunix.org/simutrans/simutrans/trunk
Repository Root: svn://tron.homeunix.org/simutrans
Repository UUID: 8aca7d54-2c30-db11-9de9-000461428c89
Revision: 4653
Node Kind: directory
Schedule: normal
Last Changed Author: turfit
Last Changed Rev: 4653
Last Changed Date: 2011-06-18 01:41:29 +0200 (Sat, 18 Jun 2011)

Da Du über Windows kompilierst, gehe ich momentan eher von Problemen im Make-System aus ...

Gruß,
Werner

[Paco_m]

Original von wernieman

Da Du über Windows kompilierst, gehe ich momentan eher von Problemen im Make-System aus ...

Also ich hab kein Windows 8o

make, gcc 4.x und SDL auf 64bit openSUSE 11.4

[Paco_m]

Hab mal was getestet...

paco@eridu:/opt/games/simutrans_src> svn co svn://tron.homeunix.org/simutrans/simutrans/trunk -r 4648
...
Ausgecheckt, Revision 4648.
paco@eridu:/opt/games/simutrans_src> cd trunk
paco@eridu:/opt/games/simutrans_src/trunk> svn info
Pfad: .
URL: svn://tron.homeunix.org/simutrans/simutrans/trunk
Basis des Projektarchivs: svn://tron.homeunix.org/simutrans
UUID des Projektarchivs: 8aca7d54-2c30-db11-9de9-000461428c89
Revision: 4648
Knotentyp: Verzeichnis
Plan: normal
Letzter Autor: dwachs
Letzte geänderte Rev: 4646
Letztes Änderungsdatum: 2011-06-16 09:59:42 +0200 (Do, 16. Jun 2011)

paco@eridu:/opt/games/simutrans_src/trunk>

Kann das daran liegen, daß dein buildsystem eventuell die letzte geänderte Rev und nicht die aktuelle Rev nimmt?
In deinem Beispiel sind die ja gerade beide gleich (4653) daher spielts hier keine Rolle.

[DirrrtyDirk]

Das Problem scheint nur dann aufzutreten, wenn die aktuellste Änderung nicht im normalen Simutrans-Bereich (=trunk?) stattgefunden hat, sondern im 3D-Branch - aus irgendeinem Grund bekommt der Nightly-Server es dann nicht richtig mit, dass die Revisionsnummer (durch Arbeiten am Branch) nochmal angestiegen ist...

[wernieman]

Also eigentlich ... Du hast Recht, das Script nimmt die "Last Changed Rev:"

Die "Changed Rev:" ist die Globale Nummer. Sorry es zu sagen, aber vom Code her ist die "Last Changed Rev:" richtig. Scheinbar verwendet allerdings das Make-System und damit Simutrans die "Changed Rev:". Währe jetzt gut, wenn ein Entwickler sich dazu äußern würde. Kann gerne das System auf die "Changed Rev:" umstellen, nur würden dann u.a. auch Simutrans-Compiliert, auch wenn "nur" die Globale-Nummer sich ändert ....

(P.S. Leuft beim Cron-Job ohne Profile und deshalb ist SVN English ....)

[Dwachs]

Werner, wie kommt denn die Revisions-Nummer in dein Buildsystem?

Wenn du in den config.* Dateien

Code:

WITH_REVISION = 1

drin hast (siehe config.template), dann wird das von svnversion genommen.

Was mich wundert ist, dass in dem Begruessungsfenster und im Fenstertitel verschiedene Nummern stehen. Das sollte nicht sein. Auch das 'M' bei der Revisionsnummer ist verdaechtig, da sind irgendwelche Dateien geaendert, die nicht geaendert werden sollten.

Kannst du mal posten, was bei

Code:

svnversion
svn st -q
svn diff

so alles ausgegeben wird?

Am Build-system hat sich seit der Umstellung auf extra build-Verzeichnisse nichts geaendert.

[wernieman]

Wie gewünscht ...

Code:

simutrans@hedwig ~/code $ svnversion
4653
simutrans@hedwig ~/code $ svn st -q
simutrans@hedwig ~/code $ svn diff
simutrans@hedwig ~/code $ svn info
Path: .
URL: svn://tron.homeunix.org/simutrans/simutrans/trunk
Repository Root: svn://tron.homeunix.org/simutrans
Repository UUID: 8aca7d54-2c30-db11-9de9-000461428c89
Revision: 4653
Node Kind: directory
Schedule: normal
Last Changed Author: turfit
Last Changed Rev: 4653
Last Changed Date: 2011-06-18 01:41:29 +0200 (Sat, 18 Jun 2011)

Als ich "damals" das System gebaut habe, gab es noch nicht die Möglichkeit mit "WITH_REVISION", deshalb habe ich da etwas "eigenes" gebaut. So wie es aussieht, muß ich die Info von svnversion übernehmen ....

Wie das "M" hinkommt, kann ich Dir nich sagen. Nur svnversion selber sagt dazu:

Code:

4168M         modified working copy

Edit:
ich glaube, ich weiß woher es kommt ...
"Damals" wurde es nicht in die simversion.h. Da es einige Probleme bei Bugmeldungen gab (Welche Version hat denn den Fehler?), hatte ich es eingebaut ... wurde jetzt von mir rausgenommen!


Edit 2:
Soll ich die Versionnummern von svnversion nehmen?
Momentan sieht es eher so aus:

Code:

version=`svn info ${simuverz} | grep "Last Changed Rev:" | cut -f2 -d ':' | sed -e "s/ //g"`

Besser wäre obiges aber mit:

Code:

`svn info | grep "Last Changed Rev:" | sed -e "s/^.*: //"`

[Dwachs]

Kannst du nicht die Versionsnummer von svnversion nehmen? Die wird auch vom Makefile genommen und erscheint als Fenstertitel etc im Programm.

[wernieman]

Siehe oben, war mein Vorschlag.

Man muß sich nur dann daran erinnern, das svnversion die "Globale" SVN-Version nimmt, nicht die vom lokalen Verzeichnis!

[wernieman]

Bitte nicht wundern, wegen eines Fehlers (Änderung nicht komplett durchgezogen) ist bei der 4656 nicht die Haupt-Nummer eingetragen. Sollte bei der nächsten Generierung wieder funktionieren.