ryandesign at macports.org
Sun May 29 16:41:09 PDT 2011
On May 29, 2011, at 16:39, Joshua Root wrote:
>> Revision: 79008
>> Author: fotanus at macports.org
>> Date: 2011-05-29 14:27:53 -0700 (Sun, 29 May 2011)
>> Log Message:
>> Added post-destroot phase to macports.
>> Currently, it does nothing.
>> Modified Paths:
>> Added Paths:
> It seems like this could easily be confused with post-destroot. Does
> your project really need a new target, or could your code be put in a
> proc run from e.g. destroot_finish?
> If you really do need a new target, I would suggest calling it something
> more distinctive, perhaps based on what it does rather than when it
> happens to run.
Yes, this commit surprised me. We already have a destroot phase, which of course has pre-destroot and post-destroot hooks, just as the other phases have pre- and post- hooks. Now you're making a new post-destroot phase... will there be pre-post-destroot and post-post-destroot hooks? What if a port has a post-destroot block -- is that a post- hook of the existing destroot phase, or does it overwrite this new post-destroot phase? This is confusing.
It seems like it should be possible for these checks to occur at the end of the destroot phase, after all its post-destroot blocks have run, and this would be better than making a whole new phase.
More information about the macports-dev