Forums » Software Development »
meta-mitydsp-l138 issue - SRCREV = "${AUTOREV}"
Added by Fred Weiser about 2 years ago
I'd like to report an issue in the meta-mitydsp-l138 (branch "daisy")repo where recipes use SRCREV = "${AUTOREV}". The specific issue I saw was caused by recent changes being pushed to the master branch of the u-boot-mitydspl138 repo, to which recipes in meta-mitydsp-l138 refer using SRCREV = "${AUTOREV}"; the changes broke the Yocto build I use as patches being applied fail. I might add that the recipe for our product was also written by Critical Link for a contract, and it also included SRCREV = "${AUTOREV}" (and the patches themselves). While functional at the time, the simple updates and pushes to u-boot-mitydspl138 broke quite a few things. My solution will be to revise the u-boot-mitydspl138 repo to have SRCREV point to the correct commit. I'd be happy to push the change for upstream if we can set up a merge request.
Meanwhile, I would recommend the general practice of always using specific commit hashes for all SRCREV commands in bitbake recipes to avoid further future problems. The practice assures any future build will exactly reproduce the build that was done originally.
Replies (2)
RE: meta-mitydsp-l138 issue - SRCREV = "${AUTOREV}" - Added by Bob Duke about 2 years ago
Fred,
I understand your concern. Your fix (set SRCREV to a specific commit) is the right one and we typically review that option with customers during delivery, although I missed reviewing that with you.
We typically have our Yocto builds use the latest versions of everything, but that can cause annoying problems if you are applying patches (as you discovered).
If you have any difficulty with adjusting the SRCREV or if you'd like me to create a separate stable branch on our support server, please let me know.
RE: meta-mitydsp-l138 issue - SRCREV = "${AUTOREV}" - Added by Fred Weiser about 2 years ago
I do hope you reconsider the practice of using AUTOREV. Meanwhile, to protect myself, I'll duplicate your repo, make a stable branch, and store them locally.
