<div dir="ltr">Sorry for the late reply. Answers inline:<div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 22, 2018 at 5:46 PM, Clément Hermann <span dir="ltr"><<a href="mailto:nodens@nodens.org" target="_blank">nodens@nodens.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi there,<br>
<br>
Ping / reminder about this RFC - since I didn't get any feedback ;)<br>
<br>
Cheers,<br>
<br>
nodens<br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
On 21/12/2017 21:05, Clément Hermann wrote:<br>
> Hi!<br>
><br>
> I'm newly joined to the team, packaging some dependancies for LXD [0]<br>
> mainly.<br>
><br>
> While working on golang-gopkg-lxc-go-lxc, Martin (Tincho) pointed to me<br>
> that I should exclude examples from source - which surprised me, because<br></div></div></blockquote><div><br></div><div>We should exclude examples from being installed as binaries (which the example below does), but it’s not clear to me why we would exclude them from being included in the sources.</div><div><br></div><div>The advantage to having them installed is that they could be compiled when testing packages for an updated package version, and thereby pinpointing an issue more quickly. E.g., assume I’m updating golang-foo 2.2 to 2.3, which is used by golang-bar, which in turn is used by cooltool. If golang-bar comes with an example program, I might catch the issue which golang-foo 2.3 introduces as a failure in the golang-bar compilation, not only in the cooltool compilation.</div><div><br></div><div>tincho, can you clarify the intention behind your advice?</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"><div class="gmail-HOEnZb"><div class="gmail-h5">
> I thought I did. But indeed, I misread the man page [1], which plainly<br>
> states that this is about excluding _targets_.<br></div></div></blockquote><div><br></div><div>By “this”, you refer to DH_GOLANG_EXCLUDES, I assume.</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"><div class="gmail-HOEnZb"><div class="gmail-h5">
><br>
> OTOH, the example debian/rule extract is:<br>
><br>
>><br>
>> # We want to ship only the library packages themselves, not the accompanying<br>
>>             # example binaries.<br>
>>             export DH_GOLANG_EXCLUDES := examples/<br>
><br>
><br>
> And in this case, we probably want to put the examples where they<br>
> belong, that is, in the documentation, not in the source.<br>
> So that mean we have to delete them from the source once they are copied.<br>
><br>
> What if we didn't copy them in the first place ?<br>
><br>
> I had a look at the code of dh-golang, and it wouldn't be difficult to<br>
> use the same logic we have in the configure sub in install.<br>
><br>
> I actually started a branch to implement it.<br>
><br>
><br>
> Now the issue is dealing with this change (if it's indeed a change, it<br>
> could be an addition):<br>
><br>
> - we could just decide to honor the excludes in install, at least for<br>
> sources and maybe bin. Then we have to deal with legacy, just in case,<br>
> so it should be possible to trigger it off, and at some point in the<br>
> future, we decide if it becomes the default or not.<br>
><br>
> - we could just add another variable for sources, and maybe also for<br>
> bin. It looks counter-intuitive to me, but then it allows to deal with<br>
> the case when you want to exclude something from source but not from<br>
> target (regex problem, similar dir names in subdirs, etc).<br>
><br>
> What do you think ?<br>
><br>
> Cheers,<br>
><br>
<br>
<br>
</div></div><div class="gmail-HOEnZb"><div class="gmail-h5">______________________________<wbr>_________________<br>
Pkg-go-maintainers mailing list<br>
<a href="mailto:Pkg-go-maintainers@lists.alioth.debian.org">Pkg-go-maintainers@lists.<wbr>alioth.debian.org</a><br>
<a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers" rel="noreferrer" target="_blank">http://lists.alioth.debian.<wbr>org/cgi-bin/mailman/listinfo/<wbr>pkg-go-maintainers</a></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Best regards,<br>Michael</div>
</div></div>