Re: backward compatibility problem with grub

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



| From: Jeff Spaleta <jspaleta@xxxxxxxxx>

| This is quite easy to resolve.. edit the grub.spec file to remove
| -Wno-pointer-sign

I would have thought that the whole point of the autoconf horror show
is to avoid wiring this stuff in.

| and it will rebuild on fc3 using the compiler tools in fc3.  Right now
| when compiling for gcc4 this warning needs to be supressed to get grub
| to build. Since gcc4 is now the default compiler in rawhide, the
| rawhide grub has that option turned on.  I don't think its
| unreasonable to have to edit a spec file on occasion to get something
| to recompile, especially when we are talking about jumping a compiler
| toolchain version.

I admit that I am naive about this.  The technical solution used might
be expeditious.  But from a distance, something looks wrong.

I don't know autoconf (I have avoided it), but the first chunk of its
documentation (in info format -- a bad sign) says:

   Autoconf is a tool for producing shell scripts that automatically
   configure software source code packages to adapt to many kinds of
   UNIX-like systems.  The configuration scripts produced by Autoconf
   are independent of Autoconf when they are run, so their users do
   not need to have Autoconf.

The .spec file used autoconf.  Surely adapting between GCC versions
should be well within the scope of autoconf.

So: hardwiring CFLAGS into a spec file seems suspect.  That is
precisely what the grub spec does.  CFLAGS specific to GCC4.  It makes
the .spec file brittle.  Portability is a good thing.

Also: usually compiler warnings should be addressed by changing the
code that provokes the warning.  Not having a gcc4 at hand, I haven't
looked at the particular code in grub that causes the warning(s) that
are being suppressed, so I don't know whether the warning is bogus or
warranted.

BTW: why does x86_64 require -static and other architectures do not?
A comment explaining this would be welcome in the .spec file.


[Index of Archives]     [Fedora Desktop]     [Fedora SELinux]     [Photo Sharing]     [Yosemite Forum]     [KDE Users]