Re: Make 4.3.90 breaks makefile of dtc

2022-09-23 Thread Dmitry Goncharov
On Thu, Sep 22, 2022 at 4:39 AM Andreas Schwab wrote: > There is some misbehaviour that creates an infinite loop trying to run > the bison rule. Interestingly, this eventually runs out of file > descriptors The latest make from master (same as 4.3.90) builds dtc-1.6.1 successfully for me with

Re: Make 4.3.90 breaks makefile of dtc

2022-09-22 Thread Dmitry Goncharov
On Thu, Sep 22, 2022 at 4:39 AM Andreas Schwab wrote: ... > Interestingly, this eventually runs out of file > descriptors, because, due to -O, make keeps opening new pipes for > synchronizing the output without closing any. Without -O, this would > probably go on forever. Make used to synchroniz

[bug #63070] posix_spawn fails to run a child process.

2022-09-20 Thread Dmitry Goncharov
Follow-up Comment #13, bug #63070 (project make): There is nothing to be sorry about, Martin. Thank you for your work, keep contributing. ___ Reply to this item at: ___

[bug #63070] posix_spawn fails to run a child process.

2022-09-20 Thread Dmitry Goncharov
Follow-up Comment #10, bug #63070 (project make): In glibc-2.17 posix_spawn returns 0 whether POSIX_SPAWN_USEVFORK is set or not. When POSIX_SPAWN_USEVFORK is not set posix_spawn returns 0 and errno is 0. When POSIX_SPAWN_USEVFORK is set posix_spawn returns 0 and errno is ENOEXEC. However, today

[bug #63070] posix_spawn fails to run a child process.

2022-09-20 Thread Dmitry Goncharov
Follow-up Comment #11, bug #63070 (project make): Patch 3 tested on glibc-2.17, glibc-2.32 and sun os. ___ Reply to this item at: ___ Message sent via S

[bug #63070] posix_spawn fails to run a child process.

2022-09-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #63070 (project make): File name: sv63070_fix3.diff Size:1 KB ___ Reply to this item at:

[bug #63070] posix_spawn fails to run a child process.

2022-09-18 Thread Dmitry Goncharov
Follow-up Comment #8, bug #63070 (project make): However, maybe we should also define _GNU_SOURCE in the configure check before including #include . Something like index ec8b4c13..38418142 100644 --- a/configure.ac +++ b/configure.ac @@ -369,6 +369,7 @@ AS_IF([test "$make_cv_posix_spawn" = y

[bug #63070] posix_spawn fails to run a child process.

2022-09-18 Thread Dmitry Goncharov
Follow-up Comment #7, bug #63070 (project make): Let me describe differently. posix_spawn calls either vfork or clone, depending on arguments being passed. See the man page referenced above. I observed that when posix_spawn calls vfork, that's is when POSIX_SPAWN_USEVFORK is set in flags, errno i

[bug #63070] posix_spawn fails to run a child process.

2022-09-18 Thread Dmitry Goncharov
Additional Item Attachment, bug #63070 (project make): File name: sv63070_fix2.diff Size:1 KB ___ Reply to this item at:

[bug #63070] posix_spawn fails to run a child process.

2022-09-18 Thread Dmitry Goncharov
Follow-up Comment #5, bug #63070 (project make): posix_spawn calls either vfork of spawn. On glibc-2.17 i observed that when posix_spawn calls vfork the return code is zero, but errno is set to ENOEXEC. When posix_spawn calls spawn, both return code and errno are zero. This man page https://man7.

[bug #63071] Race condition in .WAIT tests.

2022-09-16 Thread Dmitry Goncharov
Additional Item Attachment, bug #63071 (project make): File name: sv63071_fix.diff Size:1 KB ___ Reply to this item at:

[bug #63071] Race condition in .WAIT tests.

2022-09-16 Thread Dmitry Goncharov
Follow-up Comment #1, bug #63071 (project make): Test 11 of targets/WAIT fails due to a race. $ cat tests/work/targets/WAIT.diff.11 *** work/targets/WAIT.base.11 Wed Sep 14 18:37:08 2022 --- work/targets/WAIT.log.11Wed Sep 14 18:37:08 2022 *** *** 1,5 - start-pre1 pr

[bug #63071] Race condition in .WAIT tests.

2022-09-16 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Sat 17 Sep 2022 01:31:48 AM UTC By: Dmitry Goncharov . ___ Reply to this item at: <https://savanna

[bug #63070] posix_spawn fails to run a child process.

2022-09-16 Thread Dmitry Goncharov
Additional Item Attachment, bug #63070 (project make): File name: sv63070_fix.diff Size:1 KB ___ Reply to this item at:

[bug #63070] posix_spawn fails to run a child process.

2022-09-16 Thread Dmitry Goncharov
Follow-up Comment #1, bug #63070 (project make): posix_spawn fails to run a child process. Some versions of glibc (i observed this with glibc-2.17) fail to spawn a shell program missing a shbang and return 0 from posix_spawn. $ ls hello.sh makefile $ cat hello.sh printf "hello, world\n" $

[bug #63070] posix_spawn fails to run a child process.

2022-09-16 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Sat 17 Sep 2022 01:23:19 AM UTC By: Dmitry Goncharov . ___ Reply to this item at:

[bug #63045] Make crashes when makefile keeps the loaded shared object intact.

2022-09-10 Thread Dmitry Goncharov
Additional Item Attachment, bug #63045 (project make): File name: sv63045_fix.diff Size:11 KB File name: sv63045_test.diff Size:4 KB

[bug #63045] Make crashes when makefile keeps the loaded shared object intact.

2022-09-10 Thread Dmitry Goncharov
Follow-up Comment #1, bug #63045 (project make): Make crashes when makefile keeps the loaded shared object intact. In this makefile make loads shared object hello.so, then unloads it in order to rebuild. The rule provided by the makefile keeps the shared object intact. Make does not re-execute it

[bug #63045] Make crashes when makefile keeps the loaded shared object intact.

2022-09-10 Thread Dmitry Goncharov
: None Triage Status: None ___ Follow-up Comments: --- Date: Sun 11 Sep 2022 03:13:13 AM UTC By: Dmitry Goncharov . ___ Re

[bug #63044] Make fails to update .LOADED when the setup routine returns -1.

2022-09-10 Thread Dmitry Goncharov
Additional Item Attachment, bug #63044 (project make): File name: sv63044_fix.diff Size:0 KB File name: sv63044_test.diff Size:1 KB

[bug #63044] Make fails to update .LOADED when the setup routine returns -1.

2022-09-10 Thread Dmitry Goncharov
Follow-up Comment #1, bug #63044 (project make): Make fails to add a loaded shared object to .LOADED when the shared object setup routine returns -1. $ ls makefile timer2.c $ cat timer2.c int plugin_is_GPL_compatible; int timer2_gmk_setup (void) { return -1; } $ gcc -o timer2.so -shar

[bug #63044] Make fails to update .LOADED when the setup routine returns -1.

2022-09-10 Thread Dmitry Goncharov
: None Triage Status: None ___ Follow-up Comments: --- Date: Sun 11 Sep 2022 01:46:28 AM UTC By: Dmitry Goncharov . ___ Re

Re: New release of GNU make

2022-09-04 Thread Dmitry Goncharov
On Sun, Sep 4, 2022 at 9:53 AM Paul Smith wrote: > > On Sun, 2022-09-04 at 00:28 +, Martin Dorey wrote: > > https://git.savannah.gnu.org/cgit/make.git/commit/configure.ac?id=079 > > 3658c09a8f33581dae6dfbe2483ea279e72b1 > > > > ... imposed a dependency on autoconf 2.71. > > I'm not sure if the

[bug #62840] make --version in pipe return SIGPIPE

2022-08-29 Thread Dmitry Goncharov
Follow-up Comment #4, bug #62840 (project make): Patch sv62840_fix.diff sets stdout to line buffering after --version and --help are processed, but before make reads makefiles. ___ Reply to this item at:

[bug #62840] make --version in pipe return SIGPIPE

2022-08-29 Thread Dmitry Goncharov
Additional Item Attachment, bug #62840 (project make): File name: sv62840_fix.diff Size:2 KB ___ Reply to this item at:

[bug #62929] Normalize foo/./bar

2022-08-22 Thread Dmitry Goncharov
Follow-up Comment #4, bug #62929 (project make): Given makefile foo/bar:; @echo $@ and target foo/./bar this changeset sets $@ to foo/bar. To proceed with this, changes may be needed in addition to the one in the attachment. 1. same normalization in lookup_file and in main 2. ability to match fo

[bug #62929] Normalize foo/./bar

2022-08-22 Thread Dmitry Goncharov
Follow-up Comment #3, bug #62929 (project make): pmake behaves the same as bmake. ___ Reply to this item at: ___ Message sent via Savannah https://savan

[bug #62929] Normalize foo/./bar

2022-08-22 Thread Dmitry Goncharov
Follow-up Comment #2, bug #62929 (project make): i compared gmake, bmake, sun make, sun dmake, sun make in svr4 mode, aix make. Given makefile hello: @echo $@ gmake treats ./hello, ././hello, ././/hello or .//hello, as hello. $@ has value hello. sun make, sun dmake, sun svr4 make treat ./hel

Re: False positive "doesn't match the target pattern" error

2022-08-20 Thread Dmitry Goncharov
On Sat, Aug 20, 2022 at 8:28 PM Masahiro Yamada wrote: > build-dirs := . drivers sound net virt arch/x86/pci arch/x86/power lib > arch/x86/lib > subdir-modorder := $(addsuffix /.modules.order, $(build-dirs)) > $(sort $(subdir-modorder)): %/.modules.order: % Can you remove . from build-dirs and ad

[bug #62929] Normalize foo/./bar

2022-08-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #62929 (project make): File name: sv62929_fix.diff Size:3 KB File name: sv62929_test.diff Size:1 KB

[bug #62929] Normalize foo/./bar

2022-08-20 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62929 (project make): Make fails to normalize foo/./bar as foo/bar. The original bug report is here https://lists.gnu.org/archive/html/bug-make/2022-08/msg00064.html ___ Reply to this item at:

[bug #62929] Normalize foo/./bar

2022-08-20 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Sun 21 Aug 2022 03:38:36 AM UTC By: Dmitry Goncharov . ___ Reply to this item at: <https://savannah.gnu

Re: False positive "doesn't match the target pattern" error

2022-08-20 Thread Dmitry Goncharov
On Sat, Aug 20, 2022 at 11:36 AM Alejandro Colomar wrote: > Why does make(1) need to special-case a leading ./ ? If your makefile has a rule like foo.x: foo.z; cp $< $@ then make foo.x and make ./foo.x both produce the same file. regards, Dmitry

Re: False positive "doesn't match the target pattern" error

2022-08-20 Thread Dmitry Goncharov
On Sat, Aug 20, 2022 at 5:52 AM Masahiro Yamada wrote: > I appreciate GNU Make normalize the path > by removing "./" > > This is helpful in some cases, but I think it is a bad side-effect > in this case. Is there a reason to treat './foo.x' as different from 'foo.x'? > If this is a bug, I can f

[bug #62908] Regression. Double free if jobserver is enabled, but cannot open fifo.

2022-08-17 Thread Dmitry Goncharov
Additional Item Attachment, bug #62908 (project make): File name: sv62908_fix.diff Size:0 KB File name: sv62908_test.diff Size:1 KB

[bug #62908] Regression. Double free if jobserver is enabled, but cannot open fifo.

2022-08-17 Thread Dmitry Goncharov
: None Triage Status: None ___ Follow-up Comments: --- Date: Wed 17 Aug 2022 11:45:30 PM UTC By: Dmitry Goncharov Failure to open a fifo causes make to double free fif

Re: Some "fun" last weekend

2022-08-15 Thread Dmitry Goncharov
On Thu, Jul 21, 2022 at 10:23 AM Paul Smith wrote: > Instead, any sub-process can look at MAKEFLAGS, see the value of jobserver-auth, find the name of the pipe, open it, and start to use it. If the sub-process doesn't know anything about MAKEFLAGS, it will never know that the jobserver is relevan

Re: Macro arguments

2022-08-13 Thread Dmitry Goncharov
On Tue, Aug 9, 2022 at 11:56 PM Paul Smith wrote: > ...modifying the way all function arguments > are parsed is something that needs to be carefully considered, if > nothing else from a backward-compatibility standpoint. agree. i added an example to the manual of how to use variable to hide spec

[bug #62881] parentheses confuse make parser

2022-08-09 Thread Dmitry Goncharov
Additional Item Attachment, bug #62881 (project make): File name: sv62881_test.diff Size:1 KB File name: sv62881_fix.diff Size:0 KB

Re: Macro arguments

2022-08-09 Thread Dmitry Goncharov
On Tue, Aug 9, 2022 at 10:48 AM Gisle Vanem wrote: Thanks for your report. See here https://savannah.gnu.org/bugs/index.php?62881 regards, Dmitry

[bug #62881] parentheses confuse make parser

2022-08-09 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62881 (project make): A user reported the following here https://lists.gnu.org/archive/html/bug-make/2022-08/msg00017.html in GNU-make macros. Like in: msg = @echo "$(1)" ptest_1: $(call msg, foo (arg1, arg2, arg3)) ... A 'ma

[bug #62881] parentheses confuse make parser

2022-08-09 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Tue 09 Aug 2022 07:02:50 PM UTC By: Dmitry Goncharov . ___ Reply to this item at:

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-23 Thread Dmitry Goncharov
Follow-up Comment #5, bug #62706 (project make): sv62706_fix2.diff takes care of secondary expanding a prerequisite marked as intermediate by .SECONDARY or .INTERMEDIATE when an explicit or static pattern rule builds the prerequisite. sv62706_test2.diff adds related tests. __

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-23 Thread Dmitry Goncharov
Additional Item Attachment, bug #62706 (project make): File name: sv62706_fix2.diff Size:5 KB File name: sv62706_test2.diff Size:15 KB

Re: New feature: Post-requisites

2022-07-22 Thread Dmitry Goncharov
On Fri, Jul 22, 2022 at 9:34 AM wrote: > In your example, wouldn't the following accomplish the same thing? > > > update-mandb: $(DESTDIR)$(man3dir)/foo.3 $(DESTDIR)$(man3dir)/bar.3 i'd also be interested in the answer. Either this traditional syntax specified above, or $(DESTDIR)$(man3dir)/bar

Re: Some "fun" last weekend

2022-07-20 Thread Dmitry Goncharov
On Wed, Jul 20, 2022 at 9:33 AM Paul Smith wrote: > There are so many ways this can bite people. The last thing > GNU make needs IMO is more obscure subtleties. For example if someone > writes this: > > NRMAKE = $(MAKE) > > recurse: > MAKEFLAGS=$(MAKEFLAGS) $(NRMAKE) > > it will br

Re: Some "fun" last weekend

2022-07-19 Thread Dmitry Goncharov
On Tue, Jul 19, 2022 at 6:13 PM Paul Smith wrote: > There are at least two Savannah bugs about this very subtle and > annoying issue. Do you mean that jobserver-auth is present in the child env, even though the fds are closed? > So the first thing I tried to do was to add another setting of the

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-09 Thread Dmitry Goncharov
Follow-up Comment #3, bug #62706 (project make): Another thought on point 2 above. Posix contains the following requirement. "The make utility shall treat all prerequisites as targets themselves and recursively ensure that they are up-to-date, processing them in the order in which they appear in

[bug #62734] Have the test suite set ASAN env to ignore leaks.

2022-07-09 Thread Dmitry Goncharov
Additional Item Attachment, bug #62734 (project make): File name: sv62734.diff Size:0 KB ___ Reply to this item at:

[bug #62734] Have the test suite set ASAN env to ignore leaks.

2022-07-09 Thread Dmitry Goncharov
Status: None ___ Follow-up Comments: --- Date: Sat 09 Jul 2022 06:57:40 PM UTC By: Dmitry Goncharov There are known memory leaks in make. When make is build with ASAN checks enabled, those

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-05 Thread Dmitry Goncharov
Follow-up Comment #2, bug #62706 (project make): This is a more detailed description of point 2 above (2. Causes all prerequisites to be second expanded in the same order they are being built.) $ cat makefile .SECONDEXPANSION: all: hello.tsk bye.tsk hello.tsk: $$(info 2nd expansion of prereq

Re: [bug #62706] Restrict second expansion to targets which are being built.

2022-07-05 Thread Dmitry Goncharov
On Tue, Jul 5, 2022 at 12:34 PM Edward Welbourne wrote: > was thinking of the fact that, if you'd started with > > all: $$(info 2nd expansion for all) hello.tsk bye.tsk; echo built both > > then its info would appear first, although its rule would be run last. That's correct. $$(info 2nd expansio

Re: [bug #62706] Restrict second expansion to targets which are being built.

2022-07-05 Thread Dmitry Goncharov
On Tue, Jul 5, 2022 at 5:19 AM Edward Welbourne wrote: > I would have expected the order to be the exact reverse of the > order of building: i guess, an example will make it more clear. $ cat makefile .SECONDEXPANSION: all: hello.tsk bye.tsk hello.tsk: $$(info 2nd expansion of prereqs of $$@);

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-04 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62706 (project make): Second expand only the prerequisites of the targets being built. Avoid second expanding the prerequisites of targets which are not being built. There are multiple benefits. 1. Avoids redundant work of second expanding prerequisites of the targets

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-04 Thread Dmitry Goncharov
Additional Item Attachment, bug #62706 (project make): File name: sv62706_doc.diff Size:4 KB File name: sv62706_test.diff Size:14 KB

[bug #62706] Restrict second expansion to targets which are being built.

2022-07-04 Thread Dmitry Goncharov
Triage Status: None ___ Follow-up Comments: --- Date: Mon 04 Jul 2022 11:19:17 PM UTC By: Dmitry Goncharov . ___ Reply t

Re: Goodbye to GNU make's "build.sh" ... ?

2022-06-26 Thread Dmitry Goncharov
On Sun, Jun 26, 2022 at 5:48 PM Paul Smith wrote: > Seems like it's not so much that the patch was rejected; indeed they > seem to agree there's a bug. But they wanted more work on it and it > sort of fell apart. I have no idea how much effort it would be to push > hard enough to get a change me

Re: Goodbye to GNU make's "build.sh" ... ?

2022-06-26 Thread Dmitry Goncharov
On Sunday, June 26, 2022, Paul Smith wrote > > > I would prefer to have that fixed in gnulib, than continue to maintain > a local copy just to avoid this issue. Then there is no choice to drop gnulib and build.sh should go. Since you asked for opinions, my opinion is that maintaining a mini make

Re: Goodbye to GNU make's "build.sh" ... ?

2022-06-26 Thread Dmitry Goncharov
On Sun, Jun 26, 2022 at 10:11 AM Paul Smith wrote: > This really came up because I was trying to find a way to include the > latest gnulib globbing library. Adding "glob" to this pulls in a HUGE > number of prerequisites. Have you considered to to avoid glob from gnulib? Make has its one impl of

[bug #62650] Fix an incorrect user message.

2022-06-18 Thread Dmitry Goncharov
Additional Item Attachment, bug #62650 (project make): File name: sv62650_fix.diff Size:1 KB File name: sv62650_test.diff Size:2 KB

[bug #62650] Fix an incorrect user message.

2022-06-18 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Sat 18 Jun 2022 10:13:59 PM UTC By: Dmitry Goncharov Make reports an incorrect message when a file found through directory search is preferred over the explicitly men

[bug #62615] Fix the leaks of file->cmds and file->cmds->commands

2022-06-15 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62615 (project make): Paul, can you please ignore this patch? This bug report is correct, there is a memory leak. But the patch is not correct. The patch fails to take into account a case when multiple targets are defined on the same rule. Correctly taking care of freein

[bug #62615] Fix the leaks of file->cmds and file->cmds->commands

2022-06-11 Thread Dmitry Goncharov
Additional Item Attachment, bug #62615 (project make): File name: sv62615_fix.diff Size:1 KB File name: sv62615_test.diff Size:1 KB

[bug #62615] Fix the leaks of file->cmds and file->cmds->commands

2022-06-11 Thread Dmitry Goncharov
ge Status: None ___ Follow-up Comments: --- Date: Sat 11 Jun 2022 01:52:52 PM UTC By: Dmitry Goncharov Makefile like the following causes make to leak file->cmds along with file->cmd

[bug #62469] Make honors MAKEFLAGS= definition on the command line.

2022-05-25 Thread Dmitry Goncharov
Follow-up Comment #2, bug #62469 (project make): > I'm confident a lot of makefiles do do this, and although it's not a good idea I don't know that we want to break them. i agree, that we don't want to break existing makefiles. i also think, that a build system which relies on a MAKEFLAGS= cli de

[bug #62514] Regression. make ignores cli flags.

2022-05-23 Thread Dmitry Goncharov
Additional Item Attachment, bug #62514 (project make): File name: sv62514_fix.diff Size:4 KB File name: sv62514_test.diff Size:3 KB

[bug #62514] Regression. make ignores cli flags.

2022-05-23 Thread Dmitry Goncharov
: None ___ Follow-up Comments: --- Date: Tue 24 May 2022 12:16:45 AM UTC By: Dmitry Goncharov Commit f2771aa614d3a826b70973217c33f605fb4cd0c2 introduced function reset_switches. reset_switches wipes out c

Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-21 Thread Dmitry Goncharov
On Sat, May 21, 2022 at 12:25 PM Paul Smith wrote: > Maybe what you're saying is that make should throw an error or warning > if you try to add an order-only prerequisite to a phony target, telling > you that it will have no effect on your makefile? Having a phony target depend (usually indirectl

[bug #62496] Fix "7.3 Conditionals that Test Flags" in the manual,

2022-05-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #62496 (project make): File name: sv62496_test.diff Size:1 KB ___ Reply to this item at:

[bug #62496] Fix "7.3 Conditionals that Test Flags" in the manual,

2022-05-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #62496 (project make): File name: sv62496_doc.diff Size:1 KB ___ Reply to this item at:

[bug #62496] Fix "7.3 Conditionals that Test Flags" in the manual,

2022-05-20 Thread Dmitry Goncharov
Triage Status: None ___ Follow-up Comments: --- Date: Sat 21 May 2022 01:21:51 AM UTC By: Dmitry Goncharov In paragraph "7.3 Conditionals that Test Flags" the man

Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-18 Thread Dmitry Goncharov
On Wed, May 18, 2022 at 5:37 PM Jacob Kopczynski wrote: >> - it might make sense to state explicitly that order-only prereqs >> which are phony behave in all ways like standard prerequisites. This statement is simply incorrect. Order-only prerequisites which are phony do not behave like reg

[bug #62469] Make honors MAKEFLAGS= definition on the command line.

2022-05-16 Thread Dmitry Goncharov
Additional Item Attachment, bug #62469 (project make): File name: sv62469_doc.diff Size:1 KB File name: sv62469_test.diff Size:2 KB

[bug #62469] Make honors MAKEFLAGS= definition on the command line.

2022-05-16 Thread Dmitry Goncharov
URL: Summary: Make honors MAKEFLAGS= definition on the command line. Project: make Submitted by: dgoncharov Submitted on: Mon 16 May 2022 02:12:20 AM UTC Severity: 3 - Normal

Re: Archive Members as Targets

2022-05-10 Thread Dmitry Goncharov
On Tue, May 10, 2022 at 5:12 PM Michael Lehn wrote: > But not on the Linux boxes there make always rebuild everything. On all > machines I am using GNU Make. ... > Can anyone confirm that? i can confirm that for me on linux the latest make from git as well as make-4.3 correctly detect that libfo

[bug #62397] jobserver unavailable from $(shell $(MAKE)...)

2022-05-03 Thread Dmitry Goncharov
Follow-up Comment #3, bug #62397 (project make): For explanation of the warning see https://lists.gnu.org/archive/html/bug-make/2021-02/msg5.html. MAKEFLAGS= removes all flags. i'd rather use something like filter-out (or some other tool) to remove just the jobserver related flags.

Re: [bug #62173] Fix test features/archives.

2022-04-27 Thread Dmitry Goncharov
On Sun, Apr 24, 2022 at 2:48 PM Jeffrey Walton wrote: > Be sure to test the change on OpenBSD. In the past OpenBSD needed the > dash for options. I'm not sure if it is needed nowadays. The change that was mentioned added the dash. The current default ARFLAGS=-rv, which is also required by posix.

[bug #62174] fix test functions/shell on aix.

2022-04-27 Thread Dmitry Goncharov
Follow-up Comment #2, bug #62174 (project make): Paul, the changes that were introduced in toplevel to set LANG=C do not have desired effect, because run_make_tests.pl sets $ERR_command_not_found before &toplevel is called. Since you didn't like removing LANG from environment, i attached another

[bug #62174] fix test functions/shell on aix.

2022-04-27 Thread Dmitry Goncharov
Additional Item Attachment, bug #62174 (project make): File name: sv62174_fix2.diff Size:0 KB ___ Reply to this item at:

[bug #62173] Fix test features/archives.

2022-04-27 Thread Dmitry Goncharov
Follow-up Comment #4, bug #62173 (project make): Paul, a typo was introduced, which causes multiple tests to fail. Please have a look at sv62173_fix_typo.diff. ___ Reply to this item at: _

[bug #62173] Fix test features/archives.

2022-04-27 Thread Dmitry Goncharov
Additional Item Attachment, bug #62173 (project make): File name: sv62173_fix_typo.diff Size:0 KB ___ Reply to this item at:

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

2022-04-23 Thread Dmitry Goncharov
Follow-up Comment #3, bug #60736 (project make): > For one thing it causes a number of tests to fail. Fixed. > For another thing the docs explicitly say: > Note 'make' is smart enough not to add a prerequisite listed in '.EXTRA_PREREQS' as a prerequisite to itself. Does this behavior have t

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

2022-04-23 Thread Dmitry Goncharov
Additional Item Attachment, bug #60736 (project make): File name: sv_60736_add_avoid_circular_dep_msg_tests.diff Size:2 KB ___ Reply to

[bug #62356] Setting -R through MAKEFLAGS should set -r.

2022-04-23 Thread Dmitry Goncharov
Additional Item Attachment, bug #62356 (project make): File name: sv62356_test.diff Size:0 KB File name: sv62356_fix.diff Size:0 KB

[bug #62356] Setting -R through MAKEFLAGS should set -r.

2022-04-23 Thread Dmitry Goncharov
URL: Summary: Setting -R through MAKEFLAGS should set -r. Project: make Submitted by: dgoncharov Submitted on: Sat 23 Apr 2022 02:11:17 PM UTC Severity: 3 - Normal Item

[bug #62324] Fix cache integrity check

2022-04-16 Thread Dmitry Goncharov
Additional Item Attachment, bug #62324 (project make): File name: sv62324_test.diff Size:2 KB File name: sv62324_fix.diff Size:4 KB

[bug #62324] Fix cache integrity check

2022-04-16 Thread Dmitry Goncharov
URL: Summary: Fix cache integrity check Project: make Submitted by: dgoncharov Submitted on: Sat 16 Apr 2022 05:59:23 PM UTC Severity: 3 - Normal Item Group: Bug

[bug #62278] Fix % substitution in static pattern rules.

2022-04-09 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62278 (project make): Tested on 64 and 32 bit linux. ___ Reply to this item at: ___ Message sent via Savannah https://sav

[bug #62278] Fix % substitution in static pattern rules.

2022-04-09 Thread Dmitry Goncharov
Additional Item Attachment, bug #62278 (project make): File name: sv62278_test.diff Size:5 KB File name: sv62278_fix.diff Size:2 KB

[bug #62278] Fix % substitution in static pattern rules.

2022-04-09 Thread Dmitry Goncharov
URL: Summary: Fix % substitution in static pattern rules. Project: make Submitted by: dgoncharov Submitted on: Sat 09 Apr 2022 03:30:51 PM UTC Severity: 3 - Normal Item

[bug #62206] Fix % substitution in pattern rules.

2022-03-21 Thread Dmitry Goncharov
Follow-up Comment #4, bug #62206 (project make): Such a workaround with 2 rules could be something like .SECONDEXPANSION: all: lib/hello.o hello.o %.o: 3$$*.c; $(info $@ from $<) lib/%.o: lib/3$$*.c; $(info $@ from $<) ___ Reply

[bug #62206] Fix % substitution in pattern rules.

2022-03-21 Thread Dmitry Goncharov
Follow-up Comment #3, bug #62206 (project make): > What I usually do is recommend people use $$* instead of % in secondary expansion expressions like this as it's more clear and less confusing, and ends up being the same result. There are situations where the difference between % and $$* matters.

[bug #62206] Fix % substitution in pattern rules.

2022-03-20 Thread Dmitry Goncharov
Follow-up Comment #1, bug #62206 (project make): Tested on linux, sun and aix, 64 and 32 bits. ___ Reply to this item at: ___ Message sent via Savanna

[bug #62206] Fix % substitution in pattern rules.

2022-03-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #62206 (project make): File name: sv62206_stem_substitution_in_pattern_rules_fix.diff Size:6 KB File name: sv62206_stem_substitution_in_pattern_rules_test.dif

[bug #62206] Fix % substitution in pattern rules.

2022-03-20 Thread Dmitry Goncharov
URL: Summary: Fix % substitution in pattern rules. Project: make Submitted by: dgoncharov Submitted on: Mon 21 Mar 2022 01:04:05 AM UTC Severity: 3 - Normal Item Group:

[bug #62173] Fix test features/archives.

2022-03-13 Thread Dmitry Goncharov
Follow-up Comment #2, bug #62173 (project make): The patch contains the following changes -my $arflags = 'rv'; +my $arflags = '-rv'; +if ($^O eq 'aix') { +$arflags = "-Xany -rv"; +} That reason for the first part of the patch, specifically -my $arflags = 'rv'; +my $arflags = '-

[bug #62175] Port tests to big endian.

2022-03-12 Thread Dmitry Goncharov
Additional Item Attachment, bug #62175 (project make): File name: port_tests_to_big_endian.diff Size:5 KB ___ Reply to this item at:

[bug #62175] Port tests to big endian.

2022-03-12 Thread Dmitry Goncharov
URL: Summary: Port tests to big endian. Project: make Submitted by: dgoncharov Submitted on: Sat 12 Mar 2022 11:21:32 PM UTC Severity: 3 - Normal Item Group: Bug

[bug #62174] fix test functions/shell on aix.

2022-03-12 Thread Dmitry Goncharov
Additional Item Attachment, bug #62174 (project make): File name: test_functions_shell_fix_on_aix.diff Size:0 KB ___ Reply to this item at:

[bug #62174] fix test functions/shell on aix.

2022-03-12 Thread Dmitry Goncharov
URL: Summary: fix test functions/shell on aix. Project: make Submitted by: dgoncharov Submitted on: Sat 12 Mar 2022 11:14:40 PM UTC Severity: 3 - Normal Item Group: Bug

<    1   2   3   4   5   6   7   >