<div dir="ltr"><div class="gmail_quote"><div dir="ltr"><div class="gmail_extra">Resending with a useful subject line</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On 20 April 2015 at 13:00,  <span dir="ltr"><<a href="mailto:bash-completion-devel-request@lists.alioth.debian.org" target="_blank">bash-completion-devel-request@lists.alioth.debian.org</a>></span> wrote:</div><div class="gmail_quote"><br></div><div class="gmail_quote">[snipped digest mail]<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">Date: Sun, 19 Apr 2015 16:05:30 +0300<br>
From: Ville Skytt? <<a href="mailto:ville.skytta@iki.fi" target="_blank">ville.skytta@iki.fi</a>><br></span>Message-ID:<br>
        <<a href="mailto:CABr9L5ADihGZEx08Qp3OBhCuP0CPyuJ2yHUyMJ0Ze0U2%2BdN9tg@mail.gmail.com" target="_blank">CABr9L5ADihGZEx08Qp3OBhCuP0CPyuJ2yHUyMJ0Ze0U2+dN9tg@mail.gmail.com</a>><br></blockquote><span class=""><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Sun, Apr 5, 2015 at 5:31 PM, Tristan Wibberley<br>
<<a href="mailto:tristan.wibberley@gmail.com" target="_blank">tristan.wibberley@gmail.com</a>> wrote:<br>
<br>
> Secondly, please would you have a look at the bundle and consider applying<br>
> it?.<br>
<br>
I started looking into it. For me this is the first time I've come<br>
across git bundles and I must say that for contributions like this I<br>
find the format-patch/send-email approach which is documented in our<br>
README *much* preferrable. Could you please follow that in the future?<br></blockquote><div><br></div></span><div>Certainly, it was also the first time I used git bundles so I thought I'd try it out. I didn't  know about the README's requirements because I first found the mailing list via google.</div><span class=""><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Anyway, about the first commit. I looked into what make produces off<br>
the modified Makefile, but it did not add any whitespace to any "#<br>
Files" or "# Not a target" lines, but I do see more than one space in<br>
the "# File is an intermediate prerequisite" line.</blockquote><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Could you clarify<br>
what the two first modifications are for, i.e. do you have a case<br>
where make produces output that would require these adjustments?<br></blockquote><div><br></div></span><div>As I recall, nothing requires the modifications to the match for '# Files' and '# Not a target' except for good practice. Since the output in Ubuntu changed in way that nonobviously broke completion I thought that I would make the script robust to the wider class of format changes so that those features don't vanish - unnoticed - if somebody unifies the output of make. Ie, I applied the network rule based on this experience.</div><span class=""><div>  </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
The variable patch seems innocent enough, I've applied it.<br></blockquote><div><br></div></span><div>Great, thanks.</div><div><br></div><div><br></div><div>Any thoughts on the problem with the test case for the hidden target completion?</div><div><br></div><div><br></div><div>Thanks</div><div>--</div><div>Tristan</div></div></div></div>
</div><br></div>