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

Upgrade git-lfs to support local paths #89

Merged
merged 2 commits into from
Sep 28, 2023

Conversation

kevinbreit-garmin
Copy link
Contributor

git-lfs support was added, but unfortunately versions before 2.10 don't support local paths.

This can make testing difficult with a bitbake recipe that points to a local path for testing, when that local repository uses LFS. To fix this, we can pull in newer versions of git-lfs like 16.04 had. To make sure git-lfs is new enough, add a test for it. The minimum git-lfs release is v2.10.

@pep8speaks
Copy link

pep8speaks commented Sep 26, 2023

Hello @kevinbreit-garmin! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-09-27 15:12:14 UTC

This change upgrades git-lfs, in all images that used it, to a newer
version that is at least 2.10 that supports local paths. This enables
lfs with local remotes within the containers to function.
This adds a test of the git-lfs version to be at least
2.10, when local file path support was added.
@JoshuaWatt JoshuaWatt merged commit 62b71da into garmin:master Sep 28, 2023
21 checks passed
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.

3 participants