Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 440920 - dev-java/java-config: please migrate to distutils-r1 (effectively fixing jython loop)
Summary: dev-java/java-config: please migrate to distutils-r1 (effectively fixing jyth...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Java team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-02 10:03 UTC by Michał Górny
Modified: 2012-12-05 19:16 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch against the current ebuild (java-config-python-r1.diff,1.90 KB, patch)
2012-11-02 10:03 UTC, Michał Górny
Details | Diff
The updated ebuild (java-config-2.1.12-r1.ebuild,1.47 KB, text/plain)
2012-11-02 10:04 UTC, Michał Górny
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2012-11-02 10:03:26 UTC
Created attachment 328052 [details, diff]
Patch against the current ebuild

If you tried the following:

$ mv /usr/bin/python{,~}
$ ln -s jython2.5 /usr/bin/python

You'd notice that running 'python' results in a fork-dead-loop of jython spawning itself through java-config. This effectively means that it is impossible to set Jython as the default Python implementation.

However, there's an easy fix. If java-config was migrated to python-r1/distutils-r1, the new (and better) python-exec wrapper would not try running java-config with jython, effectively fixing the issue.

I'm attaching the new ebuild and a diff against the current one. If you have any questions, I'd be glad to answer them.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2012-11-02 10:04:01 UTC
Created attachment 328054 [details]
The updated ebuild
Comment 2 Ralph Sennhauser (RETIRED) gentoo-dev 2012-12-05 19:16:56 UTC
That's a known issue. Wanted to migrate anyway. Did a handful changes to your patch though. Thanks for the report.

*java-config-2.1.12-r1 (05 Dec 2012)

  05 Dec 2012; <sera@gentoo.org> +java-config-2.1.12-r1.ebuild:
  Migrate to distutils-r1, thanks to mgorny for an initial patch. #440920
  Drop hotfixes for #415397, #305773, #350459 as they were fixed in 2.1.12.