On 2017-09-01 09:46:44, Uwe Kleine-König wrote: > Hello, > > On Thu, Aug 31, 2017 at 11:43:53PM +0100, Ramsay Jones wrote: >> On 31/08/17 21:55, Uwe Kleine-König wrote: >> > On Wed, Aug 30, 2017 at 08:11:49PM -0400, Christopher Li wrote: >> >> That is very much like on x86_64 missing define "#weak_define __x86_64__ 1" >> >> >> >> Does cgcc work for you? In the future we do want to move the archetecture >> >> related define from cgcc into sparse by itself. For now you can set >> >> "sparse" as "cgcc -no-compile" >> > >> > Yes that works. So to address the Debian bug I can do: >> > >> > - move sparse to /usr/lib >> > - teach cgcc about the move of sparse >> > - make /usr/bin/sparse call cgcc -no-compile "$@" >> >> Hmm, I don't think that would be a good idea ... >> >> > or is it easier to teach sparse about the architecture stuff? >> >> I now understand (I think!) that you are building a sparse >> package (presumably a .deb) and you are concerned that sparse >> does not pass it's own testsuite on those platforms. > > Nearly right. I'm responsible for the sparse Debian package and the > problem at hand is https://bugs.debian.org/873508. This bug report has > "Severity: serious" wihch might eventually result in the removal of > sparse from the Debian archive. > > @anarcat: Given that cgcc seems to work, would you agree to apply the > following patch to horst: > > diff --git a/Makefile b/Makefile > index 4f924fa..d563652 100644 > --- a/Makefile > +++ b/Makefile > @@ -110,7 +110,7 @@ $(NAME): $(OBJS) > $(OBJS): .buildflags > > check: > - sparse $(CFLAGS) *.[ch] > + cgcc -no-compile $(CFLAGS) *.[ch] > > clean: > -rm -f *.o radiotap/*.o *~ > > and downgrade the bug to "important"? That would be a compromise that > buys us a bit of time. Well right now I have simply disabled the checks for those broken architectures, so sparse isn't as affected anymore. Frankly, I don't quite know what to do with this - but I'd be happy to happly that patch to sparse if it fixes the issue better. A. -- Those who would give up essential Liberty, to purchase a little temporary Safety, deserve neither Liberty nor Safety. - Benjamin Franklin, 1755 -- 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
