Skip to content
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

New version of graphviz #15933

Closed
miguelmarco opened this issue Mar 13, 2014 · 12 comments
Closed

New version of graphviz #15933

miguelmarco opened this issue Mar 13, 2014 · 12 comments

Comments

@miguelmarco
Copy link
Contributor

For some reason, i get a segfault when i try to system-wide graphviz from sage, and the current experimental spkg fails to install, so i updated it. It now installs on my systems and works ok.

https://drive.google.com/file/d/0B_B7eBQ-W5NGNVBSR3RCTlJRMEk/edit?usp=sharing

CC: @nthiery @rlmill @mwhansen @nathanncohen @jdemeyer @haraldschilly

Component: packages: experimental

Author: Miguel Marco

Reviewer: Nathann Cohen

Issue created by migration from https://trac.sagemath.org/ticket/15933

@mezzarobba

This comment has been minimized.

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Mar 13, 2014

comment:2

Hello !

Well, the spkg works (even though I expected it to be a bzip2 file instead of a gzip file), but how exactly can this spkg be used through Sage so that I may test it ?

Nathann

@nathanncohen nathanncohen mannequin added the s: needs info label Mar 13, 2014
@miguelmarco
Copy link
Contributor Author

comment:4

The situation where i used it is in the singular interface.

$ sage -singular
                     SINGULAR                                 /  Development
 A Computer Algebra System for Polynomial Computations       /   version 3-1-6
                                                           0<
 by: W. Decker, G.-M. Greuel, G. Pfister, H. Schoenemann     \   Dec 2012
FB Mathematik der Universitaet, D-67653 Kaiserslautern        \
> LIB "all.lib";;

(all of output about the libraries loaded)

> ring r=0,(x,y),dp;
> ideal i=x2-y3;
> list L=resolve(i);
> ResTree(L,collectDiv(L)[1],"output");
Currently showing graphics in separate window
Press <Return> to continue
sh: /usr/bin/X11/xv: No such file or directory
pause>
> 

It creates a text file called output, and tries to call "dot" on it. But it fails, and creates an empty file output.jpg

With this new version, it creates the file output.jpg as expected.

Once you have the file output created, you can try to call dot manually on it:

$ sage -sh

Starting subshell with Sage environment variables set.  Don't forget
to exit when you are done.  Beware:
 * Do not do anything with other copies of Sage on your system.
 * Do not use this for installing Sage packages using "sage -i" or for
   running "make" at Sage's root directory.  These should be done
   outside the Sage shell.

Bypassing shell configuration files...

Note: SAGE_ROOT=/home/mmarco/sagedevel/sage
(sage-sh) mmarco@localhost:sage$ dot -Tjpg output -o output.jpg
segmentation fault

But with the spkg installed, it just creates the output.jpg file as expected.

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Mar 14, 2014

comment:5

Your message is badly formatted, it looks like you repeated the same thing three times. Could you clean it please ?

@miguelmarco
Copy link
Contributor Author

comment:6

Yes, sorry, it is corrected now.

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Mar 14, 2014

comment:7

Okayyyyyy okay. Well, both work on my computer, but I guess it cannot hurt to update the spkg and it does solve your problem. Good ! :-)

Nathann

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Mar 14, 2014

Reviewer: Nathann Cohen

@vbraun
Copy link
Member

vbraun commented Mar 14, 2014

comment:8

Harald, can you add this to the experimental spkgs?

@haraldschilly
Copy link
Member

comment:9

ok, done.

... and PS: and for the future, could we avoid sharing spkg files via such web based tools? it's kind of tedious to move them to the sever that way :-\

@vbraun
Copy link
Member

vbraun commented Mar 21, 2014

Author: Marc Mezzarobba

@mezzarobba
Copy link
Member

Changed author from Marc Mezzarobba to Miguel Marco

@mezzarobba
Copy link
Member

comment:11

I don't remember changing the description of this ticket, though apparently I did. But I'm quite certain I didn't prepare the updated spkg!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants