Hi Jonas,<br><br><div class="gmail_quote">On Sat, May 22, 2010 at 3:55 PM, Jonas Smedegaard <span dir="ltr"><<a href="mailto:jonas@jones.dk">jonas@jones.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Thu, May 20, 2010 at 03:33:52PM -0500, Rudy Godoy wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Please add cdbs_curpkg for DEB_DH_INSTALLINIT_ARGS. Currently it<br>
does apply for each target binary package.<br>
</blockquote>
<br></div>
Ah, stupid me [asking for a patch without noticing that indeed you already provided one] :-P<br>
<br>
Thanks for the patch! It is flawed, so a good example of how small changes can go bad: You _replace_ DEB_DH_INSTALLINIT_ARGS with the package-specific variant of it. The proper would be to still use DEB_DH_INSTALLINIT_ARGS as fallback.<br>
<br></blockquote><div><br>I'm afraid I'm a bit lost on this. You can blame my re-involvement. <br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
But flawed or not - it helps me more clearly understand exactly what is your needs. Now I'll do my best to fix it to not hurt potential needs of others. :-)<br>
<br></blockquote><div><br>Great.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
...and while we are at it: It won't hurt to also add an *_ALL variant which is always applied, in addition to either the main variable or the package-specific override. I.e. use the (also recently added) cdbs_expand_curvar.<br>
<br></blockquote><div><br><br>Great. As I put on my report, the current behabiour doesn't actually hurt, however it doesn't make much sense to also pass it to packages who are not installing an init script.<br><br>
regards,<br>Rudy<br><br></div></div>-- <br>Rudy Godoy<br><a href="http://htu.pe">http://htu.pe</a> - Internet Software<br>