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

Jump to bottom

Travis-CI: add build that uses Clang as CC compiler #85

Merged
merged 2 commits into from Mar 21, 2015
Merged

Travis-CI: add build that uses Clang as CC compiler #85

merged 2 commits into from Mar 21, 2015

Conversation

zmughal
Copy link
Member

@zmughal zmughal commented Mar 18, 2015

This is in order to test against compilers other than GCC.

This fixes #13.

@zmughal
Travis-CI: add build that uses Clang as CC compiler
8690d82 
This is in order to test against compilers other than GCC.

This fixes <#13>.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.95% when pulling 8690d82 on travis-compiler-clang into aa7c613 on master.

@zmughal
Copy link
Member Author

zmughal commented Mar 19, 2015

While Travis-CI says that it passes, this is deceptive. Setting CC using the EUMM option does not work for subdirectory Makefiles.

perl Makefile.PL CC=clang; find -name Makefile -exec grep -H '^CC =' {} \;

will only set the CC=clang in the top level Makefile. All others have CC=cc.

I have an ugly fix for this using symlinks, but I would rather fix the build.

@mohawk2
Copy link
Member

mohawk2 commented Mar 19, 2015

The way forward would be to configure that Perl installation with something like

sudo perl -pie "s/(cc => ')gcc(')/${1}clang$2/" /correct/path/Config.pm

@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.95% when pulling 6ca153d on travis-compiler-clang into aa7c613 on master.

@zmughal
Travis-CI: change the value of $Config{cc} to clang instead of using …
fcb0fbe 
…PERL_MM_OPT

`PERL_MM_OPT` does not work with Makefile.PLs in subdirectories. Running

    perl Makefile.PL CC=clang; find -name Makefile -exec grep -H '^CC =' {} \;

shows that `CC = clang` in the toplevel Makefile, but `CC = cc`
elsewhere.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.95% when pulling fcb0fbe on travis-compiler-clang into aa7c613 on master.

@zmughal zmughal mentioned this pull request Mar 21, 2015
Merged
@mohawk2 mohawk2 merged commit fcb0fbe into master Mar 21, 2015
@mohawk2 mohawk2 deleted the travis-compiler-clang branch March 21, 2015 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cat:build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Travis-CI: add support for multiple compilers
3 participants
@zmughal @coveralls @mohawk2