<div dir="ltr">Hi Rafael,<div><br></div><div>No worries. Thanks for fixing it so quickly.<div><br></div><div>Just so I understand - the plan is ultimately to add these checks to sys::fs::remove? (Or add a "safe" remove variant that does the checks)?</div>
<div><br></div><div>Once that's done we should be able to re-apply your patch, right?</div><div><br></div><div>Cheers,</div><div>Lang.</div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Jun 17, 2013 at 12:56 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 17 June 2013 15:48, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>
> Hi Rafael,<br>
><br>
> This broke an internal tester. The eraseFromDisk() method contains some<br>
> checks for special files to prevent it from removing anything important.<br>
><br>
> We really want to keep those checks in place. I'm not familiar with the path<br>
> library work - what is the best way to resolve this? Should this be<br>
> reverted, or can the checks be added to sys::fs::remove?<br>
<br>
<br>
</div>Thanks for catching this. I reverted this in 184111. I will add the<br>
missing functionality to PathV2.h.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>