Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#41422 closed defect (fixed)

graphviz: missing poppler dependency

Reported by: dbevans (David B. Evans) Owned by: ryandesign (Ryan Schmidt)
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc: cooljeanius (Eric Gallager)
Port: graphviz

Description

If poppler is installed, graphviz builds plugin

/opt/local/lib/graphviz/libgvplugin_poppler.6.dylib

but poppler is not present in its dependency tree.

Change History (7)

comment:1 Changed 9 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed

Fix committed in r113653, maintainer timeout.

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

Couldn't this lead to a circular dependency with the +docs variant? Via a recursive dependency of a cairo installed with +x11:

 cairo
...
    xrender
...
      xorg-libX11
...
        xorg-libxcb
...
          doxygen
...
            graphviz

Poppler already creates enough circular dependency chains as it is, as shown by the number of duplicates of #37604 out there. Do we really need to be going and creating more?

comment:3 Changed 9 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:4 Changed 9 years ago by dbevans (David B. Evans)

It may do. However, your comment fails to address the reported problem. An alternate solution may be to disable poppler support in configure.

comment:5 in reply to:  4 Changed 9 years ago by cooljeanius (Eric Gallager)

Replying to devans@…:

It may do. However, your comment fails to address the reported problem. An alternate solution may be to disable poppler support in configure.

Or make a variant for it. That seems to be how the graphviz port has been handling optional dependencies up until this point.

comment:6 Changed 9 years ago by dbevans (David B. Evans)

Again what you're talking about is a different problem altogether from this ticket (which is being addressed in #37604 as you mention).

comment:7 in reply to:  1 Changed 9 years ago by ryandesign (Ryan Schmidt)

Replying to devans@…:

Fix committed in r113653, maintainer timeout.

Thank you but the revision should only have been increased on the main graphviz/graphviz-devel ports, not on their subports.

Note: See TracTickets for help on using tickets.