New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
optional valgrind-3.3.1 spkg doesn't build with newer GCC's #7440
Comments
comment:2
I sent this email to sage-devel:
|
comment:3
A new package with the latest version of valgrind is here: http://sage.math.washington.edu/home/timdumol/valgrind-3.5.0.p0.spkg. |
Author: Tim Dumol |
comment:4
I tried installing this on Mac OS X 10.6 and got an error during the "configure" process:
Seems to build correctly on sage.math, for what that's worth. Once the OS X issue is worked out, other people should definitely look at this; I don't think I'm qualified to review it properly. |
comment:5
Despite the message "Valgrind works on Darwin 9.x", that Valgrind spkg has never built on OS X. Valgrind was Linux only. I recently heard that it was ported to OS X, but I'm not sure if that is really the case. Mabshoff used to tell me that it would be ported any day now... |
comment:6
I believe OS X 10.6 is Darwin 10.x, which Valgrind indeed does not support. I have uploaded a new version of the package which checks for the release version here: http://sage.math.washington.edu/home/timdumol/valgrind-3.5.0.p0.spkg. |
comment:7
This has a portability bug. 'uname' will will called with the '-p' option on any non-Linux system. But '-p' is not a POSIX option for uname. http://www.opengroup.org/onlinepubs/9699919799/utilities/uname.html so there is no reason any system should support the -p option. If you need to test the processor type, make sure the test is only done on platforms where you know the -p option is supported. HP-UX is one platform where this will fail, and I expect there are others too.
Dave |
Reviewer: David Kirkby |
comment:10
FYI: Valgrind doesn't make any sense on any platforms that don't support 'uname -p'. The only supported platforms for the latest Valgrind are: X86/Linux, AMD64/Linux, PPC32/Linux, PPC64/Linux, and X86/Darwin (Mac OS X). These all support uname -p. |
comment:11
Replying to @sagetrac-drkirkby:
I do not believe this is a problem, since
|
comment:12
First, merry Christmas to you both. William asked me the other day to check what optional packages installed on Solaris. Valgind did not install, but did give a helpful error message, indicating why. That's much better than executing a command which will fail. So I believe the fact Valgind does not work on Solaris, AIX, HP-UX etc is a not an excuse for sloppy programming - there is enough of that in Sage anyway! William's use of:
in an early version of 'prereq' did cause an issue on HP-UX, with the -p option creating problems - see #7156. Tim's version is written slighlty different, using the preffered '&&' instead of '-a'. The use of '-a' is deprecated and discouraged by POSIX http://www.opengroup.org/onlinepubs/009695399/utilities/test.html so perhaps Tim's arugument is valid. On reflection, I agree with Tim. However, as my grandmother used to say, the proof of the pudding is in the eating, so I tested this on HP-UX, where sage-4.2.1 is installed.
So I would have to agree, the spkg-install does work properly, even on HP-UX. The error message is helpful too. The spkg-install does not fail as I initially expected it would, so I'm changing this to positive review. Dave |
comment:13
BTW, I just noticed that Michael Abshoff is the noted as the package maintainer in SPKG.txt. #7738 says he should be removed from all packages, and specificially lists Valgrind. Hence I believe this would be a good oppotunity to remove Michael's name. Does time want to take on the role? As such, I'm swapping this back to 'needs work'. It would seem silly to update a package, without addressing this minor issue. Otherwise, I'm happy with this, so will change it back to positive when its done. Dave |
comment:14
That was supposed to be 'does Tim want to take on the role?', not 'time' as I said. |
comment:15
Replying to @sagetrac-drkirkby:
Sure, I'll be glad to. New version of spkg up at http://sage.math.washington.edu/home/timdumol/valgrind-3.5.0.p0.spkg |
comment:16
Due to the problems with Sage (disk related I believe), I unable to download this. But since the only issue was changing someone name in a text file, I'm sure there will be no problem, so I'm changing it to positive review. |
Merged: sage-4.3.1.rc1 |
comment:18
Now this package has a problem with the new glibc version as well. I get:
I use Ubuntu 10.04, and the glibc version is 2.11 .... |
comment:20
Replying to @sagetrac-maldun:
As a general rule, don't reopen a ticket once it is closed. Open a new ticket instead. |
I tried to build the valgrind_3.3.1 spkg on ubuntu-9.10 and it quickly fails with
CC: @williamstein @sagetrac-drkirkby
Component: packages: optional
Author: Tim Dumol
Reviewer: David Kirkby
Merged: sage-4.3.1.rc1
Issue created by migration from https://trac.sagemath.org/ticket/7440
The text was updated successfully, but these errors were encountered: