Skip to content

[atk][exiv2][gdk-pixbuf][gettext][glib][libcroco][tesseract] Replace uses of unofficial-gettext with FindIntl#13762

Closed
ras0219 wants to merge 9 commits intomicrosoft:masterfrom
ras0219:dev/roschuma/gettext
Closed

[atk][exiv2][gdk-pixbuf][gettext][glib][libcroco][tesseract] Replace uses of unofficial-gettext with FindIntl#13762
ras0219 wants to merge 9 commits intomicrosoft:masterfrom
ras0219:dev/roschuma/gettext

Conversation

@ras0219
Copy link
Contributor

@ras0219 ras0219 commented Sep 27, 2020

See title. See also #13027.

We prefer supporting built-in cmake modules whenever possible instead of providing our own replacements.

@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Sep 27, 2020
@strega-nil
Copy link
Contributor

I think this might've been fixed already by @Neumann-A in #11776, no?

@PhoebeHui
Copy link
Contributor

PhoebeHui commented Dec 11, 2020

After compared with PR #11776, the changes for gettext still needed, I will help to undo changes for other ports and resolve the conflicts.

@PhoebeHui PhoebeHui force-pushed the dev/roschuma/gettext branch 2 times, most recently from 008aafa to 1018990 Compare December 11, 2020 07:46
Copy link
Contributor

@PhoebeHui PhoebeHui left a comment

Choose a reason for hiding this comment

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

@ras0219, I think we need check if unofficial-gettext exist.

Comment on lines +62 to +63
# These targets are provided for backwards compatibility with previous vcpkg versions
vcpkg_fixup_cmake_targets(CONFIG_PATH share/unofficial-gettext TARGET_PATH share/unofficial-gettext)
Copy link
Contributor

Choose a reason for hiding this comment

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

The CI failed with following errors:

Make Error at scripts/cmake/vcpkg_fixup_cmake_targets.cmake:126 (message):
  'D:/packages/gettext_x64-windows-static/debug/share/unofficial-gettext'
  does not exist.
Call Stack (most recent call first):
  ports/gettext/portfile.cmake:63 (vcpkg_fixup_cmake_targets)
  scripts/ports.cmake:136 (include)

Copy link
Contributor

Choose a reason for hiding this comment

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

it does not exists anymore since gettext was changed to native build which is make based

@ras0219 ras0219 closed this Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants