Hi Diego, The series looks good except the CFLAGS over ride change. The CFLAGS obvious have two parts. Some of it can be savely overwrite. Some of shouldn't be overwrite otherwise sparse can't compile correctly. I think it'd better seperate out the overwrite part of CFLAGS with the rest of CFLAGS, just like the git makefile. Some thing like the patch I attached. You might need to make some changes to merge with your branch. Can you rebase your series on the sparse chrisl branch? http://git.kernel.org/?p=devel/sparse/chrisl/sparse.git;a=summary Thanks Chris On Thu, Apr 7, 2011 at 6:32 PM, Diego Elio Pettenò <flameeyes@xxxxxxxxx> wrote: > Hello Christopher, > > I'm a Gentoo developer and I went to look at sparse today since it > failed to build with the new GCC 4.6 version. I've fixed the build > issue, but since I was looking at it I also made a number of changes > that makes it much easier to build in Gentoo. > > If you could look at the patches and merge those that makes sense, it'd > be lovely: > > http://git.overlays.gentoo.org/gitweb/?p=proj/sparse.git;a=summary > > Thanks in advance, > -- > Diego Elio Pettenò — Flameeyes > http://blog.flameeyes.eu/ > > -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
