<div dir="ltr"><div dir="ltr">Hi Thorsten,</div><div><br></div>Thanks for reviewing my package and thanks for the feedback! I have uploaded a new version [1] that I believe addresses all of your concerns. I took the opportunity to make a couple other minor improvements, detailed below.<div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jul 3, 2020 at 5:00 PM Thorsten Alteholz <<a href="mailto:ftpmaster@ftp-master.debian.org">ftpmaster@ftp-master.debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Hi Olek,<br>
<br>
it looks strange to have one package with a version ~exp2 and only<br>
one debian/changelog entry besides a package with version ~exp1.<br></blockquote><div><br></div><div>My mistake, I forgot to update the changelog. In the initial version I had neglected to add the "lib" prefix to the binary package name.</div><div><br></div><div>After reading your email, I took the opportunity to start the packaging from scratch because I've learned a lot about Gradle since I first worked on this package. This now has a fairly conventional build process (apart from the heavy patching required for any Gradle build). I also packaged the 3.0.1 version instead of 3.0.0 to include the latest bug fixes to the 3.x version. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Anyway, please tell upstream to put better copyright information into <br>
the software or at least start with the developers from pom.xml as<br>
copyright holders in your debian/copyright.<br></blockquote><div><br></div><div>That's a fair point, done.[2] Sadly, not unique to this package as I'm sure you've seen with other corporate-sponsored packages. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If the term "Checker Framework developers" does not come from upstream,<br>
there should be real persons/companies mentioned as copyright holders.<br></blockquote><div><br></div><div>That term is the only attribution that they give in their LICENSE.txt file. [3] However, as I mentioned, I have asked them to be more verbose. In the meantime, I took your suggestion and included the authors from the <a href="http://maven.org">maven.org</a> pom.xml files in d/copyright. You may have noticed that in their LICENSE.txt they have a rather confusing [4] explanation for which files are covered under the MIT license. Digging through the source code, they can only be referring to the source files for that build [5]. That is also now clarified in the d/copyright file. (Previously, I was pulling the source from <a href="http://maven.org">maven.org</a> and it did not include the GPL code. Pulling all the code from GitHub allows us to build additional modules from this package in the future if their dependencies become available in Debian.) </div><div><br></div><div>Finally, I clarified the updated d/copyright as well as the build system decision in a fresh d/changelog. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Thanks!<br>
 Thorsten<br>
<br>
<br>
<br>
===<br>
<br>
Please feel free to respond to this email if you don't understand why<br>
your files were rejected, or if you upload new files which address our<br>
concerns.<br></blockquote><div><br></div><div>Done :) Please let me know if you have any additional concerns!</div><div><br></div><div>-Olek</div><div><br></div><div>[1] <a href="https://ftp-master.debian.org/new/checker-framework-java_3.0.1+ds-1~exp1.html">https://ftp-master.debian.org/new/checker-framework-java_3.0.1+ds-1~exp1.html</a></div><div>[2] <a href="https://github.com/typetools/checker-framework/issues/3429">https://github.com/typetools/checker-framework/issues/3429</a></div><div>[3] <a href="https://github.com/typetools/checker-framework/blob/b323ab2d3dfcd0a3699ce61ad2c7f511db801afe/LICENSE.txt#L2">https://github.com/typetools/checker-framework/blob/b323ab2d3dfcd0a3699ce61ad2c7f511db801afe/LICENSE.txt#L2</a></div><div>[4] <a href="https://github.com/typetools/checker-framework/blob/master/LICENSE.txt#L14-L15">https://github.com/typetools/checker-framework/blob/master/LICENSE.txt#L14-L15</a></div><div>[5] <a href="https://github.com/typetools/checker-framework/blob/master/checker-qual/build.gradle#L19-L29">https://github.com/typetools/checker-framework/blob/master/checker-qual/build.gradle#L19-L29</a></div><div><br></div></div></div></div>