Skip to content

[ci] Remove rpath=ON from global configuration #19613

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

Merged
merged 3 commits into from
Aug 13, 2025
Merged

Conversation

hahnjo
Copy link
Member

@hahnjo hahnjo commented Aug 12, 2025

Also remove non-working link for rpath deprecation, which is currently very confusing for users building from source.

hahnjo added 2 commits August 12, 2025 09:30
It was deprecated in commit 26d24de and currently causes warning
messages in every build.
The release notes will only exist after 6.38 is released, making it
currently very confusing for users building from source.
Copy link
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

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

Thanks! I would prefer if the release note link would already work, like it was the case for past release cycles, but maybe that's also questionable because there is actually no release with the upcoming version number yet.

Removing the link is definitely fine too!

Copy link

github-actions bot commented Aug 12, 2025

Test Results

    17 files      17 suites   2d 19h 48m 37s ⏱️
 3 360 tests  3 342 ✅ 0 💤  18 ❌
55 652 runs  55 523 ✅ 0 💤 129 ❌

For more details on these failures, see this check.

Results for commit 1186745.

♻️ This comment has been updated with latest results.

@ferdymercury
Copy link
Collaborator

ferdymercury commented Aug 13, 2025

Related: #19626
Feel free to take it over into your PR and close that one.

@hahnjo hahnjo merged commit 77d82f6 into root-project:master Aug 13, 2025
6 of 26 checks passed
@hahnjo hahnjo deleted the rpath branch August 13, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants