<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 September 2016 at 04:11, Jonathan Dowland <span dir="ltr"><<a href="mailto:jmtd@debian.org" target="_blank">jmtd@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for your detailed response! I've re-ordered your points in reply so that<br>
I can group them according to response:<br>
<span class=""><br>
On Wed, Sep 28, 2016 at 10:52:06PM +0000, Potter, Tim (HPE Linux Support) wrote:<br>
> * remove unneeded lintian overrides<br>
</span><span class="">> * update Vcs-* fields<br>
> * set section and priority to "devel" and "extra"<br>
> * set homepage to project github page<br>
<br>
</span>Done, thanks for noticing<br>
<span class=""><br>
> * add upstream and pristine-tar branches (running "gbp import-dsc<br>
>   --pristine-tar" should do the trick)<br>
<br>
</span>Will do (once we've resolved the source pkg name, see below)<br>
<span class=""><br>
> * remove unneeded override_dh_install target from r/rules<br>
</span><span class="">> * are d/*.install files necessary?  I thought the binary and manage were<br>
>   installed automatically by other parts of dh<br>
</span><span class="">> * is dh-exec used anymore? if not remove from B-Ds<br>
<br>
</span>Grouping these three points together: w/o the .install file for the lib package,<br>
the lib package ends up empty.  w/o the override in d/rules, dh_install fails:<br></blockquote><div><br></div><div>Take /usr/lib out of the .install file, and remove the override. The ubuntu packaging of this package tries to both support building shared libraries and building not shared libraries from the same source package and has some weird bits because of this.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>    dh_install -O--buildsystem=golang<br>
> dh_install: Cannot find (any matches for) "usr/lib/" (tried in "." and "debian/tmp")<br>
> dh_install: golang-github-dustinkirkland-<wbr>golang-petname-dev missing files: usr/lib/<br>
> dh_install: missing files, aborting<br>
<br>
The install file for the binary package performs a rename (petname => golang-petname)<br>
which seems to be the way dh_install's manpage recommends this is achieved, which is<br>
why there's a dh-exec dependency.<br>
<span class=""><br>
> * maybe add remark about d/update-wordlists.sh to d/README.source?<br>
<br>
</span>Done, thanks<br>
<span class=""><br>
> * rename git repo and source package to<br>
>   golang-github-dustinkirkland-<wbr>golang-petname<br>
><br>
</span><span class="">> The source and binary package names are a bit irksome.  Technically the<br>
> source package should be golang-github-dustinkirckland-<wbr>golang-petname and the<br>
> binary package the same, but that's such an awful name for a binary package -<br>
> golang-petname is much better.  That leaves:<br>
><br>
> source package: golang-github-dustinkirckland-<wbr>golang-petname<br>
> package with binary: golang-petname<br>
> dev package: golang-github-dustinkirckland-<wbr>golang-petname-dev<br>
><br>
> Which I think isn't too bad.<br>
<br>
</span>Is renaming the source package really needed? I just scanned over the<br>
packaging.html again and the naming scheme is only mentioned in the "Library<br>
(or binary + library) packages" section, and is ambiguous in that later<br>
references are just to "package name" rather than "source package name" or<br>
"library package name", but my interpretation of it was that the naming is only<br>
necessary for the library binary package (which is the thing which will appear<br>
in dependency lines for other packages).<br></blockquote><div><br></div><div>Yeah, I think it's fine as is.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If we should/do rename source packages for lib+binary packages, then I will<br>
adjust packages.html to make that clearer (but I lack the rationale to add :))<br></blockquote><div><br></div><div>Cheers,</div><div>mwh </div></div></div></div>