| 1 |
Caster wrote: |
| 2 |
> OK tried to look at it myself, and the problems mentioned were caused |
| 3 |
> just by improper paths in the beginning of |
| 4 |
> /usr/share/eselect/modules/java.eselect |
| 5 |
> Fixed them, also fixed the "eselect java list" (didn't mark user-vm |
| 6 |
> properly) - see the attached patch. Listing and changing is now in sync |
| 7 |
> with java-config-2. |
| 8 |
> |
| 9 |
> |
| 10 |
You forgot to attach the patch :) |
| 11 |
|
| 12 |
But it's moot, since I just barely fixed it before seeing your message. |
| 13 |
> But then I've run into another problem. It allows root to set user-vm as |
| 14 |
> well as system-vm, while java-config-2 doesn't. java-config-2 then lists |
| 15 |
> it as active with java-config -L (probably causes other problems too). |
| 16 |
> There's no way to unset it but removing /root/.gentoo/java-config-2 |
| 17 |
> So, eselect java shouldn't list or allow changing of user-vm for root. |
| 18 |
> Haven't tried to fix that. |
| 19 |
> |
| 20 |
> |
| 21 |
Right you are. It should die now if you try to set a user vm as root. |
| 22 |
|
| 23 |
On a related note, it seems that the eselect module sets stuff by hand, |
| 24 |
ie by creating symlinks and whatnot. I'm not sure if this was |
| 25 |
intentional, or if it should be using java-config to do this instead. |
| 26 |
|
| 27 |
In any event, thanks for the report. This is exactly the stuff I want to |
| 28 |
be found and fixed before we merge the migration-overlay into the main tree. |
| 29 |
|
| 30 |
- Josh |
| 31 |
-- |
| 32 |
gentoo-java@g.o mailing list |
Archives