-
Notifications
You must be signed in to change notification settings - Fork 0
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
Use git source to build #50
Conversation
bot, build com.revolutionarygamesstudio.ThriveLauncher.yaml |
Queued test build for com.revolutionarygamesstudio.ThriveLauncher.yaml. |
Started test build 128989 |
Started test build 128990 |
Build 128990 failed |
Build 128989 failed |
Does git LFS work now on the build servers? |
The error is:
Also occurs locally with |
It seems it is at least attempting to perform git lfs download. |
It seems flatpak/flatpak-builder#498 is being used? |
@Doomsdayrs looks like lfs is working, your issue is further up the build log:
This indicates some objects are missing from the source repository here. See git-lfs/git-lfs#4735 (comment) You might have to ask people to run |
I tried doing that push but it did nothing. As far as I can tell the So as far as I can tell the Git LFS server serving the Thrive Launcher content works fine, at least when used with Git LFS from my desktop ( Edit: could the problem be that the builder doesn't understand custom git LFS URLS? The Thrive Launcher LFS data is hosted at: https://dev.revolutionarygamesstudio.com/api/v1/lfs/launcher |
Something is wrong, but I don't think it's in flatpak-builder. It doesn't do a "regular" clone it does a sort of mirror clone of the repo. You can reproduce the error by doing:
|
Output with GIT_TRACE does not show anything useful, you might have to ask over at git lfs or git issue tracker/mailing list.
|
It does show that it does the wrong thing, it should not ask this from github server:
My conclusion is that, that way to initialize the repo does not take |
Yea it should be reported to git-lfs |
@hhyyrylainen @Doomsdayrs please add |
I think @Doomsdayrs was working on this, I haven't touched these changes at all except looking into what's going on with the server side of things and confirming that there was/is a git client side issue. |
Started test build 140256 |
Started test build 140257 |
bot, build com.revolutionarygamesstudio.ThriveLauncher.yaml |
Queued test build for com.revolutionarygamesstudio.ThriveLauncher.yaml. |
Started test build 140258 |
Build 140258 failed |
Build 140256 failed |
Build 140257 successful
|
Closes #12