Skip to content
This repository was archived by the owner on Oct 24, 2025. It is now read-only.

Conversation

@mgreter
Copy link
Contributor

@mgreter mgreter commented Dec 21, 2014

Made windows mingw compliation working:
https://ci.appveyor.com/project/mgreter/libsass/build/1.0.25
Needs sass/sassc#83 to be merged first!

@mgreter mgreter changed the title [WIPFeature/build dll win mingw [WIP] Feature/build dll win mingw Dec 21, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) when pulling 0772828 on mgreter:feature/build-dll-win-mingw into 9852e41 on sass:master.

@mgreter mgreter force-pushed the feature/build-dll-win-mingw branch from 0772828 to f9f51f8 Compare December 21, 2014 12:35
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) when pulling f9f51f8 on mgreter:feature/build-dll-win-mingw into c4e03e9 on sass:master.

@mgreter
Copy link
Contributor Author

mgreter commented Dec 21, 2014

@darrenkopp I hope you don't mind that I have taken over your project files from https://github.com/darrenkopp/sassc-win? I think it should work correctly now. Took me quite some time to figure out why it was giving me strange results with the test runner (seems ruby 1.9 was the cause). Beside that everything seems to work now. Once the PR on sassc is merged I can remove the reference to my local sassc repository!

Also the debug target on msvc throws some debug assertions, which seem to keep the build on hold forever till it times out. So I disabled the test-runner on that target.

@mgreter mgreter changed the title [WIP] Feature/build dll win mingw Feature/build dll win mingw Dec 21, 2014
@mgreter mgreter added the Build label Dec 21, 2014
@darrenkopp
Copy link

Yup, no worries.

@am11
Copy link
Contributor

am11 commented Dec 21, 2014

Awesome! :)

@mgreter, separately, TravisCI has beta OSX testing support too. Based on this, you would need to send them an email ([email protected]) to enable Mac testing for libsass repo. I contacted them for node-sass, but they denied saying:

I am afraid we are unable to turn the multi-OS feature for you at this time, as the repository requires multiple Node.js runtimes that are not available.

I think they might enable it for libsass, as there is no such issue.

@mgreter mgreter mentioned this pull request Dec 22, 2014
12 tasks
mgreter added a commit that referenced this pull request Dec 22, 2014
@mgreter mgreter merged commit a73dda9 into sass:master Dec 22, 2014
@mgreter mgreter deleted the feature/build-dll-win-mingw branch April 6, 2015 17:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants