Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#41422 closed defect (fixed)

graphviz: missing poppler dependency

Reported by: dbevans (David B. Evans) Owned by: ryandesign (Ryan Carsten 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 11 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed

Fix committed in r113653, maintainer timeout.

comment:2 Changed 11 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 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:4 Changed 11 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 11 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 11 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 11 years ago by ryandesign (Ryan Carsten 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.