| Summary: | Cleanup package.bbclass - runstrip and split_and_strip_files checks for kernel modules | ||
|---|---|---|---|
| Product: | [Build System, Metadata & Runtime] Meta-yocto | Reporter: | Mark Hatle <mark.hatle> |
| Component: | meta-yocto | Assignee: | Sam Kent <sam> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | Medium | CC: | bluelightning, jessica.zhang, mateusz.hercun, mathieu.dubois-briand, mhalstead, paul, poky.bs.watcher, poky.watcher, randy.macleod, richard.purdie, sam |
| Version: | 1.2 | ||
| Target Milestone: | 6.1 | ||
| Hardware: | x86 | ||
| OS: | Multiple | ||
| Whiteboard: | NEWCOMER | ||
| OS type for building Yocto: | --- | Type of Regression: | --- |
| Verified: | Documentation change: | Don't know | |
|
Description
Mark Hatle
2012-04-19 14:25:59 UTC
The code has changed a bit but the idea is likely still valid. Hello. I want to fix this issue but I'm not sure what it means 'runstrip and split_and_strip_files checks do not match how they find kernel objects.' Mateusz, It's great that you want to work on this. I only have a minute but this might get you started: poky.git on master ❯ rg "def runstrip" meta/lib/oe/package.py 22:def runstrip(file, elftype, strip, extra_strip_sections=''): ❯ rg split_and_strip_files meta/classes-global/package.bbclass 22:# d) split_and_strip_files - split the files into runtime and debug and strip them. 532: oe.package.process_split_and_strip_files(d) Take a look at those files and search docs.yoctoproject.org and then come back with more questions. Hi I'm new to contributing and had a look at this to start with something straight forward whilst getting used to using a mailing list Submitted a patch (hopefully correctly) to the mailing list https://lists.openembedded.org/g/openembedded-core/topic/patch_1_2_package_py_fix/118935412 Thanks Sam. I've given this enhancement to you. We're just wrapping up the 6.0 release so your patch may not be merged until after that's done, say next week. Have you read: https://docs.yoctoproject.org/contributor-guide/index.html and did you think there was anything missing from that guide? I see your patches but I'm not the right person to review them. Welcome to the team! ../Randy Sam, Patch reviewed on weekly meeting and is okay but needs a self-test. Paul will reply on the list and point you in the right direction. Thanks, I've submitted a V2 with a self test I used Claude to help get to grips with oe-selftest and have updated the commit message according to the contributor guidelines Patch got merged: https://git.openembedded.org/openembedded-core/commit/?id=2f32e6a01cdd18c3a7ade348c7a50b0cd459590f Mark, can we close the bug entry? RP says fixed. Thanks Sam! |