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

build with -latomic on platforms that need it #6774

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

niol
Copy link
Contributor

@niol niol commented Apr 7, 2024

Following https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1067625 and failure to build transmission on specific platforms that require -latomic, here is the way to a proper fix.

@niol
Copy link
Contributor Author

niol commented Apr 7, 2024

I still need to test that this fixes the issue, but setting up qemu takes me some time.

cmake/CheckAtomic.cmake Outdated Show resolved Hide resolved
@Coeur Coeur added the type:build Changes that affect the build system label Apr 23, 2024
@Coeur
Copy link
Collaborator

Coeur commented May 21, 2024

You marked the PR as draft. Is it ready for review and tested?

@niol
Copy link
Contributor Author

niol commented May 22, 2024

Still working on it, does not compile.

@niol
Copy link
Contributor Author

niol commented May 22, 2024

All good now.

@niol niol marked this pull request as ready for review May 22, 2024 06:26
@Coeur Coeur unassigned niol May 22, 2024
@ckerr
Copy link
Member

ckerr commented May 27, 2024

CC @mikedld for CMake review

cmake/CheckAtomic.cmake Outdated Show resolved Hide resolved
cmake/CheckAtomic.cmake Outdated Show resolved Hide resolved
libtransmission/CMakeLists.txt Outdated Show resolved Hide resolved
libtransmission/CMakeLists.txt Outdated Show resolved Hide resolved
cmake/CheckAtomic.cmake Outdated Show resolved Hide resolved
@mikedld mikedld added this to the 4.1.0 milestone Jun 4, 2024
@mikedld mikedld merged commit d42d0f3 into transmission:main Jun 4, 2024
25 checks passed
@mikedld mikedld added the semver:patch make backwards compatible bug fixes label Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch make backwards compatible bug fixes type:build Changes that affect the build system
Development

Successfully merging this pull request may close these issues.

None yet

4 participants