public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Fernando Seiti Furusato <ferseiti@linux.vnet.ibm.com>
Cc: xfs@oss.sgi.com, 793495@bugs.debian.org,
	Breno Leitao <brenohl@br.ibm.com>,
	Daniel Bast <daniel.bast@gmx.net>
Subject: Re: Bug#793495: xfsprogs fails to build on debian ppc64el
Date: Wed, 5 Aug 2015 08:58:19 +1000	[thread overview]
Message-ID: <20150804225819.GV16638@dastard> (raw)
In-Reply-To: <55C0D916.1010102@linux.vnet.ibm.com>

On Tue, Aug 04, 2015 at 12:24:06PM -0300, Fernando Seiti Furusato wrote:
> Hi Dave.
> 
> I have just noticed that for config.guess and config.sub to be updated, no
> autoreconf or dh_autotools-dev is needed, because of how Makefile was made.
> 
> In Makefile, @row 80:
> 
>  LIBTOOLIZE_INSTALL = `libtoolize -n -i >/dev/null 2>/dev/null && echo -i`
> 
>  configure:
>          libtoolize -c $(LIBTOOLIZE_INSTALL) -f
>          cp include/install-sh .
>          aclocal -I m4
>          autoconf
> 
> This runs libtoolize -c -i which updates those files.
> 
> The file debian/rules runs `make default`, which at some point calls the target
> above (`make configure`). But the file configure already exists, so it does not
> run.

Right. Intentionally so.

....

> diff -Nru xfsprogs-3.2.4/Makefile xfsprogs-3.2.4ppc64el1/Makefile
> --- xfsprogs-3.2.4/Makefile	2013-10-10 17:07:16.000000000 -0400
> +++ xfsprogs-3.2.4ppc64el1/Makefile	2015-08-04 11:06:40.000000000 -0400
> @@ -116,7 +116,7 @@
>  	$(Q)$(MAKE) $(MAKEOPTS) -C $* install-qa
>  
>  distclean: clean
> -	$(Q)rm -f $(LDIRT)
> +	$(Q)rm -f $(LDIRT) $(CONFIGURE)
>  
>  realclean: distclean
>  	$(Q)rm -f $(CONFIGURE) .gitcensus

That makes distclean do the same thing that realclean does and, as
that commit I pointed out previously, has been intentionally
separated in the build system. You could make the debian/rules file
simply call 'make realclean' rather than 'make distclean' to acheive
this without changing the main makefile. Either way, however, I
don't think that's the problem.

Darrick Wong noted on #xfs that his builds of xfsprogs on ppc64el
from a tarball don't fail at all. His libtool chain is:

>       host-triplet:     powerpc64le-unknown-linux-gnu
>       libtoolize:       (GNU libtool) 2.4.2 Debian-2.4.2-1.7ubuntu1
>       automake:         automake (GNU automake) 1.14.1
>       autoconf:         autoconf (GNU Autoconf) 2.69

Which is similar to mine, but libtoolize is slightly older (and I'm
on x86_64-pc-linux-gnu). It turns out that the reason the build
works on his machines is that someone had made a small manual change
to /usr/share/aclocal/libtool.m4:

--- a/libtool.m4       2014-02-11 04:58:49.000000000 -0800
+++ b/libtool.m4  2015-07-31 13:48:23.375432372 -0700
@@ -1312,7 +1312,7 @@
   rm -rf conftest*
   ;;

-x86_64-*kfreebsd*-gnu|x86_64-*linux*|powerpc*-*linux*| \
+x86_64-*kfreebsd*-gnu|x86_64-*linux*|ppc*-*linux*|powerpc*-*linux*| \
 s390*-*linux*|s390*-*tpf*|sparc*-*linux*)
   # Find out which ABI we are using.
   echo 'int i;' > conftest.$ac_ext

i.e. the problem looks to be that the libtool package shipped in
debian/unstable does not support ppc64el correctly....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  reply	other threads:[~2015-08-04 22:58 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-31 15:21 xfsprogs fails to build on debian ppc64el Daniel Bast
2015-07-31 21:57 ` Fernando Seiti Furusato
2015-08-01 22:52   ` Dave Chinner
2015-08-02 13:06     ` Bug#793495: " Daniel Bast
2015-08-02 22:13       ` Dave Chinner
2015-08-04 15:24         ` Fernando Seiti Furusato
2015-08-04 22:58           ` Dave Chinner [this message]
2015-08-05 13:00             ` Fernando Seiti Furusato
2015-08-05 13:47               ` Dave Chinner
2015-08-05 15:52                 ` Fernando Seiti Furusato
2015-08-06  0:21                   ` Dave Chinner
2015-08-06  7:10                     ` Nathan Scott

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150804225819.GV16638@dastard \
    --to=david@fromorbit.com \
    --cc=793495@bugs.debian.org \
    --cc=brenohl@br.ibm.com \
    --cc=daniel.bast@gmx.net \
    --cc=ferseiti@linux.vnet.ibm.com \
    --cc=xfs@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox