← Back to issue list

elf: cache crawled files

View original Github issue

Metadata

Project
snapcraft
Number
#1925
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Implemented ELF file caching to reduce package prime times. Approved by two reviewers, passed CI checks, and merged into the main branch with 114 lines added across six files.

Suggested action:

No scores available.

Issue body

LP: #1746607 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Implemented ELF file caching to reduce package prime times. Approved by two reviewers, passed CI checks, and merged into the main branch with 114 lines added across six files.
qwen3.6-35b-a3b-mtp-q6 Merged to cache crawled ELF files, resolving LP #1746607. The optimization significantly reduced prime time for ros-talker-listener. After minor CI flakiness, the change passed tests and was integrated.
qwen3.6-35b-a3b-mtp-q6 Merged. Implements ELF file crawling cache to reduce build priming times. Resolved initial Travis CI LXD failures before integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1989 elf: don't parse elf more than necessary snapcraft merged Merged a single-line change optimizing ELF parsing to avoid unnecessary work, yielding a minor performance improvement. Approved by a maintainer and passed CI checks.
71%