Bruce Dubbs wrote:
DJ Lucas wrote:

you won't
have to modify fedora patches, just concatenate the three (and rediff if
you want to kill the offsets - not necessary, I know, but I did it anyway,
.orig files bug me). I can commit the patch to repo if you like.

DJ, your patch broke the coreutils build in the LFS environment. It has one chunk:

--- coreutils-8.25-orig/configure.ac   2016-01-01 07:48:50.000000000 -0600
+++ coreutils-8.25/configure.ac  2016-01-22 21:23:04.634308804 -0600
@@ -425,6 +425,8 @@ fi
 # I'm leaving it here for now.  This whole thing needs to be modernized...
 gl_WINSIZE_IN_PTEM

+gl_MBFILE
+
 gl_HEADER_TIOCGWINSZ_IN_TERMIOS_H

 if test $gl_cv_sys_tiocgwinsz_needs_termios_h = no && \
-----------

That changes the timestamp on configure.ac. The make command (not configure) then wants to call aclocal:

/sources/coreutils-8.25/build-aux/missing: line 81: aclocal-1.15: command not found
WARNING: 'aclocal-1.15' is missing on your system.
         You should only need it if you modified 'acinclude.m4' or
         'configure.ac' or m4 files included by 'configure.ac'.
         The 'aclocal' program is part of the GNU Automake package:
         <http://www.gnu.org/software/automake>
         It also requires GNU Autoconf, GNU m4 and Perl in order to run:
         <http://www.gnu.org/software/autoconf>
         <http://www.gnu.org/software/m4/>
         <http://www.perl.org/>
Makefile:5880: recipe for target 'aclocal.m4' failed

---------

Google has nothing in the last year.   I did find:

# mbfile.m4 serial 7
dnl Copyright (C) 2005, 2008-2016 Free Software Foundation, Inc.
dnl This file is free software; the Free Software Foundation
dnl gives unlimited permission to copy and/or distribute it,
dnl with or without modifications, as long as this notice is preserved.

dnl autoconf tests required for use of mbfile.h
dnl From Bruno Haible.

AC_DEFUN([gl_MBFILE],
[
  AC_REQUIRE([AC_TYPE_MBSTATE_T])
  :
])

Looking at AC_TYPE_MBSTATE_T, it is looking for mbstate_t in wchar.h. We DO have it and configure tests for it.


There are a couple of ways to fix it, The easiest is to remove the above chunk from the patch. Do you concur?
  -- Bruce



--
http://lists.linuxfromscratch.org/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page

Reply via email to