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.

Sign up for GitHub

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

Jump to bottom

remove the graphite2.exe #3102

Closed

Alisayyy-Yang wants to merge 1 commit into microsoft:master from Alisayyy-Yang:fixgraphite2

Contributor

Alisayyy-Yang commented Mar 21, 2018

1, Because the EXEs still exisited so that the Performing post-build validation failed.
2. After we make it removed .then the lib will be ok


          remove the graphite2.exe

2d1cac8

ras0219-msft closed this in

55e3859

Contributor

ras0219-msft commented Mar 23, 2018

Thanks.

Upon closer inspection, it looks like it is better to patch out the build entirely instead of building and then removing.

Alisayyy-Yang deleted the fixgraphite2 branch

March 26, 2018 08:49

spl mentioned this pull request

Building a static library on Windows silnrsi/graphite#55

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment