Issue 67911 - dmake removes tempfiles of current target after each recipe
Summary: dmake removes tempfiles of current target after each recipe
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: dmake (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: hjs
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-30 03:26 UTC by quetschke
Modified: 2013-08-07 15:34 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Patch for dmake (4.72 KB, patch)
2007-09-02 11:38 UTC, quetschke 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description quetschke 2006-07-30 03:26:29 UTC 
Follow up to issue 64583.

Dmake currently removes all tempfiles of the current target after execution of
each recipe line. This adds overhead and should only be done after the last
recipe line.
Comment 1 quetschke 2006-08-23 05:05:18 UTC 
The summary is not quite accurate. It removes the tempfiles if the end of the
recipes for the current target process is reached. For non-parallel (-P1)
execution of dmake this is always the case after each recipe line.

The folowing makefile demonstrates this:

--- makefile.mk ---
SHELL*:=/bin/sh 
SHELLFLAGS*:=-ce

all :
	+echo $(assign VERSIONTMP:=$(mktmp iii)) ; sleep 1
	+echo Z$(VERSIONTMP)Z ; ls $(VERSIONTMP)
--- makefile.mk ---

For `dmake -r -P3` the tempfile is still there at the "ls $(VERSIONTMP)", for
`dmake -r` it is already deleted.
Comment 2 hjs 2006-09-14 17:20:47 UTC 
first makefile snippet that runs with -Pn but breaks otherwise :)
Comment 3 quetschke 2007-09-02 11:38:12 UTC 
Created attachment 47911 [details]
Patch for dmake
Comment 4 quetschke 2007-09-02 11:58:30 UTC 
Committed (plus testcase) to dmake411.
Comment 5 quetschke 2007-09-02 11:58:59 UTC 
Please verify
Comment 6 hjs 2007-09-26 17:08:52 UTC 
fixes single process issue :)
Comment 7 hjs 2007-12-21 13:39:59 UTC 
dmake 4.11 is in use now