remaining tasks for 3.1.0

I’ve uploaded 3.1.0-svn5062 now, and I believe all we’re waiting for now is translation of the key added after the translators gave us the current translations. I checked each one to arrive at the following count of untranslated keys:

French: 7
Spanish: 1
German: 65
Italian: 8
Dutch: 8
Japanese: 1

Note also that the translation dialog is started by giving VASSAL the ‘–translate’ command-line option now. If any of our translators don’t know how to do that, please let me know and I’ll provide instructions.

Can you write up a blurb with the download location and how to start VASSAL with the propery command-line switch? I’ll forward it to the translators.

rk

Post generated using Mail2Forum (mail2forum.com)

Thus spake Rodney Kinney:

Windows:

  1. Open a terminal window. (Select “Run” from the Start menu, and type ‘cmd’.)
  2. cd X:\path\to\VASSAL.exe (The path will probably be
    “C:\Program Files\VASSAL-3.1.0-something”.)
  3. VASSAL.exe --translate

Linux:

  1. Open a terminal.
  2. cd to wherever you installed VASSAL.
  3. ./VASSAL.sh --translate

Mac OS X:

I can’t provide instructions for Mac OS X, because I don’t know how to open
a terminal there. Otherwise, it’s the same: Open a terminal, cd to where
the app bundle is, the run it with the --translate option.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

On Feb 6, 2009, at 2:39 PM, Joel Uckelman wrote:

Actually, it isn’t quite so easy, because the App bundle itself isn’t
something you can invoke directly on the command line and pass
arguments to. It is not an executable, so you can’t just invoke the
*.app bundle from the command line. You can start applications from
the command line by using the “open” command, but that doesn’t pass
along any arguments from the command line.
Instead it launches using an application launcher and takes any
command line arguments from the appropriate key in the Info.plist file.

For the Mac, the easiest thing to do from the user perspective along
these lines would to create a copy of the Vassal.app application and
just modify the Info.plist file to include the correct command line
argument. The appropriate Java key for that is “Arguments”.

I would think that the simplest solution would be to create a
*.command script for each Vassal version XYZ, called Translate-Vassal-
XYZ.command that you put in the native Applications folder. It will
then just assume that it is in the right place and have the following
content:

Translate-Vassal-XYZ.command:
-------- Begin file contents ---------
#! /bin/sh

java -classpath Vassal-XY.app/Contents/Resources/Java/Vengine.jar
VASSAL.i18n.TranslateVassalWindow
-------- End file contents ---------

This could use the standard Java startup and pass the command line
argument instead of invoking the specific class instead.

But as to the original question:

  • To launch a command line terminal on Mac OS X:
  • Go to Applications/Utilities.
    From the Finder this is directly accessible from the
    “Go > Utilities” menu item, but that will have a
    different name in different languages.

  • Double-click on the “Terminal” application

Sorry it isn’t any easier than this.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Okay. I pinged the translators to get these last phrases.

Am I right in remembering that the one missing phrase here is “Translate the VASSAL user interface” which is no longer needed?

rk

Post generated using Mail2Forum (mail2forum.com)

Thus spake Rodney Kinney:

No. It’s “Import module”.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Swampwallaby-work@5066 contains a partial fix for

Bug [2551552] Detect Infinite Loops

It prevents ‘Return To Deck’ infinite loops. No ‘Return To Deck’ is performed if the piece is already in the target Deck.

B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Joel,

swampwallaby-work@5067 contains the rest of the fix for

Bug [2551552] Detect Infinite Loops

This detects and reports Global Key Command loops - Direct and indirect.

I extracted the loop detection we built into DoActionButton and generalized it. I am just using one global counter to detect infinite loops of all kinds. The recursion limit is set to 50, so after 50 nested calls to Loopable components, the Error Dialog will display.

I though about having a seperate recursion limit for each type of component (i.e. count DoAction button recursion seperately from Global Key Command recursion), but decided it just wasn’t warranted. If you have 50 levels of GKC and DoAction recursion combined, your module is in deep trouble.

The only loop remaining on my list is the infinite Place Marker loop. I’ll look at this one now, but I think I can use the new InfiniteLoopDetection class for this too.

B.

*********** REPLY SEPARATOR ***********

On 8/02/2009 at 10:31 AM Brent Easton wrote:


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Still waiting for updates to the German and Italian translations. Others are complete.

rk

Post generated using Mail2Forum (mail2forum.com)

swampwallaby-work3@5086 fixes

Bug [2594850] ABR: NumberFormatException: For input string: “139 false”

I had already fixed this, but code includes a blooper. If the BadDataReport fires off, it was not supposed to also generate a Bug Report.

Also,

Bug [2551552] Detect Infinite Loops

is still waiting to be merged from swampwallaby-work@5066 & 5067

B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Thus spake “Brent Easton”:

Merged to 3.1@5087, trunk@5088.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Thus spake “uckelman”:

The only thing left of these is the German translation. Do we have an ETA
on that?

Also, here are a few newly reported bugs while we wait:

2603679: NumberFormatException in FreeRotator.mySetState()
2602968: NPE in WizardSupport.showWelcomeWizard()
2588852: NPE in BasicLogger.write()

I assigned 2603679 to Brent because this looks similar to some other
bugs he’s fixed recently. The other two are unassigned.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

No ETA yet. I pinged him again, but Torsten provided the first draft of his translation a long time ago and indicated then that he didn’t have much time, so we may have to look wider for a German translator.

rk

Post generated using Mail2Forum (mail2forum.com)

Unreproducible, no email, no details.

Appears to be a corrupted Save Game.

There are a large number of places in the code where unprotected

int = Integer.parseInt(String)

is done. Often, the data to be parsed is sourced directly from an IntConfigurer, so there is no change of getting a non-integer

However, for many of them, the data is sourced from either the buildFile or a Saved Game that may have been hand modified or corrupted. We seem to be getting about 1 a month where this has happened. These will all need to be protected with a try/catch block and Bad Data reports of some sort generated and a reasonable default used.

Do you want me to work on this for 3.1? This is our bug fix version, so I am guessing the answer is yes!

B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Thus spake “Brent Easton”:

It should be done eventually, if for no other reason than to stop the bug
reports. Since we haven’t seen this problem in great quantities in the betas,
I think we shouldn’t hold up 3.1.0 for it, but I would be shocked if we
released no 3.1.1 which this could go into.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

I have fixed the cases in VASSAL.counters in swampwallaby-work@5093 and updated the bug report to be more general.

I will work through the rest. I’ll see if I can beat the German translation ;)

B.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Thus spake “Brent Easton”:

Merged to 3.1@5099, trunk@5100.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Thus spake Rodney Kinney:

I don’t want to hold up releasing 3.1 too much longer. If we don’t have
this on the last day of February, I’d like to release 3.1.0 without it
and pick up an updated translation later.


J.


Messages mailing list
Messages@forums.vassalengine.org
forums.vassalengine.org/mailman/ … engine.org

Post generated using Mail2Forum (mail2forum.com)

Torsten got back to me. We should have it in a couple of days.

rk

Post generated using Mail2Forum (mail2forum.com)

Definitely! The hot topic on this forum 1 year ago today was

Remaining tasks before 3.1.0-beta1