-
Notifications
You must be signed in to change notification settings - Fork 464
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
Fix SmartOS compilation once and for all #1847
Conversation
|
2f0d5f0
to
687a969
Compare
@jaredmorrow @djensen47 I'd like to hear your thoughts here |
Looks like Btw. install from libtools outputs:
¯_(ツ)_/¯ |
2c6a78f
to
19d9b71
Compare
Verfied both install methods and documented my findings. Very very sure this chapter can be closed! |
Removed `#undef` occurrences that are no longer needed
Rename `Sass::SassUnitType` to `Sass::UnitClass`
19d9b71
to
7b670e4
Compare
I'm going to merge this on my responsibility. Beside CI I have tested it with Visual Studio 2013 Community, SmartOS 5.11, Gentoo Linux, Cygwin and MinGW. Not all combinations, but a fair share. |
Fix SmartOS compilation once and for all
Verified by compiling inside VMWare image (the problems with
SEC
were present):Let's see what CI thinks ... #1802