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.

Sign up for GitHub

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

Jump to bottom

Environment variable to disable git repo checks #417

Closed
SelfhostedPro opened this issue Jun 30, 2022 · 1 comment · Fixed by #529
Closed

Environment variable to disable git repo checks #417

SelfhostedPro opened this issue Jun 30, 2022 · 1 comment · Fixed by #529
Labels
enhancement New feature or request

Comments

@SelfhostedPro
Copy link

Context

I'm using a submodule to use one of my unity projects inside of a flutter app and have both built at the same time in the same repo. I'm running into the following error when building the unity side of things even though all of the files required to build the project are there:

Failed to run "git rev-list --count 1ae98cf9e5d20e8bd637b9efe21d2e25c4de0424".
 In-command error caught: Error: The process '/usr/bin/git' failed with exit code 128

Suggested solution

Add an environment variable that disables checking the git repo in the job.

Considered alternatives

Forking the project and adding this myself for my own use.

Additional details

https://discord.com/channels/710946343828455455/710946344264794219/991749950780223598
@SelfhostedPro SelfhostedPro added the enhancement New feature or request label Jun 30, 2022
@ErcinDedeoglu
Copy link

ErcinDedeoglu commented Jan 2, 2023 •
edited
Loading

I have same problem and this blocks to move forward with game-ci. Now investigating another solution... :/
x

AndrewKahr added a commit to AndrewKahr/unity-builder that referenced this issue Mar 5, 2023
@AndrewKahr
Fix for game-ci#417. When pulling in other git histories, the github.…
89cf8ec 
…sha is invalid. Thus by using HEAD we will version based on the checked out sha. Tested with PR, commits, and tags and was unable to reproduce any extra counting when comparing outputs between HEAD and explicit sha values regardless of detached head state.
@AndrewKahr AndrewKahr mentioned this issue Mar 28, 2023
Merged
3 tasks
AndrewKahr added a commit that referenced this issue Mar 28, 2023
@AndrewKahr
V3 Updates (#529)
3032a4a 
- Add missing unityLicenseServer input (Fix #480)
- Use HEAD when calculating semantic version number. This is a riskier change as this has always used `github.sha` on the runner. However, when pulling in other repos and running the action, it may not be referencing the correct commit on the repo. After testing, though, nothing appears to be broken so this in theory should work fine. (Fix #417)
- Setup private token rewrites on Windows images (Fix #428)
- Allow setting a custom workspace path within docker container with `dockerWorkspacePath`. (Fix #433)
- [Breaking Change] Remove `androidAppBundle` parameter in favor of `androidExportType`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

V3 Updates AndrewKahr/unity-builder
2 participants
@ErcinDedeoglu @SelfhostedPro