Return-Path: X-Original-To: apmail-apr-dev-archive@www.apache.org Delivered-To: apmail-apr-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id AA40BD635 for ; Wed, 21 Nov 2012 20:07:48 +0000 (UTC) Received: (qmail 70289 invoked by uid 500); 21 Nov 2012 20:07:48 -0000 Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 70158 invoked by uid 500); 21 Nov 2012 20:07:47 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Id: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 70150 invoked by uid 99); 21 Nov 2012 20:07:47 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Nov 2012 20:07:47 +0000 X-ASF-Spam-Status: No, hits=0.0 required=5.0 tests=SPF_HELO_FAIL,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of ben@reser.org designates 192.242.17.19 as permitted sender) Received: from [192.242.17.19] (HELO mail.brain.org) (192.242.17.19) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Nov 2012 20:07:38 +0000 Received: from localhost (localhost [127.0.0.1]) by mail.brain.org (Postfix) with ESMTP id 32B72179E19A for ; Wed, 21 Nov 2012 12:07:16 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at fornix.brain.org Received: from mail.brain.org ([127.0.0.1]) by localhost (fornix.brain.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Wy8DpVlRj1Au for ; Wed, 21 Nov 2012 12:07:12 -0800 (PST) Received: from mail-ie0-f178.google.com (mail-ie0-f178.google.com [209.85.223.178]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by mail.brain.org (Postfix) with ESMTPSA id 977F6179E11B for ; Wed, 21 Nov 2012 12:07:12 -0800 (PST) Received: by mail-ie0-f178.google.com with SMTP id e11so13369956iej.37 for ; Wed, 21 Nov 2012 12:07:11 -0800 (PST) MIME-Version: 1.0 Received: by 10.50.46.198 with SMTP id x6mr775139igm.28.1353528431786; Wed, 21 Nov 2012 12:07:11 -0800 (PST) Received: by 10.64.124.40 with HTTP; Wed, 21 Nov 2012 12:07:11 -0800 (PST) Date: Wed, 21 Nov 2012 12:07:11 -0800 Message-ID: Subject: APR 1.3.x branch doesn't build properly with libtool 2.4 From: Ben Reser To: dev@apr.apache.org Cc: esr@thyrsus.com Content-Type: text/plain; charset=ISO-8859-1 X-Virus-Checked: Checked by ClamAV on apache.org This came up recently on the svn-dev list. http://mail-archives.apache.org/mod_mbox/subversion-dev/201211.mbox/browser It doesn't build because libtool.m4 that comes with libtool 2.4 uses: LIBTOOL='$(SHELL) $(top_builddir)/libtool' And APR doesn't define top_builddir. In 1.4 and newer versions of APR this has been resolved by running a sed command against build/libtool.m4 to replace top_builddir with apr_builddir. Specifically, this was done in r1125475 on apr-trunk. Based on this google search quite a few people seem to have run into this: https://www.google.com/search?q=apr+1.3+LIBTOOL+top_builddir I realize that APR 1.3.x is probably not something you care about terribly much since development has moved along. However, Subversion still has places where we recommend that in our instructions (e.g. Subversion 1.7.x), which I'm going to get corrected, but obviously can't correct the existing older versions in the wild. In the interest of users though I think it might be nice to patch the 1.3.x branch so that it either builds with libtool 2.4 (e.g. adopting the fix newer versions of APR have) or make buildconf refuse to work with newer libtools. Given that the fix is already available, I'd recommend simply applying the fix.