On Tue, Sep 01, 2009 at 06:07:04PM +0200, Robert Millan wrote: > On Tue, Sep 01, 2009 at 03:44:18PM +0200, Vladimir 'phcoder' Serbinenko wrote: > > > > +AC_CACHE_CHECK([whether gcc is recent enough], grub_cv_cc_recent, [ > > + CFLAGS= > > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > > +#if defined (__GNUC__) && (__GNUC__ < 4 || __GNUC_MINOR__ < 2) > > +#error Use at least GCC 4.2 > > +#endif > > This would fail with e.g. GCC 5.0. Other than this, the patch is fine. > > If Bean wants to add a configure option (e.g. like QEMU's --disable-gcc-check) > so that this requirement can be overriden, that's ok too.
Btw, INSTALL file still says 2.95. If we do this, it should be adjusted as well. -- Robert Millan The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and how) you may access your data; but nobody's threatening your freedom: we still allow you to remove your data and not access it at all." _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel
