-
Notifications
You must be signed in to change notification settings - Fork 2
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
ffi phase: copy curl-config to dynamic folder #10
Comments
Interesting, by default we drop everything that doesn't go into the For now at least I want the Proj will have to be careful to link dynamic directories correctly, when the libraries aren't in the standard system library directories. The exact incantations aren't portable which is one of the reasons Alien-Build avoids doing it. |
If we do install the |
I think the end result will be a System libs I'll have to deal with when I get to them. A curl-config probe in Alien::proj could indicate if the system curl is static, in which case Alien::proj might need to complain that a share build is needed to get the dynamic version. I'll look at Alien-Role-Dino as well. A |
connect, just to be clear this is what my suggestion was intending.
ARD is just for background. But worth reviewing it has some commentary about the challenges with share install dynamic libs.
👍 |
This also gets the package config file, .dll.a and .la files. The former is useful, the other two should/hopefully have no real effect. Updates PerlAlien#10
This also gets the package config file, .dll.a and .la files. The former is useful, the other two should/hopefully have no real effect. Updates PerlAlien#10
This also gets the package config file, .dll.a and .la files. The former is useful, the other two should/hopefully have no real effect. Updates #10
I think we are done here? or at least some work has been done here, so if more work needs doing lets open a new ticket for that. |
Proj 7.0.0 has a new dependency on libcurl. Its configure script uses the curl-config utility to set the lib paths and cflags, and it seems not to be simple to override this.
The issue is that compilation of Alien::proj with Proj 7 fails because the curl-config utility in the bin dir refers to the static build, while proj requires a dynamic build.
Copying the curl-config utility from the ffi build dir into the dynamic dir under Alien::curl->dist_dir seems to fix the issue.
I'd submit a PR now, but in looking at the code for the ffi phase in the alienfile, I'm not actually sure which of the paths refers to the build directory. If you let me know what the best way to get the build dir then I can work up a PR if needed.
Thanks,
Shawn.
The text was updated successfully, but these errors were encountered: