> After looking at the code I think that the easiest way to avoid
> monkey patching at all would be inheritance.
Inheritance won't work :(
> * Replaces `Archive::Zip->new()` with `Local::AZM->new()`.
Because Local::AZM replaces Archive::Zip::Member not Archive::Zip.
> Monkey patching without importing other packages is also doable,
> but I think inheritance is cleaner here.
So I ended up doing a manual monkey patch:
https://salsa.debian.org/reproducible-builds/strip-nondeterminism/-/merge_requests/16
JFYI, tests pass locally, but the pipeline failed on something
that totally unrelated:
gbp:error: Cannot find pristine tar commit for archive
'strip-nondeterminism_1.13.1.orig.tar.bz2'
Hope this helps,
Alex
--
⢀⣴⠾⠻⢶⣦⠀
⣾⠁⢠⠒⠀⣿⡁ Alex Muntada <alexm@debian.org>
⢿⡄⠘⠷⠚⠋ Debian Developer 🍥 log.alexm.org
⠈⠳⣄⠀⠀⠀⠀
Attachment:
signature.asc
Description: PGP signature