Opened 11 years ago

Closed 3 years ago

#42348 closed defect (fixed)

graphviz @2.36.0 +ming: error: unknown type name 'textpara_t'; did you mean 'textspan_t'?

Reported by: davidfavor (David Favor) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc: cooljeanius (Eric Gallager), Smattr (Matthew Fernandez)
Port: graphviz, graphviz-devel

Description (last modified by ryandesign (Ryan Carsten Schmidt))

See attached log.

gvrender_ming.c:45:15: warning: unused variable 'action' [-Wunused-variable]

Workaround is to build without +ming variant.

Attachments (1)

graphviz.log (32.2 KB) - added by davidfavor (David Favor) 11 years ago.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by davidfavor (David Favor)

Attachment: graphviz.log added

comment:1 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… removed
Description: modified (diff)
Owner: changed from macports-tickets@… to ryandesign@…
Port: graphviz-devel added
Status: newassigned
Summary: Latest graphviz with +ming variant build errorsgraphviz @2.36.0 +ming: error: unknown type name 'textpara_t'; did you mean 'textspan_t'?

The warning you quoted is just a warning. The real problem is what comes after the warning:

:info:build gvrender_ming.c:94:50: error: unknown type name 'textpara_t'; did you mean 'textspan_t'?
:info:build static void ming_textpara(GVJ_t * job, pointf p, textpara_t * para)
:info:build                                                  ^~~~~~~~~~
:info:build                                                  textspan_t
:info:build ../../lib/common/textspan.h:63:7: note: 'textspan_t' declared here
:info:build     } textspan_t;
:info:build       ^
:info:build gvrender_ming.c:126:45: error: no member named 'fontsize' in 'textspan_t'
:info:build     SWFTextField_setHeight(textfield, para->fontsize);
:info:build                                       ~~~~  ^
:info:build gvrender_ming.c:133:20: error: no member named 'width' in 'textspan_t'
:info:build         offset.x = -para->width;
:info:build                     ~~~~  ^
:info:build gvrender_ming.c:137:20: error: no member named 'width' in 'textspan_t'
:info:build         offset.x = -para->width/2.;
:info:build                     ~~~~  ^
:info:build gvrender_ming.c:141:23: error: no member named 'height' in 'textspan_t'
:info:build     offset.y = -para->height + para->fontsize*.4;  /* empirically determined */
:info:build                 ~~~~  ^
:info:build gvrender_ming.c:141:38: error: no member named 'fontsize' in 'textspan_t'
:info:build     offset.y = -para->height + para->fontsize*.4;  /* empirically determined */
:info:build                                ~~~~  ^
:info:build 1 warning and 6 errors generated.

Confirmed with graphviz @2.36.0 and @2.37.20140201.0545. I've reported it to the developers: http://www.graphviz.org/mantisbt/view.php?id=2420

comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:3 Changed 3 years ago by Smattr (Matthew Fernandez)

In the migration to Gitlab, the above Mantis issue became https://gitlab.com/graphviz/graphviz/-/issues/292. This was reportedly fixed in Graphviz commit a3ef2c5a5a1077c1176c6503f5aefc4df1df15c3, which made it into 2.38.0.

comment:4 Changed 3 years ago by Smattr (Matthew Fernandez)

Cc: Smattr added

comment:5 Changed 3 years ago by mascguy (Christopher Nielsen)

Resolution: fixed
Status: assignedclosed

Since our current graphviz release is at 2.40.1 (with a pending update to 2.48.0) - and the fix was included in 2.38.0 - this is no longer an issue. Closing as resolved.

Note: See TracTickets for help on using tickets.