<html><body bgcolor="#FFFFFF"><br><blockquote type="cite"><div><span></span><span>AFAICS there is no more than 30 places.</span><br></div></blockquote><div><br></div>Yep, my grep was a bit loose :)<div><br><blockquote type="cite">
<div><span></span><span></span><span>As you describe it it would be an unnecessary abstraction and extra complexity and more verbose code without any benefit. I doubt a patch for that would be accepted. </span></div></blockquote>
<div><br></div><div>Admittedly I'm new to python, but I have a lot of experience in other languages. I don't consider moving a commonly used string into a constant an unnecessary abstraction: I'd generally call it best practice. Not trying to start an argument here: your codebase; your rules. I'm just trying to understand the objection.</div>
<br><blockquote type="cite"><div><span>A repo made by a Mercurial hacked as you describe it would no longer be a Mercurial repository. One of Mercurials core values is that the repository format only change in backward compatible ways.</span><font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#0023A3"><br>
</font></font></div></blockquote><div><br></div><div>I disagree. So long as the variable was set to ".hg" it would be perfectly backwards compatible. </div><div><br></div><div>I'm not talking about any fundamental change here: just moving a string into a variable. It's only dangerous to backwards comparability if someone chooses to change the variable in their own build and then tries to use it on a "mainstream" hg repository - which is true for any custom change to the codebase. Surely this is an inherent risk in any open source project?</div>
<br><blockquote type="cite"><div><span>Note that Mercurial is GPLv2+. If you ever distribute your application it would have to be under a compatible license. That should however not be a problem if it is for internal use within your own legal entity only.</span><br>
</div></blockquote><div><br></div>Unless I misunderstand the GPL (which is always a possiblity :)), calling a GPL'd exe places no requirements on the calling app. Of course we would have to make and changes we made to the Hg source publicly available. <span class="Apple-style-span" style="-webkit-tap-highlight-color: rgba(26, 26, 26, 0.292969); -webkit-composition-fill-color: rgba(175, 192, 227, 0.230469); -webkit-composition-frame-color: rgba(77, 128, 180, 0.230469); ">Please let me know if you think I've got the wrong end of that stick.</span></div>
<div><br></div><div>Thanks for taking the time to talk this through.</div><div><br></div><div>Cheers,</div><div><br></div><div>Ben<br><div><br></div><blockquote type="cite"><div><span></span></div></blockquote></div></body></html>