The International Simutrans Forum

Simutrans Extended => Pak128.Britain-Ex Bug Reports => Pak128.Britain-Ex => Simutrans-Extended paksets => Pak128.Britain-Ex closed bug reports => Topic started by: Vladki on July 06, 2016, 10:08:10 PM

Title: ground.outside - duplicate definition
Post by: Vladki on July 06, 2016, 10:08:10 PM
This patch got lost in the thread about experimental server. The object ground.outside is defined in a dat file, and generated on the fly from makefile. As a result pakset compiled on different computers were not consistent. Makefile generation disabled:


diff --git a/Makefile b/Makefile
index e88941d..2b5d446 100644
--- a/Makefile
+++ b/Makefile
@@ -3,7 +3,7 @@
# to get fresh and ready to deploy .tbz2 and .zip archives
#
# Change THIS to change the version string encoded in the pak file
-VERSION_STRING = "pak128.Britain-Ex-0.9.2"
+# VERSION_STRING = "pak128.Britain-Ex-0.9.2"
#
#
#
@@ -157,11 +157,11 @@ $(OUTSIDE):
        @echo "===> OUTSIDE with REVISION and grounds"
        @mkdir -p $(PAKDIR)
        @$(MAKEOBJ) PAK128 $(PAKDIR)/ $@/ > /dev/null
-       @echo -e "Obj=ground\nName=Outside\ncopyright=$(VERSION_STRING)" >$@/outsiderev.dat
-       #@svnversion >>$@/outsiderev.dat
-       @echo -e "Image[0][0]=images/ls-water-outside-128.0.0\n-" >>$@/outsiderev.dat
-       @$(MAKEOBJ) PAK128 $(PAKDIR)/ $@/outsiderev.dat > /dev/null
-       @rm $@/outsiderev.dat
+#      @echo -e "Obj=ground\nName=Outside\ncopyright=$(VERSION_STRING)" >$@/outsiderev.dat
+#      #@svnversion >>$@/outsiderev.dat
+#      @echo -e "Image[0][0]=images/ls-water-outside-128.0.0\n-" >>$@/outsiderev.dat
+#      @$(MAKEOBJ) PAK128 $(PAKDIR)/ $@/outsiderev.dat > /dev/null
+#      @rm $@/outsiderev.dat

clean:
        @echo "===> CLEAN"

Title: Re: ground.outside - duplicate definition
Post by: jamespetts on July 07, 2016, 10:05:39 PM
Thank you for the report. I am having trouble finding more than one of the following:


VERSION_STRING = "pak128.Britain-Ex-0.9.2"


May I ask where the other is located?
Title: Re: ground.outside - duplicate definition
Post by: Vladki on July 10, 2016, 07:54:35 PM
pak1file/128/outside.dat
Title: Re: ground.outside - duplicate definition
Post by: jamespetts on July 10, 2016, 10:04:11 PM
Thank you - now fixed.
Title: Re: ground.outside - duplicate definition
Post by: Vladki on December 03, 2016, 07:01:09 PM
This patch was not applied completely. Here is the missing part:


diff --git a/Makefile b/Makefile
index 5b1fa14..5753835 100644
--- a/Makefile
+++ b/Makefile
@@ -156,12 +156,7 @@ $(DIRS256):
$(OUTSIDE):
        @echo "===> OUTSIDE with REVISION and grounds"
        @mkdir -p $(PAKDIR)
-       @$(MAKEOBJ) PAK128 $(PAKDIR)/ $@/ > /dev/null
-       @echo -e "Obj=ground\nName=Outside\ncopyright=$(VERSION_STRING)" >$@/outsiderev.dat
-       #@svnversion >>$@/outsiderev.dat
-       @echo -e "Image[0][0]=images/ls-water-outside-128.0.0\n-" >>$@/outsiderev.dat
-       @$(MAKEOBJ) PAK128 $(PAKDIR)/ $@/outsiderev.dat > /dev/null
-       @rm $@/outsiderev.dat
+       @$(MAKEOBJ) quiet PAK128 $(PAKDIR)/ $@/ > /dev/null

clean:
        @echo "===> CLEAN"


I hoped it will help with the desync issue, but it does not...
Title: Re: ground.outside - duplicate definition
Post by: jamespetts on December 03, 2016, 07:04:00 PM
Splendid - thank you for that.
Title: Re: ground.outside - duplicate definition
Post by: Vladki on December 03, 2016, 08:15:04 PM
James please, remove the old makeobj-experimental from the pakset repository.

Thanks for the fix, but I think that the line:
@mkdir -p $(PAKDIR)
should not  be removed