Overeager pattern (lib_re) in libdir QA check resulting in: WARNING: QA Issue: soletta-dbg: found library in wrong location: /usr/share/gdb/auto-load/libsoletta.so.0.0.1-gdb.py [libdir]
Created attachment 3058 [details] Proposed patch Add gdb dir to INSANE_SKIP list. See attached patch. Joshua, I'm not on the soletta mailing list. If you are, can you submit? (I don't have the setup to test this currently, but from what I've read and seen, it does seem to be the correct fix.)
Apologies, the original bug description could have been much better. The bug isn't in the soletta recipe specifically. The issue is that the test is hitting false positives, likely due to the lib_re regex in insane.bbclass being too greedy. The test is matching something which clearly isn't a library — /usr/share/gdb/auto-load/libsoletta.so.0.0.1-gdb.py — because the filename begins with /lib and contains "so.", then reporting that as an error because the matched file isn't in ${libdir} or ${libexecdir}.
Created attachment 3059 [details] Revised patch to solve the real issue Ok, now that I understand the real issue, the fix is straight forward. See attachment #2 [details]. Tested the new re using the tool on this website: https://regex101.com/ and several real examples of library files.
(In reply to comment #3) > Created attachment 3059 [details] > Revised patch to solve the real issue > > Ok, now that I understand the real issue, the fix is straight forward. See > attachment #2 [details]. Tested the new re using the tool on this website: > https://regex101.com/ and several real examples of library files. Looks good to me, could you submit that to OE-Core? Thanks Bill!
Patch send to oe-core mailing list.
Created attachment 3062 [details] Third patch attempt Ok, so this checks to verify the file is an ELF file before declaring it to be in the wrong location. The re checks are loosened up (basically back to what they were originally) to allow library files with non-standard names to be included in the check. I still need to test this and verify "rel_path" is the correct entity to run the ELF check on vs. one of the other path options in that block of code.
Created attachment 3087 [details] fourth patch Verified with local testing using a real ELF library and a non-ELF file that the ELF test does work and that full_path is the proper value to pass to the ELFFile() function.
Fixed in oe-core a3ad36b9a435e7c3d97f114809561198b8abe6cf.