<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 29, 2014 at 7:42 AM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<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">
Integrating bashate into something as complicated as devstack, the file<br>
ignore problem has come up.<br>
<br>
We seem to have 3 approaches out under review right now:<br>
<br>
<a href="https://review.openstack.org/#/c/117425" target="_blank">https://review.openstack.org/#/c/117425</a> : --exclude-dirs<br>
<a href="https://review.openstack.org/#/c/115794" target="_blank">https://review.openstack.org/#/c/115794</a> : --exclude-dirs (different<br>
implementation)<br>
<a href="https://review.openstack.org/#/c/113892" target="_blank">https://review.openstack.org/#/c/113892</a> : removing hidden directories<br>
<br>
I'm actually kind of convinced now that none of these approaches are<br>
what we need, and that we should instead have a .bashateignore file in<br>
the root dir for the project instead, which would be regex that would<br>
match files or directories to throw out of the walk.<br>
<br>
I think that would handle the concerns that everyone is having, and<br>
hopefully provides a more clear set of semantics in integrating.<br>
<br>
Anyone up for taking a stab at this patch?<br></blockquote><div><br></div><div>I started the other night and ran into the usual semantic problems wrt meaning...rather than re-invent this wheel I found the pathspec module another new dependency!) that purports to do .gitignore-style handling, only it doesn't.  It's closer to  rsync include file syntax.  I managed to get it really close only to fail on handling bare directories properly.  Example:</div>
<div><br></div><div>Ignoring a doc directory in .gitignore:</div><div>doc</div></div><div><br></div><div>Ignoring a doc directory in my trial:</div><div>doc/</div><div><br></div><div>It occurs to me that fixing this too means maybe I started down the wrong path.  This matters to be because I want to also leverage the existing .gitignore files we have.</div>
<div><br></div><div>Just to join the party I pushed up the working state in <a href="https://review.openstack.org/117772">https://review.openstack.org/117772</a>.</div><div><br></div><div>dt</div><div><br></div>-- <br><br>
Dean Troyer<br><a href="mailto:dtroyer@gmail.com">dtroyer@gmail.com</a><br>
</div></div>