<div dir="ltr">I’m not opposed, please feel free to push your changes.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 14, 2016 at 3:50 PM, Martín Ferrari <span dir="ltr"><<a href="mailto:tincho@tincho.org" target="_blank">tincho@tincho.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since nobody has voiced strong concerns about this, I will merge this<br>
change later today or tomorrow and upload.<br>
<br>
I wanted to wait for Michael S. to chime in, but I guess he's not<br>
strongly opposed either :-)<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 08/05/16 05:31, Martín Ferrari wrote:<br>
> Hi all,<br>
><br>
> I have just pushed a branch (tincho_extensions) to dh-golang,<br>
> implementing a few changes that I believe are beneficial for golang<br>
> packaging. If there are no objections, I would like to merge this to<br>
> master and release 0.17.<br>
><br>
><br>
> * Export DH_GOLANG_INSTALL_EXTRA with a list of space-separated paths to<br>
> copy to the build dir, for tests and other files not automatically<br>
> installed.<br>
><br>
> This means we can stop using the INSTALL_ALL=1+rm combo, which I think<br>
> is pretty awkward. Instead, one would do (real example from the<br>
> prometheus package):<br>
><br>
> export DH_GOLANG_INSTALL_EXTRA := retrieval/discovery/fixtures \<br>
>     storage/local/fixtures config/testdata promql/testdata \<br>
>     retrieval/testdata<br>
><br>
><br>
> * Add --no-source and --no-binaries options to install target.<br>
><br>
> This avoids the need to remove debian/prometheus/usr/share/gocode, and<br>
> would go a long way to fix #814690, and I think is generally a good<br>
> idea, to make it simple to split packages in binary and sources. Again,<br>
> a real (and tested) example from prometheus:<br>
><br>
> override_dh_auto_install:<br>
>         dh_auto_install -O--buildsystem=golang -- --no-source<br>
><br>
><br>
> * Display a debug message when copying files to the build tree.<br>
><br>
> A minor change, I think that when DH_VERBOSE is set, all the copy and<br>
> symlink operations during preparation of the source tree should be<br>
> printed. It looks like this:<br>
><br>
>       Copy retrieval/testdata/server.cer -><br>
> build/src/<a href="http://github.com/prometheus/prometheus/retrieval/testdata/server.cer" rel="noreferrer" target="_blank">github.com/prometheus/prometheus/retrieval/testdata/server.cer</a><br>
>       Copy retrieval/testdata/client.cer -><br>
> build/src/<a href="http://github.com/prometheus/prometheus/retrieval/testdata/client.cer" rel="noreferrer" target="_blank">github.com/prometheus/prometheus/retrieval/testdata/client.cer</a><br>
>       Symlink /usr/share/gocode/src/<a href="http://code.google.com" rel="noreferrer" target="_blank">code.google.com</a> -> build/src/<a href="http://code.google.com" rel="noreferrer" target="_blank">code.google.com</a><br>
>       Symlink /usr/share/gocode/src/<a href="http://github.com/asaskevich" rel="noreferrer" target="_blank">github.com/asaskevich</a> -><br>
> build/src/<a href="http://github.com/asaskevich" rel="noreferrer" target="_blank">github.com/asaskevich</a><br>
><br>
><br>
> Thoughts?<br>
><br>
<br>
<br>
--<br>
Martín Ferrari (Tincho)<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Best regards,<br>Michael</div>
</div>