Opened 9 years ago
Closed 8 years ago
#15366 closed defect (duplicate)
cyggcc_s.dll causes Python random segfaults
Reported by: | jpflori | Owned by: | |
---|---|---|---|
Priority: | critical | Milestone: | sage-duplicate/invalid/wontfix |
Component: | packages: standard | Keywords: | gcc spkg cygwin |
Cc: | kcrisman, dimpase, tscrim | Merged in: | |
Authors: | Jean-Pierre 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/2013-08/msg00201.html
- http://cygwin.com/ml/cygwin/2013-07/msg00528.html
Use git branch.
Attachments (1)
Change History (21)
comment:1 Changed 9 years ago by
- Keywords gcc spkg cygwin added
comment:2 Changed 9 years ago by
- Status changed from new to needs_review
comment:3 Changed 9 years ago by
- Description modified (diff)
comment:4 Changed 9 years ago by
- Status changed from needs_review to needs_work
comment:5 Changed 9 years ago by
- Status changed from needs_work to needs_review
comment:6 Changed 9 years ago by
I guess the bug is also:
comment:7 Changed 9 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 8 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 8 years ago by
- Cc tscrim added
comment:10 Changed 8 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:11 Changed 8 years ago by
- Commit changed from 13366b752018f8aee38b4200897b2620f92d2672 to c5e9e876e87a750f4b3ffde72bad4bf241d5b007
Branch pushed to git repo; I updated commit sha1. New commits:
c5e9e87 | Merge remote-tracking branch 'trac/develop' into ticket/15366
|
comment:12 Changed 8 years ago by
- Status changed from needs_review to needs_work
This spkg-install
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 8 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 8 years ago by
- Status changed from needs_work to needs_review
Thanks, I did not pay attention to set -e
...
comment:15 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:16 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:17 Changed 8 years ago by
Is this fixed by #17169?
comment:18 Changed 8 years ago by
I think so.
comment:19 Changed 8 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 8 years ago by
- Milestone changed from sage-6.4 to sage-duplicate/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.