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 conflicts with other python libraries due to static link to libgcc and libstdc++ #244

Merged

Conversation

BewareMyPower
Copy link
Contributor

@BewareMyPower BewareMyPower commented Feb 4, 2025

Fixes #242
Fixes #243
Fixes #245

Add a patch to avoid linking to libgcc and libstdc++.

@BewareMyPower BewareMyPower added bug Something isn't working release/3.6.1 labels Feb 4, 2025
@BewareMyPower BewareMyPower self-assigned this Feb 4, 2025
@BewareMyPower BewareMyPower marked this pull request as draft February 4, 2025 07:14
@BewareMyPower BewareMyPower force-pushed the bewaremypower/fix-glibc-conflicts branch from 21727a2 to 08c5ce4 Compare February 4, 2025 11:31
@BewareMyPower BewareMyPower marked this pull request as ready for review February 7, 2025 12:50
@BewareMyPower BewareMyPower changed the title Fix conflicts with other python libraries due to static link to libstdc++ Fix conflicts with other python libraries due to static link to libgcc and libstdc++ Feb 7, 2025
@BewareMyPower BewareMyPower added this to the 3.7.0 milestone Feb 7, 2025
@@ -46,6 +47,9 @@ if [ $CPP_BINARY_TYPE == "rpm" ]; then
cd ..
./bootstrap-vcpkg.sh
cd ..
if [ $PULSAR_CPP_VERSION == "3.7.0" ]; then
patch lib/CMakeLists.txt $ROOT_DIR/pkg/manylinux2014/pulsar-client-cpp-3.7.0.patch
Copy link
Contributor

Choose a reason for hiding this comment

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

For next versions, should we just fix this in cpp repo build file? At least add a build option

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I pushed a PR here: apache/pulsar-client-cpp#474

@BewareMyPower BewareMyPower merged commit 4a4ac3f into apache:main Feb 8, 2025
11 checks passed
@BewareMyPower BewareMyPower deleted the bewaremypower/fix-glibc-conflicts branch February 8, 2025 03:44
BewareMyPower added a commit that referenced this pull request Feb 8, 2025
…dc++ (#244)

Fixes #242
Fixes #243
Fixes #245

Add a patch to avoid linking to libgcc and libstdc++.

(cherry picked from commit 4a4ac3f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants