Code review comment for lp:~lasall/ppa-purge/dryrun

Revision history for this message
Tim Lunn (darkxst) wrote :

I guess this branch is dead, but incase anyone else decides to work on it.

You don't need to back up UNIT64_MAX files, just the one. the changes should also be reverted in the trap handler. Alternatively it may be sufficent to just revert the changes i.e remove the "# " in a trap handler and exit failure.

« Back to merge proposal