From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg1-x533.google.com (mail-pg1-x533.google.com [IPv6:2607:f8b0:4864:20::533]) by sourceware.org (Postfix) with ESMTPS id 897263858401 for ; Thu, 19 Jan 2023 07:15:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 897263858401 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-pg1-x533.google.com with SMTP id 7so819826pga.1 for ; Wed, 18 Jan 2023 23:15:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=mNl1KDzbyIMC2irEWkDyGH75EpmLBwUerU48Zc7pIqI=; b=ZFOuZm64fyGiGTOJLmzNqRO1OZzZbNnTh06HomBtOZQNuBdkoYbbXAmurrjG/4yDIU 5bcFaMARUhQHUQhOUiAnDxd7XE/JNTMA/m0SbVj4eMQzYAnkNhcMA9ZUFyQKd5YCT2YW bEDPAMipFbUqNCf+CxvSkQ6kIb3Vf1v47T4zUA/8E7fEwgwf67QAuCp+ZOzerpsa2JpT Ctf/+9Ok+SJAEyMwtVUrH0AQAhiBsPrmA/0uqWC4Z/MVUoNL8p07ZAkdz/AFd6WsQmkU Wp+UqCyeedRtPz7MfJiwg67d1ZS5qZnNL8QE/3VHMb8ICDaKiere1svCEnGyOsM2G55Z u0tQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=mNl1KDzbyIMC2irEWkDyGH75EpmLBwUerU48Zc7pIqI=; b=o/gpPzVqPCrUwROP3Rictlf96nCj4wRGLZN063+g6MNJWBYmYhawiVam9Rr4uwQMpk JLNV2nZcnfnlP34xQMLvnFSmcP2t2ttGTl39sDbPM3gVFFuSGi+6Uw82Y4yez0w04vsS hUqOli1yrntKLIuwTziSdIQtZinZMbpVzfCxTO7A3BcQhJE6QWEtUulHuTDVDTv3RyV0 wX6BcW3dFJ4tR0Zwy3NH7BVGVbT2srvUbqVmXBiilGv6znlwWsLn49K59yQtt/Z/Y4dQ qrnX7kPmrFa+/x7xqAx3/TM+nCfH8UwF6rzreJASYstWHjcekZzQipkXrxfwWDD+ORim IkdA== X-Gm-Message-State: AFqh2kpa121Jj8VXfdOqwOq0IquMs34356q2ahyAcEyYfPWearlv/CMl 4PL9df3REKDlzhT904QNxZLgXaY58yc= X-Google-Smtp-Source: AMrXdXtIxndXJ+w8+W3hrrMWb+ioHmlgyiAEWvbRcvIvy2p+2H85A+HQmg91/Zo2TOxse7vDDXxZEg== X-Received: by 2002:a05:6a00:d55:b0:587:405a:421c with SMTP id n21-20020a056a000d5500b00587405a421cmr10097804pfv.7.1674112534108; Wed, 18 Jan 2023 23:15:34 -0800 (PST) Received: from squeak.grove.modra.org ([2406:3400:51d:8cc0:7fcf:af4d:5230:f70f]) by smtp.gmail.com with ESMTPSA id 125-20020a620483000000b005817fa83bcesm20489939pfe.76.2023.01.18.23.15.33 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 18 Jan 2023 23:15:33 -0800 (PST) Received: by squeak.grove.modra.org (Postfix, from userid 1000) id 60A60114206A; Thu, 19 Jan 2023 17:45:31 +1030 (ACDT) Date: Thu, 19 Jan 2023 17:45:31 +1030 From: Alan Modra To: binutils@sourceware.org Subject: PR 30022, concurrent builds can fail Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-Spam-Status: No, score=-3035.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,GIT_PATCH_0,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: So let's not copy .libs/libbfd.a to libbfd.a now that nothing in the binutils-gdb source tries to link against it. PR 30022 * Makefile.am (noinst_LIBRARIES, libbfd_a_SOURCES, stamp-lib), (libbfd.a): Delete rules. (CLEANFILES): Adjust to suit. diff --git a/bfd/Makefile.am b/bfd/Makefile.am index a745372c0a4..931b2cbdcd6 100644 --- a/bfd/Makefile.am +++ b/bfd/Makefile.am @@ -781,25 +781,6 @@ libbfd_la_DEPENDENCIES = $(OFILES) ofiles ../libsframe/libsframe.la libbfd_la_LIBADD = `cat ofiles` @SHARED_LIBADD@ $(LIBDL) $(ZLIB) $(ZSTD_LIBS) ../libsframe/libsframe.la libbfd_la_LDFLAGS += -release `cat libtool-soversion` @SHARED_LDFLAGS@ -# libtool will build .libs/libbfd.a. We create libbfd.a in the build -# directory so that we don't have to convert all the programs that use -# libbfd.a simultaneously. This is a hack which should be removed if -# everything else starts using libtool. FIXME. - -noinst_LIBRARIES = libbfd.a -libbfd_a_SOURCES = - -stamp-lib: libbfd.la - libtooldir=`$(LIBTOOL) --config | $(SED) -n -e 's/^objdir=//p'`; \ - if [ -f $$libtooldir/libbfd.a ]; then \ - cp $$libtooldir/libbfd.a libbfd.tmp; \ - $(RANLIB) libbfd.tmp; \ - $(SHELL) $(srcdir)/../move-if-change libbfd.tmp libbfd.a; \ - else true; fi - touch stamp-lib - -libbfd.a: stamp-lib ; @true - # This file holds an array associating configuration triplets and # vector names. It is built from config.bfd. It is not compiled by # itself, but is included by targets.c. @@ -976,7 +957,7 @@ stmp-lcoff-h: $(LIBCOFF_H_FILES) $(MKDOC) MOSTLYCLEANFILES += ofiles stamp-ofiles -CLEANFILES += bfd.h dep.sed stmp-bfd-h DEP DEPA DEP1 DEP2 libbfd.a stamp-lib \ +CLEANFILES += bfd.h dep.sed stmp-bfd-h DEP DEPA DEP1 DEP2 \ stmp-bin2-h stmp-lbfd-h stmp-lcoff-h DISTCLEANFILES += $(BUILD_CFILES) $(BUILD_HFILES) libtool-soversion diff --git a/bfd/Makefile.in b/bfd/Makefile.in index 1e0c2dcae1e..6fde8df6b06 100644 --- a/bfd/Makefile.in +++ b/bfd/Makefile.in @@ -33,7 +33,6 @@ # - VPATH = @srcdir@ am__is_gnu_make = { \ if test -z '$(MAKELEVEL)'; then \ @@ -143,16 +142,6 @@ mkinstalldirs = $(SHELL) $(top_srcdir)/../mkinstalldirs CONFIG_HEADER = config.h CONFIG_CLEAN_FILES = bfd-in3.h po/Makefile.in CONFIG_CLEAN_VPATH_FILES = -LIBRARIES = $(noinst_LIBRARIES) -ARFLAGS = cru -AM_V_AR = $(am__v_AR_@AM_V@) -am__v_AR_ = $(am__v_AR_@AM_DEFAULT_V@) -am__v_AR_0 = @echo " AR " $@; -am__v_AR_1 = -libbfd_a_AR = $(AR) $(ARFLAGS) -libbfd_a_LIBADD = -am_libbfd_a_OBJECTS = -libbfd_a_OBJECTS = $(am_libbfd_a_OBJECTS) am__vpath_adj_setup = srcdirstrip=`echo "$(srcdir)" | sed 's|.|.|g'`; am__vpath_adj = case $$p in \ $(srcdir)/*) f=`echo "$$p" | sed "s|^$$srcdirstrip/||"`;; \ @@ -234,8 +223,7 @@ AM_V_CCLD = $(am__v_CCLD_@AM_V@) am__v_CCLD_ = $(am__v_CCLD_@AM_DEFAULT_V@) am__v_CCLD_0 = @echo " CCLD " $@; am__v_CCLD_1 = -SOURCES = $(libbfd_a_SOURCES) $(libbfd_la_SOURCES) \ - $(EXTRA_libbfd_la_SOURCES) +SOURCES = $(libbfd_la_SOURCES) $(EXTRA_libbfd_la_SOURCES) AM_V_DVIPS = $(am__v_DVIPS_@AM_V@) am__v_DVIPS_ = $(am__v_DVIPS_@AM_DEFAULT_V@) am__v_DVIPS_0 = @echo " DVIPS " $@; @@ -501,8 +489,8 @@ zlibinc = @zlibinc@ AUTOMAKE_OPTIONS = no-dist foreign info-in-builddir no-texinfo.tex ACLOCAL_AMFLAGS = -I . -I .. -I ../config MOSTLYCLEANFILES = ofiles stamp-ofiles $(MKDOC) doc/*.o doc/*.stamp -CLEANFILES = bfd.h dep.sed stmp-bfd-h DEP DEPA DEP1 DEP2 libbfd.a \ - stamp-lib stmp-bin2-h stmp-lbfd-h stmp-lcoff-h +CLEANFILES = bfd.h dep.sed stmp-bfd-h DEP DEPA DEP1 DEP2 stmp-bin2-h \ + stmp-lbfd-h stmp-lcoff-h DISTCLEANFILES = $(BUILD_CFILES) $(BUILD_HFILES) libtool-soversion \ doc/bfd.?? doc/bfd.??? texput.log MAINTAINERCLEANFILES = $(DOCFILES) doc/bfd.info @@ -1208,13 +1196,6 @@ libbfd_la_SOURCES = $(BFD32_LIBS_CFILES) EXTRA_libbfd_la_SOURCES = $(CFILES) libbfd_la_DEPENDENCIES = $(OFILES) ofiles ../libsframe/libsframe.la libbfd_la_LIBADD = `cat ofiles` @SHARED_LIBADD@ $(LIBDL) $(ZLIB) $(ZSTD_LIBS) ../libsframe/libsframe.la - -# libtool will build .libs/libbfd.a. We create libbfd.a in the build -# directory so that we don't have to convert all the programs that use -# libbfd.a simultaneously. This is a hack which should be removed if -# everything else starts using libtool. FIXME. -noinst_LIBRARIES = libbfd.a -libbfd_a_SOURCES = BFD_H_FILES = bfd-in.h opncls.c libbfd.c \ bfdio.c bfdwin.c section.c archures.c reloc.c \ syms.c bfd.c archive.c corefile.c targets.c format.c \ @@ -1386,9 +1367,6 @@ bfd-in3.h: $(top_builddir)/config.status $(srcdir)/bfd-in2.h po/Makefile.in: $(top_builddir)/config.status $(top_srcdir)/po/Make-in cd $(top_builddir) && $(SHELL) ./config.status $@ -clean-noinstLIBRARIES: - -test -z "$(noinst_LIBRARIES)" || rm -f $(noinst_LIBRARIES) - install-bfdlibLTLIBRARIES: $(bfdlib_LTLIBRARIES) @$(NORMAL_INSTALL) @list='$(bfdlib_LTLIBRARIES)'; test -n "$(bfdlibdir)" || list=; \ @@ -2043,8 +2021,7 @@ distclean-tags: check-am: all-am check: $(BUILT_SOURCES) $(MAKE) $(AM_MAKEFLAGS) check-recursive -all-am: Makefile $(INFO_DEPS) $(LIBRARIES) $(LTLIBRARIES) $(HEADERS) \ - config.h +all-am: Makefile $(INFO_DEPS) $(LTLIBRARIES) $(HEADERS) config.h installdirs: installdirs-recursive installdirs-am: for dir in "$(DESTDIR)$(bfdlibdir)" "$(DESTDIR)$(infodir)" "$(DESTDIR)$(bfdincludedir)"; do \ @@ -2090,8 +2067,7 @@ maintainer-clean-generic: clean: clean-recursive clean-am: clean-aminfo clean-bfdlibLTLIBRARIES clean-generic \ - clean-libtool clean-noinstLIBRARIES clean-noinstLTLIBRARIES \ - mostlyclean-am + clean-libtool clean-noinstLTLIBRARIES mostlyclean-am distclean: distclean-recursive -rm -f $(am__CONFIG_DISTCLEAN_FILES) @@ -2263,24 +2239,23 @@ uninstall-am: uninstall-bfdincludeHEADERS uninstall-bfdlibLTLIBRARIES \ .PHONY: $(am__recursive_targets) CTAGS GTAGS TAGS all all-am \ am--refresh check check-am clean clean-aminfo \ clean-bfdlibLTLIBRARIES clean-cscope clean-generic \ - clean-libtool clean-noinstLIBRARIES clean-noinstLTLIBRARIES \ - cscope cscopelist-am ctags ctags-am dist-info distclean \ - distclean-compile distclean-generic distclean-hdr \ - distclean-libtool distclean-tags dvi dvi-am html html-am \ - html-local info info-am install install-am \ - install-bfdincludeHEADERS install-bfdlibLTLIBRARIES \ - install-data install-data-am install-dvi install-dvi-am \ - install-exec install-exec-am install-html install-html-am \ - install-info install-info-am install-man install-pdf \ - install-pdf-am install-ps install-ps-am install-strip \ - installcheck installcheck-am installdirs installdirs-am \ - maintainer-clean maintainer-clean-aminfo \ - maintainer-clean-generic mostlyclean mostlyclean-aminfo \ - mostlyclean-compile mostlyclean-generic mostlyclean-libtool \ - pdf pdf-am ps ps-am tags tags-am uninstall uninstall-am \ - uninstall-bfdincludeHEADERS uninstall-bfdlibLTLIBRARIES \ - uninstall-dvi-am uninstall-html-am uninstall-info-am \ - uninstall-pdf-am uninstall-ps-am + clean-libtool clean-noinstLTLIBRARIES cscope cscopelist-am \ + ctags ctags-am dist-info distclean distclean-compile \ + distclean-generic distclean-hdr distclean-libtool \ + distclean-tags dvi dvi-am html html-am html-local info info-am \ + install install-am install-bfdincludeHEADERS \ + install-bfdlibLTLIBRARIES install-data install-data-am \ + install-dvi install-dvi-am install-exec install-exec-am \ + install-html install-html-am install-info install-info-am \ + install-man install-pdf install-pdf-am install-ps \ + install-ps-am install-strip installcheck installcheck-am \ + installdirs installdirs-am maintainer-clean \ + maintainer-clean-aminfo maintainer-clean-generic mostlyclean \ + mostlyclean-aminfo mostlyclean-compile mostlyclean-generic \ + mostlyclean-libtool pdf pdf-am ps ps-am tags tags-am uninstall \ + uninstall-am uninstall-bfdincludeHEADERS \ + uninstall-bfdlibLTLIBRARIES uninstall-dvi-am uninstall-html-am \ + uninstall-info-am uninstall-pdf-am uninstall-ps-am .PRECIOUS: Makefile @@ -2318,17 +2293,6 @@ stamp-ofiles: Makefile ofiles: stamp-ofiles ; @true -stamp-lib: libbfd.la - libtooldir=`$(LIBTOOL) --config | $(SED) -n -e 's/^objdir=//p'`; \ - if [ -f $$libtooldir/libbfd.a ]; then \ - cp $$libtooldir/libbfd.a libbfd.tmp; \ - $(RANLIB) libbfd.tmp; \ - $(SHELL) $(srcdir)/../move-if-change libbfd.tmp libbfd.a; \ - else true; fi - touch stamp-lib - -libbfd.a: stamp-lib ; @true - # This file holds an array associating configuration triplets and # vector names. It is built from config.bfd. It is not compiled by # itself, but is included by targets.c. -- Alan Modra Australia Development Lab, IBM