<div dir="ltr">On Mon, Sep 25, 2017 at 2:59 AM Eli Schwartz <<a href="mailto:eschwartz@archlinux.org">eschwartz@archlinux.org</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 09/24/2017 08:52 PM, Luke Shumaker wrote:<br>
> On Sun, 24 Sep 2017 20:08:51 -0400,<br>
> Eli Schwartz wrote:<br>
>> On 09/24/2017 06:56 PM, Luke Shumaker wrote:<br>
>>> This mirrors dbscripts commit<br>
>>> 625fa02 by Pierre Schmitz <<a href="mailto:pierre@archlinux.de" target="_blank">pierre@archlinux.de</a>> at 2017-04-18 14:20:49<br>
>>> ---<br>
><br>
>>> +. /usr/share/makepkg/util.sh<br>
>><br>
>> Please just source the actual parts we need, the same way<br>
>> <a href="https://git.archlinux.org/dbscripts.git/commit/?id=625fa02" rel="noreferrer" target="_blank">https://git.archlinux.org/dbscripts.git/commit/?id=625fa02</a> also only uses<br>
>> /usr/share/makepkg/util/message.sh<br>
>> /usr/share/makepkg/util/util.sh<br>
>> /usr/share/makepkg/util/pkgbuild.sh<br>
><br>
> I was wondering if someone was going to call me on that.  I suppose I<br>
> should have noted in the commit message that 2 days later, in commit<br>
> 76f95dd, Pierre changed it to use /usr/share/makepkg/util.sh instead;<br>
> so I mimicked that.  AFAICT, that change was unrelated to the rest of<br>
> 76f95dd, but it's quite a large patch, and I didn't look at it too<br>
> closely.<br>
><br>
> <a href="https://git.archlinux.org/dbscripts.git/commit/?id=76f95dd" rel="noreferrer" target="_blank">https://git.archlinux.org/dbscripts.git/commit/?id=76f95dd</a><br>
<br>
Hah! :p<br>
<br>
Well, this seems like bloat anyway, maybe I should ask Pierre to change<br>
it back...<br></blockquote><div><br></div><div>It seems simpler to me, even if it loads more code than needed. If everyone sources util.sh instead of its components directly, makepkg can shuffle the internals around without breaking consumers. This is also a common pattern seen in C includes.</div><div><br></div><div>I'll apply this patch as-is now. We can still change it afterwards.<br></div><div><br></div><div>Luke, thanks for your continued work on devtools.<br></div></div></div>