Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 283967 - DEPEND in package.env is using colons for sepreating both packages and SLOT, if colons are used in EANT_GENTOO_CLASSPATH
Summary: DEPEND in package.env is using colons for sepreating both packages and SLOT, ...
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Java (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Java team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-07 09:56 UTC by Justin Lecher (RETIRED)
Modified: 2015-06-15 08:06 UTC (History)
0 users

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


Attachments
Fix the behaviour for eapis other than 0 1 (java-utils-2_eapi-fixes.patch,1.04 KB, patch)
2011-07-15 09:30 UTC, Ralph Sennhauser (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Lecher (RETIRED) gentoo-dev 2009-09-07 09:56:00 UTC
if one uses 

EANT_GENTOO_CLASSPATH="cglib:2.1,commons-cli:1,commons-net,javamail,log4j,xpp3,xstream"

where SLOTS are sperated by colons it is transformed to

DEPEND="cglib:2.1:commons-cli:1:commons-net:javamail:log4j:xpp3:xstream"

in package.env, which makes the launcher fail with

java_config_2.Errors.UnexistingPackageError: Package cglib was not found!

Using a hyphen works well and is transformed to

DEPEND="cglib-2.1:commons-cli-1:commons-net:javamail:log4j:xpp3:xstream"
Comment 1 Ralph Sennhauser (RETIRED) gentoo-dev 2011-07-15 09:30:15 UTC
Created attachment 280101 [details, diff]
Fix the behaviour for eapis other than 0 1

The substitution is done for EAPI=1, not for later ones.

Do the same for other ones.
Comment 2 Patrice Clement gentoo-dev 2015-06-15 00:17:49 UTC
I remember Chewi and wltjr discussing this problem in #gentoo-java. :)
Comment 3 James Le Cuirot gentoo-dev 2015-06-15 08:06:18 UTC
Although the original intention was to allow colons in EANT_GENTOO_CLASSPATH (and it did work in EAPI 1), a grep of the tree shows that this was only ever done in about 5 packages. wltjr has argued that the colon syntax is more consistent with the ebuild's DEPEND but I don't want to end up in a situation where half the ebuilds use one syntax and half the ebuilds use the other. Better to just nip this in the bug and ban the colon syntax now. It's a little confusing for the first 10 seconds but really not that hard to get your head around.