[hardy] freecol java detection flawed and not enough memory to start

[Stefan Fleiter]

Binary package hint: freecol

freecol 0.7.2-5 does not start here.
Debugging revealed two problems:
- the java detection can start java with JAVA_HOME not matching java executable
- 128M needs to be increased, I've chosen 192M

I have attached my very first ubuntu patch, so please tell me what to improve next time.
There are still some occurences of telling that 128M are needed in code and doc.

All changes regarding memory should probably be integrated in a dpatch,
I have stopped here to get some feedback first.

Thanks a lot for considering.

[Stefan Fleiter (stefan-fleiter)]

Thinking a bit more about the problem I come to the conclusion this should be solved
by a generic java wrapper.
It does not make any sense in spite of the fact that many JAVA programms require
the environment variable JAVA_HOME set the java binary is simply managed by the
/etc/alternatives mechanism.

I think there are two possible solutions:
1. Let a wrapper script set JAVA_HOME matching to the java provider chosen by
    /etc/alternatives
2. Choose JAVA dynamically by the given JAVA_HOME if set, if not set goto 1. :-)

[boucman (jeremy-rosen)]

Just confirming that I am getting that bug too...

[Stefan Fleiter (stefan-fleiter)]

Setting to confirmed based on last comment.
At least the memory part should be an easy fix.

[Launchpad Janitor (janitor)]

This bug was fixed in the package freecol - 0.7.3-2ubuntu1

---------------
freecol (0.7.3-2ubuntu1) intrepid; urgency=low

  * Merged from Debian (LP: #196264), remaining Ubuntu changes:
    + debian/control, debian/rules:
      - Replaced icedtea-java7 references with openjdk-6.
      - Changed maintainer field.

freecol (0.7.3-2) unstable; urgency=low

  * Pushing memory limit to 180M, as it does not work anymore on AMD64 with
    only 128M...

freecol (0.7.3-1) unstable; urgency=low

  [ Ansgar Burchardt ]
  * Remove deprecated Encoding key from .desktop file

  [ Vincent Fourmond ]
  * New upstream release (Closes: #465490)
    - dropped 30-language-option, included upstream
    - updated 10-fix-jar-path and 15-fix-jar-manifest
  * Now providing a debian/new-upstream to repackage upstream tarball
  * Added a NEWS file
  * Switched to java-wrappers for the wrapper script
    - 40-manpage to point the user to java-wrappers(7)

 -- Matvey Kozhev <email address hidden> Sun, 04 May 2008 14:55:20 +0700