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

Linux build version script to hide exports of dependencies #984

Merged
merged 5 commits into from
Oct 17, 2019

Conversation

Gillibald
Copy link
Contributor

Description of Change

Currently symbols of Freetype etc. are exported during the linker process. This leads to errors when used with applications that load build in third party libraries.

Bugs Fixed

cake/BuildExternals.cake Outdated Show resolved Hide resolved
Co-Authored-By: Matthew Leibowitz <mattleibow@live.com>
@mattleibow mattleibow self-requested a review October 17, 2019 08:59
mattleibow
mattleibow previously approved these changes Oct 17, 2019
Copy link
Contributor

@mattleibow mattleibow left a comment

Choose a reason for hiding this comment

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

looking good!

@mattleibow
Copy link
Contributor

I see an error:

/usr/bin/ld:/__w/1/s/native-builds/libSkiaSharp_linux/libSkiaSharp.map:7: syntax error in VERSION script

@Gillibald
Copy link
Contributor Author

Let me fire up a VM quick

@mattleibow
Copy link
Contributor

mattleibow commented Oct 17, 2019

I think it was missing the last semicolon... Updated the code, so let's see.

Took a guess from the sample in https://www.gnu.org/software/gnulib/manual/html_node/LD-Version-Scripts.html

@Gillibald
Copy link
Contributor Author

Should have tested this after I rewrote everything from scratch :(

@mattleibow
Copy link
Contributor

Do you use a VM of WSL?

@Gillibald
Copy link
Contributor Author

Using a VM and running Ubuntu 18.04 LTS. Did a fresh clone. Let me check real quick what is happening.

@mattleibow
Copy link
Contributor

Seems to have passed the build of libSkiaSharp for Linux.

@Gillibald
Copy link
Contributor Author

Y same on my machine. Looks okay now.

@Gillibald
Copy link
Contributor Author

I wonder why Fontconfig is still exported. There seems to be a version script for glibc as well.

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