Comment 2 for bug 2059303

Revision history for this message
Frank Heimes (fheimes) wrote :

Well, I already had a hard time to get the requested commits applied to noble (which is on 2.31.0).

I figured out that:
1) commit f6c6f0cc712433221fb0588c754e0d09884453dd ("rust/pv/test: Code + Certificate refactoring") is needed on top as pre-requisite, otherwise the other patches do not apply.
2) the commit id for ("libpv: Support `Armonk` in IBM signing key subject") is d7c95265cdb6217b0203efa5893c3a27838af63c (and not 5e1cb58a21ae0707d1993de3c8fc078c5cffed88 - this commit id does not exist in upstream master)
3) the commit id for ("pvattest: Fix root-ca parsing") is 2b5e7b049123aff094c7de79ba57a5df09471b2e (and not a54daf459e7504c0f42d3eb028100b7ab07894ff - again this commit id does not exist in upstream master).

I'm really wondering if it wouldn't be best to have a new minor version tagged upstream (like a 2.31.1) that includes everything needed, since I can't patch binary files with quilt (rust/pv/tests/assets/cert/der.crl and rust/pv/tests/assets/cert/der.crt), hence had to skip these hunks.