bug#70411: [bug] install(1) fails to read /dev/stdin on Darwin

2024-04-18 Thread Sergei Trofimovich
On Tue, 16 Apr 2024 22:37:58 +0100 Sergei Trofimovich wrote: > On Tue, 16 Apr 2024 12:33:44 +0100 > Pádraig Brady wrote: > > > On 16/04/2024 01:19, Alejandro Colomar wrote: > > > Hi! > > > > > > I don't own a Darwin system, so I can't help much repr

bug#70411: [bug] install(1) fails to read /dev/stdin on Darwin

2024-04-16 Thread Sergei Trofimovich
On Tue, 16 Apr 2024 12:33:44 +0100 Pádraig Brady wrote: > On 16/04/2024 01:19, Alejandro Colomar wrote: > > Hi! > > > > I don't own a Darwin system, so I can't help much reproduce. However, > > I've received a bug report to the Linux man-pages, that our build > > system (GNUmakefile-based),

[PATCH] kexec-tools: purgatory: fix build on `binutils-2.42`

2024-01-31 Thread Sergei Trofimovich
: 64bit mode not supported on `i386'. The change moves `.code32` before `.arch 386` as suggested in https://sourceware.org/PR31319 Signed-off-by: Sergei Trofimovich --- purgatory/arch/i386/entry32-16-debug.S | 2 +- purgatory/arch/i386/entry32-16.S | 2 +- 2 files changed, 2 insertions

[sr #110990] AS_IF / AC_CHECK_LIB generates invalid shell on editline-1.17.1

2023-12-23 Thread Sergei Trofimovich
Follow-up Comment #3, sr#110990 (group autoconf): Proposed `editline` change to adapt to `autoconf-2.72` as https://github.com/troglobit/editline/pull/64 ___ Reply to this item at:

[sr #110990] AS_IF / AC_CHECK_LIB generates invalid shell on editline-1.17.1

2023-12-23 Thread Sergei Trofimovich
Follow-up Comment #2, sr#110990 (group autoconf): Yeah, adding extra quotes around the second argument of `AS_IF()` does work. Thank you! ___ Reply to this item at:

[sr #110990] AS_IF / AC_CHECK_LIB generates invalid shell on editline-1.17.1

2023-12-23 Thread Sergei Trofimovich
/Linux ___ Follow-up Comments: --- Date: Sat 23 Dec 2023 06:24:04 PM UTC By: Sergei Trofimovich Noticed seeming autoconf-2.72 regression when was building https://github.com/troglobit/editline/blob/master/configure.

[PATCH] tests: fix build against upcoming `gcc-14` (`-Werror=calloc-transposed-args`)

2023-12-21 Thread Sergei Trofimovich
argument [-Werror=calloc-transposed-args] 450 | newscnbufs = calloc (sizeof (void *), newshnums); |^~~~ elfstrmerge.c:450:32: note: earlier argument should specify number of elements, later size of each element Signed-off-by: Sergei Trofimovich

[PATCH] diagnostics: fix gcc-urlifier.cc bootstrap failure [PR112379]

2023-11-04 Thread Sergei Trofimovich
From: Sergei Trofimovich Without the change `./configure --enable-checking=release` bootstrap fails as: gcc/gcc-urlifier.cc:100:1: error: 'get_url_suffix_for_quoted_text()' defined but not used [-Werror=unused-function] This happens because the helper is used only in `ASSERT

[PATCH] libelf/elf_newscn.c: fix build failure against gcc-14 (-Walloc-size)

2023-11-02 Thread Sergei Trofimovich
:12: error: allocation of insufficient size «1» for type «Elf_ScnList» with size «16» [-Werror=alloc-size] 97 | newp = calloc (sizeof (Elf_ScnList) |^ The change swaps arguments to pass larger value as a member size. Signed-off-by: Sergei Trofimovich

Re: [PATCH] libgcc: make heap-based trampolines conditional on libc presence

2023-10-23 Thread Sergei Trofimovich
On Mon, 23 Oct 2023 13:54:01 +0100 Iain Sandoe wrote: > hi Sergei, > > > On 23 Oct 2023, at 13:43, Sergei Trofimovich wrote: > > > > From: Sergei Trofimovich > > > > To build `libc` for a target one needs to build `gcc` without `libc` > > supp

[PATCH] libgcc: make heap-based trampolines conditional on libc presence

2023-10-23 Thread Sergei Trofimovich
From: Sergei Trofimovich To build `libc` for a target one needs to build `gcc` without `libc` support first. Commit r14-4823-g8abddb187b3348 "libgcc: support heap-based trampolines" added unconditional `libc` dependency and broke libc-less `gcc` builds. An example failure on `x86_

[PATCH v4] ipa-utils: avoid uninitialized probabilities on ICF [PR111559]

2023-10-05 Thread Sergei Trofimovich
places. Oh, great point! Completely forgot about it. Attached v4. If it still looks reasonable I'll check again if `python` and `profiledbootstrap` still survives it and will push. -- Sergei >From cb9852216b5b2524f72964b399c133557ec98df0 Mon Sep 17 00:00:00 2001 From: Sergei Trofimovich Date: Wed, 27 Sep 2023 14:29:12 +

Re: [PATCH v2] ipa-utils: avoid uninitialized probabilities on ICF [PR111559]

2023-10-05 Thread Sergei Trofimovich
On Thu, Oct 05, 2023 at 01:52:30PM +0200, Jan Hubicka wrote: > > From: Sergei Trofimovich > > > > r14-3459-g0c78240fd7d519 "Check that passes do not forget to define profile" > > exposed check failures in cases when gcc produces uninitialized profile >

Re: Scaling -fmacro-prefix-map= to thousands entries

2023-10-05 Thread Sergei Trofimovich via Gcc
On Thu, Oct 05, 2023 at 07:20:35AM -0400, Ben Boeckel wrote: > On Wed, Oct 04, 2023 at 22:19:32 +0100, Sergei Trofimovich via Gcc wrote: > > The prototype that creates equivalent of the following commands does > > work for smaller packages: > > > > > &

Re: Scaling -fmacro-prefix-map= to thousands entries

2023-10-05 Thread Sergei Trofimovich via Gcc
On Thu, Oct 05, 2023 at 09:19:15AM +0200, Richard Biener wrote: > On Wed, Oct 4, 2023 at 11:20 PM Sergei Trofimovich via Gcc > wrote: > > > > Hi gcc developers! > > > > Tl;DR: > > > > I would like to implement a scalable way to pass `-fmacro-prefix-m

Scaling -fmacro-prefix-map= to thousands entries

2023-10-04 Thread Sergei Trofimovich via Gcc
! -- Sergei >From b10785c1be469319a09b10bc69db21159b0599ee Mon Sep 17 00:00:00 2001 From: Sergei Trofimovich Date: Fri, 22 Sep 2023 22:41:49 +0100 Subject: [PATCH] gcc/file-prefix-map.cc: always mangle __FILE__ into invalid store path Without the change `__FILE__` used in static inl

[PATCH] Makefile.tpl: disable -Werror for feedback stage [PR111663]

2023-10-02 Thread Sergei Trofimovich
From: Sergei Trofimovich Without the change profiled bootstrap fails for various warnings on master branch as: $ ../gcc/configure $ make profiledbootstrap ... gcc/genmodes.cc: In function ‘int main(int, char**)’: gcc/genmodes.cc:2152:1: error: ‘gcc/build/genmodes.gcda

[PATCH v2] ipa-utils: avoid uninitialized probabilities on ICF [PR111559]

2023-10-01 Thread Sergei Trofimovich
From: Sergei Trofimovich r14-3459-g0c78240fd7d519 "Check that passes do not forget to define profile" exposed check failures in cases when gcc produces uninitialized profile probabilities. In case of PR/111559 uninitialized profile is generated by edges executed 0 times reported by I

[PATCH v2] rtl: fix build failure on -fchecking=2 [PR111642]

2023-09-30 Thread Sergei Trofimovich
From: Sergei Trofimovich Before the change `make bootstrap4` (or `make profiledbootstrap`) failed as: gcc/rtl-tests.cc:249:25: in ‘constexpr’ expansion of ‘poly_int<1, long int>(1, 1)’ gcc/poly-int.h:453:5: error: too many initializers for ‘long int [1]’ The failure happene

[PATCH] rtl: fix buidl failure on -fchecking=2 [PR111642]

2023-09-30 Thread Sergei Trofimovich
From: Sergei Trofimovich Before the change `make bootstrap4` (or `make profiledbootstrap`) failed as: gcc/rtl-tests.cc:249:25: in ‘constexpr’ expansion of ‘poly_int<1, long int>(1, 1)’ gcc/poly-int.h:453:5: error: too many initializers for ‘long int [1]’ The failure happene

[PATCH] ggc: do not wipe out unrelated data via gt_ggc_rtab

2023-09-28 Thread Sergei Trofimovich
From: Sergei Trofimovich There are 3 GC root tables: gt_ggc_rtab gt_ggc_deletable_rtab gt_pch_scalar_rtab `deletable` and `scalar` tables are both simple: each element always contains a pointer to the beginning of the object and it's size is the full object. `rtab` is different: it's

[PATCH] ipa-utils: avoid generating uninitialized probabilities on merges.

2023-09-27 Thread Sergei Trofimovich
From: Sergei Trofimovich r14-3459-g0c78240fd7d519 "Check that passes do not forget to define profile" exposed check failures in cases when gcc produces uninitialized profile probabilities. In case of PR/111559 uninitialized profile is generated by edges executed 0 times duri

bug#65679: Request to cut new automake release to include new pythons

2023-09-01 Thread Sergei Trofimovich
Hi automake developers! Projects using automake build system occasionally get bug reports that python 3.11 and above do not get automatically detected by AM_PATH_PYTHON([3.7]) Recent example: https://github.com/skvadrik/re2c/issues/456 It was fixed a while ago in

pathological bash-malloc behaviour

2023-08-18 Thread Sergei Trofimovich
Hi bash developers! I think I encountered an unusual pathological behaviour of `bash-malloc`. I have a directory with ~500_000 files and directories in it. I tried to iterate through it with a single `for` loop using `*` glob and noticed that glob expansion takes more time that actual loop

[PATCH] Drop unused enum vrp_mode.

2023-08-16 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich Follow removal of EVRP and clean up unused defines. gcc/ * flag-types.h (vrp_mode): Remove unused. --- gcc/flag-types.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/gcc/flag-types.h b/gcc/flag-types.h index 36305de589e..7466c1106f2 100644 --- a/gcc

Re: [PATCH] Makefile.in: add missing intl dependency for unwind_prot.o

2023-07-25 Thread Sergei Trofimovich
On Tue, 25 Jul 2023 10:12:23 -0400 Chet Ramey wrote: > On 7/24/23 6:33 PM, Sergei Trofimovich wrote: > > As reported by Robert Scott in https://github.com/NixOS/nixpkgs/pull/245066 > > without the change `make -j8` build of `make` occasionally fails to > > buildin par

[PATCH] Makefile.in: add missing intl dependency for unwind_prot.o

2023-07-24 Thread Sergei Trofimovich
As reported by Robert Scott in https://github.com/NixOS/nixpkgs/pull/245066 without the change `make -j8` build of `make` occasionally fails to buildin parallel. The simplest reproducer is: $$ ./configure $$ make unwind_prot.o ... In file included from unwind_prot.c:51: In

[PATCH] mh-mingw: drop unused BOOT_CXXFLAGS variable

2023-07-21 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich gcc's build system has BOOT_CFLAGS and various STAGE_C{,XX}FLAGS variables. BOOT_CXXFLAGS is not handled anywhere. config/ * mh-mingw: Drop assignment of unused BOOT_CXXFLAGS variable. --- config/mh-mingw | 1 - 1 file changed, 1 deletion(-) diff --git

Re: 'exec -a' and $0 substitution

2023-01-26 Thread Sergei Trofimovich
On Thu, 26 Jan 2023 00:35:27 -0500 Lawrence Velázquez wrote: > On Wed, Jan 25, 2023, at 6:37 PM, Sergei Trofimovich wrote: > > I fear it's a side-effect of the way 'bash' gets executed via shebang by > > kernel. But maybe not? Somehow direct script execution still manages

'exec -a' and $0 substitution

2023-01-25 Thread Sergei Trofimovich
Hello bash maintainers! nixpkgs package collection likes to wrap binaies and shell scripts to pass extra environment variables via wrappers. One example is dejagnu's runtest: $ cat `which runtest` | unnix #! /<>/bash-5.2-p15/bin/bash -e PATH=... export PATH exec -a "$0"

Re: gnulib's obstack_* symbols in dynamic exports on glibc. Intentional?

2022-12-30 Thread Sergei Trofimovich
On Fri, 30 Dec 2022 19:26:48 +0100 Bruno Haible wrote: > Sergei Trofimovich wrote: > > to my surprise it did have one a set of dynamically exported symbols: > > > > $ nm -D `which ls` | grep -v '^ ' > > 004c0d40 T _obstack_allocated_p

gnulib's obstack_* symbols in dynamic exports on glibc. Intentional?

2022-12-30 Thread Sergei Trofimovich
Hello bug-gnulib (and libc-alpha, CCed)! The other day I was looking at simplest programs to make sure they don't normally have dynamic exports. I picked coreutils-9.1 'ls'. And to my surprise it did have one a set of dynamically exported symbols: $ nm -D `which ls` | grep -v '^ '

Re: [PATCH v2] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-10-13 Thread Sergei Trofimovich via Elfutils-devel
On Thu, 13 Oct 2022 16:40:36 +0200 Mark Wielaard wrote: > Hi, > > On Mon, 2022-08-08 at 01:17 +0200, Mark Wielaard wrote: > > On Sun, Aug 07, 2022 at 07:31:38PM +0100, Sergei Trofimovich via > > Elfutils-devel wrote: > > > binutils started producing 0-sized

PING [PATCH] gcc/config/t-i386: add build dependencies on i386-builtin-types.inc

2022-10-03 Thread Sergei Trofimovich via Gcc-patches
On Thu, 22 Sep 2022 22:07:52 +0100 Sergei Trofimovich wrote: > On Fri, 16 Sept 2022 at 19:49, Sergei Trofimovich wrote: > > > > From: Sergei Trofimovich > > > > i386-builtin-types.inc is included indirectly via i386-builtins.h > > into 4 files: i386.cc i38

[bug #63098] make-4.3.90 regression of unexpected dependencies in pattern rules with multiple targets

2022-09-23 Thread Sergei Trofimovich
Follow-up Comment #3, bug #63098 (project make): Aha, behavior change makes sense. Let's close it as intended then. Looks like opensp's case is even more nuanced. `%.h %.cxx %.rc: %.msg` produces `.cxx` files only for a subset of `.msg` files: `.cxx` generation happens only where file contents

[bug #63098] make-4.3.90 regression of unexpected dependencies in pattern rules with multiple targets

2022-09-23 Thread Sergei Trofimovich
elease: None Triage Status: None ___ Follow-up Comments: --- Date: Fri 23 Sep 2022 07:05:25 PM UTC By: Sergei Trofimovich Initially noticed the problem in OpenSP-1.5.2 build f

Re: [PATCH] gcc/config/t-i386: add build dependencies on i386-builtin-types.inc

2022-09-22 Thread Sergei Trofimovich via Gcc-patches
On Fri, 16 Sept 2022 at 19:49, Sergei Trofimovich wrote: > > From: Sergei Trofimovich > > i386-builtin-types.inc is included indirectly via i386-builtins.h > into 4 files: i386.cc i386-builtins.cc i386-expand.cc i386-features.cc > > Only i386.cc dependency was present

Re: [Mingw-w64-public] [PATCH 1/2] mingw-w64-crt: fix parallel build failures in dlltool calls

2022-09-22 Thread Sergei Trofimovich
On Thu, 22 Sep 2022 11:23:38 +0300 (EEST) Martin Storsjö wrote: > On Sun, 18 Sep 2022, Sergei Trofimovich wrote: > > > Before the change massively parallel builds failed when Makefile > > attempted to build both lib64/libd3dx9_38.a and lib32/libd3dx9_38.a: > > > >

[Mingw-w64-public] [PATCH 1/2] mingw-w64-crt: fix parallel build failures in dlltool calls

2022-09-21 Thread Sergei Trofimovich
Before the change massively parallel builds failed when Makefile attempted to build both lib64/libd3dx9_38.a and lib32/libd3dx9_38.a: x86_64-w64-mingw32-dlltool --as-flags=--64 -m i386:x86-64 ... \ --output-lib lib64/libd3dx9_38.a \ --temp-prefix $(basename lib64/libd3dx9_38.a .a)

[Mingw-w64-public] [PATCH 2/2] Makefile.am: add missing msvcr${V} dependency on msvcr${V}_extra

2022-09-21 Thread Sergei Trofimovich
Without the change build fails with missing dependency on _extra library similar to the following: cd lib32/ && x86_64-w64-mingw32-ar -M < /build/mingw-w64/mingw-w64-crt/lib32/msvcr71.mri x86_64-w64-mingw32-ar: libmsvcr71_extra.a: No such file or directory make[3]: ***

[PATCH] gcc/config/t-i386: add build dependencies on i386-builtin-types.inc

2022-09-16 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich i386-builtin-types.inc is included indirectly via i386-builtins.h into 4 files: i386.cc i386-builtins.cc i386-expand.cc i386-features.cc Only i386.cc dependency was present in gcc/config/t-i386 makefile. As a result parallel builds occasionally fail as: g++ ... -o

[bug #63047] --shuffle mode does not work on .SECONDEXPANSION targets

2022-09-11 Thread Sergei Trofimovich
Follow-up Comment #1, bug #63047 (project make): Posted the patch for review as https://lists.gnu.org/archive/html/bug-make/2022-09/msg00069.html ___ Reply to this item at:

[bug #63047] --shuffle mode does not work on .SECONDEXPANSION targets

2022-09-11 Thread Sergei Trofimovich
Additional Item Attachment, bug #63047 (project make): File name: 0001-Fix-shuffle-for-SECONDEXPANSION-prerequisites.patch Size:3 KB

[PATCH] Fix --shuffle for SECONDEXPANSION prerequisites

2022-09-11 Thread Sergei Trofimovich
From: Sergei Trofimovich Since commit 07eea3aa4 ("[SV 62706] Only second-expand targets that might be built") `make --shuffle` stopped shuffling prerequisites that use .SECONDEXPANSION feature for Makefiles like: .SECONDEXPANSION: all: $$(var) %_: ; @echo $@ var

[bug #63047] --shuffle mode does not work on .SECONDEXPANSION targets

2022-09-11 Thread Sergei Trofimovich
Status: None ___ Follow-up Comments: --- Date: Sun 11 Sep 2022 08:13:12 PM UTC By: Sergei Trofimovich Since commit 07eea3aa496184bb763b7306e9de6a40a94605c9 ("[SV 62706] Only second-exp

Re: [PATCH] autoconf: fix build on upcoming GNU make-4.4

2022-09-09 Thread Sergei Trofimovich
On Fri, 9 Sep 2022 16:59:21 -0500 Paul Eggert wrote: > Thanks for the bug report. I installed the attached, which isn't the > same as what you suggested, but which should fix the problem. You version works for me as well. Thank you! -- Sergei

[bug #63040] autoconf-2.69 and 2.71 fail to build after $(shell ...) environment handlign change

2022-09-09 Thread Sergei Trofimovich
Follow-up Comment #1, bug #63040 (project make): Proposed possible fix for autoconf as https://lists.gnu.org/archive/html/autoconf-patches/2022-09/msg7.html ___ Reply to this item at:

[PATCH] autoconf: fix build on upcoming GNU make-4.4

2022-09-09 Thread Sergei Trofimovich
>From https://savannah.gnu.org/bugs/?63040: GNU make 4.4 will slightly change the rules it uses to export variables to $(shell ...) calls. `autoconf` uses unportable variant of assignment: export PATH = $(shell echo "`pwd`/tests:$$PATH") This causes build failure on `make` from `git`:

[RESEND PATCH] autoconf: store autom4te request keys in sorted order

2022-09-09 Thread Sergei Trofimovich
Notced unstable key order when debugging unrelated bug. Data::Dumper's SortKeys() makes ordering stable and decreases diffs from run to run. No functional change otherwise. * lib/Autom4te/C4che.pm * lib/Autom4te/Request.pm: Sort request keys in serialization. --- lib/Autom4te/C4che.pm | 2 +-

[bug #63040] autoconf-2.69 and 2.71 fail to build after $(shell ...) environment handlign change

2022-09-09 Thread Sergei Trofimovich
: None Triage Status: None ___ Follow-up Comments: --- Date: Fri 09 Sep 2022 10:35:02 AM UTC By: Sergei Trofimovich Tried to use `make` from today's git (

[PATCH] Makefile.def: drop remnants of unused libelf

2022-08-18 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich Use of libelf was removed from gcc in misc/cutover-cvs2svn-32781-g48215350c24 ("re PR lto/46273 (Failed to bootstrap)") around 2010, before gcc-4.6.0. This change removes unused references to libelf from top-level condifure and Makefile. / * Makefile

[PATCH] Add libgo dependency on libbacktrace.

2022-08-18 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich Noticed missing dependency when regenerated Makefile.in for unrelated change with 'autoget Makefile.def'. The change was lost in basepoints/gcc-12-6861-gaeac414923a ("Revert "Fix PR 67102: Add libstdc++ dependancy to libffi" [PR67102]"). /

[PATCH] driver: fix environ corruption after putenv() [PR106624]

2022-08-16 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich The bug appeared afte r13-2010-g1270ccda70ca09 "Factor out jobserver_active_p" slightly changed `putenv()` use from allocating to non-allocating: -xputenv (concat ("MAKEFLAGS=", dup, NULL)); +xputenv (jinfo.skipped_makeflags.c_str ()); `xpu

[PATCH v2] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-08-07 Thread Sergei Trofimovich via Elfutils-devel
sh (exit status: 255) To work it around let's allow lowpc == highpc special case. https://sourceware.org/bugzilla/show_bug.cgi?id=29450 Signed-off-by: Sergei Trofimovich --- tests/ChangeLog | 4 tests/low_high_pc.c | 8 +--- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git

[PATCH] tests: run-low_high_pc.sh: drop redundant 'lx' suffix

2022-08-06 Thread Sergei Trofimovich via Elfutils-devel
Noticed when debugged test failure: lowpc: 8049000, highpc: 8049000lx ../sysdeps/i386/crti.S: [2def] '_init' highpc <= lowpc Signed-off-by: Sergei Trofimovich --- tests/ChangeLog | 4 tests/low_high_pc.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --

[PATCH] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-08-06 Thread Sergei Trofimovich via Elfutils-devel
sh (exit status: 255) To work it around let's allow lowpc == highpc special case. https://sourceware.org/bugzilla/show_bug.cgi?id=29450 Signed-off-by: Sergei Trofimovich --- tests/ChangeLog | 4 tests/low_high_pc.c | 3 +++ 2 files changed, 7 insertions(+) diff --git a/tests/ChangeLog

Re: 'make -j install' fails for missing depends

2022-06-29 Thread Sergei Trofimovich
On Wed, 29 Jun 2022 09:16:17 +0200 Daniel Sahlberg wrote: > Den lör 25 juni 2022 kl 00:41 skrev Sergei Trofimovich : > > > On Fri, 24 Jun 2022 15:14:23 -0400 > > Nathan Hartman wrote: > > > > > On Thu, Jun 23, 2022 at 5:58 AM Sergei Trofimovich > >

[PATCH v2] jit: avoid calloc() poisoning on musl [PR106102]

2022-06-28 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich On musl uses calloc() (via ). jit/ includes it directly and exposes use of poisoned calloc(): /build/build/./prev-gcc/xg++ ... ../../gcc-13-20220626/gcc/jit/jit-playback.cc make[3]: *** [Makefile:1143: jit/libgccjit.o] Error 1 make[3]: *** Waiting

[PATCH] jit: avoid calloc() poisoning on musl [PR106102]

2022-06-27 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich On musl uses calloc() (via ). jit/ includes it directly and exposes use of poisoned calloc(): /build/build/./prev-gcc/xg++ ... ../../gcc-13-20220626/gcc/jit/jit-playback.cc make[3]: *** [Makefile:1143: jit/libgccjit.o] Error 1 make[3]: *** Waiting

[PATCH] c++: avoid poisoning on musl [PR106102]

2022-06-27 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich On musl uses calloc() (via ). includes it indirectly and exposes use of poisoned calloc() when module code is built: /build/build/./prev-gcc/xg++ ... ../../gcc-13-20220626/gcc/cp/mapper-resolver.cc In file included from /<>/musl-1.2.3-dev/i

Re: 'make -j install' fails for missing depends

2022-06-24 Thread Sergei Trofimovich
On Fri, 24 Jun 2022 15:14:23 -0400 Nathan Hartman wrote: > On Thu, Jun 23, 2022 at 5:58 AM Sergei Trofimovich wrote: > > > > Hi subversion developers! > > > > I'm working on fixing parallel build (or install) faliures in upstream > > packages used in NixOS li

'make -j install' fails for missing depends

2022-06-23 Thread Sergei Trofimovich
Hi subversion developers! I'm working on fixing parallel build (or install) faliures in upstream packages used in NixOS linux distribution. Mostly to speed package builds up on continuous build farm. That usually means that $ ./configure && make && make install is changed to $

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-05-22 Thread Sergei Trofimovich
Additional Item Attachment, bug #62100 (project make): File name: v7-0001-Add-shuffle-argument-support.patch Size:35 KB ___ Reply to this

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-05-22 Thread Sergei Trofimovich
Follow-up Comment #6, bug #62100 (project make): Adding a v7-0001-Add-shuffle-argument-support.patch which 2 extra changes: 1. fixed 'if' indentation in src/main.c noticed by Fangrui Song 2. tweaked --help output to '--shuffle[={SEED|random|identity|reverse|none}]' noticed by Fangrui Song

[gnugo-devel] [patch #10208] fix gcc-10 build failure (-fno-common)

2022-05-21 Thread Sergei Trofimovich
mments: --- Date: Sat 21 May 2022 09:49:16 AM UTC By: Sergei Trofimovich gcc-10 changed the default from -fcommon to fno-common: https://gcc.gnu.org/PR85678 As a result build fails as: ld: dfa.o:/build/gnugo-3.8/patterns/../engine/liberty.h:861: multiple defi

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-05-03 Thread Sergei Trofimovich
Follow-up Comment #4, bug #62100 (project make): Caught bugs in a few upstream projects. Two nice simple examples: - vim: https://github.com/vim/vim/pull/9978 - groff: https://savannah.gnu.org/bugs/?62084 ___ Reply to this item at:

[bug #62084] Parallel build failure: fatal error: cannot load 'DESC' description file for device 'ps'

2022-05-03 Thread Sergei Trofimovich
Follow-up Comment #3, bug #62084 (project groff): Survived 15 rebuilds for me on GNU make with --shuffle option (https://savannah.gnu.org/bugs/index.php?62100). Thank you! ___ Reply to this item at:

Re: rx2660 + debian

2022-04-27 Thread Sergei Trofimovich
On Tue, 26 Apr 2022 08:18:25 + Pedro Miguel Justo wrote: > > On 2022/Apr/26, at 01:01, Sergei Trofimovich wrote: > > > > On Tue, 26 Apr 2022 02:43:00 + > > Pedro Miguel Justo wrote: > > > >>> On 2022/Apr/25, at 14:27, Pedro Miguel Justo wr

Re: rx2660 + debian

2022-04-25 Thread Sergei Trofimovich
On Mon, 25 Apr 2022 15:07:58 + Pedro Miguel Justo wrote: > > On 2022/Apr/25, at 01:22, Pedro Miguel Justo wrote: > > > > > > > >> On 2022/Apr/25, at 01:14, Frank Scheiner wrote: > >> > >> Hi guys, > >> > >> On 25.04.22 10:09, John Paul Adrian Glaubitz wrote: > From what I can

[PATCH, gcc-11 backport] gcov-profile: Allow negative counts of indirect calls [PR105282]

2022-04-19 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich TOPN metrics are histograms that contain overall count and per-bucket count. Overall count can be negative when two profiles merge and some of per-bucket metrics are disacarded. Noticed as an ICE on python PGO build where gcc crashes as: during IPA pass: modref

[PATCH] gcov-profile: Allow negavive counts of indirect calls [PR105282]

2022-04-15 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich TOPN metrics are histograms that contain overall count and per-bucket count. Overall count can be nevative when two profiles merge and some of per-bucket metrics are dropped. Noticed as an ICE on python PGO build where gcc crashes as: during IPA pass: modref

Re: gcc inserts __builtin_popcount, causes 'modpost: "__popcountdi2" ... amdgpu.ko] undefined'

2022-04-13 Thread Sergei Trofimovich
On Mon, Apr 11, 2022 at 10:08:15PM +0100, Sergei Trofimovich wrote: > Current linux-5.17.1 on fresh gcc-12 fails to build with errors like: > > ERROR: modpost: "__popcountdi2" > [drivers/net/ethernet/broadcom/bnx2x/bnx2x.ko] undefined! > ERROR: modpost: "_

gcc inserts __builtin_popcount, causes 'modpost: "__popcountdi2" ... amdgpu.ko] undefined'

2022-04-11 Thread Sergei Trofimovich
Current linux-5.17.1 on fresh gcc-12 fails to build with errors like: ERROR: modpost: "__popcountdi2" [drivers/net/ethernet/broadcom/bnx2x/bnx2x.ko] undefined! ERROR: modpost: "__popcountdi2" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! It is occasionally seen by others on

[PATCH] libgcc: IA64: don't compile glibc-based unwinder without libc headers

2022-04-07 Thread Sergei Trofimovich via Gcc-patches
From: Sergei Trofimovich In --without-headers mode gcc fails to bootstrap on libgcc as: /build/build/./gcc/xgcc -B/build/build/./gcc/ ... -Dinhibit_libc -c fde-glibc.c ../../../gcc-12-20220403/libgcc/config/ia64/fde-glibc.c:33:10: fatal error: stdlib.h: No such file

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-03-11 Thread Sergei Trofimovich
Follow-up Comment #3, bug #62100 (project make): Uploaded v6-0001-Add-shuffle-argument-support.patch. It's the rebased v5 against today's make.git where a small merge conflict is fixed around added option. Otherwise identical to v5. ___

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-03-11 Thread Sergei Trofimovich
Additional Item Attachment, bug #62100 (project make): File name: v6-0001-Add-shuffle-argument-support.patch Size:35 KB ___ Reply to this

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-02-23 Thread Sergei Trofimovich
Follow-up Comment #1, bug #62100 (project make): Changes from v4: presence of `.NOTPARALLEL:` target disables shuffling as makefile is already known to be broken in target reorder caused by -j parallelism. Example of real package is `netpbm`. Upstream does not plan to fix it.

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-02-23 Thread Sergei Trofimovich
Additional Item Attachment, bug #62100 (project make): File name: v5-0001-Add-shuffle-argument-support.patch Size:35 KB ___ Reply to this

[bug #62100] [PATCH] Add '--shuffle' argument support

2022-02-22 Thread Sergei Trofimovich
URL: Summary: [PATCH] Add '--shuffle' argument support Project: make Submitted by: slyfox Submitted on: Wed 23 Feb 2022 07:51:47 AM UTC Severity: 3 - Normal Item Group:

Re: [PATCH] Fix src/function.c build failure on gcc-12.

2022-02-22 Thread Sergei Trofimovich
On Mon, 21 Feb 2022, 15:29 Paul Smith, wrote: > > On Mon, 2022-02-21 at 08:59 +, Edward Welbourne wrote: > > and that's assuming a 32-bit int; the signed range is from - > > 2,147,483,647 to 2,147,483,648. However, may I suggest the following > > (which I know included in the GPL'd cfengine

[PATCH] Test -flto on both compiler and linker

2022-02-20 Thread Sergei Trofimovich
Before the change ./configure incorrectly enabled -flto on toolchains that support -flto on compiler side but don't support -flto on linker side. This caused incorrect type size detection on nixpkgs' Darwin: configure:54594: checking size of size_t configure:54600: clang -std=gnu11 -o conftest

[PATCH v4] Add '--shuffle' argument support

2022-02-20 Thread Sergei Trofimovich
On Sun, Feb 20, 2022 at 10:30:10AM +, Sergei Trofimovich wrote: > On Sat, Feb 19, 2022 at 09:57:13AM +0200, Eli Zaretskii wrote: > > I think NEWS should also call out the new feature. Forgot the actual NEWS entry. Attached v4. -- Sergei >From 6eb8c759f6e917ff82326769e509505f

Re: [PATCH v2] Add '--shuffle' argument support

2022-02-20 Thread Sergei Trofimovich
orse than naive recursion in update_file_1. > > --- /dev/null > > +++ b/tests/scripts/options/shuffle-reverse > > This test doesn't seem to test the random option. I understand why > this is not easy, but shouldn't it still be tested, if only to make > sure the option works

[bug #62084] Parallel build failure: fatal error: cannot load 'DESC' description file for device 'ps'

2022-02-19 Thread Sergei Trofimovich
URL: Summary: Parallel build failure: fatal error: cannot load 'DESC' description file for device 'ps' Project: GNU troff Submitted by: slyfox Submitted on: Sat 19 Feb 2022 10:13:26 AM UTC

make re-exec regression in 'make -sf -' mode

2022-02-19 Thread Sergei Trofimovich
Noticed the regression on lowdown-0.10.0 upstream package. Here is a complete trigger: $ printf 'all:\n\techo $(CC)' | make -sf - [good] GNU make 4.3 works as expected: $ printf 'all:\n\techo $(CC)' | make -sf - cc [bad] GNU make from git loops indefinitely in re-execution: $

Re: [PATCH] RFC: add --shuffle[=seed] argument support

2022-02-18 Thread Sergei Trofimovich
On Sun, 6 Feb 2022 09:34:22 + Sergei Trofimovich wrote: > On Sat, 05 Feb 2022 18:39:41 -0500 > Paul Smith wrote: > > > Nice work! > > > > On Sat, 2022-02-05 at 22:04 +, Sergei Trofimovich wrote: > > > Some high level TODOs: > >

[PATCH v2] Add '--shuffle' argument support

2022-02-18 Thread Sergei Trofimovich
From: Sergei Trofimovich The idea of the change is to introduce non-deterministic ordering into goal and prerequisite traversal to imitate non-deterministic ordering of 'make -j g1 g2 g3' mode. The implementation is to introduce second order into each dependency chain: 1. existing order

[PATCH] Fix src/function.c build failure on gcc-12.

2022-02-18 Thread Sergei Trofimovich
From: Sergei Trofimovich Upcoming gcc-12 detects possible buffer overflow for 1 byte: src/function.c: In function 'func_call': src/function.c:2781:24: error: '__builtin___sprintf_chk' may write a terminating nul past the end of the destination [-Werror=format-overflow=8] 2781

Re: [PATCH] RFC: add --shuffle[=seed] argument support

2022-02-06 Thread Sergei Trofimovich
On Sat, 05 Feb 2022 18:39:41 -0500 Paul Smith wrote: > Nice work! > > On Sat, 2022-02-05 at 22:04 +0000, Sergei Trofimovich wrote: > > Some high level TODOs: > > For this amount of change it's likely that you'll need to provide > copyright assignment paperwork. L

[PATCH] RFC: add --shuffle[=seed] argument support

2022-02-05 Thread Sergei Trofimovich
The idea of the change is to introduce non-deterministic ordering into goal and prerequisite traversal to imitate non-deterministic ordering of 'make -j' mode. The implementation is to reorder lists of goals to build and list of dependencies attached to files reachable via goals. tested on the

Re: Idea of triggering bugs in users' Makefiles

2022-02-04 Thread Sergei Trofimovich
On Fri, 04 Feb 2022 10:27:01 -0500 Paul Smith wrote: > On Fri, 2022-02-04 at 09:13 +0000, Sergei Trofimovich wrote: > > 1. Enable parallel builds by GNU make by default > > > > 2. Do not run dependencies in deterministic order by default: > > GNU make (unlike N

Idea of triggering bugs in users' Makefiles

2022-02-04 Thread Sergei Trofimovich
Individual developers more often than not build large projects in parallel to speed the process up with 'make -j '. OS distributions would also prefer to enable parallel builds by default. But there is a problem of deciding if it's safe to build a package in general. In small projects developers

Re: hello-2.11 fails multibyte-1 and atexit-1 on musl

2022-01-31 Thread Sergei Trofimovich
On Mon, 31 Jan 2022 11:39:47 + Reuben Thomas wrote: > Thanks very much for the analysis and suggested patch. > > I don't think however that a code change is needed. Arguably, the > correct fix would be to check the return value of wprintf, but in fact > close_stdout() already checks if

Re: hello-2.11 fails multibyte-1 and atexit-1 on musl

2022-01-29 Thread Sergei Trofimovich
On Thu, 27 Jan 2022 22:08:57 + Reuben Thomas wrote: > On Wed, 26 Jan 2022 at 12:59, Sergei Trofimovich wrote: > > > > Thanks for the report! > > > From what I understand at > > > > https://wiki.musl-libc.org/functional-differences-from-glibc.htm

hello-2.11 fails multibyte-1 and atexit-1 on musl

2022-01-26 Thread Sergei Trofimovich
Hi hello maintainers! I attempted to package hello-2.11 in NixOS and stumbled on two test failures on x86_64-unknown-linux-musl: FAIL: tests/atexit-1 FAIL: tests/multibyte-1 Detailed log: FAIL: tests/atexit-1 1c1 < hello: write error: No space

[dm-devel] [PATCH 4/4] multipath-tools: allow passing non-standard linux-headers location

2022-01-09 Thread Sergei Trofimovich
On NixOS nothing is installed in /usr/include and instead lives in it's own prefix. The change switches linux/nvme_ioctl.h discovery to user provided path. CC: Martin Wilck CC: Benjamin Marzinski Signed-off-by: Sergei Trofimovich --- Makefile.inc | 5 + libmultipath

[dm-devel] [PATCH 1/4] multipath-tools: avoid using GZIP Makefile variable

2022-01-09 Thread Sergei Trofimovich
n GZIP environment variable Try `gzip --help' for more information. Fix build by renaming GZIP variable to GZIP_PROG to avoid collision. CC: Martin Wilck CC: Benjamin Marzinski Signed-off-by: Sergei Trofimovich --- Makefile.inc | 2 +- kpartx/Makefile | 2 +- libmpathp

[dm-devel] [PATCH 3/4] multipath-tools: autodiscover libudev.h headers

2022-01-09 Thread Sergei Trofimovich
switches libudev.h discovery to pkg-config provided path. CC: Martin Wilck CC: Benjamin Marzinski Signed-off-by: Sergei Trofimovich --- Makefile.inc | 8 libmultipath/Makefile | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/Makefile.inc b/Makefile.inc index

[dm-devel] [PATCH 2/4] multipath-tools: autodiscover libdevmapper.h headers

2022-01-09 Thread Sergei Trofimovich
switches libdevmapper.h discovery to pkg-config provided path. CC: Martin Wilck CC: Benjamin Marzinski Signed-off-by: Sergei Trofimovich --- Makefile.inc | 8 kpartx/Makefile | 2 +- libmultipath/Makefile | 10 +- multipathd/Makefile | 2 +- 4 files changed, 15

[dm-devel] [PATCH v2 1/2] kpartx/devmapper.c: fix unused-but-set variable error

2022-01-09 Thread Sergei Trofimovich
declaration under #ifdef that uses it. CC: Martin Wilck CC: Benjamin Marzinski Signed-off-by: Sergei Trofimovich --- kpartx/devmapper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kpartx/devmapper.c b/kpartx/devmapper.c index 3efd6dfc..511c090f 100644 --- a/kpartx

  1   2   3   4   5   6   7   8   9   10   >