<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-09-04 15:43 GMT+02:00 Stephan Bergmann <span dir="ltr"><<a href="mailto:sbergman@redhat.com" target="_blank">sbergman@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 09/04/2017 03:30 PM, Tamas Zolnai wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I just noticed that for makefiles (*.mk) git does not check whether the commit contains tab indentation. As I know we use spaces also for makefiles for indentation, right? So I added a patch changing pre-commit hook to check tab also for makfiles:<br>
<a href="https://gerrit.libreoffice.org/#/c/41897/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org<wbr>/#/c/41897/</a><br>
I hope it's ok. I don't know who used to care of git stuff.<br>
</blockquote>
<br></span>
I think temporary GNU make still needs recipe lines to start with tab characters?  And in any case, our makefiles are apparently written with lots of tab characters all over the place.  I'd suggest to revert (at least for now).</blockquote><div><br></div><div>Yeah, I see that tabs and spaces are mixed in makefiles. That's why I thought it would be good to make it consistent similar to other source files.</div><div>I did not push it yet. It's on gerrit. So I can abandon it.<br></div></div><br></div></div>