Opened 13 years ago

Closed 13 years ago

#32514 closed enhancement (fixed)

fix multiple definition of graphviz dependency

Reported by: cjones051073 (Chris Jones) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch maintainer Cc: mf2k (Frank Schima)
Port: root

Description

The last update introduced a multiple declaration of a dependency on graphviz. I only noticed when I started getting warnings like 

Change: http://trac.macports.org/changeset/87901
Portfile: root

Warning: Dependency path:bin/dot:graphviz specified multiple times in depends_lib

Attached patch fixes this.

I didn't update the version or revision, as no need to force a reinstallation for users already running 5.32.00_1

Chris

Chris

Attachments (1)

root.diff​ (1.1 KB) - added by cjones051073 (Chris Jones) 13 years ago.

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by cjones051073 (Chris Jones)

Attachment: root.diff​ added

comment:1 Changed 13 years ago by mf2k (Frank Schima)

Note, please don't edit the $Id: line. Can you run an update and remake your patch? The patch does not apply for me on the latest version. I noticed that ryandesign had updated the portfile after my commit, so your patch might not be against the latest portfile revision. 

$ sudo patch Portfile root.diff 
patching file Portfile
Hunk #1 FAILED at 1.
Hunk #2 FAILED at 31.
Hunk #3 FAILED at 141.
3 out of 3 hunks FAILED -- saving rejects to file Portfile.rej

comment:2 Changed 13 years ago by mf2k (Frank Schima)

Cc: macsforever2000@… added

Cc Me!

comment:3 Changed 13 years ago by cjones051073 (Chris Jones)

Apologies. I forgot to update my local working copy after my last diff was committed, so the ID lines was wrong...

I just synced, and it seems Ryan already fixed this, so this patch is no longer needed....

comment:4 Changed 13 years ago by mf2k (Frank Schima)

Resolution: → fixed
Status: new → closed

Sounds good.

Note: See TracTickets for help on using tickets.