Opened 9 years ago
Closed 9 years ago
#13804 closed defect (fixed)
libfplll fails to build in parallel on Cygwin
Reported by: | jpflori | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-5.6 |
Component: | porting: Cygwin | Keywords: | libfplll spkg cygwin |
Cc: | kcrisman, dimpase | Merged in: | sage-5.6.rc0 |
Authors: | Jean-Pierre Flori | Reviewers: | Karl-Dieter Crisman |
Report Upstream: | None of the above - read trac for reasoning. | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
On my installation of Cywgin (on 64 bits Windows 7), building libfplll fails because make complains there is no rule to build fplll if I use MAKE=make -j4". Indeed there is one for fplll$(EXEEXT), but no for fplll and $(EXEEXT) is non empty on Cygwin. Nonetheless, if the file fplll.exe exists, then "make fplll" will happily tell you that its up to date and go on with its job.
This does not happen with a sequential build.
Use the spkg at http://boxen.math.washington.edu/home/jpflori/libfplll-3.0.12.p3.spkg
And by the way the libfplll we ship is old and the problem is not present in the most recent version. See #12835. So I won't report this upstream.
Attachments (1)
Change History (8)
comment:1 Changed 9 years ago by
comment:2 Changed 9 years ago by
- Description modified (diff)
- Summary changed from libfplll fails to build on recent Cygwin to libfplll fails to build in parallel on Cygwin
Groumpf, I got it to install by not using a parallel build.
comment:3 Changed 9 years ago by
- Component changed from PLEASE CHANGE to cygwin
- Type changed from PLEASE CHANGE to defect
Yeah, I've never seen this because my machine is certainly always with one thread. I guess I never even thought to try it in parallel, but as it's XP I'm sure that wouldn't be very useful in any case.
comment:4 Changed 9 years ago by
- Description modified (diff)
- Status changed from new to needs_review
comment:5 Changed 9 years ago by
Brief look at diff seems okay, and this installed fine on XP with passed (self, SAGE_CHECK=yes) tests. I obviously can't check whether this actually works in terms of multiple threads, though this is the right command to disable parallel make, and is only on Cygwin, so we should probably let it go through.
comment:6 Changed 9 years ago by
- Reviewers set to Karl-Dieter Crisman
- Status changed from needs_review to positive_review
Same result on Mac and sage.math, and after ./sage -b
passes the sage/libs/fplll
tests as well. Let's get this in.
comment:7 Changed 9 years ago by
- Merged in set to sage-5.6.rc0
- Resolution set to fixed
- Status changed from positive_review to closed
Or maybe Cywgin make used to automagically translate targets to add .exe extension and does not anymore.