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

libcap/2.73: Bump version #26246

Merged
merged 1 commit into from
Dec 30, 2024
Merged

libcap/2.73: Bump version #26246

merged 1 commit into from
Dec 30, 2024

Conversation

bobrofon
Copy link
Contributor

Summary

Changes to recipe: libcap/2.73

Motivation

A new version of libcap is available. Full diff between 2.70 and 2.73: https://git.kernel.org/pub/scm/libs/libcap/libcap.git/diff/?id=libcap-2.73&id2=libcap-2.70

Details

Conan-specific patches are the same as for the previous versions. They are only rebased on version 2.73.


@bobrofon bobrofon marked this pull request as ready for review December 22, 2024 14:22

# Always build libcap sources this way:
-CFLAGS += -fPIC -D_LIBPSX_PTHREAD_LINKAGE
+CFLAGS += -D_LIBPSX_PTHREAD_LINKAGE
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @bobrofon, Why is this line added if nothing similar was added in the previous patch? I couldn’t find anything
upstream—could you help me? 😄

https://github.com/conan-io/conan-center-index/blob/master/recipes/libcap/all/patches/2.57/0001-libcap-Remove-hardcoded-fPIC.patch#L21

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In the previous patch, the -fPIC flag was removed from CFLAGS. In this patch, the -fPIC flag is also removed from CFLAGS. Honestly, I don't see any difference.

-CFLAGS += -fPIC

vs

-CFLAGS += -fPIC -D_LIBPSX_PTHREAD_LINKAGE
+CFLAGS += -D_LIBPSX_PTHREAD_LINKAGE

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The associated change in the upstream: https://git.kernel.org/pub/scm/libs/libcap/libcap.git/diff/libcap/Makefile?id=libcap-2.73&id2=libcap-2.70

-CFLAGS += -fPIC
+CFLAGS += -fPIC -D_LIBPSX_PTHREAD_LINKAGE

Copy link
Contributor

@ErniGH ErniGH 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!

@jcar87 jcar87 merged commit aee5cf0 into conan-io:master Dec 30, 2024
8 checks passed
@bobrofon bobrofon deleted the libcap/2.73 branch December 30, 2024 17:42
OMGtechy pushed a commit to OMGtechy/conan-center-index that referenced this pull request Dec 31, 2024
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.

3 participants