Fix for regression in Gantt Project jnlp
Omair Majid
omajid at redhat.com
Fri Aug 24 15:10:44 PDT 2012
Hi Deepak,
On 08/24/2012 04:34 PM, Deepak Bhole wrote:
> The old patch seems to broad and does not catch the circularity error
> anyway :( Attached is a refined patch that catches just the error we are
> seeing.
I am fine with the patch going in into 1.3, assuming there are no other
regressions. But please avoid this for HEAD.
Is it possible to add a reproducer along with the fix?
> Once again, ignoring this error is OK because ignoring it implies
> permission denied, which is acceptable. This is only a stop-gap measure
> and we are working on a proper fix for 1.4 (and perhaps a backport to
> 1.3 later).
The error can probably be ignored safely here (though it may break an
application if it really needs the permission). But I am afraid this is
really an indication of a deeper bug in the classloader. This fix may
make this particular application work, but other applications may remain
broken if the circularity error is detected in some other part of code.
I would prefer to fix the root cause, if at all possible, instead.
Thanks,
Omair
More information about the distro-pkg-dev
mailing list