On Thu, Jan 17, 2008 at 02:06:54PM -0500, Johnny C. Lam wrote:
> Joerg Sonnenberger wrote:
>> whoever cares enough about undo should finish the attached patch.
>> E.g. tarup the package before replace and run pkg_add -u with it for
>> undo. As Alistair objects the patch without that and I don't care enough
>> to waste more time on it, it won't get committed.
>
> FWIW, I think the whole "replace" code needs to be ripped out and
> rewritten. When I wrote the current "replace" module, I foolishly tried to
> templatize the "replace" action so that different package tools could
> override the steps as needed. Clearly, it's much too complex as-is -- I
> barely understand the code there anymore.
The destdir case with this patch is just pkg_add -u ${pkgname}.
Joerg