<div dir="ltr">Hi Miguel, hi Emmanuel !<div><br></div><div>Sorry to come back to you so late...</div><div class="gmail_extra"><br><div class="gmail_quote">2014-05-03 19:50 GMT+02:00 Miguel Landaeta <span dir="ltr"><<a href="mailto:nomadium@debian.org" target="_blank">nomadium@debian.org</a>></span>:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On Sat, May 03, 2014 at 07:30:20PM +0200, Emmanuel Bourg wrote:<br>


> Le 03/05/2014 18:41, Miguel Landaeta a écrit :<br>
><br>
> > Well, the bug was not introduced by upstream as I thought.<br>
> > I can only reproduce this bug when patch szzepiq_jar_resources.diff is applied.<br>
> > When I remove that patch, jarjar works OK for me.<br>
><br>
> Interesting, the question is to know what issue this patch fixed. There<br>
> is no mention of a bug in the changelog.<br>
><br>
<br>
</div>However, the patch itself contains these headers:<br>
<br>
Description: Create patch from fork at <URL:<a href="https://github.com/szczepiq/jarjar" target="_blank">https://github.com/szczepiq/jarjar</a>>.<br>
 which resolve the issue with jarjar not updating the fully qualified class names<br>
 in the jar's resources.<br>
 For instance, with this patch jarjar is able to replace classname (based on requested<br>
 rules) in META-INF/plexus/components.xml files.<br>
Source: <a href="https://github.com/szczepiq/jarjar" target="_blank">https://github.com/szczepiq/jarjar</a><br>
Author: Szczepan Faber<br>
Author: Damien Raude-Morvan <<a href="mailto:drazzib@debian.org">drazzib@debian.org</a>><br>
Last-Update: 2013-01-08<br>
<br>
Damien, do you remember what issue fixed this patch in jarjar package?<br>
<br>
I think there is a serious issue with this patch. Any file not<br>
matching .class, .java or MANIFEST.MF ends being treated like a text<br>
file so it ends modifying .groovy, .png, .properties, anything under<br>
META-INF, etc.</blockquote><div><br></div><div>I've cherry-picked this patch after having issues with some maven plugins but you're right it might introduce more issues than it solved.</div><div>They were repacking themself using jarjar but it won't update references in META-INF/plexus/components.xml.</div>

<div><br></div><div>As you can see from Gradle debian/changelog :</div><div><br></div><div>gradle (1.4-1) unstable; urgency=low<br></div><div><div>  * d/control: Build-Depends on libjarjar-java (>= 1.4+svn142-1) to build</div>

<div>    Gradle with jarjar which handle correctly updating the fully qualified</div><div>    class names in the jar's resources. Otherwise, we get errors during plexus</div><div>    startup.</div></div><div><br></div>

<div>Maybe recent releases of Gradle might not needed this tricks anymore.</div><div><br></div></div><div>Cheers,</div>-- <br><div dir="ltr"><div>Damien - Debian Developper</div><div><a href="http://wiki.debian.org/DamienRaudeMorvan" target="_blank">http://wiki.debian.org/DamienRaudeMorvan</a></div>

</div>
</div></div>