bugmake - Bugs: bug #63248, Ignore sigpipe.

 
 

bug #63248: Ignore sigpipe.

Submitter:  Dmitry Goncharov <dgoncharov>
Submitted:  Thu 20 Oct 2022 09:47:11 PM UTC
   
 
Severity:  3 - Normal Item Group:  Bug
Status:  Fixed Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Component Version:  4.4 Operating System:  POSIX-Based
Fixed Release:  4.4 Triage Status:  Small Effort
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 02 Nov 2022 03:13:04 PM UTC, comment #5: 

Thanks, I have submitted #63307 for it.

Anonymous
Wed 02 Nov 2022 02:02:02 PM UTC, comment #4: 

Make should not pass the ignore to children; if it does that's a bug in make and should be reported.

Paul D. Smith <psmith>
Group administrator
Wed 02 Nov 2022 01:58:54 PM UTC, comment #3: 

Hi, this change does not just mean make ignores SIGPIPE, it also means everything spawned by make ignores SIGPIPE. Is this intentional? I am seeing a failure in another package as a result of this after updating to make 4.4 and do not know whether to submit this as a make bug, or as a bug against that package.

Anonymous
Fri 21 Oct 2022 11:30:52 PM UTC, comment #2: 

Oddly I cannot reproduce this problem on my system; apparently my SIGPIPE is set to IGN always.  I've looked high and low for something that is configured to do this and can find nothing.  But, the only way I can reproduce this issue is to explicitly change the make code to set SIGPIPE to SIG_DFL, then I get the failure to happen.

Anyway this change looks good to me so I applied it, thanks Dmitry!

Paul D. Smith <psmith>
Group administrator
Thu 20 Oct 2022 09:50:54 PM UTC, comment #1: 

A user reported that make leaves temporary files behind when killed by sigpipe.



$ ls tmp/
$ echo 'all:; sleep 2' | ~/src/gmake/make/l64/make -f- -j2 -O |:
$ ls tmp
GmIDqB2Q  GmyW1CcO
$ rm tmp/*
$ # this make ignores sigpipe
$ echo 'all:; sleep 2' | ~/src/gmake/make/l64/make -f- -j2 -O |:
fwrite(): Broken pipe
make: write error: stdout
$ ls tmp/
$


The bug report is here
https://lists.gnu.org/archive/html/bug-make/2022-10/msg00160.html

Dmitry Goncharov <dgoncharov>
Thu 20 Oct 2022 09:47:11 PM UTC, original submission:  

.

Dmitry Goncharov <dgoncharov>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #53888:  sv63248.diff added by dgoncharov (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by psmith (Posted a comment)
  • -email is unavailable- added by dgoncharov (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-10-24 psmith StatusNone Fixed
        Open/ClosedOpen Closed
        Fixed ReleaseNone 4.4
        Triage StatusNone Small Effort
    2022-10-20 dgoncharov Attached File- Added sv63248.diff, #53888

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code