]> code.ossystems Code Review - openembedded-core.git/commitdiff
grub-efi: remove configure patch, simply seed the cache
authorRoss Burton <ross.burton@intel.com>
Mon, 13 Jan 2014 17:26:51 +0000 (17:26 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 14 Jan 2014 11:54:06 +0000 (11:54 +0000)
grub-2.00-disable-help2man patches configure.ac to disable the help2man check
because we don't need the man pages.  It also then patches configure itself so
that autogen.sh doesn't have to be called for performance reasons.  However,
do_configure causes a full autoreconf so this optimisation is moot, and can
cause patch failures when an existing build tree is re-used.

Instead, simply use CACHED_CONFIGUREVARS to tell configure that it can't find
help2man.

Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch [deleted file]
meta/recipes-bsp/grub/grub-efi_2.00.bb

diff --git a/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch b/meta/recipes-bsp/grub/files/grub-2.00-disable-help2man.patch
deleted file mode 100644 (file)
index a2e42c2..0000000
+++ /dev/null
@@ -1,148 +0,0 @@
-Upstream-Status: Inappropriate (Bitbake build environment)
-
-We do not need the man pages when building just the EFI payload for the target,
-all the tools are built for the host. This does not trigger GRUB's built-in
-cross-compilation check, so force it. After the change to configure.ac,
-autogen.sh was run in devshell and the resulting configure was used to generate
-the configure diff. The configure diff was included to avoid having to add
-autogen.sh to a do_configure_prepend() routine which would slow down the build
-unnecessarily.
-
-Signed-off-by: Darren Hart <dvhart@linux.intel.com>
-
-Index: grub-2.00/configure.ac
-===================================================================
---- grub-2.00.orig/configure.ac
-+++ grub-2.00/configure.ac
-@@ -310,11 +310,12 @@ if test x$USE_NLS = xno; then
-   HOST_CFLAGS="$HOST_CFLAGS -fno-builtin-gettext"
- fi
--if test "x$cross_compiling" = xyes; then
--  AC_MSG_WARN([cannot generate manual pages while cross compiling])
--else
--  AC_PATH_PROG(HELP2MAN, help2man)
--fi
-+# Force behaving as though we are cross-compiling with respect to HELP2MAN
-+#if test "x$cross_compiling" = xyes; then
-+AC_MSG_WARN([cannot generate manual pages while cross compiling])
-+#else
-+#  AC_PATH_PROG(HELP2MAN, help2man)
-+#fi
- # Check for functions and headers.
- AC_CHECK_FUNCS(posix_memalign memalign asprintf vasprintf getextmntent)
-Index: grub-2.00/configure
-===================================================================
---- grub-2.00.orig/configure
-+++ grub-2.00/configure
-@@ -1215,7 +1215,6 @@ GL_COND_LIBTOOL_FALSE
- GL_COND_LIBTOOL_TRUE
- BUILD_CC
- LIBUTIL
--HELP2MAN
- POSUB
- LTLIBINTL
- LIBINTL
-@@ -3069,7 +3068,6 @@ _ACEOF
- # Let the site file select an alternate cache file if it wants to.
- # Prefer an explicitly selected file to automatically selected ones.
- ac_site_file1=NONE
--ac_site_file2=NONE
- if test -n "$CONFIG_SITE"; then
-   # We do not want a PATH search for config.site.
-   case $CONFIG_SITE in #((
-@@ -3077,14 +3075,8 @@ if test -n "$CONFIG_SITE"; then
-     */*) ac_site_file1=$CONFIG_SITE;;
-     *)   ac_site_file1=./$CONFIG_SITE;;
-   esac
--elif test "x$prefix" != xNONE; then
--  ac_site_file1=$prefix/share/config.site
--  ac_site_file2=$prefix/etc/config.site
--else
--  ac_site_file1=$ac_default_prefix/share/config.site
--  ac_site_file2=$ac_default_prefix/etc/config.site
- fi
--for ac_site_file in "$ac_site_file1" "$ac_site_file2"
-+for ac_site_file in $ac_site_file1
- do
-   test "x$ac_site_file" = xNONE && continue
-   if test /dev/null != "$ac_site_file" && test -r "$ac_site_file"; then
-@@ -3371,8 +3363,9 @@ case $target_os in *\ *) target_os=`echo
- # The aliases save the names the user supplied, while $host etc.
- # will get canonicalized.
- test -n "$target_alias" &&
--  test "$program_prefix$program_suffix$program_transform_name" = \
--    NONENONEs,x,x, &&
-+  test "$target_alias" != "$host_alias" &&
-+    test "$program_prefix$program_suffix$program_transform_name" = \
-+      NONENONEs,x,x, &&
-   program_prefix=${target_alias}-
- am__api_version='1.11'
-@@ -9633,51 +9626,13 @@ if test x$USE_NLS = xno; then
-   HOST_CFLAGS="$HOST_CFLAGS -fno-builtin-gettext"
- fi
--if test "x$cross_compiling" = xyes; then
--  { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: cannot generate manual pages while cross compiling" >&5
-+# Force behaving as though we are cross-compiling with respect to HELP2MAN
-+#if test "x$cross_compiling" = xyes; then
-+{ $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: cannot generate manual pages while cross compiling" >&5
- $as_echo "$as_me: WARNING: cannot generate manual pages while cross compiling" >&2;}
--else
--  # Extract the first word of "help2man", so it can be a program name with args.
--set dummy help2man; ac_word=$2
--{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5
--$as_echo_n "checking for $ac_word... " >&6; }
--if ${ac_cv_path_HELP2MAN+:} false; then :
--  $as_echo_n "(cached) " >&6
--else
--  case $HELP2MAN in
--  [\\/]* | ?:[\\/]*)
--  ac_cv_path_HELP2MAN="$HELP2MAN" # Let the user override the test with a path.
--  ;;
--  *)
--  as_save_IFS=$IFS; IFS=$PATH_SEPARATOR
--for as_dir in $PATH
--do
--  IFS=$as_save_IFS
--  test -z "$as_dir" && as_dir=.
--    for ac_exec_ext in '' $ac_executable_extensions; do
--  if as_fn_executable_p "$as_dir/$ac_word$ac_exec_ext"; then
--    ac_cv_path_HELP2MAN="$as_dir/$ac_word$ac_exec_ext"
--    $as_echo "$as_me:${as_lineno-$LINENO}: found $as_dir/$ac_word$ac_exec_ext" >&5
--    break 2
--  fi
--done
--  done
--IFS=$as_save_IFS
--
--  ;;
--esac
--fi
--HELP2MAN=$ac_cv_path_HELP2MAN
--if test -n "$HELP2MAN"; then
--  { $as_echo "$as_me:${as_lineno-$LINENO}: result: $HELP2MAN" >&5
--$as_echo "$HELP2MAN" >&6; }
--else
--  { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5
--$as_echo "no" >&6; }
--fi
--
--
--fi
-+#else
-+#  AC_PATH_PROG(HELP2MAN, help2man)
-+#fi
- # Check for functions and headers.
- for ac_func in posix_memalign memalign asprintf vasprintf getextmntent
-@@ -20477,6 +20432,7 @@ $as_echo_n "checking whether \`$CC' gene
- cat confdefs.h - <<_ACEOF >conftest.$ac_ext
- /* end confdefs.h.  */
-+void g (int);
- void f (int (*p) (void));
- void g (int i)
- {
index deb95149b30423e9d5e419626b6a32f7af6ab235..e02def3e5f44140965c92e9ebda103ea3e2d2033 100644 (file)
@@ -22,7 +22,6 @@ SRC_URI = "ftp://ftp.gnu.org/gnu/grub/grub-${PV}.tar.gz \
            file://cfg \
            file://grub-2.00-fpmath-sse-387-fix.patch \
            file://grub-2.00-fix-enable_execute_stack-check.patch \
-           file://grub-2.00-disable-help2man.patch \
            file://check-if-liblzma-is-disabled.patch \
            file://grub-no-unused-result.patch \
            file://grub-2.00-ignore-gnulib-gets-stupidity.patch \
@@ -56,6 +55,7 @@ inherit autotools
 inherit gettext
 inherit deploy
 
+CACHED_CONFIGUREVARS += "ac_cv_path_HELP2MAN="
 EXTRA_OECONF = "--with-platform=efi --disable-grub-mkfont \
                 --enable-efiemu=no --program-prefix='' \
                 --enable-liblzma=no --enable-device-mapper=no --enable-libzfs=no"