From MAILER-DAEMON Mon Aug 07 11:32:20 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dek0x-0007Nw-St for mharc-bug-automake@gnu.org; Mon, 07 Aug 2017 11:32:19 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45767) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dek0q-0007HR-G5 for bug-automake@gnu.org; Mon, 07 Aug 2017 11:32:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dek0l-0000fe-FC for bug-automake@gnu.org; Mon, 07 Aug 2017 11:32:12 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43420) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dek0l-0000fH-8b for bug-automake@gnu.org; Mon, 07 Aug 2017 11:32:07 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dek0g-00025L-Ux for bug-automake@gnu.org; Mon, 07 Aug 2017 11:32:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Resent-From: "Ghulam, Sean (Nokia - FI/Espoo)" Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Mon, 07 Aug 2017 15:32:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 28002 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: 28002@debbugs.gnu.org X-Debbugs-Original-To: "bug-automake@gnu.org" Received: via spool by submit@debbugs.gnu.org id=B.15021198877955 (code B ref -1); Mon, 07 Aug 2017 15:32:02 +0000 Received: (at submit) by debbugs.gnu.org; 7 Aug 2017 15:31:27 +0000 Received: from localhost ([127.0.0.1]:46094 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dek04-00024C-OR for submit@debbugs.gnu.org; Mon, 07 Aug 2017 11:31:27 -0400 Received: from eggs.gnu.org ([208.118.235.92]:55808) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ded86-0006vP-Ae for submit@debbugs.gnu.org; Mon, 07 Aug 2017 04:11:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ded80-0004mT-3j for submit@debbugs.gnu.org; Mon, 07 Aug 2017 04:11:09 -0400 Received: from lists.gnu.org ([2001:4830:134:3::11]:56811) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ded80-0004mM-12 for submit@debbugs.gnu.org; Mon, 07 Aug 2017 04:11:08 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46768) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ded7y-000376-GO for bug-automake@gnu.org; Mon, 07 Aug 2017 04:11:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ded7u-0004cS-62 for bug-automake@gnu.org; Mon, 07 Aug 2017 04:11:06 -0400 Received: from mail-eopbgr10092.outbound.protection.outlook.com ([40.107.1.92]:3164 helo=EUR02-HE1-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ded7t-0004bC-MZ for bug-automake@gnu.org; Mon, 07 Aug 2017 04:11:02 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=qVTx11xRXAkMZnIVOivy1247meTgZtD1s3aueCdIXBI=; b=Ndgu8TKCiLTYadWAcygqEhuYrLPwoout+RC096BRvbgCmECc1+s5WF9lmjVK/uHf0Ap6+NQj2OduOITmkcrHZOSY9Jni9TTyxj6gJu5gq/qKEWyvSjhrxwxeVkYbYKbo4di5wjeK4ip3Zk2VW7ExeTRBr60EE+ezrJPSw+emW7g= Received: from HE1PR07MB3418.eurprd07.prod.outlook.com (10.170.247.33) by HE1PR07MB1164.eurprd07.prod.outlook.com (10.163.178.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1320.10; Mon, 7 Aug 2017 08:10:58 +0000 Received: from HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da]) by HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da%13]) with mapi id 15.01.1341.010; Mon, 7 Aug 2017 08:10:58 +0000 From: "Ghulam, Sean (Nokia - FI/Espoo)" Thread-Topic: automake prog_error when appending to configure.ac a colon-separated list of input files Thread-Index: AdMPUwbqUBwPBsLjRTyQKDH/+UJXPg== Date: Mon, 7 Aug 2017 08:10:58 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=sean.ghulam@nokia.com; x-originating-ip: [131.228.32.176] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; HE1PR07MB1164; 6:I3pxoYySTdEZj8Ek4HcZNzEBiU1T0dpx+ax+kibzUs6KEAmYtczd9NFEO9N8FRHd0pWBIaEH8Y6K1dNxH/x9MDaIB5lTeFD5+jADcqfPYTh6Qr/YPzEKGZkSr64ElsXfeAHc/C/m324ZfN4LODf8Lh7rMHLTcU+yz3gpV0tDkU7ONRAH9ZVvCz/mcYrKTFkyvkbSzoJt0UDuZilxrg71yuryoA+VFzKBqTSzN4wQvq0B0b+SvlR/ntHz+yE2rijkz4Ot+YltvYxbR1RTvKkZgS30sBGQEQwe0azP6PlBXb6ufcFBicwsd1qxgOgzRY0kSpKKla2frZiy4HFnpDCDBg==; 5:JofbMv8eg8u20p8WdCNYw/Ukfmkdz2Itb5Ur4Qo1OTmbh4T685A0ttTjbWGrXCXQNuW+halIbJGM0bTOd1W6xdfK7/NIRZl01ndstPBfMnLl33UnTGVyQ/tIBehTH6S0Mma7n9qjR6lFX7/O7us5uQ==; 24:xh1Wld7Maz7NfHbaJADYsFuTXrKAvubsKbek9T7PA2FgjBq7sHt0216vgLJ61V+u+aZULZOkk/fSB5f3HEdUKoUtsiV3Mzv/MAAEyRoEt9U=; 7:rSqMVJxpzOphJ0IfnzD5i/YBHTlf3Xv1k/57zTiTK+k2KFrmTSz+XXEs2YsGbclUlz1L8cHlOoRX34XQ99jQzYuxTpOvir5L3YZZunlVpI8hsCkruFVmuKOTwftsU8rDU8YvObiseirHbcbT77Ri0sKpdYA80CX5ERBAU8rBQbF7C0uuTmhhWon5m5ShVZ1NmmnDhT/jWMQKkWkHEruD17TOvxBQpsBskQEC03uG/3Q= x-ms-office365-filtering-correlation-id: ace3d4d2-f85c-425a-d143-08d4dd6bd5e4 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603031)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:HE1PR07MB1164; x-ms-traffictypediagnostic: HE1PR07MB1164: x-exchange-antispam-report-test: UriScan:(22074186197030); x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(100000703101)(100105400095)(3002001)(10201501046)(93006095)(93001095)(6055026)(6041248)(20161123562025)(20161123560025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123555025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:HE1PR07MB1164; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:HE1PR07MB1164; x-forefront-prvs: 0392679D18 x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39850400002)(39840400002)(39450400003)(39400400002)(39410400002)(39860400002)(189002)(199003)(86362001)(7696004)(74316002)(2906002)(97736004)(50986999)(54356999)(478600001)(5250100002)(8936002)(101416001)(9686003)(53936002)(5640700003)(5660300001)(55016002)(99286003)(6436002)(110136004)(38730400002)(2501003)(189998001)(3846002)(102836003)(6116002)(3280700002)(66066001)(81156014)(81166006)(305945005)(68736007)(25786009)(3660700001)(33656002)(7736002)(106356001)(14454004)(6506006)(2351001)(105586002)(6916009)(8676002)(2900100001)(19627235001); DIR:OUT; SFP:1102; SCL:1; SRVR:HE1PR07MB1164; H:HE1PR07MB3418.eurprd07.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; received-spf: None (protection.outlook.com: nokia.com does not designate permitted sender hosts) spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: nokia.com X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Aug 2017 08:10:58.3142 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB1164 X-detected-operating-system: by eggs.gnu.org: Windows 7 or 8 [fuzzy] X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Mailman-Approved-At: Mon, 07 Aug 2017 11:31:23 -0400 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Aug 2017 15:32:18 -0000 $ automake automake: error: src/Makefile:append/append.mk in @other_input_files automake: @other_input_files =3D (src/Makefile:append/append.mk) automake: Please contact . at /usr/share/automake-1.13/Automake/Channels.pm line 662. Automake::Channels::msg('automake', '', 'src/Makefile:append/append= .mk in @other_input_files\x{a}@other_in...') called at /usr/share/automake-= 1.13/Automake/ChannelDefs.pm line 212 Automake::ChannelDefs::prog_error('src/Makefile:append/append.mk in= @other_input_files\x{a}@other_in...') called at /bin/automake line 4214 Automake::handle_configure('Makefile.am', 'Makefile.in', 'Makefile'= , 'Makefile.in') called at /bin/automake line 7873 Automake::generate_makefile('Makefile.am', 'Makefile.in') called at= /bin/automake line 8189 Automake::handle_makefile('Makefile.in') called at /bin/automake li= ne 8203 Automake::handle_makefiles_serial() called at /bin/automake line 83= 37 Tested versions: automake-1.13 and automake-1.15 How to reproduce: Extract /usr/share/doc/automake-1.13.4/amhello-1.0.tar.gz Create directory amhello-1.0/append Create file amhello-1.0/append/append.mk Add any text to amhello-1.0/append/append.mk Edit configure.ac to include append/append.mk in AC_CONFIG_FILES as follows= : AC_CONFIG_FILES([ Makefile src/Makefile:append/append.mk ]) Documentation: AC_CONFIG_FILES=20 You can override an input file name by appending to file a colon-separated = list of input files. Examples: AC_CONFIG_FILES([Makefile:boiler/top.mk:boiler/bot.mk] [lib/Makefile:boiler/lib.mk]) Doing this allows you to keep your file names acceptable to DOS variants, o= r to prepend and/or append boilerplate to the file. Best Regards, Sean From MAILER-DAEMON Mon Aug 07 19:20:13 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1derJl-0005P0-AQ for mharc-bug-automake@gnu.org; Mon, 07 Aug 2017 19:20:13 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35965) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1derJf-0005LE-Dp for bug-automake@gnu.org; Mon, 07 Aug 2017 19:20:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1derJa-0006Qu-Ab for bug-automake@gnu.org; Mon, 07 Aug 2017 19:20:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:42196) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1derJa-0006Qe-6K for bug-automake@gnu.org; Mon, 07 Aug 2017 19:20:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1derJZ-00059p-Ll for bug-automake@gnu.org; Mon, 07 Aug 2017 19:20:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Resent-From: Hans-Bernhard =?UTF-8?Q?Br=C3=B6ker?= Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Mon, 07 Aug 2017 23:20:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 28002 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: "Ghulam, Sean (Nokia - FI/Espoo)" , 28002@debbugs.gnu.org Received: via spool by 28002-submit@debbugs.gnu.org id=B28002.150214799419807 (code B ref 28002); Mon, 07 Aug 2017 23:20:01 +0000 Received: (at 28002) by debbugs.gnu.org; 7 Aug 2017 23:19:54 +0000 Received: from localhost ([127.0.0.1]:50877 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1derJO-00059M-97 for submit@debbugs.gnu.org; Mon, 07 Aug 2017 19:19:54 -0400 Received: from mailout09.t-online.de ([194.25.134.84]:54306) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1derJI-000599-Uh for 28002@debbugs.gnu.org; Mon, 07 Aug 2017 19:19:48 -0400 Received: from fwd10.aul.t-online.de (fwd10.aul.t-online.de [172.20.26.152]) by mailout09.t-online.de (Postfix) with SMTP id B190742708ED; Tue, 8 Aug 2017 01:19:42 +0200 (CEST) Received: from [192.168.2.28] (VrPCdiZQYh-qV+WAP4P945HA0kmYnOb+fh3ONpkZF20nDU4l7pxzQYCclLfH8M2ZML@[91.59.4.132]) by fwd10.t-online.de with (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384 encrypted) esmtp id 1derJE-0unyPA0; Tue, 8 Aug 2017 01:19:40 +0200 References: From: Hans-Bernhard =?UTF-8?Q?Br=C3=B6ker?= Message-ID: <9348af1d-f58b-f509-1f0e-58e4aafa6280@t-online.de> Date: Tue, 8 Aug 2017 01:19:39 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: de-DE Content-Transfer-Encoding: 7bit X-ID: VrPCdiZQYh-qV+WAP4P945HA0kmYnOb+fh3ONpkZF20nDU4l7pxzQYCclLfH8M2ZML X-TOI-MSGID: fa9f92c2-f70d-4e78-9fcf-03e7b0ea63ae X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Aug 2017 23:20:12 -0000 Am 07.08.2017 um 10:10 schrieb Ghulam, Sean (Nokia - FI/Espoo): > AC_CONFIG_FILES([ > Makefile> src/Makefile:append/append.mk> ]) Your bug report might be more convincing if you hadn't got that syntax wrong. Compare your example with the documentation example you found, and particularly note where the []s and are: > AC_CONFIG_FILES([Makefile:boiler/top.mk:boiler/bot.mk] > [lib/Makefile:boiler/lib.mk]) Wouldn't you agree that your example should have been AC_CONFIG_FILES( [Makefile] [src/Makefile:append/append.mk] ) ? From MAILER-DAEMON Tue Aug 08 02:05:14 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dexdi-00011k-J0 for mharc-bug-automake@gnu.org; Tue, 08 Aug 2017 02:05:14 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50395) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dexdb-0000we-Bx for bug-automake@gnu.org; Tue, 08 Aug 2017 02:05:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dexdW-0000fm-9T for bug-automake@gnu.org; Tue, 08 Aug 2017 02:05:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:42317) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dexdW-0000fe-3R for bug-automake@gnu.org; Tue, 08 Aug 2017 02:05:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dexdV-0001S4-OS for bug-automake@gnu.org; Tue, 08 Aug 2017 02:05:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Resent-From: "Ghulam, Sean (Nokia - FI/Espoo)" Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 08 Aug 2017 06:05:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 28002 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: Hans-Bernhard =?UTF-8?Q?Br=C3=B6ker?= , "28002@debbugs.gnu.org" <28002@debbugs.gnu.org> Received: via spool by 28002-submit@debbugs.gnu.org id=B28002.15021722465511 (code B ref 28002); Tue, 08 Aug 2017 06:05:01 +0000 Received: (at 28002) by debbugs.gnu.org; 8 Aug 2017 06:04:06 +0000 Received: from localhost ([127.0.0.1]:50998 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dexcY-0001Qi-Pl for submit@debbugs.gnu.org; Tue, 08 Aug 2017 02:04:06 -0400 Received: from mail-db5eur01on0108.outbound.protection.outlook.com ([104.47.2.108]:24383 helo=EUR01-DB5-obe.outbound.protection.outlook.com) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dexcW-0001QE-NZ for 28002@debbugs.gnu.org; Tue, 08 Aug 2017 02:04:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=ZkTpGdZOT6za1GQ2XyV5QBbBLyyKCm4V4YWYmODlrNc=; b=i1Lme7GTRJMXpLsumjkbqt7cuVJvDO50nivQecIaNGSd79EVPmxR01pip5i0tICgnxcoW68osoSsO3kBheFTWicXG7szLDgZod5kbm8GmRZl2BObscJ/yzf1w5BYl81gbm4wfyNcDU2dcKxuD6PBwiqb25li4vxDmZoNan9KgKM= Received: from HE1PR07MB3418.eurprd07.prod.outlook.com (10.170.247.33) by HE1PR07MB3404.eurprd07.prod.outlook.com (10.170.247.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1341.9; Tue, 8 Aug 2017 06:03:53 +0000 Received: from HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da]) by HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da%13]) with mapi id 15.01.1341.013; Tue, 8 Aug 2017 06:03:53 +0000 From: "Ghulam, Sean (Nokia - FI/Espoo)" Thread-Topic: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Thread-Index: AdMPUwbqUBwPBsLjRTyQKDH/+UJXPgAgJ02AAA1fLmQ= Date: Tue, 8 Aug 2017 06:03:53 +0000 Message-ID: References: , <9348af1d-f58b-f509-1f0e-58e4aafa6280@t-online.de> In-Reply-To: <9348af1d-f58b-f509-1f0e-58e4aafa6280@t-online.de> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=sean.ghulam@nokia.com; x-originating-ip: [131.228.2.16] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; HE1PR07MB3404; 6:ZLGJF27Gr61eRNw4uANFbDxNAUYG9UMWLhTuwdUZxpyAZYPzhR+nxv2lVeSl6twFFddNxaQ07lvR1gcb4GxRz0UJJjqDhoj9cWduo9r4CAwVFaXI6N02aeT6oEiQiglKaWLl6G3bRJg7cND7Ph4VEVba8VzSu3oBVyt56CoOujr3MT10MepjfcFEkgPFcbB1LM3DRsMm38eNPh3dFRjWrRTwgvQQeVFXAfa7NR6AmYU9uJSRWs7FIT0BNv+SdouCfNsHH+9jps8XQPLPBtGeTqMPy3tofGMHIfCuEYyS8p4HwFVckroL/5w4VnlpwKh4n2W5oVhBFo6By1nmnuEQXg==; 5:m+QKnlOareZfbNKdlrK1GiYUDrx3o/eh+722DCAsLCJvZTHgO4bdcZnDGSztScHOf4F1U3biGrYEJM6Dr2ukeqCxfrssQh8ikOkguWYTRQVlwQtm5viSIeY/pXqLrnxr9E1dYEgqN6loWAmgm4bAhA==; 24:m+C25Dx1KaE92rfiSr67KyJdZszxZvtdTCMnWl69Tyw1jAMlxxYYlnkTWa18Rt2fHxo6lVBisZ0Sw6Y1ehNapBBgJlfC1fSQ/VliRayO/fk=; 7:mq0AxJ2UqSKkKE7yufhQCosC0nbfVBD1NNcmBwAJNEwL2u/efe96LiONd1z4KPs9Xg7oKu9Tb9RKk80G1eC6Wy1jTa7RAJ/OjxM4DkWJnwbSnyhuelV6x7UlxKl10bKeRHn5qa7tTJwuUjGPbUGfb7bJfPJvC3OgANDy9CPBKCB6+l19dm79eJMPbV8vmqeQmpI6vGGi2YPvmswGqotHrSYd/MWgXrKJuKGy5vrVmW0= x-ms-office365-filtering-correlation-id: f97b0327-22a1-431f-822f-08d4de233f63 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603031)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:HE1PR07MB3404; x-ms-traffictypediagnostic: HE1PR07MB3404: x-exchange-antispam-report-test: UriScan:(22074186197030); x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(93006095)(93001095)(10201501046)(3002001)(6055026)(6041248)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123562025)(20161123560025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:HE1PR07MB3404; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:HE1PR07MB3404; x-forefront-prvs: 03932714EB x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39410400002)(39400400002)(39850400002)(39450400003)(39860400002)(39840400002)(189002)(377454003)(199003)(2950100002)(101416001)(102836003)(6606003)(25786009)(478600001)(3846002)(6116002)(2501003)(5250100002)(97736004)(105586002)(54356999)(2900100001)(189998001)(50986999)(53546010)(76176999)(19627405001)(106356001)(3280700002)(99286003)(86362001)(3660700001)(5660300001)(8676002)(66066001)(7736002)(8936002)(6246003)(38730400002)(74316002)(14454004)(53936002)(229853002)(9686003)(33656002)(6512007)(54896002)(68736007)(6436002)(6486002)(2906002)(6506006)(81156014)(81166006); DIR:OUT; SFP:1102; SCL:1; SRVR:HE1PR07MB3404; H:HE1PR07MB3418.eurprd07.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; received-spf: None (protection.outlook.com: nokia.com does not designate permitted sender hosts) spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: multipart/alternative; boundary="_000_AM4PR07MB340926A6C5063E555682659DFA8A0AM4PR07MB3409eurp_" MIME-Version: 1.0 X-OriginatorOrg: nokia.com X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Aug 2017 06:03:53.4008 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB3404 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Aug 2017 06:05:12 -0000 --_000_AM4PR07MB340926A6C5063E555682659DFA8A0AM4PR07MB3409eurp_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Hi, Prior to writing the report, I tried it with: AC_INIT([amhello], [1.0], [bug-automake@gnu.org]) AM_INIT_AUTOMAKE([-Wall -Werror foreign]) AC_PROG_CC AC_CONFIG_HEADERS([config.h]) AC_CONFIG_FILES( [Makefile] [src/Makefile:append/append.mk] ) AC_OUTPUT I have tried it once again and still receive the same error. Best Regards, Sean ________________________________ From: Hans-Bernhard Br=F6ker Sent: Tuesday, August 8, 2017 2:19 AM To: Ghulam, Sean (Nokia - FI/Espoo); 28002@debbugs.gnu.org Subject: Re: bug#28002: automake prog_error when appending to configure.ac = a colon-separated list of input files Am 07.08.2017 um 10:10 schrieb Ghulam, Sean (Nokia - FI/Espoo): > AC_CONFIG_FILES([ > Makefile> src/Makefile:append/append.mk> ]) Your bug report might be more convincing if you hadn't got that syntax wrong. Compare your example with the documentation example you found, and particularly note where the []s and are: > AC_CONFIG_FILES([Makefile:boiler/top.mk:boiler/bot.mk] > [lib/Makefile:boiler/lib.mk]) Wouldn't you agree that your example should have been AC_CONFIG_FILES( [Makefile] [src/Makefile:append/append.mk] ) ? --_000_AM4PR07MB340926A6C5063E555682659DFA8A0AM4PR07MB3409eurp_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable

Hi,


Prior to writing the report, I tried it with:

AC_INIT([amhello], [1.0], [bug-automake@gnu.org])
AM_INIT_AUTOMAKE([-Wall -Werror foreign])
AC_PROG_CC
AC_CONFIG_HEADERS([config.h])
AC_CONFIG_FILES(
[Makefile]
[src/Makefile:append/append.mk]
)
AC_OUTPUT

I have tried it once again and still receive the same error.

Best Regards,
Sean




From: Hans-Bernhard Br= =F6ker <HBBroeker@t-online.de>
Sent: Tuesday, August 8, 2017 2:19 AM
To: Ghulam, Sean (Nokia - FI/Espoo); 28002@debbugs.gnu.org
Subject: Re: bug#28002: automake prog_error when appending to config= ure.ac a colon-separated list of input files
 
Am 07.08.2017 um 10:10 schrieb Ghulam, Sean (Nokia= - FI/Espoo):

> AC_CONFIG_FILES([ >   Makefile>   src/Makefi= le:append/append.mk> ])
Your bug report might be more convincing if you hadn't got that syntax
wrong.  Compare your example with the documentation example you found,=
and particularly note where the []s and are:

> AC_CONFIG_FILES([Makefile:boiler/top.mk:boiler/bot.mk]
> [lib/Makefile:boiler/lib.mk])

Wouldn't you agree that your example should have been

AC_CONFIG_FILES(
   [Makefile]
   [src/Makefile:append/append.mk]
)

?
--_000_AM4PR07MB340926A6C5063E555682659DFA8A0AM4PR07MB3409eurp_-- From MAILER-DAEMON Tue Aug 08 14:46:09 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1df9W5-0007V5-Bg for mharc-bug-automake@gnu.org; Tue, 08 Aug 2017 14:46:09 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35647) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1df9W2-0007Uo-CF for bug-automake@gnu.org; Tue, 08 Aug 2017 14:46:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1df9Vy-0000St-F4 for bug-automake@gnu.org; Tue, 08 Aug 2017 14:46:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43528) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1df9Vy-0000Sj-AJ for bug-automake@gnu.org; Tue, 08 Aug 2017 14:46:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1df9Vx-0006dd-Ol for bug-automake@gnu.org; Tue, 08 Aug 2017 14:46:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Resent-From: Hans-Bernhard =?UTF-8?Q?Br=C3=B6ker?= Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 08 Aug 2017 18:46:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 28002 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: "28002@debbugs.gnu.org" <28002@debbugs.gnu.org> Received: via spool by 28002-submit@debbugs.gnu.org id=B28002.150221792625468 (code B ref 28002); Tue, 08 Aug 2017 18:46:01 +0000 Received: (at 28002) by debbugs.gnu.org; 8 Aug 2017 18:45:26 +0000 Received: from localhost ([127.0.0.1]:52208 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1df9VK-0006cY-4C for submit@debbugs.gnu.org; Tue, 08 Aug 2017 14:45:26 -0400 Received: from mailout11.t-online.de ([194.25.134.85]:44534) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1df9VE-0006cK-2f for 28002@debbugs.gnu.org; Tue, 08 Aug 2017 14:45:20 -0400 Received: from fwd17.aul.t-online.de (fwd17.aul.t-online.de [172.20.27.64]) by mailout11.t-online.de (Postfix) with SMTP id F1A1B4241413 for <28002@debbugs.gnu.org>; Tue, 8 Aug 2017 20:45:13 +0200 (CEST) Received: from [192.168.2.28] (Gn40cwZQQh6b-uyGsjNVFsNyTcaVs6Z0vOBhVdwvo7cpI+CbXil1wIsbO126aQrZ8Y@[91.59.1.247]) by fwd17.t-online.de with (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384 encrypted) esmtp id 1df9V4-2EYRrk0; Tue, 8 Aug 2017 20:45:06 +0200 References: <9348af1d-f58b-f509-1f0e-58e4aafa6280@t-online.de> From: Hans-Bernhard =?UTF-8?Q?Br=C3=B6ker?= Message-ID: <821ff125-aa40-015c-0828-a1b06ac4e17f@t-online.de> Date: Tue, 8 Aug 2017 20:45:04 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: de-DE Content-Transfer-Encoding: 7bit X-ID: Gn40cwZQQh6b-uyGsjNVFsNyTcaVs6Z0vOBhVdwvo7cpI+CbXil1wIsbO126aQrZ8Y X-TOI-MSGID: 7fb7990d-1989-4a09-8d46-64ab754de4fe X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Aug 2017 18:46:07 -0000 Am 08.08.2017 um 08:03 schrieb Ghulam, Sean (Nokia - FI/Espoo): > Prior to writing the report, I tried it with: > AC_CONFIG_FILES( > [Makefile] > [src/Makefile:append/append.mk] > ) On closer examination, the problem is that you've been reading only autoconf documenation to find out how to prepare input used by both autoconf and automake. The correct syntax is actually in the automake manual: AC_CONFIG_FILES( [Makefile] [src/Makefile:src/Makefile.in:append/append.mk] ) The trick being that for automake to grok multi-part input, you have to name not just the final output file, but also the intermediate (the one automake is to build from src/Makefile.am). From MAILER-DAEMON Tue Aug 08 17:18:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dfBt8-0003b7-6W for mharc-bug-automake@gnu.org; Tue, 08 Aug 2017 17:18:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51855) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfBt5-0003Zj-Ss for bug-automake@gnu.org; Tue, 08 Aug 2017 17:18:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfBt4-00007g-EK for bug-automake@gnu.org; Tue, 08 Aug 2017 17:18:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43618) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dfBt4-00007Z-Be for bug-automake@gnu.org; Tue, 08 Aug 2017 17:18:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dfBt4-0001ps-70 for bug-automake@gnu.org; Tue, 08 Aug 2017 17:18:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 08 Aug 2017 21:18:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15022270426991 (code B ref 27781); Tue, 08 Aug 2017 21:18:02 +0000 Received: (at 27781) by debbugs.gnu.org; 8 Aug 2017 21:17:22 +0000 Received: from localhost ([127.0.0.1]:52296 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfBsM-0001oc-ML for submit@debbugs.gnu.org; Tue, 08 Aug 2017 17:17:22 -0400 Received: from eggs.gnu.org ([208.118.235.92]:60434) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfBsK-0001oN-0T for 27781@debbugs.gnu.org; Tue, 08 Aug 2017 17:17:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfBsD-0008EA-Mp for 27781@debbugs.gnu.org; Tue, 08 Aug 2017 17:17:10 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:54696) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfBsD-0008E6-JN; Tue, 08 Aug 2017 17:17:09 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=45366 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dfBsC-0007tX-Ue; Tue, 08 Aug 2017 17:17:09 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> Date: Tue, 08 Aug 2017 23:17:06 +0200 In-Reply-To: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> (Michael Haubenwallner's message of "Fri, 21 Jul 2017 11:07:56 +0200") Message-ID: <87fud1ahf1.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Aug 2017 21:18:05 -0000 Hello, Michael Haubenwallner writes: > Hi, > > in flex-2.6.4, the $(LIBOBJDIR) does contain source files only, > eventually required via LDADD=$(LIBOBJ) in another directory. > > Now with --disable-dependency-tracking, an out of source build fails > because $top_builddir/$(LIBOBJDIR) is not created by anyone, see > the discussion in https://github.com/westes/flex/issues/244 and > http://lists.gnu.org/archive/html/autoconf-patches/2017-07/msg00002.html > > I have no idea for how to fix automake for this situation, but it > feels like automake should add this line to src/Makefile.in here: > > $(LIBOBJS): $(LIBOBJDIR)$(am__dirstamp) > > Thanks! > /haubi/ I confirm that this is an Automake issue. After some digging I think 'automake' should call the 'require_build_directory' subroutine for the LIBOBJDIR variable. However I don't have time to investigate more right now. Thanks for the report. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 From MAILER-DAEMON Tue Aug 08 18:08:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dfCfW-0002Au-Ah for mharc-bug-automake@gnu.org; Tue, 08 Aug 2017 18:08:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45258) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfCfT-0002AV-5w for bug-automake@gnu.org; Tue, 08 Aug 2017 18:08:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfCfR-0005wF-TO for bug-automake@gnu.org; Tue, 08 Aug 2017 18:08:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43654) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dfCfR-0005w9-QB for bug-automake@gnu.org; Tue, 08 Aug 2017 18:08:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dfCfR-0004tX-LV for bug-automake@gnu.org; Tue, 08 Aug 2017 18:08:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27773: [PATCH] Allow to override build date Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 08 Aug 2017 22:08:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27773 X-GNU-PR-Package: automake X-GNU-PR-Keywords: patch To: "Bernhard M. Wiedemann" Cc: 27773@debbugs.gnu.org Received: via spool by 27773-submit@debbugs.gnu.org id=B27773.150223007018798 (code B ref 27773); Tue, 08 Aug 2017 22:08:01 +0000 Received: (at 27773) by debbugs.gnu.org; 8 Aug 2017 22:07:50 +0000 Received: from localhost ([127.0.0.1]:52335 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfCfG-0004t8-6z for submit@debbugs.gnu.org; Tue, 08 Aug 2017 18:07:50 -0400 Received: from eggs.gnu.org ([208.118.235.92]:54145) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfCfE-0004sw-Od for 27773@debbugs.gnu.org; Tue, 08 Aug 2017 18:07:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfCf8-0005nL-4g for 27773@debbugs.gnu.org; Tue, 08 Aug 2017 18:07:43 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:57111) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfCf8-0005nF-1P; Tue, 08 Aug 2017 18:07:42 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=45500 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dfCf7-0002zX-Ga; Tue, 08 Aug 2017 18:07:41 -0400 From: Mathieu Lirzin References: <20170720132136.16394-1-bwiedemann@suse.de> Date: Wed, 09 Aug 2017 00:07:39 +0200 In-Reply-To: <20170720132136.16394-1-bwiedemann@suse.de> (Bernhard M. Wiedemann's message of "Thu, 20 Jul 2017 15:21:36 +0200") Message-ID: <87a839af2s.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Aug 2017 22:08:04 -0000 Hello, "Bernhard M. Wiedemann" writes: > and use gmtime to not be influenced by timezone settings > in order to make builds reproducible. > > While working on the reproducible builds effort, I found that > when building the automake package for openSUSE Linux, > there were slight differences between each build > > See https://reproducible-builds.org/ for why this is matters > and https://reproducible-builds.org/specs/source-date-epoch/ > for the definition of this variable. > --- > doc/help2man | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/doc/help2man b/doc/help2man > index e651b8d2d..4a9f75064 100755 > --- a/doc/help2man > +++ b/doc/help2man > @@ -217,7 +217,7 @@ $version_text ||= get_option_value $ARGV[0], $version_option; > # the English version expands to the month as a word and the full year. It > # is used on the footer of the generated manual pages. If in doubt, you may > # just use %x as the value (which should be the full locale-specific date). > -my $date = enc strftime _("%B %Y"), localtime; > +my $date = enc strftime _("%B %Y"), gmtime($ENV{SOURCE_DATE_EPOCH} || time()); > (my $program = $ARGV[0]) =~ s!.*/!!; > my $package = $program; > my $version; 'help2man' is a bundled script that is maintained outside of Automake repository. Unfortunately it has not being synced with upstream for a while. Latest version of GNU Help2man [1] seems to already fix this reproducibility issue [2]. So instead of applying your patch I have simply upgraded the bundled script in commit 9322f409a957f153b38ff37ba79ddf4c19cff6ca. Hopefully that will fix the issue. If that is not the case feel free to reopen this bug. Thanks for the report. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 [1] https://ftp.gnu.org/gnu/help2man/help2man-1.47.4.tar.xz [2] https://anonscm.debian.org/git/users/bod/help2man.git/commit/?id=8448110e8b7567147b70ff54ddf771306a56d39d From MAILER-DAEMON Thu Aug 10 06:46:12 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dfkyh-0003Bu-Sv for mharc-bug-automake@gnu.org; Thu, 10 Aug 2017 06:46:11 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43373) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfkyf-0003Aa-Ov for bug-automake@gnu.org; Thu, 10 Aug 2017 06:46:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfkya-0007Ni-RN for bug-automake@gnu.org; Thu, 10 Aug 2017 06:46:09 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:45331) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dfkya-0007NY-Js for bug-automake@gnu.org; Thu, 10 Aug 2017 06:46:04 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dfkyY-0002h6-An for bug-automake@gnu.org; Thu, 10 Aug 2017 06:46:04 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Resent-From: "Ghulam, Sean (Nokia - FI/Espoo)" Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Thu, 10 Aug 2017 10:46:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 28002 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: "28002@debbugs.gnu.org" <28002@debbugs.gnu.org> Received: via spool by 28002-submit@debbugs.gnu.org id=B28002.150236190210275 (code B ref 28002); Thu, 10 Aug 2017 10:46:02 +0000 Received: (at 28002) by debbugs.gnu.org; 10 Aug 2017 10:45:02 +0000 Received: from localhost ([127.0.0.1]:54012 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfkxZ-0002fO-Kw for submit@debbugs.gnu.org; Thu, 10 Aug 2017 06:45:02 -0400 Received: from mail-db5eur01on0133.outbound.protection.outlook.com ([104.47.2.133]:48674 helo=EUR01-DB5-obe.outbound.protection.outlook.com) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dfkxV-0002eq-HD for 28002@debbugs.gnu.org; Thu, 10 Aug 2017 06:44:59 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=23/wg3ZmN5ixezqKm0YaWridV/JhjdPUYadnoDlNUEY=; b=PxpnQ3ot3sED44EjhUJf8yFIZZQ7YKHHrncwGyJPbyjFvOKm54FJWDH6lVeF0Rlkh0iPQco0J/geaI3ae5/NDOCofMP0dPqY6jaaettbvcCtKcQGbEhWYV1IahdBBtm5kJ99KzHFr2ENS32Wr+nWmsxi8vl4cULhkyI5yUQrQFk= Received: from HE1PR07MB3418.eurprd07.prod.outlook.com (10.170.247.33) by HE1PR07MB1114.eurprd07.prod.outlook.com (10.163.177.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1341.9; Thu, 10 Aug 2017 10:44:50 +0000 Received: from HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da]) by HE1PR07MB3418.eurprd07.prod.outlook.com ([fe80::495d:5adf:320a:74da%13]) with mapi id 15.01.1341.013; Thu, 10 Aug 2017 10:44:50 +0000 From: "Ghulam, Sean (Nokia - FI/Espoo)" Thread-Topic: bug#28002: automake prog_error when appending to configure.ac a colon-separated list of input files Thread-Index: AdMPUwbqUBwPBsLjRTyQKDH/+UJXPgAgJ02AAA1fLmQAbvJXlw== Date: Thu, 10 Aug 2017 10:44:49 +0000 Message-ID: References: , <9348af1d-f58b-f509-1f0e-58e4aafa6280@t-online.de>, In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=sean.ghulam@nokia.com; x-originating-ip: [131.228.2.24] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; HE1PR07MB1114; 6:kDQys6CcxYB5kVXLC0hBA85/t6wYgYWVRY35Jgk/+RlwYJsUibGIx4uhiz8oXXmW+HsoTMJ8G/BLLlriNBkBL2EG5MRmTo2n40B6eHUE/X88CmR3JDKXyphIK9zd6A1G+ZhpCfdJM0WDiKuHM4fA66eno0wCjJKS9SxwUGCUBzkI+C0zfAyOX4H4fVEbZhKFSIXhsCX/PlHPsyiPfPN/WLW6Di/RgnS2C0BovG7t1iNBiWXFR3eTZdzmPvW4rsxpLez8yaB+KWxU4rHd2fwzZaZTeeBpRDnMiqLsBFiMK5Jh/CV/V6uXJhvpx8JqMUVtN5m/eLxE8Rillm1lkOauqw==; 5:KhXsQKISa2U6QSo6IQMrp41fbG+DehzJkG8ZDyuO643R/xaat9W3BnzNEoLFc+jD2Xi+oT+SQSxFfR+fHXzIQONt9cY7NJuKle6R9oJS/W6NPyvxPodx68xaZn/mzcK1tcblIWwF5Q1feX9G3zD6fg==; 24:W4mKvp/aqckVSxjgEKOGU+qxkNTkzMJiDX50vnStkaSDPWNKz6OrSdkTybQxep47Pc2Xg54gwY/E7cBrTxRYg+dhZCxvoXs91Ys/hda/t4E=; 7:MQyBtPJpvro+R3FFTab6mUNQ8GkIGjJfDJ+JYEzy5q3jMgc5+aS//ADYD6wgFVM5IdHHeUDI3/cub0DRg96L7cqzk6dd+e03V93z4B3tGgAXBFvh92H3vduA50kTS5DA5BLG9T3H2NzZ0Em6D4DwuzD72VJfV4ik+SN+25+tp4cRGhAcdZBLE9yBhsGDqfRqfTKHR2pqX8MWSB8OpK5c9uozosyMxlPkdFZzeHd7PgU= x-ms-office365-filtering-correlation-id: f7703422-8459-4cb9-56b6-08d4dfdcd38c x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(300000502095)(300135100095)(22001)(2017030254152)(48565401081)(300000503095)(300135400095)(2017052603031)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:HE1PR07MB1114; x-ms-traffictypediagnostic: HE1PR07MB1114: x-exchange-antispam-report-test: UriScan:; x-microsoft-antispam-prvs: x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(8121501046)(5005006)(10201501046)(93006095)(93001095)(3002001)(100000703101)(100105400095)(6055026)(6041248)(20161123560025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123555025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:HE1PR07MB1114; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:HE1PR07MB1114; x-forefront-prvs: 03950F25EC x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(39860400002)(39450400003)(39840400002)(39400400002)(39410400002)(39850400002)(199003)(189002)(19627405001)(5660300001)(2900100001)(81156014)(55016002)(7696004)(6916009)(2950100002)(2351001)(86362001)(2906002)(6606003)(189998001)(5640700003)(8936002)(99286003)(97736004)(105586002)(74316002)(6436002)(106356001)(81166006)(25786009)(8676002)(14454004)(7736002)(3280700002)(102836003)(3846002)(6116002)(66066001)(3660700001)(558084003)(68736007)(2501003)(6506006)(9686003)(50986999)(478600001)(53336002)(5250100002)(33656002)(54896002)(53936002)(101416001)(110136004)(38730400002)(76176999)(54356999); DIR:OUT; SFP:1102; SCL:1; SRVR:HE1PR07MB1114; H:HE1PR07MB3418.eurprd07.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; received-spf: None (protection.outlook.com: nokia.com does not designate permitted sender hosts) spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: multipart/alternative; boundary="_000_HE1PR07MB34180FA4CD31DE0BF9EB1612FA880HE1PR07MB3418eurp_" MIME-Version: 1.0 X-OriginatorOrg: nokia.com X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Aug 2017 10:44:49.9821 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5d471751-9675-428d-917b-70f44f9630b0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB1114 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 10 Aug 2017 10:46:11 -0000 --_000_HE1PR07MB34180FA4CD31DE0BF9EB1612FA880HE1PR07MB3418eurp_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable The following goes through: AC_CONFIG_FILES( [Makefile] [src/Makefile:src/Makefile.in:append/append.mk] ) Thanks --_000_HE1PR07MB34180FA4CD31DE0BF9EB1612FA880HE1PR07MB3418eurp_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable

The following goes through:


AC_CONFIG_FILES(
[Makefile]
[src/Makefile:src/Makefile.in:append/append.mk]
)

Thanks

--_000_HE1PR07MB34180FA4CD31DE0BF9EB1612FA880HE1PR07MB3418eurp_-- From MAILER-DAEMON Wed Aug 16 12:37:07 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1di1Jb-0008CK-9i for mharc-bug-automake@gnu.org; Wed, 16 Aug 2017 12:37:07 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39665) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1di1JZ-0008By-R7 for bug-automake@gnu.org; Wed, 16 Aug 2017 12:37:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1di1JW-0006f2-KG for bug-automake@gnu.org; Wed, 16 Aug 2017 12:37:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:60950) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1di1JW-0006et-HQ for bug-automake@gnu.org; Wed, 16 Aug 2017 12:37:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1di1JW-0000fY-8R for bug-automake@gnu.org; Wed, 16 Aug 2017 12:37:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Wed, 16 Aug 2017 16:37:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15029014072552 (code B ref 27781); Wed, 16 Aug 2017 16:37:02 +0000 Received: (at 27781) by debbugs.gnu.org; 16 Aug 2017 16:36:47 +0000 Received: from localhost ([127.0.0.1]:41398 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1di1JG-0000f5-Lv for submit@debbugs.gnu.org; Wed, 16 Aug 2017 12:36:46 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:12234) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1di1JE-0000er-RS for 27781@debbugs.gnu.org; Wed, 16 Aug 2017 12:36:45 -0400 X-IronPort-AV: E=Sophos;i="5.41,383,1498514400"; d="scan'208,223";a="2644918" X-IPAS-Result: A2GzAwB1c5RZ/+shHKxdHAEBBAEBCgEBhD6BFaARmCskgRWEDgKFBxQBAgEBAQEBAQEDgRCEAls7AQEBAQIBI1YQCw4DAwECASoCAk8IBg0GAgEBiiSrJIImi1kBAQEBAQEBAQEBAQEBAQEBEw+DKIVcgkg0hD5VgnOCYQWgRYIoggqDIo9Xj3ORG4UANmFLd4d6dIchgkEBAQE Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Aug 2017 18:36:37 +0200 Received: from s01en24.wamas.com ([172.28.41.101]) by mailhost.salomon.at with esmtp (Exim 4.77) (envelope-from ) id 1di1J7-0003xX-Gt; Wed, 16 Aug 2017 18:36:37 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> From: Michael Haubenwallner Message-ID: <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> Date: Wed, 16 Aug 2017 18:36:37 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <87fud1ahf1.fsf@gnu.org> Content-Type: multipart/mixed; boundary="------------75582FD14204CDDB6BE3397F" Content-Language: en-US X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Aug 2017 16:37:07 -0000 This is a multi-part message in MIME format. --------------75582FD14204CDDB6BE3397F Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Hi, On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> >> I have no idea for how to fix automake for this situation, but it >> feels like automake should add this line to src/Makefile.in here: >> >> $(LIBOBJS): $(LIBOBJDIR)$(am__dirstamp) > > I confirm that this is an Automake issue. After some digging I think > 'automake' should call the 'require_build_directory' subroutine for the > LIBOBJDIR variable. However I don't have time to investigate more right > now. In this case let me come up with attached patch for now (without deeper knowledge of automake internals though). Thanks! /haubi/ --------------75582FD14204CDDB6BE3397F Content-Type: text/x-patch; name="0001-automake-Depend-on-LIBOBJDIR-for-LIBOBJS.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0001-automake-Depend-on-LIBOBJDIR-for-LIBOBJS.patch" >From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 From: Michael Haubenwallner Date: Wed, 16 Aug 2017 18:16:12 +0200 Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. This change fixes automake bug#27781. * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for each LIBOBJS/ALLOCA variable used. --- bin/automake.in | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/bin/automake.in b/bin/automake.in index 9c4cb86..b82e8c5 100644 --- a/bin/automake.in +++ b/bin/automake.in @@ -2153,10 +2153,19 @@ sub handle_LIBOBJS_or_ALLOCA $dir = backname ($relative_dir) . "/$dir" if $relative_dir ne '.'; define_variable ('LIBOBJDIR', "$dir", INTERNAL); + my $dirstamp = ''; + # Abusing $clean_files{"$(VAR)"} as indicator for whether + # we have added the "$(VAR): $dirstamp" dependency already. + $dirstamp = require_build_directory ($dir) + if ! defined $clean_files{"\$($var)"}; $clean_files{"\$($var)"} = MOSTLY_CLEAN; + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); # If LTLIBOBJS is used, we must also clear LIBOBJS (which might # be created by libtool as a side-effect of creating LTLIBOBJS). - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; + if ($var =~ s/^LT//) { + $clean_files{"\$($var)"} = MOSTLY_CLEAN; + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); + } } return $dir; -- 2.10.2 --------------75582FD14204CDDB6BE3397F-- From MAILER-DAEMON Wed Aug 16 19:31:11 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1di7mJ-00077n-P2 for mharc-bug-automake@gnu.org; Wed, 16 Aug 2017 19:31:11 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:32841) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1di7mF-00076f-ST for bug-automake@gnu.org; Wed, 16 Aug 2017 19:31:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1di7mA-0001U4-3Q for bug-automake@gnu.org; Wed, 16 Aug 2017 19:31:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:32995) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1di7m9-0001U0-W1 for bug-automake@gnu.org; Wed, 16 Aug 2017 19:31:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1di7m9-0001F7-LC for bug-automake@gnu.org; Wed, 16 Aug 2017 19:31:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Wed, 16 Aug 2017 23:31:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15029262041103 (code B ref 27781); Wed, 16 Aug 2017 23:31:01 +0000 Received: (at 27781) by debbugs.gnu.org; 16 Aug 2017 23:30:04 +0000 Received: from localhost ([127.0.0.1]:41676 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1di7lD-0000HL-LI for submit@debbugs.gnu.org; Wed, 16 Aug 2017 19:30:04 -0400 Received: from eggs.gnu.org ([208.118.235.92]:41700) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1di7lC-0000Fs-AL for 27781@debbugs.gnu.org; Wed, 16 Aug 2017 19:30:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1di7l6-0000G4-7w for 27781@debbugs.gnu.org; Wed, 16 Aug 2017 19:29:57 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:35336) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1di7l6-0000G0-3h; Wed, 16 Aug 2017 19:29:56 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=37002 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1di7l5-0002r2-LM; Wed, 16 Aug 2017 19:29:55 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> Date: Thu, 17 Aug 2017 01:29:54 +0200 In-Reply-To: <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> (Michael Haubenwallner's message of "Wed, 16 Aug 2017 18:36:37 +0200") Message-ID: <87inhnf5vx.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Aug 2017 23:31:08 -0000 Hi, Michael Haubenwallner writes: > On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >>> >>> I have no idea for how to fix automake for this situation, but it >>> feels like automake should add this line to src/Makefile.in here: >>> >>> $(LIBOBJS): $(LIBOBJDIR)$(am__dirstamp) > >> >> I confirm that this is an Automake issue. After some digging I think >> 'automake' should call the 'require_build_directory' subroutine for the >> LIBOBJDIR variable. However I don't have time to investigate more right >> now. > > In this case let me come up with attached patch for now > (without deeper knowledge of automake internals though). > > Thanks! > /haubi/ > > From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 > From: Michael Haubenwallner > Date: Wed, 16 Aug 2017 18:16:12 +0200 > Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. > > This change fixes automake bug#27781. > > * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for > each LIBOBJS/ALLOCA variable used. > --- Excellent! To ensure this bug is not reintroduced later, I think it is important to combine this patch with a test. Are you willing to look into adding one? Thank you -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 From MAILER-DAEMON Thu Aug 17 08:36:14 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1diK22-0004TO-Ph for mharc-bug-automake@gnu.org; Thu, 17 Aug 2017 08:36:14 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35417) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1diK1w-0004Mz-5Z for bug-automake@gnu.org; Thu, 17 Aug 2017 08:36:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1diK1q-0006VM-54 for bug-automake@gnu.org; Thu, 17 Aug 2017 08:36:08 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:33392) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1diK1q-0006VI-0q for bug-automake@gnu.org; Thu, 17 Aug 2017 08:36:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1diK1p-0004li-Q1 for bug-automake@gnu.org; Thu, 17 Aug 2017 08:36:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Thu, 17 Aug 2017 12:36:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150297331918280 (code B ref 27781); Thu, 17 Aug 2017 12:36:01 +0000 Received: (at 27781) by debbugs.gnu.org; 17 Aug 2017 12:35:19 +0000 Received: from localhost ([127.0.0.1]:42073 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1diK11-0004kd-U0 for submit@debbugs.gnu.org; Thu, 17 Aug 2017 08:35:19 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:8770) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1diK0r-0004jp-P1 for 27781@debbugs.gnu.org; Thu, 17 Aug 2017 08:35:10 -0400 X-IronPort-AV: E=Sophos;i="5.41,387,1498514400"; d="scan'208,223";a="2680762" X-IPAS-Result: A2FuBQCAjJVZ/+shHKxdHAEBBAEBCgEBhD6BFaASk0cIglqBQUMsgQ1TgzsChRkVAQIBAQEBAQEBA4EQgmEEOA4pAwEBAQEBAQEBAQEBAQEBAQEBAQEBAQEVAQEBAQEBAQEBAQEBAQEBARoCDSIkFwEBAQEDeRALDgMDAQIBLk8IBg0GAgEBBYo3rD+LYgEBAQEBAQEBAQEBAQEBAQEBEg+DKIVcgnyEOlmCNoMeBZ93UoIoggqDIoMMjEuIX4cUjGiENYUANYEtd16HHHSKJAEBAQ Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Aug 2017 14:34:53 +0200 Received: from [172.28.53.67] by mailhost.salomon.at with esmtps (UNKNOWN:AES128-SHA:128) (Exim 4.77) (envelope-from ) id 1diK0j-0003P6-C5; Thu, 17 Aug 2017 14:34:53 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <87inhnf5vx.fsf@gnu.org> From: Michael Haubenwallner Message-ID: <5c7cfc72-ce5a-2b77-340b-ab10dd6a1a74@ssi-schaefer.com> Date: Thu, 17 Aug 2017 14:34:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <87inhnf5vx.fsf@gnu.org> Content-Type: multipart/mixed; boundary="------------BBE64B86E4F72E86E8BF4E5C" Content-Language: en-US X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Aug 2017 12:36:13 -0000 This is a multi-part message in MIME format. --------------BBE64B86E4F72E86E8BF4E5C Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 7bit On 08/17/2017 01:29 AM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: >>> Michael Haubenwallner writes: >> >> This change fixes automake bug#27781. >> >> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >> each LIBOBJS/ALLOCA variable used. >> --- > > Excellent! > > To ensure this bug is not reintroduced later, I think it is important to > combine this patch with a test. Are you willing to look into adding > one? Ok - like this one? Thanks! /haubi/ --------------BBE64B86E4F72E86E8BF4E5C Content-Type: text/x-patch; name="0001-automake-add-test-for-PR-27781.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0001-automake-add-test-for-PR-27781.patch" >From 5c6e104ae5f2fa6d276fcc662d27ef046e77750d Mon Sep 17 00:00:00 2001 From: Michael Haubenwallner Date: Thu, 17 Aug 2017 14:12:20 +0200 Subject: [PATCH] automake: add test for PR 27781 Add test for LIBOBJ_DIR containing nothing else but LIBOBJS source files, in out of source build with dependency tracking disabled, reported as PR 27781. * t/pr27781.sh: New file. * t/list-of-tests.mk (handwritten_TESTS): Add t/pr27781.sh. --- t/list-of-tests.mk | 1 + t/pr27781.sh | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 65 insertions(+) create mode 100644 t/pr27781.sh diff --git a/t/list-of-tests.mk b/t/list-of-tests.mk index 051ac0c..8680483 100644 --- a/t/list-of-tests.mk +++ b/t/list-of-tests.mk @@ -886,6 +886,7 @@ t/pr307.sh \ t/pr401.sh \ t/pr401b.sh \ t/pr401c.sh \ +t/pr27781.sh \ t/prefix.sh \ t/preproc-basics.sh \ t/preproc-c-compile.sh \ diff --git a/t/pr27781.sh b/t/pr27781.sh new file mode 100644 index 0000000..15e2da1 --- /dev/null +++ b/t/pr27781.sh @@ -0,0 +1,64 @@ +#! /bin/sh +# Copyright (C) 2017 Free Software Foundation, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2, or (at your option) +# any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# Test for PR 27781. +# +# == Description == +# Suppose you have some LIBOBJS source files in a separate LIBOBJ_DIR, +# but nothing else, even not some Makefile. +# Now in an out of source build, with dependency tracking disabled, +# there is nothing else that requires the LIBOBJ_DIR within the +# build directory - except for the LIBOBJS files. +# +# This has happened with flex-2.6.4 when malloc.o was required, +# reported as https://github.com/westes/flex/issues/244. +# +# We have to make sure the LIBOBJ_DIR is a prerequisite of the +# LIBOBJS files in the Makefile using LIBOBJS. +# + +. test-init.sh + +cat >> configure.ac << 'END' +AC_PROG_CC +AC_CONFIG_LIBOBJ_DIR(foo) +AC_LIBOBJ(foo) +AC_OUTPUT +END + +cat > Makefile.am << 'END' +bin_PROGRAMS = helldl +LDADD = $(LIBOBJS) +.PHONY: test-pr27781 +test-pr27781: $(LIBOBJS) +END + +mkdir foo +cat > foo/foo.c << 'END' +int foo() { return 0; } +END + +mkdir build + +$ACLOCAL +$AUTOCONF +$AUTOMAKE + +cd build +../configure --disable-dependency-tracking +run_make test-pr27781 + +: -- 2.10.2 --------------BBE64B86E4F72E86E8BF4E5C-- From MAILER-DAEMON Sun Aug 20 08:36:05 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1djPSX-000333-7V for mharc-bug-automake@gnu.org; Sun, 20 Aug 2017 08:36:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58550) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djPSV-00032e-B1 for bug-automake@gnu.org; Sun, 20 Aug 2017 08:36:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djPSU-0000dH-Fg for bug-automake@gnu.org; Sun, 20 Aug 2017 08:36:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:37488) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1djPSU-0000d9-BW for bug-automake@gnu.org; Sun, 20 Aug 2017 08:36:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1djPSU-0003h2-26 for bug-automake@gnu.org; Sun, 20 Aug 2017 08:36:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#28160: Support newer version of python Resent-From: Bastien ROUCARIES Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Sun, 20 Aug 2017 12:36:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 28160 X-GNU-PR-Package: automake X-GNU-PR-Keywords: To: 28160@debbugs.gnu.org X-Debbugs-Original-To: bug-automake@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.150323251514140 (code B ref -1); Sun, 20 Aug 2017 12:36:01 +0000 Received: (at submit) by debbugs.gnu.org; 20 Aug 2017 12:35:15 +0000 Received: from localhost ([127.0.0.1]:46169 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djPRi-0003g0-Tq for submit@debbugs.gnu.org; Sun, 20 Aug 2017 08:35:15 -0400 Received: from eggs.gnu.org ([208.118.235.92]:39292) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djPRh-0003fn-9D for submit@debbugs.gnu.org; Sun, 20 Aug 2017 08:35:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djPRb-0007cB-6z for submit@debbugs.gnu.org; Sun, 20 Aug 2017 08:35:08 -0400 Received: from lists.gnu.org ([2001:4830:134:3::11]:40082) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1djPRb-0007c1-48 for submit@debbugs.gnu.org; Sun, 20 Aug 2017 08:35:07 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58496) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djPRa-0002lN-5D for bug-automake@gnu.org; Sun, 20 Aug 2017 08:35:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djPRZ-0007ar-Bs for bug-automake@gnu.org; Sun, 20 Aug 2017 08:35:06 -0400 Received: from mail-wr0-x22d.google.com ([2a00:1450:400c:c0c::22d]:38347) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1djPRZ-0007ZS-4y for bug-automake@gnu.org; Sun, 20 Aug 2017 08:35:05 -0400 Received: by mail-wr0-x22d.google.com with SMTP id p8so13052733wrf.5 for ; Sun, 20 Aug 2017 05:35:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=EQN6bciBZhshYwrda12g5jfmhaQZN6E/itSKRB1JX4o=; b=snyu47WOiqy+iympzM9w6j2Q61u/zPdH0yn5sOfKX+A1eworbROMKEWOjAQxF2yE96 xjKalxtun8LByiVHD6gYmn2wUVZDC3o82Y4pNWGlABUBtoHJPmMjcwWB9QeDyJvI/FNO qwWUGoA/FBRWDgCt1JCOe2cB7QvDb9nEdg5YASt2K1sKMStn6COU4KSZRYBBLZlSRO1M wUlDqcn0oUv1GFkaDvgX86Tm8pKF87gZTihlxtP3bjnAs0hvzwbMBmMQwHp/VaI9jE+9 spAy7BuxOjxyTDDv2RaTAX9C7Hl3olei2ZnLHw7kR+Y5xanhh/dFTd7WKV/WdraKSYfR Vl0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=EQN6bciBZhshYwrda12g5jfmhaQZN6E/itSKRB1JX4o=; b=CUxBWJ8BXBe9zWjlJarnuRBvqCNDMXG7q26/QrYT17woBSGHeqMNdq9EC4dKg/AiXq x/nRgbnLXKKCjNPsWUtXaP7PaW0SFdqXo+7We3P7tLYM9f9KAfF9MsXyyqdBgCe4PGbN zByGjqu13ssRuDzkBgIOWid4QOlQVvavQ/o2GIP3VX/XJ37v2yYm8koGt77pOUii7lXd S3mH/Ux+T3wvkMccnbNyhvDd1ucKBdKu4Em4UTV/6N67LW62k/Oe1YI5+aRH9/Jk537v NnKHwHStPjC1EAIphx9OGso2TFyTDoEI/ZL8Xg6CJmNkP4/BXRxOGtUR7+I7YM0aiHBa 4zDQ== X-Gm-Message-State: AHYfb5gqbQm5w6N+rwF10O2gOJlkwsgfX/qGyCSqBlhlS9hLNbFxJ2HO 71BdKDDGTv31RushlrwG4uDdmw33Admm X-Received: by 10.28.236.208 with SMTP id h77mr191120wmi.162.1503232501616; Sun, 20 Aug 2017 05:35:01 -0700 (PDT) MIME-Version: 1.0 Received: by 10.28.209.143 with HTTP; Sun, 20 Aug 2017 05:34:30 -0700 (PDT) From: Bastien ROUCARIES Date: Sun, 20 Aug 2017 14:34:30 +0200 Message-ID: Content-Type: text/plain; charset="UTF-8" X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 20 Aug 2017 12:36:04 -0000 Hi, Following https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=872052 could you suppoort python3.8 python3.7 and pyhton3.6 ? Thanks From MAILER-DAEMON Mon Aug 21 18:25:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1djv86-0006Ih-SG for mharc-bug-automake@gnu.org; Mon, 21 Aug 2017 18:25:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56609) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djv83-0006Hp-O1 for bug-automake@gnu.org; Mon, 21 Aug 2017 18:25:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djv82-0000Lu-5h for bug-automake@gnu.org; Mon, 21 Aug 2017 18:25:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:39615) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1djv82-0000Lq-2E for bug-automake@gnu.org; Mon, 21 Aug 2017 18:25:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1djv81-0004bH-OL for bug-automake@gnu.org; Mon, 21 Aug 2017 18:25:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Mon, 21 Aug 2017 22:25:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150335425117625 (code B ref 27781); Mon, 21 Aug 2017 22:25:01 +0000 Received: (at 27781) by debbugs.gnu.org; 21 Aug 2017 22:24:11 +0000 Received: from localhost ([127.0.0.1]:48296 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djv70-0004Zj-7P for submit@debbugs.gnu.org; Mon, 21 Aug 2017 18:24:10 -0400 Received: from eggs.gnu.org ([208.118.235.92]:36071) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djv6q-0004ZP-KR for 27781@debbugs.gnu.org; Mon, 21 Aug 2017 18:23:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djv6i-00004r-93 for 27781@debbugs.gnu.org; Mon, 21 Aug 2017 18:23:43 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:45250) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djv6i-0008WR-4C; Mon, 21 Aug 2017 18:23:40 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=44196 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1djv6h-0000vy-Fs; Mon, 21 Aug 2017 18:23:39 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <87inhnf5vx.fsf@gnu.org> <5c7cfc72-ce5a-2b77-340b-ab10dd6a1a74@ssi-schaefer.com> Date: Tue, 22 Aug 2017 00:23:36 +0200 In-Reply-To: <5c7cfc72-ce5a-2b77-340b-ab10dd6a1a74@ssi-schaefer.com> (Michael Haubenwallner's message of "Thu, 17 Aug 2017 14:34:39 +0200") Message-ID: <87bmn8k1av.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Aug 2017 22:25:06 -0000 --=-=-= Content-Type: text/plain Michael Haubenwallner writes: > On 08/17/2017 01:29 AM, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >>> On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: >>>> Michael Haubenwallner writes: > >>> >>> This change fixes automake bug#27781. >>> >>> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >>> each LIBOBJS/ALLOCA variable used. >>> --- >> >> Excellent! >> >> To ensure this bug is not reintroduced later, I think it is important to >> combine this patch with a test. Are you willing to look into adding >> one? > > Ok - like this one? > > Thanks! > /haubi/ > Yes, I have made some changes to your original patch (i.e. changing the test name and the description). If you are OK with those I will push it. --=-=-= Content-Type: text/x-patch Content-Disposition: attachment; filename=0001-automake-Ensure-that-LIBOBJ_DIR-builddir-is-created.patch >From 236cb06666a44f23157aa95adc26450fdec92126 Mon Sep 17 00:00:00 2001 From: Michael Haubenwallner Date: Thu, 17 Aug 2017 14:12:20 +0200 Subject: [PATCH] automake: Ensure that LIBOBJ_DIR builddir is created * t/libobj-no-dependency-tracking.sh: New test reproducing the issue described in automake bug#27781. * t/list-of-tests.mk (handwritten_TESTS, XFAIL_TESTS): Add it. --- t/libobj-no-dependency-tracking.sh | 56 ++++++++++++++++++++++++++++++++++++++ t/list-of-tests.mk | 2 ++ 2 files changed, 58 insertions(+) create mode 100644 t/libobj-no-dependency-tracking.sh diff --git a/t/libobj-no-dependency-tracking.sh b/t/libobj-no-dependency-tracking.sh new file mode 100644 index 0000000..1f02405 --- /dev/null +++ b/t/libobj-no-dependency-tracking.sh @@ -0,0 +1,56 @@ +#! /bin/sh +# Copyright (C) 2017 Free Software Foundation, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2, or (at your option) +# any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# Ensure that LIBOBJS source files are properly built without dependency +# tracking when using out of tree builds. +# +# This is a non regression test which is following an issue in flex-2.6.4 when +# "malloc.o" was required, see . + +. test-init.sh + +# The LIBOBJS are in a separate LIBOBJ_DIR directory without anything else in +# it to not trigger the creation of the build directory accidentally. +cat >> configure.ac << 'END' +AC_PROG_CC +AC_CONFIG_LIBOBJ_DIR([foo]) +AC_LIBOBJ([foo]) +AC_OUTPUT +END + +cat > Makefile.am << 'END' +AUTOMAKE_OPTIONS = subdir-objects +bin_PROGRAMS = helldl +LDADD = $(LIBOBJS) +.PHONY: dummy +dummy: $(LIBOBJS) +END + +mkdir foo +cat > foo/foo.c << 'END' +int foo() { return 0; } +END + +mkdir build + +$ACLOCAL +$AUTOCONF +$AUTOMAKE + +cd build +../configure --disable-dependency-tracking +run_make dummy +: diff --git a/t/list-of-tests.mk b/t/list-of-tests.mk index dacdc8d..dab4a7c 100644 --- a/t/list-of-tests.mk +++ b/t/list-of-tests.mk @@ -38,6 +38,7 @@ t/override-conditional-pr13940.sh \ t/dist-pr109765.sh \ t/instdir-cond2.sh \ t/java-nobase.sh \ +t/libobj-no-dependency-tracking.sh \ t/objext-pr10128.sh \ t/remake-timing-bug-pr8365.sh \ t/lex-subobj-nodep.sh \ @@ -602,6 +603,7 @@ t/lflags.sh \ t/lflags-cxx.sh \ t/libexec.sh \ t/libobj-basic.sh \ +t/libobj-no-dependency-tracking.sh \ t/libobj2.sh \ t/libobj3.sh \ t/libobj4.sh \ -- 2.9.5 --=-=-= Content-Type: text/plain -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 --=-=-=-- From MAILER-DAEMON Mon Aug 21 18:41:05 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1djvNZ-0004A1-AW for mharc-bug-automake@gnu.org; Mon, 21 Aug 2017 18:41:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42162) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djvNW-00049V-Vw for bug-automake@gnu.org; Mon, 21 Aug 2017 18:41:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djvNW-0001eH-2u for bug-automake@gnu.org; Mon, 21 Aug 2017 18:41:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:39624) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1djvNV-0001e0-UV for bug-automake@gnu.org; Mon, 21 Aug 2017 18:41:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1djvNV-0004xa-Kz for bug-automake@gnu.org; Mon, 21 Aug 2017 18:41:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Mon, 21 Aug 2017 22:41:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150335524319033 (code B ref 27781); Mon, 21 Aug 2017 22:41:01 +0000 Received: (at 27781) by debbugs.gnu.org; 21 Aug 2017 22:40:43 +0000 Received: from localhost ([127.0.0.1]:48305 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djvND-0004wv-9b for submit@debbugs.gnu.org; Mon, 21 Aug 2017 18:40:43 -0400 Received: from eggs.gnu.org ([208.118.235.92]:51006) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1djvNB-0004wj-JC for 27781@debbugs.gnu.org; Mon, 21 Aug 2017 18:40:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1djvN5-00014j-78 for 27781@debbugs.gnu.org; Mon, 21 Aug 2017 18:40:36 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:45546) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1djvN5-00014I-2D; Mon, 21 Aug 2017 18:40:35 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=44204 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1djvN4-0007rU-BB; Mon, 21 Aug 2017 18:40:34 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> Date: Tue, 22 Aug 2017 00:40:32 +0200 In-Reply-To: <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> (Michael Haubenwallner's message of "Wed, 16 Aug 2017 18:36:37 +0200") Message-ID: <874lt0k0in.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Aug 2017 22:41:04 -0000 Michael Haubenwallner writes: > In this case let me come up with attached patch for now > (without deeper knowledge of automake internals though). I have tested this patch, and confirm that it fixes the bug. However I have on question regarding the added code. > From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 > From: Michael Haubenwallner > Date: Wed, 16 Aug 2017 18:16:12 +0200 > Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. > > This change fixes automake bug#27781. > > * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for > each LIBOBJS/ALLOCA variable used. > --- > bin/automake.in | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/bin/automake.in b/bin/automake.in > index 9c4cb86..b82e8c5 100644 > --- a/bin/automake.in > +++ b/bin/automake.in > @@ -2153,10 +2153,19 @@ sub handle_LIBOBJS_or_ALLOCA > $dir = backname ($relative_dir) . "/$dir" > if $relative_dir ne '.'; > define_variable ('LIBOBJDIR', "$dir", INTERNAL); > + my $dirstamp = ''; > + # Abusing $clean_files{"$(VAR)"} as indicator for whether > + # we have added the "$(VAR): $dirstamp" dependency already. I am not sure to understand what this comment really means. Can you explain it in other words? > + $dirstamp = require_build_directory ($dir) > + if ! defined $clean_files{"\$($var)"}; > $clean_files{"\$($var)"} = MOSTLY_CLEAN; > + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); > # If LTLIBOBJS is used, we must also clear LIBOBJS (which might > # be created by libtool as a side-effect of creating LTLIBOBJS). > - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; > + if ($var =~ s/^LT//) { > + $clean_files{"\$($var)"} = MOSTLY_CLEAN; > + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); > + } > } > > return $dir; Thank you. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 From MAILER-DAEMON Tue Aug 22 03:05:12 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dk3FP-0006J4-VL for mharc-bug-automake@gnu.org; Tue, 22 Aug 2017 03:05:11 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50863) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dk3FK-0006Ce-QD for bug-automake@gnu.org; Tue, 22 Aug 2017 03:05:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dk3FH-0005rq-0Z for bug-automake@gnu.org; Tue, 22 Aug 2017 03:05:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:39824) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dk3FG-0005ri-U3 for bug-automake@gnu.org; Tue, 22 Aug 2017 03:05:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dk3FG-0001NS-GO for bug-automake@gnu.org; Tue, 22 Aug 2017 03:05:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 22 Aug 2017 07:05:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15033854465226 (code B ref 27781); Tue, 22 Aug 2017 07:05:02 +0000 Received: (at 27781) by debbugs.gnu.org; 22 Aug 2017 07:04:06 +0000 Received: from localhost ([127.0.0.1]:48505 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dk3EM-0001MD-Hv for submit@debbugs.gnu.org; Tue, 22 Aug 2017 03:04:06 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:29707) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dk3EJ-0001Lh-VE for 27781@debbugs.gnu.org; Tue, 22 Aug 2017 03:04:04 -0400 X-IronPort-AV: E=Sophos;i="5.41,411,1498514400"; d="scan'208";a="2822370" X-IPAS-Result: A2E/BQA31ptZ/+shHKxcGgEBAQECAQEBAQgBAQEBhVOgGJYfghGFRwKEXBEBAgEBAQEBAQEDgRCFGQEFeRALDgouVwYNBgIBAblei2MBAQEBAQEBAwEBAQEkgyqFXIJ8imcFkRiHB4gzgiiIOZUshxWWJzkBgSh3h3p0iiYBAQE Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Aug 2017 09:03:57 +0200 Received: from [172.28.53.104] by mailhost.salomon.at with esmtps (UNKNOWN:AES128-SHA:128) (Exim 4.77) (envelope-from ) id 1dk3EC-0004Xc-Us; Tue, 22 Aug 2017 09:03:57 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <87inhnf5vx.fsf@gnu.org> <5c7cfc72-ce5a-2b77-340b-ab10dd6a1a74@ssi-schaefer.com> <87bmn8k1av.fsf@gnu.org> From: Michael Haubenwallner Message-ID: <2ce25cf1-2d5e-2ea5-8b93-ea3eb12a1737@ssi-schaefer.com> Date: Tue, 22 Aug 2017 09:03:37 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <87bmn8k1av.fsf@gnu.org> Content-Type: text/plain; charset=iso-8859-15 Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Aug 2017 07:05:10 -0000 On 08/22/2017 12:23 AM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> On 08/17/2017 01:29 AM, Mathieu Lirzin wrote: >>> Michael Haubenwallner writes: >>>> On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: >>>>> Michael Haubenwallner writes: >>>> >>>> This change fixes automake bug#27781. >>>> >>>> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >>>> each LIBOBJS/ALLOCA variable used. >>>> --- >>> >>> Excellent! >>> >>> To ensure this bug is not reintroduced later, I think it is important to >>> combine this patch with a test. Are you willing to look into adding >>> one? >> >> Ok - like this one? > > Yes, I have made some changes to your original patch (i.e. changing the > test name and the description). If you are OK with those I will push it. > YesPlease! Thanks! /haubi/ From MAILER-DAEMON Tue Aug 22 03:31:10 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dk3eY-000397-HI for mharc-bug-automake@gnu.org; Tue, 22 Aug 2017 03:31:10 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58073) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dk3eW-00038W-Ni for bug-automake@gnu.org; Tue, 22 Aug 2017 03:31:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dk3eS-0004mC-2d for bug-automake@gnu.org; Tue, 22 Aug 2017 03:31:08 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:39845) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dk3eR-0004iK-V4 for bug-automake@gnu.org; Tue, 22 Aug 2017 03:31:04 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dk3eQ-0001ys-Ii for bug-automake@gnu.org; Tue, 22 Aug 2017 03:31:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Tue, 22 Aug 2017 07:31:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15033870597605 (code B ref 27781); Tue, 22 Aug 2017 07:31:02 +0000 Received: (at 27781) by debbugs.gnu.org; 22 Aug 2017 07:30:59 +0000 Received: from localhost ([127.0.0.1]:48526 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dk3eN-0001ya-18 for submit@debbugs.gnu.org; Tue, 22 Aug 2017 03:30:59 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:12597) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dk3eK-0001yM-K8 for 27781@debbugs.gnu.org; Tue, 22 Aug 2017 03:30:57 -0400 X-IronPort-AV: E=Sophos;i="5.41,411,1498514400"; d="scan'208";a="2824225" X-IPAS-Result: A2HkAQA93ZtZ/+shHKxTChoBAQEBAgEBAQEIAQEBAYQ+gRWdc4IlmDAkgRWEDgKEWRQBAgEBAQEBAQEDgRCEAls7AQEBAQN5EAsOAwMBAgEuTwgGDQYCAQG6BYtiAQEBAQEBAQEBAQEBAQEBI4MqhVyCfIQ+CoYfBYl3hyGBZ41TgiiFLY9XiGGHFZEnhQA2gSx3h3p0h2WCQQEBAQ Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Aug 2017 09:30:23 +0200 Received: from [172.28.53.104] by mailhost.salomon.at with esmtps (UNKNOWN:AES128-SHA:128) (Exim 4.77) (envelope-from ) id 1dk3dl-00054r-Hw; Tue, 22 Aug 2017 09:30:21 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> From: Michael Haubenwallner Message-ID: <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> Date: Tue, 22 Aug 2017 09:30:02 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <874lt0k0in.fsf@gnu.org> Content-Type: text/plain; charset=iso-8859-15 Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Aug 2017 07:31:09 -0000 On 08/22/2017 12:40 AM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: > >> In this case let me come up with attached patch for now >> (without deeper knowledge of automake internals though). > > I have tested this patch, and confirm that it fixes the bug. However I > have on question regarding the added code. > >> From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 >> From: Michael Haubenwallner >> Date: Wed, 16 Aug 2017 18:16:12 +0200 >> Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. >> >> This change fixes automake bug#27781. >> >> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >> each LIBOBJS/ALLOCA variable used. >> --- >> bin/automake.in | 11 ++++++++++- >> 1 file changed, 10 insertions(+), 1 deletion(-) >> >> diff --git a/bin/automake.in b/bin/automake.in >> index 9c4cb86..b82e8c5 100644 >> --- a/bin/automake.in >> +++ b/bin/automake.in >> @@ -2153,10 +2153,19 @@ sub handle_LIBOBJS_or_ALLOCA >> $dir = backname ($relative_dir) . "/$dir" >> if $relative_dir ne '.'; >> define_variable ('LIBOBJDIR', "$dir", INTERNAL); >> + my $dirstamp = ''; >> + # Abusing $clean_files{"$(VAR)"} as indicator for whether >> + # we have added the "$(VAR): $dirstamp" dependency already. > > I am not sure to understand what this comment really means. Can you > explain it in other words? > >> + $dirstamp = require_build_directory ($dir) Thing is, $dirstamp is calculated only when ... >> + if ! defined $clean_files{"\$($var)"}; ... $clean_files{} was defined, which is done ... >> $clean_files{"\$($var)"} = MOSTLY_CLEAN; ... here. Subsequently, $output_rules is extended only when $dirstamp was calculated: >> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); But the name "clean_files" feels somewhat unrelated to the dirstamp dependency (as in causing a different output by itself), because I use it to make sure the dirstamp rule is added only once to $output_rules: Within flex, I've seen handle_LIBOBJS_or_ALLOCA to be called thrice, but this dirstamp rule is enough once, as "\$($var)" holds all three object files. While I have also thought about adding this dependency for each explicit object file separately - which would not need the "clean_files" indicator, I've failed to identify where to get the correct object file name from. >> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >> # be created by libtool as a side-effect of creating LTLIBOBJS). >> - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; >> + if ($var =~ s/^LT//) { >> + $clean_files{"\$($var)"} = MOSTLY_CLEAN; >> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); >> + } >> } >> >> return $dir; Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile line: If I remember correctly, there have been (non-GNU) make implementations that choke on this rule when LIBOBJS is an empty variable: Am I wrong here, or is GNU make required anyway these days? Thanks! /haubi/ From MAILER-DAEMON Wed Aug 23 16:16:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkc4M-0001My-4F for mharc-bug-automake@gnu.org; Wed, 23 Aug 2017 16:16:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40668) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkc4J-0001K9-Cd for bug-automake@gnu.org; Wed, 23 Aug 2017 16:16:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkc4I-00020V-GV for bug-automake@gnu.org; Wed, 23 Aug 2017 16:16:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43069) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkc4I-000205-C5 for bug-automake@gnu.org; Wed, 23 Aug 2017 16:16:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkc4I-0003mg-5u for bug-automake@gnu.org; Wed, 23 Aug 2017 16:16:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Wed, 23 Aug 2017 20:16:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150351932714500 (code B ref 27781); Wed, 23 Aug 2017 20:16:02 +0000 Received: (at 27781) by debbugs.gnu.org; 23 Aug 2017 20:15:27 +0000 Received: from localhost ([127.0.0.1]:51750 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkc3i-0003lo-Rr for submit@debbugs.gnu.org; Wed, 23 Aug 2017 16:15:27 -0400 Received: from eggs.gnu.org ([208.118.235.92]:49571) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkc3f-0003lb-CM for 27781@debbugs.gnu.org; Wed, 23 Aug 2017 16:15:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkc3Y-0000u1-OI for 27781@debbugs.gnu.org; Wed, 23 Aug 2017 16:15:17 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:55786) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkc3Y-0000tv-Kj; Wed, 23 Aug 2017 16:15:16 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=33500 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dkc3Y-0008W2-1z; Wed, 23 Aug 2017 16:15:16 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <87inhnf5vx.fsf@gnu.org> <5c7cfc72-ce5a-2b77-340b-ab10dd6a1a74@ssi-schaefer.com> <87bmn8k1av.fsf@gnu.org> <2ce25cf1-2d5e-2ea5-8b93-ea3eb12a1737@ssi-schaefer.com> Date: Wed, 23 Aug 2017 22:15:13 +0200 In-Reply-To: <2ce25cf1-2d5e-2ea5-8b93-ea3eb12a1737@ssi-schaefer.com> (Michael Haubenwallner's message of "Tue, 22 Aug 2017 09:03:37 +0200") Message-ID: <87fucidory.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Aug 2017 20:16:04 -0000 Michael Haubenwallner writes: > On 08/22/2017 12:23 AM, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >>> On 08/17/2017 01:29 AM, Mathieu Lirzin wrote: >>>> Michael Haubenwallner writes: >>>>> On 08/08/2017 11:17 PM, Mathieu Lirzin wrote: >>>>>> Michael Haubenwallner writes: >>>>> >>>>> This change fixes automake bug#27781. >>>>> >>>>> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >>>>> each LIBOBJS/ALLOCA variable used. >>>>> --- >>>> >>>> Excellent! >>>> >>>> To ensure this bug is not reintroduced later, I think it is important to >>>> combine this patch with a test. Are you willing to look into adding >>>> one? >>> >>> Ok - like this one? >> >> Yes, I have made some changes to your original patch (i.e. changing the >> test name and the description). If you are OK with those I will push it. >> > > YesPlease! pushed as commit 236cb06666a44f23157aa95adc26450fdec92126 -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 From MAILER-DAEMON Wed Aug 23 17:26:05 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkdA5-0001Tm-Ly for mharc-bug-automake@gnu.org; Wed, 23 Aug 2017 17:26:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53515) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkdA3-0001Tg-T0 for bug-automake@gnu.org; Wed, 23 Aug 2017 17:26:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkdA2-0001XY-JX for bug-automake@gnu.org; Wed, 23 Aug 2017 17:26:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43116) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkdA2-0001XS-FS for bug-automake@gnu.org; Wed, 23 Aug 2017 17:26:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkdA2-0005Ww-9V for bug-automake@gnu.org; Wed, 23 Aug 2017 17:26:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Wed, 23 Aug 2017 21:26:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150352350821190 (code B ref 27781); Wed, 23 Aug 2017 21:26:02 +0000 Received: (at 27781) by debbugs.gnu.org; 23 Aug 2017 21:25:08 +0000 Received: from localhost ([127.0.0.1]:51797 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkd9A-0005Vh-AW for submit@debbugs.gnu.org; Wed, 23 Aug 2017 17:25:08 -0400 Received: from eggs.gnu.org ([208.118.235.92]:34168) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkd98-0005VA-EU for 27781@debbugs.gnu.org; Wed, 23 Aug 2017 17:25:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkd91-0000eH-M9 for 27781@debbugs.gnu.org; Wed, 23 Aug 2017 17:25:01 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:56960) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkd91-0000eC-II; Wed, 23 Aug 2017 17:24:59 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=33552 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dkd91-0000t3-0L; Wed, 23 Aug 2017 17:24:59 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> Date: Wed, 23 Aug 2017 23:24:56 +0200 In-Reply-To: <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> (Michael Haubenwallner's message of "Tue, 22 Aug 2017 09:30:02 +0200") Message-ID: <87d17mdljr.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Aug 2017 21:26:05 -0000 Michael Haubenwallner writes: > On 08/22/2017 12:40 AM, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >> >>> In this case let me come up with attached patch for now >>> (without deeper knowledge of automake internals though). >> >> I have tested this patch, and confirm that it fixes the bug. However I >> have on question regarding the added code. >> >>> From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 >>> From: Michael Haubenwallner >>> Date: Wed, 16 Aug 2017 18:16:12 +0200 >>> Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. >>> >>> This change fixes automake bug#27781. >>> >>> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >>> each LIBOBJS/ALLOCA variable used. >>> --- >>> bin/automake.in | 11 ++++++++++- >>> 1 file changed, 10 insertions(+), 1 deletion(-) >>> >>> diff --git a/bin/automake.in b/bin/automake.in >>> index 9c4cb86..b82e8c5 100644 >>> --- a/bin/automake.in >>> +++ b/bin/automake.in >>> @@ -2153,10 +2153,19 @@ sub handle_LIBOBJS_or_ALLOCA >>> $dir = backname ($relative_dir) . "/$dir" >>> if $relative_dir ne '.'; >>> define_variable ('LIBOBJDIR', "$dir", INTERNAL); >>> + my $dirstamp = ''; >>> + # Abusing $clean_files{"$(VAR)"} as indicator for whether >>> + # we have added the "$(VAR): $dirstamp" dependency already. >> >> I am not sure to understand what this comment really means. Can you >> explain it in other words? >> >>> + $dirstamp = require_build_directory ($dir) > > Thing is, $dirstamp is calculated only when ... > >>> + if ! defined $clean_files{"\$($var)"}; > > ... $clean_files{} was defined, which is done ... > >>> $clean_files{"\$($var)"} = MOSTLY_CLEAN; > > ... here. Subsequently, $output_rules is extended only when $dirstamp was > calculated: > >>> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); > > But the name "clean_files" feels somewhat unrelated to the dirstamp dependency > (as in causing a different output by itself), because I use it to make sure > the dirstamp rule is added only once to $output_rules: > Within flex, I've seen handle_LIBOBJS_or_ALLOCA to be called thrice, but this > dirstamp rule is enough once, as "\$($var)" holds all three object files. > While I have also thought about adding this dependency for each explicit > object file separately - which would not need the "clean_files" indicator, > I've failed to identify where to get the correct object file name from. OK so this is mostly an optimization, since having multiple time the same line shouldn't hurt. Let's first fix the without such optimization. > >>> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >>> # be created by libtool as a side-effect of creating LTLIBOBJS). >>> - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; >>> + if ($var =~ s/^LT//) { >>> + $clean_files{"\$($var)"} = MOSTLY_CLEAN; >>> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); >>> + } >>> } >>> >>> return $dir; > Regarding that part of the code even before you made it seems to me that the conditional second '$clean_files{"\$($var)"} = MOSTLY_CLEAN' statement is useless. Am I overlooking something? Here is the snippet of the code before your changed. --8<---------------cut here---------------start------------->8--- define_variable ('LIBOBJDIR', "$dir", INTERNAL); $clean_files{"\$($var)"} = MOSTLY_CLEAN; # If LTLIBOBJS is used, we must also clear LIBOBJS (which might # be created by libtool as a side-effect of creating LTLIBOBJS). $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; --8<---------------cut here---------------end--------------->8--- > Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile line: > If I remember correctly, there have been (non-GNU) make implementations that > choke on this rule when LIBOBJS is an empty variable: > Am I wrong here, or is GNU make required anyway these days? GNU make is not required for current Automake version. I didn't find any reference of this issue with empty variable target in Autoconf "Portable Make programming" section [1] but if it is actually the case then it would be nice to find a solution for that. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 [1] https://www.gnu.org/software/autoconf/manual/autoconf-2.65/html_node/Portable-Make.html From MAILER-DAEMON Wed Aug 23 18:06:08 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkdmq-0003Zm-LC for mharc-bug-automake@gnu.org; Wed, 23 Aug 2017 18:06:08 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58603) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkdmo-0003Za-Ct for bug-automake@gnu.org; Wed, 23 Aug 2017 18:06:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkdmk-0005dJ-JF for bug-automake@gnu.org; Wed, 23 Aug 2017 18:06:06 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43133) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkdmk-0005dB-Fm for bug-automake@gnu.org; Wed, 23 Aug 2017 18:06:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkdmk-0006SE-2V for bug-automake@gnu.org; Wed, 23 Aug 2017 18:06:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Nick Bowler Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Wed, 23 Aug 2017 22:06:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org, Michael Haubenwallner Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150352591224755 (code B ref 27781); Wed, 23 Aug 2017 22:06:02 +0000 Received: (at 27781) by debbugs.gnu.org; 23 Aug 2017 22:05:12 +0000 Received: from localhost ([127.0.0.1]:51814 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkdlu-0006RB-MJ for submit@debbugs.gnu.org; Wed, 23 Aug 2017 18:05:12 -0400 Received: from mail-lf0-f43.google.com ([209.85.215.43]:32937) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkdlt-0006Qz-4G for 27781@debbugs.gnu.org; Wed, 23 Aug 2017 18:05:09 -0400 Received: by mail-lf0-f43.google.com with SMTP id d17so6292624lfe.0 for <27781@debbugs.gnu.org>; Wed, 23 Aug 2017 15:05:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=draconx-ca.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=zwvVm/JO135rD5qzNrHOhp4ym0Ae4YUAtJCg7Ro0v6Q=; b=NC95mGtZWSQvk6YIkJGJcS5IWyYa1fAmNQZUt9HALoSVXbd83bo+4DKEynefMEyl9j +vKoW6W3UEkM1llsiE4v48dMJtxiZFLXS3ZZq1t+l54U82xEonmPOjVaSqFE7wTSxXvJ 2uddJMmbg/TwzFYQehZ7ff5ADhDKaJNTimJk7v3mabzd5XwIpiLOoTXqy4wPI6AepBqX aBu1B2DlKnak6ISJKRknT4O390Juyos10DXZlPy0zmb8XU7B7pxUDSAkTDx1cjQcyc+R vwRNXx9a+VqQom0aSqCjX+jgl0S095Sub7BrTHmJWCZd16pvLZiR2F2pRsTVpM1b7YLd E37g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=zwvVm/JO135rD5qzNrHOhp4ym0Ae4YUAtJCg7Ro0v6Q=; b=b47AMrUZwJZ6WFZi79fm/NMiOP939jKWQnOq6aYOeJLDF6uoBT41x/A2VdDKNszQAW C8TwxyJXEgf2gbn2JLEHNBJTaAre9HofK2lFQhhIj4oAxkQY3yJiaLZ9Za+jfMCv+wMO tmkykAziH5axUKvHcuUZXu8Wb2u00VI3mIx+ZD2NNHvwpj1D2P8vXS8nkm7pftba3eJ5 1wCzwakLsLlKOAuGkK7H5K3HErD5/jOxL+0VZpX3xu7TB0TQUECi+X5eQlyBUmqgZC5h VHjeO9O7tXP/IRZXsL5pYAAbXSyhCgb74/LEWHJadFOT6YcLxOpu4K0wFLhhiKAnJzsK GPqw== X-Gm-Message-State: AHYfb5hyGyBW/Rn0qO/PY2CiuMnUENAEMgIsoFCMjqHclQkx+iES4svN Q6g4K+HDCp6wldHDOiscQFK4oCXSbDGy X-Received: by 10.46.82.29 with SMTP id g29mr1608925ljb.19.1503525903239; Wed, 23 Aug 2017 15:05:03 -0700 (PDT) MIME-Version: 1.0 Received: by 10.179.5.244 with HTTP; Wed, 23 Aug 2017 15:05:02 -0700 (PDT) X-Originating-IP: [24.156.181.89] In-Reply-To: <87d17mdljr.fsf@gnu.org> References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> From: Nick Bowler Date: Wed, 23 Aug 2017 18:05:02 -0400 Message-ID: Content-Type: text/plain; charset="UTF-8" X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Aug 2017 22:06:07 -0000 On 8/23/17, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile >> line: If I remember correctly, there have been (non-GNU) make >> implementations thatchoke on this rule when LIBOBJS is an empty >> variable: Am I wrong here, or is GNU make required anyway these days? > > GNU make is not required for current Automake version. I didn't find > any reference of this issue with empty variable target in Autoconf > "Portable Make programming" section [1] but if it is actually the case > then it would be nice to find a solution for that. POSIX requires that target rules specify at least one target. At least dmake will fail to parse the makefile if it contains an target rule with no targets: % cat >Makefile <<'EOF' FOO = dummy: ; $(FOO): ; EOF % dmake dmake: Makefile: line 3: Error: -- Missing targets or attributes in rule If the target list is a macro which can reasonably be empty then one solution is to add a bogus target name that won't be used anywhere else, along the lines of: gobbledegook $(FOO): ; Cheers, Nick From MAILER-DAEMON Thu Aug 24 05:36:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkoYX-0004EN-Ty for mharc-bug-automake@gnu.org; Thu, 24 Aug 2017 05:36:05 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48791) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkoYV-0004E6-H8 for bug-automake@gnu.org; Thu, 24 Aug 2017 05:36:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkoYU-00037k-EN for bug-automake@gnu.org; Thu, 24 Aug 2017 05:36:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43458) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkoYU-00037W-B3 for bug-automake@gnu.org; Thu, 24 Aug 2017 05:36:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkoYU-0007kl-5c for bug-automake@gnu.org; Thu, 24 Aug 2017 05:36:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Thu, 24 Aug 2017 09:36:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Nick Bowler Cc: 27781@debbugs.gnu.org, Michael Haubenwallner Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150356732529759 (code B ref 27781); Thu, 24 Aug 2017 09:36:02 +0000 Received: (at 27781) by debbugs.gnu.org; 24 Aug 2017 09:35:25 +0000 Received: from localhost ([127.0.0.1]:52139 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkoXt-0007jv-C2 for submit@debbugs.gnu.org; Thu, 24 Aug 2017 05:35:25 -0400 Received: from eggs.gnu.org ([208.118.235.92]:57705) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkoXr-0007jg-IK for 27781@debbugs.gnu.org; Thu, 24 Aug 2017 05:35:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkoXl-0002FV-ID for 27781@debbugs.gnu.org; Thu, 24 Aug 2017 05:35:18 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:35389) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkoXh-000241-F0; Thu, 24 Aug 2017 05:35:13 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=35550 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dkoXa-0003iM-Rv; Thu, 24 Aug 2017 05:35:07 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> Date: Thu, 24 Aug 2017 11:35:04 +0200 In-Reply-To: (Nick Bowler's message of "Wed, 23 Aug 2017 18:05:02 -0400") Message-ID: <87poblia0n.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Aug 2017 09:36:04 -0000 Nick Bowler writes: > On 8/23/17, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >>> Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile >>> line: If I remember correctly, there have been (non-GNU) make >>> implementations thatchoke on this rule when LIBOBJS is an empty >>> variable: Am I wrong here, or is GNU make required anyway these days? >> >> GNU make is not required for current Automake version. I didn't find >> any reference of this issue with empty variable target in Autoconf >> "Portable Make programming" section [1] but if it is actually the case >> then it would be nice to find a solution for that. > > POSIX requires that target rules specify at least one target. At least > dmake will fail to parse the makefile if it contains an target rule with > no targets: > > % cat >Makefile <<'EOF' > FOO = > dummy: ; > $(FOO): ; > EOF > % dmake > dmake: Makefile: line 3: Error: -- Missing targets or attributes in rule > > If the target list is a macro which can reasonably be empty then one > solution is to add a bogus target name that won't be used anywhere > else, along the lines of: > > gobbledegook $(FOO): ; Instead of this dummy target, I would rather prefer adding the dirstamp dependency for each explicit object file separately. this should be computed from the '%libsources' variable. However after a quick look in the code, it seems that this variable is not properly populated by the 'scan_autoconf_traces' subroutine. It only contains the files that are explicitely defined by scanning the AC_LIBSOURCE macro and not by AC_LIBOBJ or AC_LIBSOURCES in 'configure.ac'. Michael: Are you interested in looking into this? Thanks for your input. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 From MAILER-DAEMON Thu Aug 24 06:36:10 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkpUf-00008u-Ra for mharc-bug-automake@gnu.org; Thu, 24 Aug 2017 06:36:09 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59506) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkpUd-00007k-M9 for bug-automake@gnu.org; Thu, 24 Aug 2017 06:36:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkpUY-0005NE-Vk for bug-automake@gnu.org; Thu, 24 Aug 2017 06:36:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:43482) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkpUY-0005N7-Qp for bug-automake@gnu.org; Thu, 24 Aug 2017 06:36:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkpUY-0000ld-DF for bug-automake@gnu.org; Thu, 24 Aug 2017 06:36:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Thu, 24 Aug 2017 10:36:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15035709502931 (code B ref 27781); Thu, 24 Aug 2017 10:36:02 +0000 Received: (at 27781) by debbugs.gnu.org; 24 Aug 2017 10:35:50 +0000 Received: from localhost ([127.0.0.1]:52163 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkpUM-0000lD-6i for submit@debbugs.gnu.org; Thu, 24 Aug 2017 06:35:50 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:53327) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkpUJ-0000ky-M5 for 27781@debbugs.gnu.org; Thu, 24 Aug 2017 06:35:48 -0400 X-IronPort-AV: E=Sophos;i="5.41,421,1498514400"; d="scan'208";a="2933083" X-IPAS-Result: A2E0AwB8q55Z/+shHKxTChsBAQEDAQEBCQEBAYQ+gRWDd5l8gieNSoptJIEVhA4ChQwUAQIBAQEBAQEBA4EQhAJbOwEBAQECASMEUgULCw4DAwECAQICJgICTwgGDQYCAQEXig6uKoFtOotgAQEBAQEBAQEBAQEBAQEBI4ENgh2FXIJINIQ+CoM+gmEFiXcHhx2BZ4UhiDaCKYUtj1qIY4cWkTCFADaBLHeHenSEP4JBAQEB Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Aug 2017 12:35:40 +0200 Received: from [172.28.41.101] by mailhost.salomon.at with esmtp (Exim 4.77) (envelope-from ) id 1dkpUC-0003aW-6R; Thu, 24 Aug 2017 12:35:40 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> From: Michael Haubenwallner Message-ID: Date: Thu, 24 Aug 2017 12:35:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <87d17mdljr.fsf@gnu.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Aug 2017 10:36:09 -0000 On 08/23/2017 11:24 PM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote: >>> Michael Haubenwallner writes: >>> >>>> In this case let me come up with attached patch for now >>>> (without deeper knowledge of automake internals though). >>> >>> I have tested this patch, and confirm that it fixes the bug. However I >>> have on question regarding the added code. >>> >>>> From c3d51adb53400fc8bf65c0e003d810b2e7166d0d Mon Sep 17 00:00:00 2001 >>>> From: Michael Haubenwallner >>>> Date: Wed, 16 Aug 2017 18:16:12 +0200 >>>> Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. >>>> >>>> This change fixes automake bug#27781. >>>> >>>> * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for >>>> each LIBOBJS/ALLOCA variable used. >>>> --- >>>> bin/automake.in | 11 ++++++++++- >>>> 1 file changed, 10 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/bin/automake.in b/bin/automake.in >>>> index 9c4cb86..b82e8c5 100644 >>>> --- a/bin/automake.in >>>> +++ b/bin/automake.in >>>> @@ -2153,10 +2153,19 @@ sub handle_LIBOBJS_or_ALLOCA >>>> $dir = backname ($relative_dir) . "/$dir" >>>> if $relative_dir ne '.'; >>>> define_variable ('LIBOBJDIR', "$dir", INTERNAL); >>>> + my $dirstamp = ''; >>>> + # Abusing $clean_files{"$(VAR)"} as indicator for whether >>>> + # we have added the "$(VAR): $dirstamp" dependency already. >>> >>> I am not sure to understand what this comment really means. Can you >>> explain it in other words? >>> >>>> + $dirstamp = require_build_directory ($dir) >> >> Thing is, $dirstamp is calculated only when ... >> >>>> + if ! defined $clean_files{"\$($var)"}; >> >> ... $clean_files{} was defined, which is done ... >> >>>> $clean_files{"\$($var)"} = MOSTLY_CLEAN; >> >> ... here. Subsequently, $output_rules is extended only when $dirstamp was >> calculated: >> >>>> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); >> >> But the name "clean_files" feels somewhat unrelated to the dirstamp dependency >> (as in causing a different output by itself), because I use it to make sure >> the dirstamp rule is added only once to $output_rules: >> Within flex, I've seen handle_LIBOBJS_or_ALLOCA to be called thrice, but this >> dirstamp rule is enough once, as "\$($var)" holds all three object files. >> While I have also thought about adding this dependency for each explicit >> object file separately - which would not need the "clean_files" indicator, >> I've failed to identify where to get the correct object file name from. > > OK so this is mostly an optimization, since having multiple time the > same line shouldn't hurt. Agreed. > Let's first fix the without such optimization. I'm going to look into the (other subthread's) AC_LIBOBJ thing, for adding the rule per explicit object file instead. >> >>>> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >>>> # be created by libtool as a side-effect of creating LTLIBOBJS). >>>> - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; >>>> + if ($var =~ s/^LT//) { >>>> + $clean_files{"\$($var)"} = MOSTLY_CLEAN; >>>> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); >>>> + } >>>> } >>>> >>>> return $dir; >> > > Regarding that part of the code even before you made it seems to me that > the conditional second '$clean_files{"\$($var)"} = MOSTLY_CLEAN' > statement is useless. Am I overlooking something? It took a while for me too to understand what's going on here: Consider $var = 'LTLIBOBJS'; > Here is the snippet of the code before your changed. > > --8<---------------cut here---------------start------------->8--- > define_variable ('LIBOBJDIR', "$dir", INTERNAL); > $clean_files{"\$($var)"} = MOSTLY_CLEAN; here, unconditionally: $clean_files{'$(LTLIBOBJS)'} = MOSTLY_CLEAN; > # If LTLIBOBJS is used, we must also clear LIBOBJS (which might > # be created by libtool as a side-effect of creating LTLIBOBJS). > $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; Crucial here is that trailing { if $var =~ s/^LT// }: First, leading 'LT' eventually is dropped: $var = 'LIBOBJS'; If the leading 'LT' was actually found: $clean_files{'LIBOBJS'} = MOSTLY_CLEAN; > --8<---------------cut here---------------end--------------->8--- > >> Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile line: >> If I remember correctly, there have been (non-GNU) make implementations that >> choke on this rule when LIBOBJS is an empty variable: >> Am I wrong here, or is GNU make required anyway these days? > > GNU make is not required for current Automake version. I didn't find > any reference of this issue with empty variable target in Autoconf > "Portable Make programming" section [1] but if it is actually the case > then it would be nice to find a solution for that. > /haubi/ From MAILER-DAEMON Thu Aug 24 10:24:11 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dkt3L-00012e-GQ for mharc-bug-automake@gnu.org; Thu, 24 Aug 2017 10:24:11 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55263) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dkt3I-00012J-BW for bug-automake@gnu.org; Thu, 24 Aug 2017 10:24:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dkt3B-0007fi-Vm for bug-automake@gnu.org; Thu, 24 Aug 2017 10:24:08 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:44315) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dkt3B-0007fc-Rk for bug-automake@gnu.org; Thu, 24 Aug 2017 10:24:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dkt3B-0001ZW-Kj for bug-automake@gnu.org; Thu, 24 Aug 2017 10:24:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Thu, 24 Aug 2017 14:24:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin , Nick Bowler Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15035845965980 (code B ref 27781); Thu, 24 Aug 2017 14:24:01 +0000 Received: (at 27781) by debbugs.gnu.org; 24 Aug 2017 14:23:16 +0000 Received: from localhost ([127.0.0.1]:52996 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkt2R-0001YO-S0 for submit@debbugs.gnu.org; Thu, 24 Aug 2017 10:23:16 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:63271) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dkt2O-0001Y8-01 for 27781@debbugs.gnu.org; Thu, 24 Aug 2017 10:23:12 -0400 X-IronPort-AV: E=Sophos;i="5.41,421,1498514400"; d="scan'208,223";a="2950155" X-IPAS-Result: A2FMAwCD4J5Z/+shHKxdGwEBAQMBAQEJAQEBhD6BFYN3nAEid4xTim0kgRWEDgKFDRQBAgEBAQEBAQEDgRCEAls7AQEBAQMjBFIQCw4DAwECASoCAk8IBgEMBgIBARe5XYFtOoteAQEBAQEBAQEBAQEBAQEBAQESD4MqhTErC4I9NIUTgnOCYQWJfocdjz6CKYILgyKPWohjhxaRMIUANoEsd16HHHSLMAEBAQ Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Aug 2017 16:23:01 +0200 Received: from [172.28.41.101] by mailhost.salomon.at with esmtp (Exim 4.77) (envelope-from ) id 1dkt2C-00089Y-Bj; Thu, 24 Aug 2017 16:23:00 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> <87poblia0n.fsf@gnu.org> From: Michael Haubenwallner Message-ID: Date: Thu, 24 Aug 2017 16:23:00 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <87poblia0n.fsf@gnu.org> Content-Type: multipart/mixed; boundary="------------036B8BDC981B1EE9C49982CF" Content-Language: en-US X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Aug 2017 14:24:09 -0000 This is a multi-part message in MIME format. --------------036B8BDC981B1EE9C49982CF Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit On 08/24/2017 11:35 AM, Mathieu Lirzin wrote: > Nick Bowler writes: >> On 8/23/17, Mathieu Lirzin wrote: >>> Michael Haubenwallner writes: >>>> Another thought about the final "$(LIBOBJS): .../.dirstamp" Makefile >>>> line: If I remember correctly, there have been (non-GNU) make >>>> implementations thatchoke on this rule when LIBOBJS is an empty >>>> variable: Am I wrong here, or is GNU make required anyway these days? >>> >>> GNU make is not required for current Automake version. I didn't find >>> any reference of this issue with empty variable target in Autoconf >>> "Portable Make programming" section [1] but if it is actually the case >>> then it would be nice to find a solution for that. >> >> POSIX requires that target rules specify at least one target. At least >> dmake will fail to parse the makefile if it contains an target rule with >> no targets: >> >> % cat >Makefile <<'EOF' >> FOO = >> dummy: ; >> $(FOO): ; >> EOF >> % dmake >> dmake: Makefile: line 3: Error: -- Missing targets or attributes in rule >> >> If the target list is a macro which can reasonably be empty then one >> solution is to add a bogus target name that won't be used anywhere >> else, along the lines of: >> >> gobbledegook $(FOO): ; > > Instead of this dummy target, I would rather prefer adding the dirstamp > dependency for each explicit object file separately. this should be computed > from the '%libsources' variable. However after a quick look in the code, it > seems that this variable is not properly populated by the > 'scan_autoconf_traces' subroutine. It only contains the files that are > explicitely defined by scanning the AC_LIBSOURCE macro and not by AC_LIBOBJ or > AC_LIBSOURCES in 'configure.ac'. > > Michael: Are you interested in looking into this? > Here's an attempt to use the explicit AC_LIBSOURCE values to depend on the LIBOBJDIR dirstamp. IMHO, tracing AC_LIBOBJ would not provide additional information over AC_LIBSOURCE, as AC_LIBOBJ by itself uses AC_LIBSOURCE. But then there is _AC_LIBOBJS_NORMALIZE, adding $U to {,LT}LIBOBJS values (seems referring to "Automake's ANSI2KNR"): What is this about? /haubi/ --------------036B8BDC981B1EE9C49982CF Content-Type: text/x-patch; name="0001-automake-Depend-on-LIBOBJDIR-for-LIBOBJS.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0001-automake-Depend-on-LIBOBJDIR-for-LIBOBJS.patch" >From 4465bb9e31c12a8f59c199c61aaef17966e1ffc4 Mon Sep 17 00:00:00 2001 From: Michael Haubenwallner Date: Wed, 16 Aug 2017 18:16:12 +0200 Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. This change fixes automake bug#27781. * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for each LIBOBJS/ALLOCA source file found. --- bin/automake.in | 28 ++++++++++++++++------------ 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/bin/automake.in b/bin/automake.in index 9c4cb86..66619d2 100644 --- a/bin/automake.in +++ b/bin/automake.in @@ -2129,13 +2129,14 @@ sub handle_lib_objects return $seen_libobjs; } -# handle_LIBOBJS_or_ALLOCA ($VAR) -# ------------------------------- +# handle_LIBOBJS_or_ALLOCA ($VAR, $BASE) +# -------------------------------------- # Definitions common to LIBOBJS and ALLOCA. # VAR should be one of LIBOBJS, LTLIBOBJS, ALLOCA, or LTALLOCA. +# BASE should be one base file name from AC_LIBSOURCE, or alloca. sub handle_LIBOBJS_or_ALLOCA { - my ($var) = @_; + my ($var, $base) = @_; my $dir = ''; # If LIBOBJS files must be built in another directory we have @@ -2153,10 +2154,13 @@ sub handle_LIBOBJS_or_ALLOCA $dir = backname ($relative_dir) . "/$dir" if $relative_dir ne '.'; define_variable ('LIBOBJDIR', "$dir", INTERNAL); - $clean_files{"\$($var)"} = MOSTLY_CLEAN; - # If LTLIBOBJS is used, we must also clear LIBOBJS (which might - # be created by libtool as a side-effect of creating LTLIBOBJS). - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; + if ($dir && !defined $clean_files{"$dir$base.\$(OBJEXT)"}) { + my $dirstamp = require_build_directory ($dir); + $output_rules .= "$dir$base.\$(OBJEXT): $dirstamp\n"; + $output_rules .= "$dir$base.lo: $dirstamp\n" if $var =~ /^LT/; + } + $clean_files{"$dir$base.\$(OBJEXT)"} = MOSTLY_CLEAN; + $clean_files{"$dir$base.lo"} = MOSTLY_CLEAN if $var =~ /^LT/; } return $dir; @@ -2171,14 +2175,14 @@ sub handle_LIBOBJS $var->requires_variables ("\@${lt}LIBOBJS\@ used", $lt . 'LIBOBJS') if ! keys %libsources; - my $dir = handle_LIBOBJS_or_ALLOCA "${lt}LIBOBJS"; - foreach my $iter (keys %libsources) { - if ($iter =~ /\.[cly]$/) + my $dir = ''; + if ($iter =~ /^(.*)(\.[cly])$/) { - saw_extension ($&); + saw_extension ($2); saw_extension ('.c'); + $dir = handle_LIBOBJS_or_ALLOCA ("${lt}LIBOBJS", $1); } if ($iter =~ /\.h$/) @@ -2206,7 +2210,7 @@ sub handle_ALLOCA my ($var, $cond, $lt) = @_; my $myobjext = $lt ? 'lo' : 'o'; $lt ||= ''; - my $dir = handle_LIBOBJS_or_ALLOCA "${lt}ALLOCA"; + my $dir = handle_LIBOBJS_or_ALLOCA ("${lt}ALLOCA", "alloca"); $dir eq '' and $dir = './'; $var->requires_variables ("\@${lt}ALLOCA\@ used", $lt . 'ALLOCA'); -- 2.10.2 --------------036B8BDC981B1EE9C49982CF-- From MAILER-DAEMON Sun Aug 27 10:09:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dlyFO-0005u0-DE for mharc-bug-automake@gnu.org; Sun, 27 Aug 2017 10:09:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56524) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dlyFL-0005sG-Bs for bug-automake@gnu.org; Sun, 27 Aug 2017 10:09:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dlyFK-0002MU-31 for bug-automake@gnu.org; Sun, 27 Aug 2017 10:09:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:49272) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dlyFJ-0002MQ-Uz for bug-automake@gnu.org; Sun, 27 Aug 2017 10:09:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dlyFJ-00076Q-MX for bug-automake@gnu.org; Sun, 27 Aug 2017 10:09:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Sun, 27 Aug 2017 14:09:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.150384289127239 (code B ref 27781); Sun, 27 Aug 2017 14:09:01 +0000 Received: (at 27781) by debbugs.gnu.org; 27 Aug 2017 14:08:11 +0000 Received: from localhost ([127.0.0.1]:57953 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dlyEV-00075H-Cv for submit@debbugs.gnu.org; Sun, 27 Aug 2017 10:08:11 -0400 Received: from eggs.gnu.org ([208.118.235.92]:37192) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dlyES-000751-UJ for 27781@debbugs.gnu.org; Sun, 27 Aug 2017 10:08:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dlyEM-00026B-EC for 27781@debbugs.gnu.org; Sun, 27 Aug 2017 10:08:03 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:49051) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dlyEM-000267-9z; Sun, 27 Aug 2017 10:08:02 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=55284 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dlyEL-0008Nm-RQ; Sun, 27 Aug 2017 10:08:02 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> Date: Sun, 27 Aug 2017 16:07:58 +0200 In-Reply-To: (Michael Haubenwallner's message of "Thu, 24 Aug 2017 12:35:39 +0200") Message-ID: <87val9p0ht.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 27 Aug 2017 14:09:04 -0000 --=-=-= Content-Type: text/plain Michael Haubenwallner writes: > On 08/23/2017 11:24 PM, Mathieu Lirzin wrote: >> Michael Haubenwallner writes: >>> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote: >>>> Michael Haubenwallner writes: >>>> >>>>> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >>>>> # be created by libtool as a side-effect of creating LTLIBOBJS). >>>>> - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; >>>>> + if ($var =~ s/^LT//) { >>>>> + $clean_files{"\$($var)"} = MOSTLY_CLEAN; >>>>> + $output_rules .= "\$($var): $dirstamp\n" if ($dirstamp); >>>>> + } >>>>> } >>>>> >>>>> return $dir; >>> >> >> Regarding that part of the code even before you made it seems to me that >> the conditional second '$clean_files{"\$($var)"} = MOSTLY_CLEAN' >> statement is useless. Am I overlooking something? > It took a while for me too to understand what's going on here: > > Consider $var = 'LTLIBOBJS'; > >> Here is the snippet of the code before your changed. >> >> --8<---------------cut here---------------start------------->8--- >> define_variable ('LIBOBJDIR', "$dir", INTERNAL); >> $clean_files{"\$($var)"} = MOSTLY_CLEAN; > > here, unconditionally: $clean_files{'$(LTLIBOBJS)'} = MOSTLY_CLEAN; > >> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >> # be created by libtool as a side-effect of creating LTLIBOBJS). >> $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; > > Crucial here is that trailing { if $var =~ s/^LT// }: > First, leading 'LT' eventually is dropped: $var = 'LIBOBJS'; > If the leading 'LT' was actually found: $clean_files{'LIBOBJS'} = MOSTLY_CLEAN; I overlooked the regexp substitution. This code is too complex for what it achieves. I have pushed the following patch in commit 5521219348c55af354878583b99c5f9d66d6d38a --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=0001-automake-Handle-LTLIBOBJS-more-specifically.patch >From 5521219348c55af354878583b99c5f9d66d6d38a Mon Sep 17 00:00:00 2001 From: Mathieu Lirzin Date: Sun, 27 Aug 2017 15:27:00 +0200 Subject: [PATCH] automake: Handle LTLIBOBJS more specifically * bin/automake.in (handle_LIBOBJS_or_ALLOCA): Remove complex regexp substitution when handling LTLIBOBJS. --- bin/automake.in | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bin/automake.in b/bin/automake.in index d126836..cdea388 100644 --- a/bin/automake.in +++ b/bin/automake.in @@ -2306,9 +2306,8 @@ sub handle_LIBOBJS_or_ALLOCA if $relative_dir ne '.'; define_variable ('LIBOBJDIR', "$dir", INTERNAL); $clean_files{"\$($var)"} = MOSTLY_CLEAN; - # If LTLIBOBJS is used, we must also clear LIBOBJS (which might - # be created by libtool as a side-effect of creating LTLIBOBJS). - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; + # libtool might create LIBOBJS as a side-effect of using LTLIBOBJS. + $clean_files{"\$(LIBOBJS)"} = MOSTLY_CLEAN if $var eq "LTLIBOBJS"; } else { -- 2.9.5 --=-=-= Content-Type: text/plain Thanks for the explanation. -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 --=-=-=-- From MAILER-DAEMON Sun Aug 27 11:25:06 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dlzQw-0002DZ-2G for mharc-bug-automake@gnu.org; Sun, 27 Aug 2017 11:25:06 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38536) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dlzQt-0002DQ-KZ for bug-automake@gnu.org; Sun, 27 Aug 2017 11:25:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dlzQs-0002TY-6a for bug-automake@gnu.org; Sun, 27 Aug 2017 11:25:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:49310) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dlzQs-0002TT-37 for bug-automake@gnu.org; Sun, 27 Aug 2017 11:25:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dlzQr-0000MZ-U4 for bug-automake@gnu.org; Sun, 27 Aug 2017 11:25:01 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Mathieu Lirzin Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Sun, 27 Aug 2017 15:25:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Michael Haubenwallner Cc: Nick Bowler , 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15038474531335 (code B ref 27781); Sun, 27 Aug 2017 15:25:01 +0000 Received: (at 27781) by debbugs.gnu.org; 27 Aug 2017 15:24:13 +0000 Received: from localhost ([127.0.0.1]:57991 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dlzQ5-0000LT-39 for submit@debbugs.gnu.org; Sun, 27 Aug 2017 11:24:13 -0400 Received: from eggs.gnu.org ([208.118.235.92]:47498) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dlzQ3-0000LG-8x for 27781@debbugs.gnu.org; Sun, 27 Aug 2017 11:24:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dlzPu-0002G5-VA for 27781@debbugs.gnu.org; Sun, 27 Aug 2017 11:24:06 -0400 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:53322) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dlzPu-0002Fx-RB; Sun, 27 Aug 2017 11:24:02 -0400 Received: from [2a01:e35:2ec2:e580:491c:541:7a4a:37d9] (port=55304 helo=localhost.localdomain) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1dlzPu-0003WK-Ap; Sun, 27 Aug 2017 11:24:02 -0400 From: Mathieu Lirzin References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> <87poblia0n.fsf@gnu.org> Date: Sun, 27 Aug 2017 17:23:59 +0200 In-Reply-To: (Michael Haubenwallner's message of "Thu, 24 Aug 2017 16:23:00 +0200") Message-ID: <87shgdowz4.fsf@gnu.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 27 Aug 2017 15:25:05 -0000 Michael Haubenwallner writes: > On 08/24/2017 11:35 AM, Mathieu Lirzin wrote: > >> Instead of this dummy target, I would rather prefer adding the dirstamp >> dependency for each explicit object file separately. this should be computed >> from the '%libsources' variable. However after a quick look in the code, it >> seems that this variable is not properly populated by the >> 'scan_autoconf_traces' subroutine. It only contains the files that are >> explicitely defined by scanning the AC_LIBSOURCE macro and not by AC_LIBOBJ or >> AC_LIBSOURCES in 'configure.ac'. >> >> Michael: Are you interested in looking into this? >> > > Here's an attempt to use the explicit AC_LIBSOURCE values to depend on the > LIBOBJDIR dirstamp. IMHO, tracing AC_LIBOBJ would not provide additional > information over AC_LIBSOURCE, as AC_LIBOBJ by itself uses AC_LIBSOURCE. I stand corrected. This is indeed working properly since 'scan_autoconf_traces' doesn't simply scan "configure.ac" but the output 'autoconf --trace=...'. > But then there is _AC_LIBOBJS_NORMALIZE, adding $U to {,LT}LIBOBJS values > (seems referring to "Automake's ANSI2KNR"): What is this about? Those are legacy features of Automake that were removed 6 years ago: --8<---------------cut here---------------start------------->8--- [...] commit 0cbcf393dbdaa2ff36b11cf2516675e78eada49f Author: Stefano Lattarini Date: Sun Aug 28 15:06:59 2011 +0200 docs: remove description of de-ANSI-fication support from manual * doc/automake.texi (Auxiliary Programs): Remove mention of `ansi2knr.c' and `ansi2knr.1'. (Obsolete Macros): Remove mention of `AM_C_PROTOTYPES'. Do not index it nor variables `ANSI2KNR' and `U' anymore. (List of Automake options): Remove mention of `ansi2knr' option, and obsolete entries for concept and option indexes. (Optional): Remove entry about `AM_C_PROTOTYPES'. (@c LocalWords): Remove `ansi' and `knr'. (@menu, @detailmenu): Remove entries about de-ANSI-fication support. [...] --8<---------------cut here---------------end--------------->8--- So I guess you shouldn't worry about those thing. ;) >>From 4465bb9e31c12a8f59c199c61aaef17966e1ffc4 Mon Sep 17 00:00:00 2001 > From: Michael Haubenwallner > Date: Wed, 16 Aug 2017 18:16:12 +0200 > Subject: [PATCH] automake: Depend on LIBOBJDIR for LIBOBJS. > > This change fixes automake bug#27781. > > * bin/automake.in: Add Makefile dependency on LIBOBJDIR/dirstamp for > each LIBOBJS/ALLOCA source file found. > --- This patch can't be applied on the 'minor' branch, which corresponds to the branch of the next release (I know this is confusing [1]). Could resend it after rebasing onto the 'minor' branch? -- Mathieu Lirzin GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37 [1] https://debbugs.gnu.org/cgi/bugreport.cgi?msg=199;bug=13578 From MAILER-DAEMON Mon Aug 28 03:55:09 2017 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dmEt3-0006cA-ID for mharc-bug-automake@gnu.org; Mon, 28 Aug 2017 03:55:09 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54233) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dmEt1-0006an-NO for bug-automake@gnu.org; Mon, 28 Aug 2017 03:55:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dmEsw-00036x-Ra for bug-automake@gnu.org; Mon, 28 Aug 2017 03:55:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:49912) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dmEsw-00036q-Ny for bug-automake@gnu.org; Mon, 28 Aug 2017 03:55:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dmEsw-0001RP-DV for bug-automake@gnu.org; Mon, 28 Aug 2017 03:55:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR Resent-From: Michael Haubenwallner Original-Sender: "Debbugs-submit" Resent-CC: bug-automake@gnu.org Resent-Date: Mon, 28 Aug 2017 07:55:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27781 X-GNU-PR-Package: automake X-GNU-PR-Keywords: confirmed patch To: Mathieu Lirzin Cc: 27781@debbugs.gnu.org Received: via spool by 27781-submit@debbugs.gnu.org id=B27781.15039068815510 (code B ref 27781); Mon, 28 Aug 2017 07:55:02 +0000 Received: (at 27781) by debbugs.gnu.org; 28 Aug 2017 07:54:41 +0000 Received: from localhost ([127.0.0.1]:58593 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dmEsb-0001Qn-KN for submit@debbugs.gnu.org; Mon, 28 Aug 2017 03:54:41 -0400 Received: from atfriesa01.ssi-schaefer.com ([193.186.16.100]:27953) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dmEsa-0001Qb-6x for 27781@debbugs.gnu.org; Mon, 28 Aug 2017 03:54:40 -0400 X-IronPort-AV: E=Sophos;i="5.41,440,1498514400"; d="scan'208";a="3042772" X-IPAS-Result: A2EXEQBvy6NZ/+shHKxbGgEBAQECAQEBAQgBAQEBhVODd5wBIpYnghGFRwKEMxEBAgEBAQEBAQEDgRCFGQEFIwRSEAsOCgICJgICVwYNBgIBAbw2gW06i1kBAQEBAQEBAwEBAQEkgQ2CHYUzKwuCcogIgmEFigGWY4Ipkh+LT4cXlj05C4Ehd4d7dIpdAQEB Received: from samail03.wamas.com (HELO mailhost.salomon.at) ([172.28.33.235]) by atfriesa01.ssi-schaefer.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Aug 2017 09:54:33 +0200 Received: from [172.28.41.101] by mailhost.salomon.at with esmtp (Exim 4.77) (envelope-from ) id 1dmEsT-0008PG-An; Mon, 28 Aug 2017 09:54:33 +0200 References: <76a2c423-1b66-6394-31ee-9a1b8a1763ef@ssi-schaefer.com> <87fud1ahf1.fsf@gnu.org> <4b3d2b8a-19fb-52fb-4caa-9e0dfdba7a96@ssi-schaefer.com> <874lt0k0in.fsf@gnu.org> <3c6a7481-fff6-7c1f-cadf-c66bd44207b8@ssi-schaefer.com> <87d17mdljr.fsf@gnu.org> <87val9p0ht.fsf@gnu.org> From: Michael Haubenwallner Message-ID: <69188400-6b39-5210-df7e-490b9f835931@ssi-schaefer.com> Date: Mon, 28 Aug 2017 09:54:33 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <87val9p0ht.fsf@gnu.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-automake@gnu.org List-Id: Bug reports for Automake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Aug 2017 07:55:08 -0000 On 08/27/2017 04:07 PM, Mathieu Lirzin wrote: > Michael Haubenwallner writes: >> On 08/23/2017 11:24 PM, Mathieu Lirzin wrote: >>> Michael Haubenwallner writes: >>>> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote: >>>>> Michael Haubenwallner writes: >>> --8<---------------cut here---------------start------------->8--- >> >>> # If LTLIBOBJS is used, we must also clear LIBOBJS (which might >>> # be created by libtool as a side-effect of creating LTLIBOBJS). >>> $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; >> >> Crucial here is that trailing { if $var =~ s/^LT// }: >> First, leading 'LT' eventually is dropped: $var = 'LIBOBJS'; >> If the leading 'LT' was actually found: $clean_files{'LIBOBJS'} = MOSTLY_CLEAN; > > I overlooked the regexp substitution. This code is too complex for what it > achieves. I have pushed the following patch in commit > 5521219348c55af354878583b99c5f9d66d6d38a > > - $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//; > + $clean_files{"\$(LIBOBJS)"} = MOSTLY_CLEAN if $var eq "LTLIBOBJS"; One more note: IMHO, the idea was to also support "LTALLOCA" here - even if AC_FUNC_ALLOCA does not (yet?). /haubi/