Opened 7 years ago
Closed 6 years ago
#15366 closed defect (duplicate)
cyggcc_s.dll causes Python random segfaults
Reported by:  jpflori  Owned by:  

Priority:  critical  Milestone:  sageduplicate/invalid/wontfix 
Component:  packages: standard  Keywords:  gcc spkg cygwin 
Cc:  kcrisman, dimpase, tscrim  Merged in:  
Authors:  JeanPierre Flori  Reviewers:  
Report Upstream:  Fixed upstream, in a later stable release.  Work issues:  
Branch:  u/jpflori/ticket/15366 (Commits, GitHub, GitLab)  Commit:  46e6ac63cec260b90c27865caf91a80650473151 
Dependencies:  Stopgaps: 
Description (last modified by )
We should include the corresponding fix to our GCC. See:
 http://cygwin.com/ml/cygwin/201308/msg00201.html
 http://cygwin.com/ml/cygwin/201307/msg00528.html
Use git branch.
Attachments (1)
Change History (21)
comment:1 Changed 7 years ago by
 Keywords gcc spkg cygwin added
comment:2 Changed 7 years ago by
 Status changed from new to needs_review
comment:3 Changed 7 years ago by
 Description modified (diff)
comment:4 Changed 7 years ago by
 Status changed from needs_review to needs_work
comment:5 Changed 7 years ago by
 Status changed from needs_work to needs_review
comment:6 Changed 7 years ago by
I guess the bug is also:
comment:7 Changed 7 years ago by
 Report Upstream changed from N/A to Fixed upstream, but not in a stable release.
Ok the previous bug was unrelated. The right one was here:
and has been fixed upstream after my bumping.
comment:8 Changed 7 years ago by
 Branch set to u/jpflori/ticket/15366
 Commit set to 13366b752018f8aee38b4200897b2620f92d2672
 Description modified (diff)
New commits:
13366b7  cyggcc_s.dll causes Python random segfaults on Cygwin.

comment:9 Changed 7 years ago by
 Cc tscrim added
comment:10 Changed 7 years ago by
 Milestone changed from sage6.1 to sage6.2
comment:11 Changed 7 years ago by
 Commit changed from 13366b752018f8aee38b4200897b2620f92d2672 to c5e9e876e87a750f4b3ffde72bad4bf241d5b007
Branch pushed to git repo; I updated commit sha1. New commits:
c5e9e87  Merge remotetracking branch 'trac/develop' into ticket/15366

comment:12 Changed 7 years ago by
 Status changed from needs_review to needs_work
This spkginstall
uses set e
. So, you should remove all the extra checking you added, and you might want to move the set e
down to after the patching.
comment:13 Changed 7 years ago by
 Commit changed from c5e9e876e87a750f4b3ffde72bad4bf241d5b007 to 46e6ac63cec260b90c27865caf91a80650473151
Branch pushed to git repo; I updated commit sha1. New commits:
46e6ac6  Correctly deal with errors in gcc install script.

comment:14 Changed 7 years ago by
 Status changed from needs_work to needs_review
Thanks, I did not pay attention to set e
...
comment:15 Changed 7 years ago by
 Milestone changed from sage6.2 to sage6.3
comment:16 Changed 7 years ago by
 Milestone changed from sage6.3 to sage6.4
comment:17 Changed 6 years ago by
Is this fixed by #17169?
comment:18 Changed 6 years ago by
I think so.
comment:19 Changed 6 years ago by
 Report Upstream changed from Fixed upstream, but not in a stable release. to Fixed upstream, in a later stable release.
 Status changed from needs_review to positive_review
comment:20 Changed 6 years ago by
 Milestone changed from sage6.4 to sageduplicate/invalid/wontfix
 Resolution set to duplicate
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Hum, I put the patch in the wrong direction.