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 debug postfix in pkgconfig file #363

Merged
merged 1 commit into from
Jul 15, 2020

Conversation

c72578
Copy link
Contributor

@c72578 c72578 commented Jul 15, 2020

In case of USE_POSTFIX, the POSTFIX is dependent
on the CMAKE_BUILD_TYPE.
Use the correct POSTFIX also in the generated pugixml.pc file.

This results in the following contents of pugixml.pc:

  • Release:
    Libs: -L${libdir} -lpugixml
  • RelWithDebInfo
    Libs: -L${libdir} -lpugixml_r
  • MinSizeRel:
    Libs: -L${libdir} -lpugixml_m
  • Debug:
    Libs: -L${libdir} -lpugixml_d

In case of USE_POSTFIX, the POSTFIX is dependent
on the CMAKE_BUILD_TYPE.
Use the correct POSTFIX also in the generated pugixml.pc file.

This results in the following contents of pugixml.pc:
- Release:
  Libs: -L${libdir} -lpugixml
- RelWithDebInfo
  Libs: -L${libdir} -lpugixml_r
- MinSizeRel:
  Libs: -L${libdir} -lpugixml_m
- Debug:
  Libs: -L${libdir} -lpugixml_d
c72578 added a commit to c72578/vcpkg that referenced this pull request Jul 15, 2020
- Add missing vcpkg_fixup_pkgconfig(), which was detected in
  microsoft#12151
- Fix debug POSTFIX in pugixml.pc file using
  pugixml-v1.10_fix_debug_pkgconfig.patch
  based on upstream: zeux/pugixml#363
  This results in the following contents of pugixml.pc:
  Release:
    Libs: -L${libdir} -lpugixml
  Debug:
    Libs: -L${libdir} -lpugixml_d
@zeux zeux merged commit 22401ba into zeux:master Jul 15, 2020
@zeux
Copy link
Owner

zeux commented Jul 15, 2020

Thanks!

ras0219-msft pushed a commit to microsoft/vcpkg that referenced this pull request Jul 15, 2020
- Add missing vcpkg_fixup_pkgconfig(), which was detected in
  #12151
- Fix debug POSTFIX in pugixml.pc file using
  pugixml-v1.10_fix_debug_pkgconfig.patch
  based on upstream: zeux/pugixml#363
  This results in the following contents of pugixml.pc:
  Release:
    Libs: -L${libdir} -lpugixml
  Debug:
    Libs: -L${libdir} -lpugixml_d
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.

2 participants