On Mon, 2022-08-08 at 11:35 +0200, Alexander Kanavin wrote: > On Mon, 8 Aug 2022 at 10:23, Alexander Kanavin via > lists.openembedded.org <[email protected]> > wrote: > > Usage of patchelf in uninative seems to be contained in > > uninative_changeinterp, which can be tweaked > > to handle r/o files before invoking patchelf. Was it that all usages > > of it need to be tweaked similarly? > > Ok, I couldnt find other usages of it that would require similar > tweaking (icecc uses host patchelf if available, rust.inc/cargo.inc > operate on only a few static filenames), so maybe it's ok to move what > the patch does into uninative_changeinterp (add +r before invocation, > restore original permissions after). Still better than carrying a > rejected patch.
It feels like there is something we're missing. I wish I could remember what :(. Note that the testing cycle for changing patchelf is a pain as we need to merge, create a uninative release and upgrade to before we have the full cycle and know any change is fully working. Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#169078): https://lists.openembedded.org/g/openembedded-core/message/169078 Mute This Topic: https://lists.openembedded.org/mt/92886974/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
