Hi Mike, Right! This I can follow lol! Thank you yes I can proceed further now lol.
When you originally talked about "FreeColMessages.properties" I thought you were talking about some line value within some master translation file that I had no idea where to find lol. But now I see you meant FreeColMessages.properties IS the file name lol. Alright well I'll go cancel my Pull Request and redo the job correctly when I have time again (prob next weekend). I've gotta say with all due respect it seems strange that the whole map usage system requires manual entry into some translation list instead of storing the required extra info in the map files themselves and simply having a map reader that pulls the info out of the map files. As then all this special work and list management isn't required when regular users want to add maps to the game. In fact interestingly someone else has put in a pull request that has the map editor ask and store such extra info (eg author) in the map file itself. I know this will have no effect on the map reader but maybe I should hold off my PR until this PR is either accepted or denied by you guys since if its approved then I'll probably wanna refresh all the maps through the editor again and use the new description & author fields he's added.https://github.com/FreeCol/freecol/pull/58 Anyway sorry, I know you don't wanna be bothered about this stuff but it doesn't seem like there's anyone else left to talk about pull requests with anymore now. I did start a whole convo on the forums in Mazim's map thread about file names and what they should be and a number of people gave their thoughts but no one seemed to know about all this special naming and translation list stuff so I wasn't warned. What impresses me about all these new 2020 pull requests is that there are 5 different users behind them and I don't see their names in the old closed requests.. so that's presumably a few different new programmers wanting to help you which is really good (maybe the new welcome thread attracted their help & sent them in the right direction, or a least that's what I'm going to tell myself haha). Hopefully it leads to future team expansion and once again taking away some of the load from you. :) Regards Blake On Sunday, 15 March 2020, 09:15:17 pm ACDT, Michael T. Pope <mp...@computer.org> wrote: > Just letting you know the good news that a number of different people are > making contributions to FreeCol on github... Quite so. Pity they are a bit down the list. I was away last week, and am now stuck at github issue#32 which is proving to be a PITA, and there is a new sourceforge bug to look at... after that I plan to deal with the more problematic pull requests... > Mine PR has been sitting there for weeks too... ...like yours:-). Your map reworking has problems... Lets try again. 1. The name of the map file is *not* what is supposed to be displayed on the map selection panel. Yes, we want to credit the authors, but crufting them into the file name is not how to do it. Worse yet, renaming the existing map files is breaking the test suite, which is why the github page for your PR is showing failed checks. 2. File names with spaces tend to cause breakage in the build scripts, please avoid them. 3. The string we display under the map thumbnail comes from the strings database. This is a file called "FreeColMessages.properties" which is in the strings directory, right next to the maps directory you have been hacking around in. It is a plain old text file, and contains lines like "freecol.map.Australia=Australia". To credit someone, you would change that, for example, to "freecol.map.Australia=Australia by Drongo Bruce" 4. For every file in the maps directory, you need a line in the properties file. The left hand side *must* have the freecol.map. prefix, after which is the *file name* without the .fsm extension. Once again, no spaces in the file name as that can stuff up the routines that read the properties file. 5. If you do *not* do this, freecol goes through the map directory, finds a file called foo.fsm, tries to look in the strings database for a key of freecol.map.foo, fails to find it, and defaults to just displaying the key freecol.map.foo. Which is what you were complaining about in https://sourceforge.net/p/freecol/bugs/3182/ 6. Yes I read your "but hunting down master translation stuff somewhere in something.. umm yeah.. I got no idea sorry." response in that bug. Humour me, eh? I am stretched pretty thin here, and really appreciate it when people work stuff out from what I admit is a terse explanation on my part. Hopefully this was a bit clearer and/or enough detail to get things moving again? Cheers, Mike Pope
_______________________________________________ Freecol-developers mailing list Freecol-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freecol-developers