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

fix: update download link for boost v1.74.0 #65

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

Shin-kyoto
Copy link
Contributor

@Shin-kyoto Shin-kyoto commented Jan 8, 2025

Description

As you see in boostorg/boost#997, https://boostorg.jfrog.io is out of service. So I updated the link.

Tests performed

I did NOT check this PR by running ansible.
@softyanija , @TetsuKawa , @Owen-Liuyuxuan, can you check this PR by running ansible?

Effects on system behavior

This PR may affect the environment, especially related to boost.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Copy link

@Owen-Liuyuxuan Owen-Liuyuxuan left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you

Copy link
Collaborator

@manato manato left a comment

Choose a reason for hiding this comment

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

@Shin-kyoto
Thank you for the prompt handling of the Boost downloading issue. LGTM.

Copy link
Contributor

@softyanija softyanija left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@Shin-kyoto
Copy link
Contributor Author

Shin-kyoto commented Jan 8, 2025

@softyanija
How did you check this PR by running ansible? I want to know the result 🙏

@Shin-kyoto Shin-kyoto merged commit b83b9f0 into main Jan 8, 2025
10 of 12 checks passed
@Shin-kyoto Shin-kyoto deleted the fix/boost-download-link branch January 8, 2025 06:27
@softyanija
Copy link
Contributor

@Shin-kyoto
I applied the url changes to the branch used by x2 gen2 (after apply) and built with evaluator, and the build completed successfully!
build result : https://evaluation.ci.tier4.jp/evaluation/reports/94b00440-fe1d-578a-b48b-a79137ce1774?project_id=x2_dev

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.

4 participants