Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#28426 closed enhancement (fixed)

doxygen @1.7.3 patch for Tcl support

Reported by: l2g@… Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: doxygen

Description

A developer named Rene Zaumseil has written code to add support for Tcl to Doxygen. He's submitted a patch to the Doxygen devs, but they have not approved it yet. Fortunately, it looks like he's been dedicated enough to update it whenever there's a new Doxygen release.

I thought it might be of interest to other MacPorts developers as a possible alternative to tcl-dox (the author has orphaned it).

There is probably a better way to package this than I have--especially if someone can figure out how to have it fetched from Sourceforge mirrors rather than included in the other port files.

Attachments (2)

doxygen+tcl.patch (78.7 KB) - added by l2g@… 13 years ago.
Modified version of Rene Zaumseil's patch for Doxygen Tcl support
Portfile.diff (1.8 KB) - added by l2g@… 13 years ago.
Add a patch for Tcl support in Doxygen (the other patch attached to this ticket can be ignored)

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by l2g@…

Attachment: doxygen+tcl.patch added

Modified version of Rene Zaumseil's patch for Doxygen Tcl support

comment:1 Changed 13 years ago by l2g@…

Owner: changed from macports-tickets@… to css@…

comment:2 in reply to:  description ; Changed 13 years ago by jmroot (Joshua Root)

Replying to l2g@…:

There is probably a better way to package this than I have--especially if someone can figure out how to have it fetched from Sourceforge mirrors rather than included in the other port files.

You'd do this by not putting the patch in files/, setting patch_sites, and setting checksums for the patchfile.

comment:3 in reply to:  2 Changed 13 years ago by l2g@…

Replying to jmr@…:

I tried this but was confounded in my first attempts. Now I think I've finally figured it out, so thank you for your prompting.

The patch is downloaded from here:

http://sourceforge.net/projects/kbskit/files/doxygen%2Btcl/

...so I was trying to set patch_sites to "sourceforge:kbskit" and play with the value of patch_sites.mirror_subdir. I finally figured out that it would work if I just set patch_sites to "sourceforge:kbskit/doxygen\+tcl".

I'll post revised changes.

Changed 13 years ago by l2g@…

Attachment: Portfile.diff added

Add a patch for Tcl support in Doxygen (the other patch attached to this ticket can be ignored)

comment:4 Changed 13 years ago by cssdev

Status: newassigned

Thanks for the patch! I'll give it a try.

comment:5 Changed 13 years ago by cssdev

Resolution: fixed
Status: assignedclosed

added in r77277. thanks!

comment:6 Changed 13 years ago by jmroot (Joshua Root)

Hardcoding the distfile name in the checksums isn't a great idea; use ${distname}${extract.suffix}.

Note: See TracTickets for help on using tickets.