Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Update to the new Travis CI infrastructure. #1117

Merged
merged 1 commit into from
Sep 1, 2015

Conversation

@xzyfer xzyfer self-assigned this Aug 31, 2015
@xzyfer xzyfer added this to the next.patch milestone Aug 31, 2015
@saper
Copy link
Member

saper commented Aug 31, 2015

Er, looks like we are not getting the compiler we want. My suggestion for priority: low :)

@saper
Copy link
Member

saper commented Aug 31, 2015

According to "C++0x/C++11 Support in GCC" table template alias feature should be supported in g++ 4.7. Which compiler are we getting in the end?

Interestingly, libsass seems to compile just fine with the default gcc 4.6.3. So the new V8 is quite picky about the compiler. Can we ask for g++ 4.7 only for io.js 3 in the new infrastructure? (besides it does not seem to work)

@xzyfer xzyfer force-pushed the feat/new-travis branch 3 times, most recently from 06783c5 to 3a86027 Compare September 1, 2015 00:21
@xzyfer
Copy link
Contributor Author

xzyfer commented Sep 1, 2015

It looks like I can install gcc/gcc 4.7 but I can't alias gcc/g++ to the newer versions without sudo.

I agree this isn't a high priority. I just wanted to take a run at it before going to bed. The slow Travis times are also starting to bug me.

@xzyfer xzyfer modified the milestones: Unknown, next.patch Sep 1, 2015
@saper
Copy link
Member

saper commented Sep 1, 2015

That should be relatively easy, then. If the compilers are named gcc47 and g++47 we just need to call /usr/bin/env CC=gcc47 CXX=g++48 LINK=gcc47 LINKXX=g++47 npm install

@xzyfer xzyfer force-pushed the feat/new-travis branch 2 times, most recently from b104b60 to 0a1b807 Compare September 1, 2015 07:51
@xzyfer
Copy link
Contributor Author

xzyfer commented Sep 1, 2015

Looks like this works now. I've some intermittent Travis failures which don't instill me with confidence.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants