On 2015-03-18 at 18:07:27 -0500, Zakariyya Mughal wrote: > On 2015-03-18 at 15:25:05 -0600, Derek Lamb wrote: > > I just created a branch clang_warnings that is focused on squashing > > the sometimes nit-picky warnings clang generates when PDL is compiled. > > In fact, probably >90% of the warnings have already been squashed. > > Before rebasing onto master, I would appreciate tests and code review > > from both clang and non-clang users; a lot of this is messing around > > with low-level C stuff, Perl macros, etc, so I want to make sure I > > didn't do anything dumb. > > Excellent! > > I just opened a PR on GitHub for that branch so that it can run under > Travis-CI <https://github.com/PDLPorters/pdl/pull/84>. > > I have this issue open on GH to use more compilers to the Travis-CI > tests <https://github.com/PDLPorters/pdl/issues/13>. I might have some > tuits to add a clang build.
And now Travis-CI is also testing a Clang build on Perl v5.20! Before the Travis build was combined with `clang_warnings`, the build timed out because all the Clang warnings caused the build log to reach over 4 MB. So now the fixes are in master. Also, using CCFLAGS in Makefile.PL doesn't work because EUMM will use CCFLAGS to override the compiler flags. The correct way to add arbitrary flags is to set INC. Ed made a change that demonstrates that here[^1]. Cheers, - Zaki Mughal [^1]: <https://github.com/PDLPorters/pdl/commit/3928b95fc5433235a8fa6eb648e7044472036933> > > Cheers, > - Zaki Mughal > > > > > Derek > > ------------------------------------------------------------------------------ > > Dive into the World of Parallel Programming The Go Parallel Website, > > sponsored > > by Intel and developed in partnership with Slashdot Media, is your hub for > > all > > things parallel software development, from weekly thought leadership blogs > > to > > news, videos, case studies, tutorials and more. Take a look and join the > > conversation now. http://goparallel.sourceforge.net/ > > _______________________________________________ > > pdl-devel mailing list > > pdl-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/pdl-devel ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ pdl-devel mailing list pdl-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/pdl-devel
