Skip to content

[elfutils] Add libdebuginfod feature#26107

Closed
BigRedEye wants to merge 3 commits into
microsoft:masterfrom
BigRedEye:master
Closed

[elfutils] Add libdebuginfod feature#26107
BigRedEye wants to merge 3 commits into
microsoft:masterfrom
BigRedEye:master

Conversation

@BigRedEye
Copy link
Copy Markdown
Contributor

What does your PR fix?

Allows to use libdebuginfod from elfutils

Does your PR follow the maintainer guide?

Yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

Yes

github-actions[bot]
github-actions Bot previously approved these changes Aug 1, 2022
@Adela0814 Adela0814 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Aug 2, 2022
Comment thread ports/elfutils/portfile.cmake Outdated
@Adela0814 Adela0814 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Aug 2, 2022
vcpkg_install_make()
file(REMOVE "${CURRENT_PACKAGES_DIR}/debug/lib/pkgconfig/libdebuginfod.pc" "${CURRENT_PACKAGES_DIR}/lib/pkgconfig/libdebuginfod.pc") #--disable-debuginfod
file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/etc/debuginfod" "${CURRENT_PACKAGES_DIR}/etc/debuginfod")
if (WITH_LIBDEBUGINFOD)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems like it can't be correct: Why are you deleting the bits backing the feature if the feature is on? (Is the sense of this if flipped?)

PATCHES configure.ac.patch
PATCHES
configure.ac.patch
install-libdebuginfod.patch
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you explain in detail what this patch is doing?

Has it been submitted upstream?

@BillyONeal BillyONeal added requires:author-response and removed info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. labels Aug 5, 2022
@Adela0814
Copy link
Copy Markdown
Contributor

Adela0814 commented Sep 7, 2022

Convert this PR to draft since there is no progress. Please ping us if this PR is ready for review again.

@Adela0814 Adela0814 marked this pull request as draft September 7, 2022 06:25
@Adela0814
Copy link
Copy Markdown
Contributor

Closing this PR since it seems that no progress is being made. Please ping us to reopen if work is still being done.

@Adela0814 Adela0814 closed this Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants