The idea was that this would sit in the head which is the final resting place for changesets. You wouldn't pull changesets in and out all the time, and if you're going forwards or backwards within revisions from the head you probably want to recompile anyway. The revisions you're making locally wouldn't have the hook applied and wouldn't change until after you'd (hopefully) finalized everything. I'm not trying to push the idea, but it doesn't seem like having to recompile unnecessarily is anything you'd actually have to worry about.

Gabe

Ali Saidi wrote:
This diff stamps a file adds the mercurial id to the output, however Nate complained that if the revision in the working directory changed (which could happen if you qdeleted a patch) then you would have to recompile (although not much just a re-linking). Something that could fix that (that I didn't try) would be to remove the build dependency from that file, but you would have to add a dependency to every other file in the build that would execute some python that would cause that file to regenerated.

Ali



On May 21, 2008, at 4:08 PM, Gabe Black wrote:

I've been thinking about how this could actually be done, and it seems to me that there could be a hook in the head (hooks are propagated, right?) which adjusted a header file every time a changeset was pushed into the head to have that hex value in a variable. The adjustment to the header file would be part of the changeset so that moving around revisions would keep it consistent, and I don't know if you could really do it any other way. That would ensure that unless the downstream users explicitly modify that header file for some reason, which would be a little weird, the output would reflect the last changeset that was stamped by the root. One downside to all this is that that file would change constantly and clutter the history and the repository metadata.

Gabe
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev


------------------------------------------------------------------------

_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to