<div dir="ltr"><div>Done, thank you<br><br></div>Paolo<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 23, 2016 at 2:23 PM, Michael Banck <span dir="ltr"><<a href="mailto:mbanck@debian.org" target="_blank">mbanck@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, Oct 26, 2014 at 10:47:33PM +0100, Michael Banck wrote:<br>
> On Sun, Oct 26, 2014 at 09:44:42PM +0100, Paolo Giannozzi wrote:<br>
> > > 2. The copyright violations in TDDFPT/ColorCalculator have not been<br>
> > > fixed it seems.<br>
<br>
The latest development snapshot still seems to have class-files without<br>
corresponding code in it:<br>
<br>
$ tar tf espresso-r12956-2016-09-11.<wbr>tar.gz | grep TDDFPT.*class$ | wc -l<br>
685<br>
$ tar tf espresso-r12956-2016-09-11.<wbr>tar.gz | grep TDDFPT.*java$ | wc -l<br>
5<br>
<br>
> > you mean: the presence of the two binaries colorCalculator4.jar and<br>
> > ColorCalculator5.jar? Removed, there are instructions on how to compile<br>
> > them.<br>
><br>
> Well that, and all the .class files which do not have a corresponding<br>
> source (.java) file, as they are from 3rd-party modules.<br>
><br>
> I had a quick look, and the two Debian package libjcommon-java and<br>
> libjfreechart-java seem to provide those .class files in their .jars.<br>
><br>
> So there are two options after removing the .class files:<br>
><br>
> 1. Adjust README/Install to tell the user to download those .jars from<br>
> elsewhere and add them to the classpath when assembling the jar.<br>
><br>
> 2.  Include the corresponding .java files for those.  However, that'll<br>
> make compilation a bit more difficult, but maybe 'javac `find . -name<br>
> "*.java"`' would suffice already.<br>
<br>
Or<br>
<br>
3. Adjust README/Install to download the jars automatically if they are<br>
not present.<br>
<br>
I've done that now in the attached patch which works for me.  With that,<br>
TDDFPT/ColorCalculator/src/{<wbr>com,org} can be removed from the source<br>
tree.<br>
<br>
The alternative would be to just remove the ColorCalculator program<br>
entirely.<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
Michael<br>
</font></span><br>______________________________<wbr>_________________<br>
Q-e-developers mailing list<br>
<a href="mailto:Q-e-developers@qe-forge.org">Q-e-developers@qe-forge.org</a><br>
<a href="http://qe-forge.org/mailman/listinfo/q-e-developers" rel="noreferrer" target="_blank">http://qe-forge.org/mailman/<wbr>listinfo/q-e-developers</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Paolo Giannozzi, Dip. Scienze Matematiche Informatiche e Fisiche,<br>Univ. Udine, via delle Scienze 208, 33100 Udine, Italy<br>Phone +39-0432-558216, fax +39-0432-558222<br><br></div></div></div></div></div>
</div>