Bug 28319 - [PATCH] restored %_datadir/java/gcj-endorsed
Summary: [PATCH] restored %_datadir/java/gcj-endorsed
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gcc4.7-java (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Sergey Bolshakov
QA Contact: qa-sisyphus
URL: http://git.altlinux.org/people/viy/pa...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-09 01:34 MSK by viy
Modified: 2013-01-10 02:13 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description viy 2013-01-09 01:34:14 MSK
Please, fetch commit 
http://git.altlinux.org/people/viy/packages/?p=gcc4.7.git;a=commit;h=9b73c8c2bbaea20334f4005a21414611410b598f
and publish a new release of gcc
to restore %_datadir/java/gcj-endorsed according to Java Policy.
Comment 1 Dmitry V. Levin 2013-01-09 02:09:54 MSK
That commit is called "java bugfix", but I don't understand why.
You said it has something to do with Java Policy but I don't see the link.

Please help me to understand what's the matter.
Comment 2 Dmitry V. Levin 2013-01-09 02:20:50 MSK
Unversioned gcj directories should go to a version agnostics package, such as libgcj-common.
Comment 3 Dmitry V. Levin 2013-01-09 02:21:39 MSK
s/agnostics/agnostic/
Comment 4 viy 2013-01-09 02:36:17 MSK
Суть патча не в упаковке каких либо каталогов.
оба варианта можно безболезненно выкинуть из спека 
-%_datadir/java/gcj%gcc_branch-endorsed
+%_datadir/java/gcj-endorsed
а %_datadir/java/gcj-endorsed запаковать в другом пакете, я только за.

Суть патча в 
--      -DGCJ_ENDORSED_DIRS="\"$(jardir)/gcj-endorsed\"" \
-+      -DGCJ_ENDORSED_DIRS="\"$(jardir)/gcj$(gcc_version)-endorsed\"" \
без этого gcj не научится обратно смотреть в %_datadir/java/gcj-endorsed,
как ему во всем мире и положено.
Comment 5 viy 2013-01-09 02:45:22 MSK
 $(jardir)/gcj$(gcc_version)-endorsed не нужна,но если хочется ее видеть из эстетических или философских соображений, можно сделать так

--      -DGCJ_ENDORSED_DIRS="\"$(jardir)/gcj-endorsed\"" \
-+      -DGCJ_ENDORSED_DIRS="\"$(jardir)/gcj-endorsed\":\"$(jardir)/gcj$(gcc_version)-endorsed\"" \

т.е. я согласен на любой мусор в DGCJ_ENDORSED_DIRS, лишь бы там присутствовал стандартный путь $(jardir)/gcj-endorsed.
Comment 6 viy 2013-01-09 02:49:43 MSK
в частности, gcj без этого не видит bouncycastle-* jars,
которые у нас лежат в /usr/share/java/gcj-endorsed/.
Comment 7 viy 2013-01-09 02:52:12 MSK
Раньше это было не смертельно, так как раньше gcj находил их в classpath,
а в gcj4.7 уже прикрутили гайки по безопастности, и теперь 
GCJ_ENDORSED_DIRS должно принимать осмысленное значение.
Comment 8 viy 2013-01-09 02:56:26 MSK
как бонус, опять станет собираться itext
(сейчас он не собирается, так как криптомодули bouncycastle-* теперь 
ищутся только в GCJ_ENDORSED_DIRS, а в GCJ_ENDORSED_DIRS пути 
%_datadir/java/gcj-endorsed, где лежат  bouncycastle-*, нет.
Comment 9 viy 2013-01-09 03:00:50 MSK
Резюмируя, в gcc4.7 закрутили гайки по безопастности,
и теперь доверенный код откуда попало не загружается.
Для этого кода у нас выделен каталог
%_datadir/java/gcj-endorsed.

Проблема в том, что у нас GCJ_ENDORSED_DIRS инициализируются левым значением,
поэтому gcj жоверенный код не находит.
надо, чтобы GCJ_ENDORSED_DIRS содержали %_datadir/java/gcj-endorsed.
Comment 10 viy 2013-01-09 03:03:25 MSK
(В ответ на комментарий №9)
> Резюмируя, в gcc4.7 закрутили гайки по безопастности,
> и теперь доверенный код откуда попало не загружается.
> Для этого кода у нас выделен каталог
> %_datadir/java/gcj-endorsed.
> 
> Проблема в том, что у нас GCJ_ENDORSED_DIRS инициализируются левым значением,
> поэтому gcj жоверенный код не находит.
> надо, чтобы GCJ_ENDORSED_DIRS содержали %_datadir/java/gcj-endorsed.

Оно у нас традиционно инициализировались левым значением,
но раньше это не сказывалось на работе.
Comment 11 viy 2013-01-09 13:54:13 MSK
упростил патч, прошу взять здесь

http://git.altlinux.org/people/viy/packages/?p=gcc4.7.git;a=commit;h=04b87129a56ecfe15b8e85f9150302dade0241a7

и выпусить новый релиз.
Comment 12 Repository Robot 2013-01-10 01:56:43 MSK
gcc4.7-4.7.2-alt6 -> sisyphus:

* Wed Jan 09 2013 Dmitry V. Levin <ldv@altlinux> 4.7.2-alt6
- libgcj: fixed GCJ_ENDORSED_DIRS (by Igor Vlasenko; closes: #28319).
- libgcj-jar: dropped /usr/share/java/gcj4.7-endorsed/.
Comment 13 viy 2013-01-10 02:13:32 MSK
спасибо!