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

[bazel] updated to allow pinning of bazel version #1283

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

[bazel] updated to allow pinning of bazel version #1283

wants to merge 2 commits into from

Conversation

cjac
Copy link
Contributor

@cjac cjac commented Dec 24, 2024

  • now using environment variables more
  • using gpg --import instead of gpg --dearmor
  • must set default bazel using update-alternatives or bazel is not in PATH, only bazel-7.4.0
    • Although this could possibly be solved by replacing all calls to bazel in our code with bazel-7.4.0 this does not hold true for others' code which does not know to change the abi

@cjac
updated to allow pinning of bazel version
274f717 
* now using environment variables more
* using gpg --import instead of gpg --dearmor
* must set default bazel using update-alternatives or bazel is not in
  PATH, only bazel-7.4.0
  * Although this could possibly be solved by replacing all calls to
    bazel in our code with bazel-7.4.0 this does not hold true for
    others' code which does not know to change the abi
@cjac cjac self-assigned this Dec 24, 2024
@cjac
Copy link
Contributor Author

cjac commented Dec 24, 2024

/gcbrun

@cjac
Copy link
Contributor Author

cjac commented Dec 24, 2024

/gcbrun

cjac added a commit to LLC-Technologies-Collier/initialization-actions that referenced this pull request Dec 24, 2024
@cjac
see GoogleCloudDataproc#1283
be0af99
@cjac cjac mentioned this pull request Dec 24, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayadeep-jayaraman jayadeep-jayaraman Awaiting requested review from jayadeep-jayaraman

At least 1 approving review is required to merge this pull request.

Assignees

@cjac cjac

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cjac