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

License script - fix dependencies installation #33839

Merged
merged 1 commit into from
Feb 3, 2025
Merged

License script - fix dependencies installation #33839

merged 1 commit into from
Feb 3, 2025

Conversation

stankiewicz
Copy link
Contributor

@stankiewicz stankiewicz commented Feb 3, 2025 •
edited
Loading

fixes #33836

@stankiewicz stankiewicz changed the title fix source activation. License script - fix source activation. Feb 3, 2025
@github-actions github-actions bot added the build label Feb 3, 2025
@stankiewicz
Copy link
Contributor Author

Task :sdks:java:container:pullLicenses FAILED
./license_scripts/license_script.sh: 43: python3.9: not found

@github-actions github-actions bot removed the build label Feb 3, 2025
@stankiewicz stankiewicz changed the title License script - fix source activation. License script - fix dependencies installation Feb 3, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 3, 2025

Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:

R: @robertwb for label java.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

lostluck
lostluck approved these changes Feb 3, 2025
View reviewed changes
Copy link
Contributor

@lostluck lostluck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@lostluck lostluck merged commit edf7c90 into apache:master Feb 3, 2025
17 checks passed
tomstepp pushed a commit to tomstepp/apache-beam that referenced this pull request Feb 3, 2025
@stankiewicz @tomstepp
fix activate (apache#33839)
6c20493
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lostluck lostluck lostluck approved these changes

Assignees
No one assigned
Labels
docker java Next Action: Reviewers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug]: Beam's licence check is failing
2 participants
@stankiewicz @lostluck