Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#37367 - Graphviz 2.34 doesn't show LLVM graphs correctly

Attached to Project: Arch Linux
Opened by Marcello Maggioni (Kariddi) - Thursday, 17 October 2013, 11:05 GMT
Last edited by Gaetan Bisson (vesath) - Wednesday, 23 October 2013, 15:20 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Evangelos Foutras (foutrelis)
Gaetan Bisson (vesath)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

After recently upgrading to graphviz 2.34 LLVM generated graphs (for debugging LLVM) are not shown correctly anymore with the error:


dotty.lefty: draw language parser error: t


Additional info:
* package version(s) graphviz 2.34.0
* config and/or log files etc.


Steps to reproduce:

Compile a file with llvm and generate a graph, like with "-mllvm -view-isel-dags". Trying to see the .dot file shows the problem.
Graphviz 2.30 works fine.
This task depends upon

Closed by  Gaetan Bisson (vesath)
Wednesday, 23 October 2013, 15:20 GMT
Reason for closing:  Fixed
Additional comments about closing:  graphviz-2.34.0-2 in [extra]
Comment by Gaetan Bisson (vesath) - Thursday, 17 October 2013, 17:05 GMT
Can you extract from LLVM one of the DOT files it tries to feed to graphviz without success?
Comment by Marcello Maggioni (Kariddi) - Friday, 18 October 2013, 10:09 GMT
Here is a graph that presents the problem + a screenshot of what happens if I try to show it.
Comment by Emden R. Gansner (emden) - Tuesday, 22 October 2013, 15:30 GMT
The output format used by dotty was extended in 2.34, but the necessary change was not made to dotty. This has now been fixed in Graphviz. The patch is


--- b/cmd/dotty/dotty_layout.lefty
+++ a/cmd/dotty/dotty_layout.lefty
@@ -5,7 +5,7 @@ dotty.grablserver = function (lserver) {
local fd;

if (~dotty.lservers[lserver] | tablesize (dotty.lservers[lserver]) == 0) {
- if (~((fd = openio ('pipe', lserver, 'r+', '%e -Txdot1.2')) >= 0)) {
+ if (~((fd = openio ('pipe', lserver, 'r+', '%e -Txdot')) >= 0)) {
dotty.message (0, concat ('cannot start ', lserver));
return null;
}
@@ -438,8 +438,6 @@ dotty.protogt.unpackdraw = function (gt, attr) {
}
} else if (t[i] == 'I') {
i = i + 7;
- } else if (t[i] == 't') {
- i = i + 2;
} else {
dotty.message (0, concat ('draw language parser error: ', t[i]));
return null;
Comment by Gaetan Bisson (vesath) - Wednesday, 23 October 2013, 01:06 GMT
Thanks a lot for the patch!
Comment by Gaetan Bisson (vesath) - Wednesday, 23 October 2013, 01:51 GMT
Marcello, could you confirm that your problem is fixed with graphviz-2.34.0-2 from [testing]?
Comment by Marcello Maggioni (Kariddi) - Wednesday, 23 October 2013, 09:24 GMT
I confirm that the problem is solved in testing with the package 2.34.0-2 !

Loading...