graphviz: missing poppler dependency

[dbevans (David B. Evans)]

If poppler is installed, graphviz builds plugin

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

but poppler is not present in its dependency tree.

[dbevans (David B. Evans)]

Fix committed in r113653, maintainer timeout.

[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?

[cooljeanius (Eric Gallager)]

Cc Me!

[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.

[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.

[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).

[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.