#8951 closed defect (duplicate)
Clear /tmp/ECL* after building on Solaris + sort out minor SAGE64 issue.
Reported by: | drkirkby | Owned by: | drkirkby |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | porting: Solaris | Keywords: | |
Cc: | was, mvngu, juanjose.garciaripoll@…, kcrisman, nbruin | Merged in: | |
Authors: | David Kirkby | Reviewers: | Nils Bruin |
Report Upstream: | Fixed upstream, in a later stable release. | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
As reported here
http://groups.google.com/group/sage-devel/browse_thread/thread/54544d8649bd027a
there is a problem when temp files created during the build of ECL. A correct fix requires changes made to ECL source code, but as a temporary fix, it may be sufficient to remove /tmp/ECL* after building ecl.
Attachments (1)
Change History (23)
comment:1 Changed 12 years ago by
- Status changed from new to needs_review
comment:2 Changed 12 years ago by
Note #8808 as another ECL spkg update.
Also, the patch about the 'yes' and '1' still says
# Compile for 64-bit if SAGE64 is set to 'yes' or '1'
although of course the code behaves as Dave describes... but maybe that's ok?
comment:3 Changed 12 years ago by
- Status changed from needs_review to needs_work
- Summary changed from Clear /tmp/ECL* after building ECL to Update ECL, clear /tmp/ECL* after building + sort out minor SAGE64 issue.
I'll create a new package based on Williams at #8808, which address all issues about ECL
comment:4 Changed 12 years ago by
- Summary changed from Update ECL, clear /tmp/ECL* after building + sort out minor SAGE64 issue. to Clear /tmp/ECL* after building on Solaris + sort out minor SAGE64 issue.
comment:5 Changed 12 years ago by
- Cc was added
- Status changed from needs_work to needs_review
Here's a revised ecl-10.4.1.
http://boxen.math.washington.edu/home/kirkby/patches/ecl-10.4.1/ecl-10.4.1.spkg
It includes
- The latest release of ecl - taken from the package posted at #8808
- Removes /tmp/ECL* on Solaris.
- Corrected minor issue with SAGE64, which would have in theory worked if SAGE64 was set to '1', but earlier bits of sage force SAGE64 to be only 'yes' or 'no', so there was no point checking for this.
- Comment in the code about this SAGE64 change is now more accurate.
I've tested this on sage.math and it works fine.
real 2m20.869s user 1m47.690s sys 0m23.480s Successfully installed ecl-10.4.1 kirkby@sage:~/sage-4.4.2.alpha0$ uname -a Linux sage.math.washington.edu 2.6.24-26-server #1 SMP Tue Dec 1 18:26:43 UTC 2009 x86_64 GNU/Linux
Note, this code to remove /tmp/ECL* is not a perfect solution. Once ECL is fixed, this should be removed. Dave
comment:6 follow-up: ↓ 7 Changed 12 years ago by
Note to release manager
- Apply the patch at #8808
- Apply the patch attacked here.
- Copy the file http://boxen.math.washington.edu/home/kirkby/patches/ecl-10.4.1/ecl-10.4.1.spkg to the new source.
- Do NOT use the .spkg at http://wstein.org/home/wstein/patches/ecl-10.4.1.spkg as that lacks the patches here, whereas this ticket includes the updated ECL at http://wstein.org/home/wstein/patches/ecl-10.4.1.spkg.
Dave
comment:7 in reply to: ↑ 6 ; follow-up: ↓ 11 Changed 12 years ago by
comment:8 follow-up: ↓ 16 Changed 12 years ago by
As a reminder here, this should *only* be merged simultaneously with the new maxima at #8731.
comment:9 Changed 12 years ago by
comment:10 Changed 12 years ago by
- Cc kcrisman added
New spkg up at http://sage.math.washington.edu/home/jason/ecl-10.4.1.spkg
I'm not sure who can review it, since I fixed it, to follow Nils' spkg, but David also fixed something on it.
Maybe kcrisman can review it, in conjunction with the new maxima update?
comment:11 in reply to: ↑ 7 Changed 12 years ago by
Replying to jason:
Replying to drkirkby:
Note to release manager
- Apply the patch at #8808
- Apply the patch attacked here.
Both of these patches are included in your spkg, right? In that case, the release manager would just use your spkg, and wouldn't apply any patches anywhere.
My package should work fine.
My reason for saying to add #8808 first is because my patch was based on the patch applied in #8088, so assumed the wording of SPKG.txt in #8088 as a starting point and used the Mercurial repository in #8088 as a starting point. Also, since William updated the package, he should get credit for that ticket. It has already got positive review.
From the point of view of actual code, it would have made no difference whatsoever.
Anyway, you have now posted another version, based on #8645. Someone needs to review it. I looked over spkg-install and SPKG.txt and they look OK to me, but I can hardly review it.
comment:12 Changed 12 years ago by
- Cc nbruin added
I positive reviewed #8808, which was a mistake. That's why I posted a new spkg, giving credit to Nils, William, and you, but using #8645 as a base for your patch. Well, actually I just took your spkg and removed the appropriate directories, since that should end up with the same final result.
If Nils agrees that the changes are good (i.e., the right directories were removed, I think we can mark this as positive review, because then at least 3 pairs of eyes will have looked at the spkg.
comment:13 Changed 12 years ago by
- Status changed from needs_review to positive_review
I confirm that the src directories in
/home/nbruin/ecl-10.4.1.spkg
and
/home/jason/ecl-10.4.1.spkg
agree according to diff -r
. Differences are only in
spkg-install
and SPKG.txt
, as expected (and in .hg).
comment:14 Changed 12 years ago by
- Reviewers set to Nils Buin
comment:15 Changed 12 years ago by
- Reviewers changed from Nils Buin to Nils Bruin
comment:16 in reply to: ↑ 8 Changed 12 years ago by
Replying to jason:
As a reminder here, this should *only* be merged simultaneously with the new maxima at #8731.
Jason,
what is there about this ticket that means it can only be merged simultaneously with the new Maxima at #8731? That Maxima ticket has not been updated for 3 weeks. If ECL & Maxima need to be updated together, that leaves a whole load of issues unresolved about ECL. Some I can think of are
- The original aim of the ticket, which was to clear the tmp files - a trivial change, that does not need ECL updated, but allows Sage to build more relieably on 't2'
- The minor SAGE64 issue - again, a trivial (cosmetic) change that does not need ECL updated.
- It stops the building of spkg's in parallel, as the ECL makefile needs a trivial edit for that - see #9187. Again, that does not need a new version of ECL for this.
- It stops a build of ECL on 64-bit OpenSolaris? - see #8089. Again, the option to permit this does not need a new upstream version of ECL.
It seems to me we have three choices here:
- Update ECL, without updating Maxima, which I think you are saying is not possible.
- Update both Maxima and ECL to the latest versions quickly.
- Apply all the other very small changes to the ECL 10.2.1 that is in Sage now. So leaving updating ECL to 10.4.1 until a later date.
I've created #9264 to update ECL to 10.4.1 and apply all changes.
It might however be wiser to create another ticket to just apply all the small changes to ECL 10.2.1.
Dave
comment:17 Changed 12 years ago by
- Status changed from positive_review to needs_work
comment:18 Changed 12 years ago by
- Status changed from needs_work to needs_info
comment:19 Changed 12 years ago by
comment:20 Changed 12 years ago by
- Resolution set to duplicate
- Status changed from needs_info to closed
comment:21 Changed 12 years ago by
- Milestone changed from sage-4.5 to sage-duplicate/invalid/wontfix
comment:22 Changed 11 years ago by
- Report Upstream changed from Reported upstream. Developers acknowledge bug. to Fixed upstream, in a later stable release.
For what it's worth, jjgarcia reports the original issue with /tmp/ECLINIT.c
being used instead of a unique tempfile has been fixed in ECL. It appears to be in the latest ECL release already, but I haven't checked personally if it now works on Solaris.
I've attached a patch. It removes the files only on Solaris. It also has a couple of minor corrections
These two are just very minor changes - the main change is to delete /tmp/ECL*
What I find a bit odd, is that I don't see these tmp files on my own Sun Blade 2000 SPARC, which runs Solaris 10 update 8 (05/2009). I can only assume they are created them removed by ecl, so why this should not happen on 't2' is a mystery.
http://boxen.math.washington.edu/home/kirkby/Solaris-fixes/ecl-10.2.1.p0.spkg
Dave