<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Sep 27, 2013 at 7:26 AM, "Martin v. Löwis" <span dir="ltr"><<a href="mailto:martin@v.loewis.de" target="_blank">martin@v.loewis.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am 26.09.13 15:52, schrieb Eli Bendersky:<br>
<div class="im">> * Should I always check-in Python-ast.h and Python-ast.c when I touch<br>
> asdl* ? The generated files are unchanged, it's only the timestamp that<br>
> changed.<br>
<br>
</div>If they really didn't change, I don't think it matters much. I believe<br>
there is a fundamental problem in Mercurial which fails to get it<br>
"right" - with the defense that one shouldn't check in generated files<br>
in the first place.<br>
<br>
"hg touch" is supposed to work around this limitation. So when you only<br>
check in the generator, anybody updating should do "make touch" after<br>
the update, which should touch the generated files even though they<br>
didn't change.<br>
<div class="im"><br>
> * Can we, in theory, use new Pythons for asdl* code, because<br>
> Python-ast.* are, in fact, checked in so they don't have to be rebuilt<br>
> by the bots or users?<br>
<br>
</div>I'd say yes. That's the point of checking in generated files, so that<br>
users don't need to run the generator.<br></blockquote><div><br></div><div>Thank you for the explanation, Martin.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


<div class="im"><br>
> While we're at it, it seems that .hgtouch is wrong:<br>
><br>
> Include/ast.h: Parser/Python.asdl Parser/asdl.py Parser/asdl_c.py<br>
> Python/Python-ast.c: Include/ast.h<br>
><br>
> The file Include/ast.h is not,<br>
<br>
</div>It may well be incorrect - feel free to fix it.<br></blockquote></div><br></div><div class="gmail_extra">Thanks. There are a couple of things to address before we can try to actually run 'make touch' on all bots. I opened <a href="http://bugs.python.org/issue19106">http://bugs.python.org/issue19106</a> to track this.<br>

<br>Eli<br><br><br><br></div><div class="gmail_extra"><br><br></div></div>