Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

helpers: Add option to build MW at specific revisions #288

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JamiKettunen
Copy link
Contributor

The devel repos for many packages can move fast at times and cause breakages when building againt the current latest SFOS release, so having a convenient switch to quickly checkout a specific revision of a middleware component simply makes sense.

This works for both git tags as well as commit hashes.

helpers/build_packages.sh Outdated Show resolved Hide resolved
helpers/util.sh Show resolved Hide resolved
@JamiKettunen
Copy link
Contributor Author

Would it make sense to also git clean -df when switching revision? I know some building some packages leaves residue files in-tree that could potentially cause issues when building another version of the package.

@JamiKettunen JamiKettunen requested a review from Thaodan December 15, 2021 13:38
helpers/util.sh Outdated Show resolved Hide resolved
The devel repos for many packages can move fast at times and cause
breakages when building againt the current latest SFOS release, so
having a convenient switch to quickly checkout a specific revision of a
middleware component simply makes sense.

This works for both git tags as well as commit hashes.
Without this any submodules could get out of sync.
@0312birdzhang
Copy link
Contributor

I would ask when will this pr merged?
On 5.0.0.29 hybris-14.1 armv7hl base, the lastest geoclue-providers-hybris failed to build and the lastest sensorfw failed to start.
Really need specific revisions support on MW, not some hacks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants