<div dir="ltr"><div>Dear Michael,<br><br></div>I responded inline:<br><div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-06 19:38 GMT-04:00 Michael Banck <span dir="ltr"><<a href="mailto:mbanck@debian.org" target="_blank">mbanck@debian.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Sebastian,<br>
<br>
sorry for the delay, I've now partly reviewed your packaging, see below.<br>
<span class=""><br>
On Mon, May 04, 2015 at 06:42:04PM -0400, Sebastian Wouters wrote:<br>
> I updated the git repository on debichem with the new upstream v1.5:<br>
> <a href="http://anonscm.debian.org/cgit/debichem/packages/libchemps2.git/" target="_blank">http://anonscm.debian.org/cgit/debichem/packages/libchemps2.git/</a><br>
><br>
> If there's anything else I can do, please let me know.<br>
><br>
</span><span class="">> 2015-05-04 13:15 GMT-04:00 Sebastian Wouters <<a href="mailto:sebastianwouters@gmail.com">sebastianwouters@gmail.com</a>>:<br>
> > Hi Michael,<br>
> ><br>
> > A little update:<br>
> > I released v1.5 for psi4. chemps2 is going to be part of their alpha<br>
> > release.<br>
<br>
</span>Do you have any insight on their releases and/or release schedule? Their<br>
last release was beta5, so I hope they did not go back to alpha?  Also,<br>
a new beta (or gamma, or final) release looks overdue, I hope it happens<br>
soonsih.<br>
<br>
I haven't tried to build your package yet, but as there is "only" a<br>
library and development library, how exactly is the PSI4 integration<br>
working?  I see a few relevant source files in integrals/psi4plugins<br>
which are also mentioned on your website.  Does this mean the current<br>
packaging does not actually include the integration, or is this<br>
something we should do on the PSI4 package side?<br></blockquote><div><br></div><div>Lori Burns from the psi4 project told me she e-mailed you in response to this question:<br><br></div><div>* They will go from "psi4 beta 5 or 6" to "psi4 1.0.0" soon. They are cleaning up their code right now.<br></div><div><br>* The cmake build system of psi4 (in their private github branch) currently downloads the v1.5-tagged version of chemps2 from github, builds the static library, and uses this library in the project. Ideally, when psi4 1.x.y gets packaged for debian, it will rely on the libchemps2 which is (might be) already in the repository.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
> > Now that Jessie is released, would you or one of your colleagues have some<br>
> > time to help me with the packaging<br>
<br>
</span>So here are my remarks:<br>
<br>
1. I would sugget you put "Debichem Team <<a href="mailto:debichem-devel@lists.alioth.debian.org">debichem-devel@lists.alioth.debian.org</a>>"<br>
in the Uploaders: field and yourself in Maintainers:.  That still<br>
ensures that you are maintaining it, but bug reports will go to the<br>
debichem-devel lists so others can act on it as well.<br></blockquote><div><br></div><div>Done.<br> <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>
2. You should add a directory debian/upstream and a file<br>
debian/upstream/metadata in it.  In there, you can add (i) the homepage,<br>
(ii) a reasonable amount of papers you think are useful to your users<br>
and possibly a link to a page with further papers. You can check the<br>
CP2K's upstream metadata file as a template here:<br>
<br>
<a href="http://anonscm.debian.org/viewvc/debichem/unstable/cp2k/debian/upstream/metadata?revision=5604&view=markup" target="_blank">http://anonscm.debian.org/viewvc/debichem/unstable/cp2k/debian/upstream/metadata?revision=5604&view=markup</a><br>
<br>
I think <a href="http://sebwouters.github.io/CheMPS2/publications.html" target="_blank">http://sebwouters.github.io/CheMPS2/publications.html</a> would be a<br>
good candidate for the Other-References: field.<br></blockquote><div><br></div><div>Done.<br> <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>
3. Drop the second item from the changelog ;)<br></blockquote><div><br></div><div>Done.<br> <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>
4. I think we should go all-in and build the HTML documentation out of<br>
sphinx instead of just shipping .rst files.<br>
<br>
In any case, I think it would be better to attach the docs explictly to<br>
one package, right now I believe they go in every package?<br></blockquote><div><br></div><div>Done, and done.<br><br></div><div>One question though:<br>Lintian raises an "embedded-javascript-library" warning, because debian's sphinx-build generates its own jquery.js and underscore.js.<br>But apparently, other people have encountered this before and state it's the documentation generator's fault: <a href="https://groups.google.com/forum/#!topic/linux.debian.devel.mentors/PN7oKgnrB8A">https://groups.google.com/forum/#!topic/linux.debian.devel.mentors/PN7oKgnrB8A</a> .<br></div><div>Do you have any opinion / tips on this?<br><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>
5. What about PyCheMPS2? I guess this can be added later.<br>
<br></blockquote><div> </div><div>I agree. It might also be better to just have it as a part of python's pip.<br></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">
<br>
That's it for now (I might have some more comments later one), but<br>
really, let's get this uploaded!<br>
<span class=""><font color="#888888"><br>
<br>
Michael<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Thank you for taking a look :-).<br><br></div><div class="gmail_extra">Let me know how the compilation works out and if you have further questions!<br><br></div><div class="gmail_extra">Best wishes,<br></div><div class="gmail_extra">Sebastian<br></div><div class="gmail_extra"><div dir="ltr"><div><div dir="ltr"><div><br style="font-family:courier new,monospace"><span style="font-family:courier new,monospace"></span></div></div></div></div>
</div></div></div>