Re: idea: make --show-interpretation

On Mon, May 6, 2024 at 8:26 PM Dan Jacobson wrote: > Have a command that reads in a makefile and outputs the makefile saying > what each part evaluated to. i'd use -p and -n switches. regards, Dmitry

Re: Treat "Circular dependency...dropped" as a hard error instead?

On Wed, May 8, 2024 at 2:04 PM JZB wrote: >... perhaps a command line option or a ".CIRCULARERROR:" rule > or some way to allow this to be turned on by those who need it? There is currently no mechanism (option or otherwise) to fail on a circular dependency. On top of that, there are situations w

[bug #65739] Add warnings circular-dep and circular-extra-dep.

Status: None ___ Follow-up Comments: --- Date: Tue 14 May 2024 01:40:55 AM UTC By: Dmitry Goncharov . ___ Reply to this i

[bug #65739] Add warnings circular-dep and circular-extra-dep.

Follow-up Comment #1, bug #65739 (group make): Patch sv65739_circular_dep_warn_part1.diff adds warning circular-dep. This warning controls how makes handles circular dependencies. The behavior is to silently drop the dependency, print a message or print a message and exit with an error code. The d

[bug #65739] Add warnings circular-dep and circular-extra-dep.

Additional Item Attachment, bug #65739 (group make): File name: sv65739_circular_dep_warn_part1.diff Size: 6KiB File name: sv65739_circular_dep_warn_part2.diff Size: 6KiB

[bug #65739] Add warnings circular-dep and circular-extra-dep.

Follow-up Comment #2, bug #65739 (group make): Here is an example $ cat makefile hello: hello $ make make: circular hello <- hello dependency dropped make: Nothing to be done for 'hello'. $ make --warn=circular-dep:ignore make: Nothing to be done for 'hello'. $ make --warn=circular-dep:error ma

Re: Treat "Circular dependency...dropped" as a hard error instead?

On Wed, May 8, 2024 at 2:04 PM JZB wrote: > > I have a GNU make-based build system wherein I would like for any/all > GNU make-detected circular dependencies to be treated as a hard error, > i.e., fail [immediately], rather than a "dropped and we go on without > it..." situation. > > Is there a wa

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #5, bug #60736 (group make): Attached an alternative patch to https://savannah.gnu.org/bugs/index.php?65739. ___ Reply to this item at: __

[bug #65759] handling of "-" and "--" on command line

Follow-up Comment #2, bug #65759 (group make): - is a valid filename. i think, everybody would be better off, with one and only one syntax of having make read from stdin. ___ Reply to this item at:

Re: [PATCH] Fix biased shuffle by avoiding already "struck" elements

On Tue, Jun 18, 2024 at 5:38 PM Sergei Trofimovich wrote: > After the change probabilities are not as biased: > > 0 1 2 3 > _ _ _ _ > 0 | 24.99 24.99 25.01 25.01 > 1 | 24.99 25.04 24.99 24.99 > 2 | 25.01 25.00 25.00 24.99 > 3 | 25.01

Re: [PATCH] Fix biased shuffle by avoiding already "struck" elements

On Wed, Jun 19, 2024 at 5:37 PM Sergei Trofimovich wrote: > Glancing at tests/ all the tests exercise user-facing `make` API. What > would be the best way to validate probabilities? For this type of code like shuffle, i like testing the function itself, directly. Just calling the function in a t

Re: [PATCH] Fix biased shuffle by avoiding already "struck" elements

On Fri, Jun 21, 2024 at 10:30 AM Paul Smith wrote: > > On Fri, 2024-06-21 at 09:11 -0400, Dmitry Goncharov wrote: > > i hope, Paul approves adding tests of this nature. > > I have no problems with adding unit tests, as long as we can find a way > to integrate it int

[bug #66011] dependencies after a non-existing file are silently ignored

Follow-up Comment #1, bug #66011 (group make): This rule %.o : %.c $(HEADERS) Makefile does not apply because nonexisting.h does not exist. To make there is simply no rule in this makefile to build *.o files. You want to do something like this instead hello.o: $(HEADERS) Makefile %.o : %.c

[bug #66011] dependencies after a non-existing file are silently ignored

Follow-up Comment #3, bug #66011 (group make): If you need such a prerequisite and the prerequisite is missing, then the makefile needs to have a rule to build such a prerequisite. Depending on your situation this rule can be either a proper rule or dummy rule like nonexisting.h:; In the end,

[bug #66037] An infinite loop with MAKEFLAGS on the command line.

: None ___ Follow-up Comments: --- Date: Sun 28 Jul 2024 11:27:44 PM UTC By: Dmitry Goncharov . ___ Reply to this i

[bug #66037] An infinite loop with MAKEFLAGS on the command line.

Additional Item Attachment, bug #66037 (group make): File name: sv66037_fix.diff Size: 4KiB File name: sv66037_test.diff Size: 2KiB

[bug #66037] An infinite loop with MAKEFLAGS on the command line.

Follow-up Comment #1, bug #66037 (group make): Make enters an infinite loop when some option and MAKEFLAGS= are specified on the command line. Sometimes rather than entering an infinite loop, make exits with an error message, because internal state of getopt is messed up. Here is an example of a

[bug #66073] $? is set incorrectly in the case of grouped targets.

: None ___ Follow-up Comments: --- Date: Sat 10 Aug 2024 02:15:45 AM UTC By: Dmitry Goncharov . ___ Reply to this i

[bug #66073] $? is set incorrectly in the case of grouped targets.

Follow-up Comment #1, bug #66073 (group make): A user reported an issue here https://lists.gnu.org/archive/html/help-make/2024-08/msg1.html. Here is a copy of that email which contains a good description of the issue. Hi. I have two similar Makefiles. [Makefile1] .PHONY: all all: foo.

Re: [bug #56484] [RFE] compile modified files first

On Wed, Jun 12, 2019 at 05:07:41PM -0400, anonymous wrote: > > Problem: if a project has 300 C files and a single header that's included > everywhere, and then someone changes 2 files: the header and any C file, most > likely one wants to know whether the changes caused build errors. However, > af

gmake-4.2.90 regression (error message missing)

Good morning. Certian implementations of posix_spawn return 0 on failure and then exit the child process with 127. I observed this with older glibc (2.12) and on aix. This results in gmake not printing related error message "no such file" or "permission denied" when the command is missing or not

gmake-4.2.90 still crashes on $(wildcard hello*/)

$ echo 'all: ; @echo $(wildcard hello*/)' > /tmp/1.mk $ ./make -f /tmp/1.mk Segmentation Fault A patch was provided here https://lists.gnu.org/archive/html/bug-make/2017-12/msg6.html Any chance to have this applied for 4.3? regards, Dmitry ___ Bug

gmake-4.2.90 patch

This patch fixes a compilation error when time.h does not get indirectly included. regards, Dmitry diff --git a/src/file.c b/src/file.c index c20fcf8..f58e855 100644 --- a/src/file.c +++ b/src/file.c @@ -17,6 +17,7 @@ this program. If not, see . */ #include "makei

gmake-4.2.90 regression (segmentation fault in sum_up_to_nul )

Good morning. sum_up_to_nul reads 4 bytes starting from the passed string 'p'. 'p' can have fewer than 4 bytes. Usually there more allocated space after 'p', which prevents this reading from manifesting itself. This reading manifests itself visibly when 'p' points to the end of the allocated block

Re: gmake-4.2.90 patch

_GL_UNUSED __attribute__ ((__unused__)) #else # define _GL_UNUSED #endif /* The name _UNUSED_PARAMETER_ is an earlier spelling, although the name is a misnomer outside of parameter lists. */ #define _UNUSED_PARAMETER_ _GL_UNUSED /* gcc supports the "unused" attribute on possibly unused

Re: gmake-4.2.90 regression (error message missing)

support cross compilation. regards, Dmitry On Mon, Sep 2, 2019 at 9:44 AM Paul Smith wrote: > > On Sun, 2019-09-01 at 15:33 -0400, Dmitry Goncharov via Bug reports and > discussion for GNU make wrote: > > This patch executes posix_spawnp at configuration time and then > > di

patch to fix features/archives test 10 on sunos

Good morning. ar add output is system dependent. This makes features/archives test 10 fail on sunos. This patch uses the output of `ar rv` rather than a hardcoded string. linux $ ar rv libxx.a 1234567890123456b a - 1234567890123456b sunos $ ar rv libxx.a 1234567890123456b a - 1234567890123456b a

patch to fix grouped_targets test 1

Good morning. echo -n is not portable. Certain implementations of echo everything, including -n. This makes test 1 of grouped_targets fail. regards, Dmitry diff --git a/tests/scripts/features/grouped_targets b/tests/scripts/features/grouped_targets index ef9366b..4d2c9d2 100644 --- a/tests/scrip

vpathplus test 2 patch

The new hashing mechanism hashes differently on big vs little endian. This makes vpathplus test 2 fail, because the order of intermediate files inter.b and inter.c is now dependent on endiannes. regards, Dmitry diff --git a/tests/scripts/features/vpathplus b/tests/scripts/features/vpathplus index

Re: vpathplus test 2 patch

On Wed, Sep 11, 2019 at 2:51 AM Andreas Schwab wrote: > > > The new hashing mechanism hashes differently on big vs little endian. > > Should this be considered as a bug? I don't think the order or removal of intermediate files was specified. Do you have a specific use case in mind, Andreas? rega

patch out of source builds

Good morning. This patch makes the test suite look for config.status in the -make_path directory to allow testing out of source builds. regards, Dmitry diff --git a/tests/README b/tests/README index 0cb7eac..5ecf392 100644 --- a/tests/README +++ b/tests/README @@ -82,12 +82,9 @@ ChangeLog in the

patch realpath on aix

aix implementation of realpath returns a path with a trailing slash given a path with 2 or more trailing slashes. This makes realpath test fail. $ echo 'all: ; @echo $(realpath /tmp/)' | ../i32/make -f- /tmp $ echo 'all: ; @echo $(realpath /tmp//)' | ../i32/make -f- /tmp/ Since both posix and gma

patch locale settings

Multiple tests fail when locale related env variables (such as LANG or LC_MESSAGES) are set to some regional setting. regards, Dmitry diff --git a/tests/run_make_tests b/tests/run_make_tests index b68b784..7b3719e 100755 --- a/tests/run_make_tests +++ b/tests/run_make_tests @@ -1,2 +1,3 @@ #!/bi

Re: patch out of source builds

On Sat, Sep 14, 2019 at 10:19:56AM +0300, Eli Zaretskii wrote: > > This patch makes the test suite look for config.status in the -make_path > > directory to allow testing out of source builds. > > Thanks, but I don't think relying on config_status is good enough, > because some builds don't have i

Re: patch locale settings

On Sat, Sep 14, 2019 at 10:22 AM Paul Smith wrote: > and, I've checked with a makefile invoked by the test suite and this > setting is in effect, even if the shell invoking run_make_tests.pl has > a different setting for LC_ALL. This should be sufficient. > Furthermore, we clear out the environm

Re: patch locale settings

On Sat, Sep 14, 2019 at 6:27 PM Paul Smith wrote: > Your setting of LANG or LC_MESSAGES > cannot make any difference to the test suite because the test suite > does not pass those values to the make that it invokes. Paul, that's the output from the perl program (called base) that is affected by t

Re: patch out of source builds

On Sat, Sep 14, 2019 at 6:39 PM Paul Smith wrote: > Rather, the above comment refers to testing like this: ... > That is, running the tests from within the remote build directory > "tests", not from within the source directory "tests". Thank you. This indeed clarifies it. The purpose of the patch

Re: patch locale settings

On Sun, Sep 15, 2019 at 08:31:39AM -0400, Paul Smith wrote: > That's because before we try to detect these messages we explicitly set > the locale in Perl; see test_driver.pl: > > my $loc = undef; > if ($has_POSIX) { > $loc = POSIX::setlocale(POSIX::LC_MESSAGES); > POSIX::setlocale

Re: patch locale settings

On Sun, Sep 15, 2019 at 10:14:07AM -0400, Paul Smith wrote: > I see. I wonder why it works for me. i was thinking your system couldn't do anything other than the default, but your german test demonstrates it can. > > > Let me come up with a fix for 'perl run_make_tests.pl'. if you set use stric

patch, test suite fails to run valgrind

Good morning. The test suite fails to run valgrind with an error message like the following. $ ./run_make_tests -memcheck -make_path ../l64/make features/archives ... ok (12 passed) features/comments ... *** Test died (

Re: patch, test suite fails to run valgrind

On Sat, Sep 21, 2019 at 04:54:59PM -0400, Paul Smith wrote: > I pushed a different fix for this issue. Thanks for noticing! Thank you. Do you intentionally avoid $! in the "Cannot execute" error message? regards, Dmitry > ___ Bug-make mailing list Bu

Re: gmake-4.2.90 regression (segmentation fault in sum_up_to_nul )

:01 PM Paul Smith wrote: > > On Tue, 2019-09-03 at 04:14 +, Dmitry Goncharov via Bug reports and > discussion for GNU make wrote: > > sum_up_to_nul reads 4 bytes starting from the passed string 'p'. > > 'p' can have fewer than 4 bytes. Usually there

Re: gmake-4.2.90 regression (segmentation fault in sum_up_to_nul )

On Tue, Sep 24, 2019 at 1:01 PM Paul Smith wrote: > The reason for the "special" code here is > performance, and unfortunately the solution proposed will reduce > performance by a measurable amount (not huge but measurable). Paul, is this call to strlen that you are concerned with? It is possible

Re: [bug #57014] make-4.2.91 segfaults under Solaris 10 when many files are involved

On Mon, Oct 7, 2019 at 9:03 AM anonymous wrote: > With the following Makefile, make-4.2.91 segfaults on my Solaris 10: ... > include /dev/null > dummy: subdir/*.c > include /dev/null This reproduces for me. This is the same read past the end of the string inside sum_up_to_nul that is already fixe

Re: [bug #57014] make-4.2.91 segfaults under Solaris 10 when many files are involved

On Mon, Oct 7, 2019 at 11:33 AM Paul Smith wrote: > Unfortunately it's not easy to build from Git unless you have a full > suite of autotools available. Denis, if you cannot built from git, can you enable core dumps and run dbx and post the backtrace here? This is what is look like for me $ dbx

Re: [PATCH] Port functions/shell to Solaris 10

On Fri, Oct 11, 2019 at 4:25 PM Paul Eggert wrote: > One possibility is that the version of Perl shipped with Solaris is so old > that it predates this Perl standardization. It is a bit of a mess. Actually this patch fixes the same issue on aix with perl-5.10.1 regards, Dmitry _

patch, new test features/exec.

Good morning. This patch adds a new test features/exec. features/exec tests that make can execute binaries as well as scripts with various shabangs and without a shebang and also with various values of SHELL=. The test demonstrates that the current (4.2.92) implementation fails to execute a shell

make-4.2.93 patch fix compilation in -ansi/-std=c99 mode

Good morning. job.c fails to compile on glibc when -ansi or -std=c99 is specified. ../src/job.c: In function 'reap_children': ../src/job.c:753: error: incompatible type for argument 1 of 'wait' /usr/include/sys/wait.h:116: note: expected '__WAIT_STATUS' but argument is of type 'int *' This happen

make-4.2.93 patch enhance error reporting from the test suite

Good morning. This patch enhances error reporting from the test suite. E.g. Clearing work... cannot unlink work/features/.nfs00c262d501ec: Device or resource busy cannot unlink work/features: Is a directory ./run_make_tests.pl: 697: Couldn't wipe out work: Is a directory regards, Dmi

make-4.2.93 patch port to c90

Good morning. This patch replaced a c99 piece of code with c90 code. This c99 piece of code does not compile with the default ./configure && make. Also, -std=c99 removes __attribute__. regards, Dmitry diff --git a/src/file.c b/src/file.c index 2f1425e..acc8c0c 100644 --- a/src/file.c +++ b/src/

Re: patch, new test features/exec.

On Fri, Jan 3, 2020 at 12:10 PM Paul Smith wrote: > On Mon, 2019-10-14 at 22:21 -0400, Dmitry Goncharov via Bug reports and > discussion for GNU make wrote: > > This patch adds a new test features/exec. > > I added this test with a few adjustments. Thank you. i am glad to

Re: make-4.2.93 patch fix compilation in -ansi/-std=c99 mode

On Sat, Jan 4, 2020 at 3:23 PM Paul Smith wrote: > Thanks, I did something similar to this but not quite the same. Have you pushed? > It's a bit strange because I don't see that same error on my system and in > fact, I don't have that code in my sys/wait.h file; I'm using GNU libc > 2.30. Maybe

Re: Phony targets not being made

Good morning. Your makefile is incorrect. Make does not search implicit rules for phony targets. regards, Dmitry On Thu, Feb 13, 2020 at 2:31 PM wrote: > > Hi > > I have the following makefile > -- > PHON1 := rr1 rr2 > PHON2 := rr1-sim > > .PHONY: all $(PHON1) > # remove below line to get rr1-s

Re: Phony targets not being made

On Sun, Feb 16, 2020 at 5:35 PM Peter Dons Tychsen wrote: > There are no implicit rules used in the example AFAIK. '%-sim: %' is the intended implicit rule that make does not look for because rr1-sim is phony. > They are all phony, but not implicit unless i am missing something. Targets could b

Re: Defining and exporting and a variable undefined in a foreach loop cause a Segmentation fault with Make 4.2.1 on Debian bullseye/sid x64

On Thu, Mar 5, 2020 at 10:45 AM Xavier Sanchez wrote: > > Hi, I've got a segfault in __strlen_avx2 () at > ../sysdeps/x86_64/multiarch/strlen-avx2.S:65 > > While defining a variable that has been undefined in a foreach loop using Make > 4.2.1 on my Bullseye/sid workstation. Does not reproduce for

Re: make-4.3: wildcard test #9 fails

> On Sun, 2020-04-05 at 16:19 +0200, Bruno Haible wrote: > > > > Building GNU make 4.3 on Ubuntu 16.04, produces one failing test: On filesystems which don't populate dirent::d_type (or on old glibc) test 7 also fails. I just asked gnulib mailing list to look at a patch. https://lists.gnu.org/arch

Re: simple assignment of target-specific variables

On Tue, Jun 9, 2020 at 9:10 AM David Boyce wrote: > TARGETS := aa bb cc dd > $(TARGETS): at := $(shell set -x; date) Despite the same name "at" those are different variables. Given that there are 4 simple variables defined, 4 calls to date take place. $ cat makefile .PHONY: all all: aa bb aa: at

Re: simple assignment of target-specific variables

On Tue, Jun 9, 2020 at 9:46 AM Edward Welbourne wrote: > I presume the workaround is > > TGTDATE := $(shell date) > $(TARGETS): DATE := $(TGTDATE) > > at the slightly annoying price of calling the command once even if > making some target *not* in the TARGETS list. It is possible to avoid this pr

Re: simple assignment of target-specific variables

On Tue, Jun 9, 2020 at 12:01 PM David Boyce wrote: > When I say "needed" here I mean specifically that the recipe runs. Clearly a > target-specific variable is 'needed' iff the target is built. It's not a > question of how many places the variable is expanded, it's a boolean thing: > the target

Re: simple assignment of target-specific variables

On Tue, Jun 09, 2020 at 09:36:43AM -0700, David Boyce wrote: > On Tue, Jun 9, 2020 at 9:21 AM Dmitry Goncharov > wrote: > > You mean for target specific simple variable memory is allocated and > > the variable is initialized only if the target is built, right? > Yes, that&#

Re: make4.3/make4.2.1 - solaris 11 x86 issues

On Mon, Jun 29, 2020 at 11:20 AM Paul Smith wrote: > Unfortunately, Dmitry has been so active with help that I'm not able to > accept any more of his patches until/unless some sort of copyright > assignment is executed. That process was definitely begun but I > haven't heard anything about it for

Re: export vs $(origin )

On Thu, Jul 2, 2020 at 8:49 AM Jan Beulich wrote: > is it really intended for "export" > without any assignment done at the same time to change the origin of a > previously undefined variable from "undefined" to "file"? "export" without any assignment done at the same time sets the origin to "env

Re: export vs $(origin )

On Thu, Jul 2, 2020 at 11:22 AM Jan Beulich wrote: > Not according to my observations. You are right. regards, Dmitry

Re: GNU Make 4.3: Makefile rule spooky action at a distance

On Sun, Oct 4, 2020 at 9:32 AM Danny Milosavljevic wrote: > I just got a build failure in lz4. I've isolated it and made a minimal test > case, see below. ... I opened https://savannah.gnu.org/bugs/index.php?59230 and attached 2 patches which fix this issue that you observed and a similar issue

Re: embedded newlines in shell function variable expansion

On Fri, Oct 9, 2020 at 11:11 AM Byrnes, Robert wrote: > > If I use this Makefile ... > > > bash$ cat Makefile > FOO := $(shell echo $(ENTRIES) ; ) > BAR := $(shell echo $(ENTRIES) ) > > all: > @echo FOO = $(FOO) > @echo BAR = $(BAR) > > .PHONY: all >

Re: [bug #59399] Expanding implicit macros in gmake (enhancement request)

On Sun, Nov 29, 2020 at 6:02 PM Jörg Schilling wrote: > I just verified that expanding $< and $* for explicit rules at all > was an invention from SunPro Make. These macros have not been > expanded by the classical UNIX make. The seminal make manual of 1978 contains the following "Before issuin

Re: AW: Issue with VPN

> Question. Is there a change to optimize the function cp and rm? In bash (in recipe lines) you can replace cp with echo "$(dst in make you can replace cp with $(file >dst,$(file

Re: gmake and ccache conspiring together in creating gremlins

On Mon, Feb 8, 2021 at 12:51 PM Dmitry Goncharov wrote: > > On Mon, Feb 8, 2021 at 12:36 PM Edward Welbourne > wrote: > > Sounds to me like that's a bug: when the descriptors are closed, the > > part of MAKEFLAGS that claims they're make's jobserver file desc

Re: gmake and ccache conspiring together in creating gremlins

On Mon, Feb 8, 2021 at 12:36 PM Edward Welbourne wrote: > Sounds to me like that's a bug: when the descriptors are closed, the > part of MAKEFLAGS that claims they're make's jobserver file descriptors > should be removed, since that's when the claim stops being true. make uses posix_spawn by defa

Re: gmake and ccache conspiring together in creating gremlins

On Tue, Feb 9, 2021 at 5:31 AM Edward Welbourne wrote: > Rather than removing the jobserver-auth data, you could amend the > MAKEFLAGS to includ jobserver-auth data with plainly invalid fds, i like jobserver-auth data with plainly invalid fds, because it lets older binaries fail on parsing jobse

Re: Intermediate file removal ... considered harmful??

On Wed, Mar 31, 2021 at 5:39 PM Paul Smith wrote: > Third, there's not a flexible way to disable intermediate files: agree. > I think it is still useful to keep intermediate files as a concept, > where if an intermediate file in a chain of implicit rules doesn't > exist but the mod time of the s

Re: .SECONDARY: unwanted effect of not causing rebuilds when intermediate files missing

On Mon, Apr 5, 2021 at 7:28 PM Britton Kerin wrote: > Even if this behavior of .SECONDARY: is considered desirable Whether or not it is desirable, it is there and there are makefiles which use it and a change would break them. >, it would > be nice to have some way of telling make what I want t

Re: Manual: Multiple outputs

On Mon, May 3, 2021 at 8:57 AM Paul Smith wrote: > data.h: data.c ; > > (note the extra semicolon). Now it will work. Ofcourse, when data.c is present and data.h is missing, then make will not recreate data.h. It'll run the empty rule. Your original makefile data.h data.c: data.foo to

Re: Keeping intermediate files

On Thu, May 20, 2021 at 4:29 PM Frank Heckenbach wrote: >- just ".SECONDARY:" with no prerequisites which makes all > targets secondary (like I want), but also makes all targets > immediate, e.g. with this Makefile, if y exists and x doesn't, > it will remake x, but not y, whic

Re: [bug #60798] Make does not compile with GCC 11.1.0

On Sat, Jun 19, 2021 at 10:31 AM Dmitrii Pasechnik wrote: > It's undefined behaviour in C to point to such a location, isn't it? > Is this hack really needed? There is no hack. It is pointer arithmetic in c. p[-1] is the same as *(p-1). regards, Dmitry

Re: [bug #60841] misleading statement in the manual

On Mon, Jun 28, 2021 at 10:23 AM Andreas Schwab wrote: > > "When make runs a recipe, variables defined in the makefile are placed into > > the environment of each shell." > > Please also read the rest of the paragraph. This statement should say "variables defined in the makefile can be placed..."

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Saturday, July 24, 2021, Eli Zaretskii wrote: > Follow-up Comment #5, bug #60774 (project make): > > The MS-Windows port of GNU Make doesn't lock stdout (or any other standard > device). Instead, it uses a mutex to synchronize output. So I think this > problem cannot happen on Windows. Tru

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Saturday, July 24, 2021, David Boyce wrote: > Wouldn’t a less intrusive solution be to check the inode/device of stdout > and if it’s the same as that of /dev/null just forego locking? That would help this specific case indeed. What about a case when make's stdout is redirected to another, n

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Sat, Jul 24, 2021 at 3:02 PM David Boyce wrote: > The point is that only /dev/null *should* be shared. Any other file *might* > be locked by an unrelated process but that would represent an error condition > in which incorrect results should be expected. i think, it is less likely that an u

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

Wanted to test that windows this still works with this change. So, i followed instructions in README.W32. Installed mingw gcc. Copied config.h.W32.template to config.h.w32. Ran build_W32.bat gcc. It fails with C:\dgoncharov\make>gcc -mthreads -Wall -std=gnu99 -gdwarf-2 -g3 -O2 -DMAKE_MAINT AINER_

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Sun, Jul 25, 2021 at 3:49 AM Eli Zaretskii wrote: > > Follow-up Comment #8, bug #60774 (project make): > > Why did you remove the code which reused the mutex passed from the parent > Make? Eli, i don't think i removed any code, other than the global variable mutex_handle. Atleast, not intentio

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Sun, Jul 25, 2021 at 10:03 AM Paul Smith wrote: > Writing to > things like /dev/lp0 SHOULD be locked, IMO: you don't want multiple > concurrent writers to your printers or your terminal! The user intentionally redirects make to a printer along with another tool. Should make be smarter than the

Re: [bug #60774] make hangs on fcntl lock when using -O and stdout is /dev/null

On Sun, Jul 25, 2021 at 3:25 PM Paul Smith wrote: > I have no problem with make refusing to write to a device that's locked > in general. If users don't want that to happen, they can just not use > the -O option. True, as long as the user knows that when -O is specified make is going to lock std

Re: Make language

On Tue, Oct 12, 2021 at 10:38 AM Bartol Hrg wrote: > > Greetings > > I installed make 3.81 for Windows > > It's in d*mn Croatian! > > Now, I don't know if you noticed, but I speak English. > > I'm really displeased and agitated. > > Best regards > B. Helium Dobar dan. Vaš pošta je na Engleskom je

Re: [bug #61226] A regression prevents generation of missing included dependency files.

On Sun, Oct 17, 2021 at 3:05 PM Britton Kerin wrote: > If I understand right that the idea is a special case for .d files The question being discussed is what to do when make cannot include a makefile. regards, Dmitry

Re: -V, --verbose, as opposite of -s, --silent, --quiet

On Fri, Oct 22, 2021 at 9:02 PM Alejandro Colomar (man-pages) wrote: > The output would be something like: > > CC foo.o > CC bar.o > ... > > But then, if something goes wrong (e.g., bar.c stops compiling into > bar.o), I'd like to be able to debug the build by showing the full command.

Re: [bug #61226] A regression prevents generation of missing included dependency files.

On Mon, Oct 25, 2021 at 6:12 AM Edward Welbourne wrote: > Surely that would solve your problem, without invasive surgery in GNU > make, or special-case handling of .d files anywhere but your make file ? Actually, my makefiles do not suffer from this. i use the technique described here https://g

Re: [bug #61226] A regression prevents generation of missing included dependency files.

On Sun, Oct 17, 2021 at 11:17 PM Britton Kerin wrote: > I got that but it sounds like the proposal is for it to do one thing > if the file is a .d and otherwise something else? The proposal is to see if the recipe is empty. There is this statement in the manual "If a rule has no prerequisites or

Re: [bug #61226] A regression prevents generation of missing included dependency files.

On Monday, October 25, 2021, Alejandro Colomar (man-pages) < alx.manpa...@gmail.com> wrote: > > > Why do I do this? Because, if you remove a file from your tree, an old .d > file will require that file and make your build fail, requiring you to > clean before making again. > > This has been solve

Re: [bug #61226] A regression prevents generation of missing included dependency files.

On Monday, October 25, 2021, Alejandro Colomar (man-pages) < alx.manpa...@gmail.com> wrote: > However, there's a downside to this: The payment for this robustness is a > non-negligible time cost. The single-process time for a no-op make is > around 10 s > > 10s is about 100 to 1000 times long

<    2   3   4   5   6   7