On 1/24/2016 1:49 AM, Bruce Dubbs wrote:
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
Grr, it was there in the logs, I just didn't catch it because it was in
the check log (doesn't run until make root-tests). I don't see where
where this could be necessary if the check is already there.
--DJ
--
http://lists.linuxfromscratch.org/listinfo/lfs-dev
FAQ: http://www.linuxfromscratch.org/faq/
Unsubscribe: See the above information page
