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

FontConfig misnamed DLL (x64-windows) #14944

Closed
gnieboer opened this issue Dec 4, 2020 · 1 comment · Fixed by #14946
Closed

FontConfig misnamed DLL (x64-windows) #14944

gnieboer opened this issue Dec 4, 2020 · 1 comment · Fixed by #14946
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@gnieboer
Copy link

gnieboer commented Dec 4, 2020

Host Environment

  • OS: x64-windows
  • Compiler: VS2019?
  • vcpkg date of last pull: 12/3/2020

To Reproduce
Steps to reproduce the behavior:
./vcpkg install cairo
cairo will install fontconfig and then link against it
Run depends.exe or similar on cairo.dll found in .../installed/x64-windows/bin
cairo.dll will be dependent on "libfontconfig-1.dll"
But the fontconfig package installs "fontconfig-1.dll"
so load of the DLL will fail

Failure logs
image

Additional context
The Cairo cmake cache shows it finding the correct fontconfig.lib file, but in the /lib directory, fontconfig.def ends with "LIBRARY libfontconfig-1.dll", and the .lib file references libfontconfig-1.dll so Cairo is linking to the DLL fontconfig provides, but it doesn't exist as FontConfig actually outputs fontconfig-1.dll (which would be the correct windows naming convention).

Recommend altering the fontconfig build to target that naming standard.

@gnieboer gnieboer changed the title Cairo build failure (misnamed linked dependency DLL Win64) FontConfig misnamed DLL (x64-windows) Dec 4, 2020
@Neumann-A
Copy link
Contributor

src/Makefile.am line 58 probably needs to be patched from

fontconfig.lib : libfontconfig.la
	lib -name:libfontconfig-@[email protected] -def:fontconfig.def -out:$@

to

fontconfig.lib : libfontconfig.la
	lib -name:fontconfig-@[email protected] -def:fontconfig.def -out:$@

@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Dec 7, 2020
strega-nil added a commit to strega-nil/vcpkg that referenced this issue May 5, 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 a pull request may close this issue.

3 participants