Hi Christoph,
apologies for the delay. I was away at a conference.
Ch Lamprecht wrote:
Steffen Mueller wrote:
if anyone would like to take a look at this, checkout from
http://svn.openfoundry.org/par/Module-ScanDeps/branches/caching
I moved code to Module-ScanDeps, added tests and pod. The option for
ScanDeps is now named 'cache_file', scandeps.pl gets a switch '-C
CACHEFILE'.
I still do not know how the dependencies-cache-file option to pp should
be named. I used cd|cachedeps but I think it's a poor choice because
it's not obvious, that it requires a filename as an argument.
I just did the following:
1) Looked through your M::SD changes and ran tests. Everything seems to
be fine.
2) Release Module::ScanDeps one last time without your changes.
3) Merged over intermediate M::SD changes to your branch.
Next up are the following:
1) Merge over your M::SD changes to trunk and remove the branch.
2) Release a new Module::ScanDeps with caching capability.
3) Check the PAR::Packer changes and think about the pp option name.
4) Merge over your changes to PAR::Packer.
5) PAR::Packer developer release.
6) Wait.
7) Release stable PAR::Packer with caching.
8) Celebrate!
Thanks for your contribution! It's a very good idea and the
implementation is solid. The packaging process could really become
unnervingly slow at times.
Best regards,
Steffen
