<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 8, 2017 at 10:33 AM, Niels Thykier <span dir="ltr"><<a href="mailto:niels@thykier.net" target="_blank">niels@thykier.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Michael Stapelberg:<br>
<span class="">> On Sat, Apr 8, 2017 at 7:54 AM, Niels Thykier <<a href="mailto:niels@thykier.net">niels@thykier.net</a>> wrote:<br>
><br>
>> Michael Stapelberg:<br>
>>> On Mon, Apr 3, 2017 at 1:07 PM, Niels Thykier <<a href="mailto:niels@thykier.net">niels@thykier.net</a>> wrote:<br>
>>><br>
>>>> [...]<br>
>>><br>
>>><br>
>>> Thanks a lot!<br>
>>><br>
>>> In turn, I created a branch in the freeradius packaging:<br>
>>> <a href="https://anonscm.debian.org/cgit/pkg-freeradius/freeradius.git/log/?h=dh_" rel="noreferrer" target="_blank">https://anonscm.debian.org/<wbr>cgit/pkg-freeradius/<wbr>freeradius.git/log/?h=dh_</a><br>
>> missing<br>
>>><br>
>><br>
>> Thanks for trying it out. :)<br>
>><br>
>>> Maybe I did the wrong changes (?), but when building, I get an error:<br>
>>><br>
>>> [...]<br>
>>><br>
>>> Can you reproduce the issue?<br>
>>><br>
>>><br>
>>><br>
>>> [...]<br>
>><br>
>> I think it is a bug in the freeradius changes.  In this commit:<br>
>><br>
>> <a href="https://anonscm.debian.org/cgit/pkg-freeradius/" rel="noreferrer" target="_blank">https://anonscm.debian.org/<wbr>cgit/pkg-freeradius/</a><br>
>> freeradius.git/commit/?h=dh_<wbr>missing&id=<wbr>58e693ff7dd6603598cb132d44c9e9<br>
>> a547f099ed<br>
>><br>
>> The dh_install call is replaced by dh_missing, but AFAICT that<br>
>> dh_install is inside an override_dh_install, so dh_install is never called.<br>
>><br>
><br>
> Thanks, you are spot on. After re-adding the dh_install command, most of<br>
> the errors vanish. Interestingly enough, this one remains:<br>
><br>
</span>> [...]<br>
<span class="">><br>
> I’ve added it to debian/freeradius.install and updated the branch.<br>
><br>
<br>
</span>Sounds good.  I am a little surprised that this gave issues now and not<br>
before.  This smells like it might cause breakage with this change.<br>
<br>
Speaking of stuff that might break: We had forgotten to pass --sourcedir<br>
to dh_missing when dh_install calls it (fixed now).<br>
<span class=""><br>
> The only remaining issue as far as the freeradius packaging is concerned is<br>
> dh_installdocs not yet reporting which files it installs — hence the build<br>
> fails at<br>
> <a href="https://anonscm.debian.org/cgit/pkg-freeradius/freeradius.git/commit/?h=dh_missing&id=afbb47b0f8646de9fc116d8f494608ef4cef6a90" rel="noreferrer" target="_blank">https://anonscm.debian.org/<wbr>cgit/pkg-freeradius/<wbr>freeradius.git/commit/?h=dh_<wbr>missing&id=<wbr>afbb47b0f8646de9fc116d8f494608<wbr>ef4cef6a90</a><br>
><br>
<br>
</span>Ah.  I have deliberately omitted dh_installdocs as I thought people<br>
would normally use dh_install for that kind of thing.  I think I will<br>
keep that behaviour for a while...<br>
<br>
Maybe with a documentation change for dh_installdocs that dh_install<br>
might be better for documentation installed by upstream's build system.<br></blockquote><div><br></div><div>Ah, this was news to me. I’ve changed the dh_missing branch to install the files using dh_install, and now everything works fine :).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Either way, what you have in the dh_missing branch right now is already<br>
> useful :).<br>
><br>
><br>
</span>>[...]<br>
<br>
Thanks for the initial patches and the testing. :)<br>
<span class="HOEnZb"><font color="#888888"><br>
~Niels<br>
<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Best regards,<br>Michael</div>
</div></div>