The International Simutrans Forum

Development => Patches & Projects => Incorporated Patches and Solved Bug Reports => Topic started by: ansgar on August 21, 2011, 05:29:28 PM

Title: Height maps are loaded as "maps/save/X*.*" not "save/file.ppm"
Post by: ansgar on August 21, 2011, 05:29:28 PM
Hi,

I'm forwarding a bug report from Debian (filed for version 102.2.2, also present in 110.0.1 and the relevant code parts don't seem to have changed since):

Quote
open("maps/save/zzz*.*", O_RDONLY)      = -1 ENOENT (No such file or directory)

I couldn't get height maps to load, so I ran strace while telling it to load
a map named "zzz".  The above is the relevant line of strace output.

According to all the online documentation the height maps should go in the
game save directory, I think this was a bad decision and maps/save may be
better, but it needs to be documented.

Opening a file with "*.*" appended to the name might work in Windows, but is
really the wrong thing to do on Linux.  Probably ".ppm" should be appended.

The original report is available at http://bugs.debian.org/637624. You can add comments by writing to 637624@bugs.debian.org, please note that you need to include 637624-submitter@bugs.debian.org if you want the submitter to be reached as well.

Ansgar
Title: Re: Height maps are loaded as "maps/save/X*.*" not "save/file.ppm"
Post by: Dwachs on September 01, 2011, 09:07:51 AM
a fix is under way
Title: Re: Height maps are loaded as "maps/save/X*.*" not "save/file.ppm"
Post by: prissi on September 01, 2011, 12:26:44 PM
The height maps should go into /maps/ ... at least this was the intention.
Title: Re: Height maps are loaded as "maps/save/X*.*" not "save/file.ppm"
Post by: Dwachs on September 01, 2011, 12:33:51 PM
the bug is: if one types the name of the file instead of clicking on the button then the file is not searched at the wrong location (the same happens for scenarios).

edit: should be fixed with r4794.