AyanSinhaMahapatra
commented
3 weeks ago From #1339:
There seem to be too many "requires-review" as a first glance in the summary report from a d2d project for ELF.
https://github.com/aboutcode-org/back2source-data/blob/main/data/archive.fedoraproject.org/pub/fedora/linux/releases/40/Everything/x86_64/debug/tree/a/atomix-debugsource-44.0-4.fc40.x86_64/d2d-summary.json#L36 has only one requires review
https://github.com/aboutcode-org/back2source-data/blob/main/data/archive.fedoraproject.org/pub/fedora/linux/releases/40/Everything/x86_64/debug/tree/a/atomix-debugsource-44.0-4.fc40.x86_64/d2d-details.json#L3535 has many correctly mapped paths that are marked as requires-review likely incorrectly
Also we should not include directories and
<artificial>and<built-in>(and this may something we could filter out in the elf-inspector ?)
- like in https://github.com/aboutcode-org/back2source-data/blob/main/data/archive.fedoraproject.org/pub/fedora/linux/releases/40/Everything/x86_64/debug/tree/a/atomix-debugsource-44.0-4.fc40.x86_64/d2d-details.json#L3398
- or in https://github.com/aboutcode-org/back2source-data/blob/main/data/archive.fedoraproject.org/pub/fedora/linux/releases/40/Everything/x86_64/debug/tree/a/atomix-debugsource-44.0-4.fc40.x86_64/d2d-details.json#L3493
Update pipelines steps for the binary to source analysis for native ELF using strings and symbols, and not only DWARFs