Opened 17 years ago

Closed 16 years ago

Last modified 16 years ago

#14603 closed update (fixed)

UPDATE: doxygen 1.5.4->1.5.5

Reported by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version: 1.6.0
Keywords: Cc: cssdev, MarcusCalhoun-Lopez (Marcus Calhoun-Lopez), ryandesign (Ryan Carsten Schmidt)
Port: doxygen

Description

Doxygen has been updated to 1.5.5.

The update takes into account the iconv difference on Leopard.

Attachments (5)

Portfile.diff (1.7 KB) - added by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) 17 years ago.
Portfile2.diff (3.9 KB) - added by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) 17 years ago.
qfiledefs_p.patch (482 bytes) - added by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) 17 years ago.
Portfile3.diff (3.9 KB) - added by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) 17 years ago.
Portfile-1.5.5_command_line.diff (1.8 KB) - added by cssdev 17 years ago.
Builds the 1.5.5 command line tool

Download all attachments as: .zip

Change History (22)

Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: Portfile.diff added

comment:1 Changed 17 years ago by cssdev

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

Well that certainly makes the port much cleaner!

Unfortunately I get a compile error on Tiger:

c++ -c -pipe -D__FreeBSD__=6 -DQT_NO_CODECS -DQT_LITE_UNICODE -Wall -W -O2 -I. -o ../objects/qdir.o qdir.cpp
qfiledefs_p.h:258: error: field 'st' has incomplete type
make[2]: *** [../objects/qdir.o] Error 1
make[1]: *** [all] Error 2
make: *** [all] Error 2

comment:2 Changed 17 years ago by cssdev

It looks like a patch can resolve the problem, although it might have other implications.

I tried defining it as (MAC_OS_X_VERSION_MAX_ALLOWED>=MAC_OS_X_VERSION_10_4), but I don't know whether that breaks things for Leopard.

comment:3 Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

The patch to qfiledefs_p.h (which I have attached for completeness) has no effect in Leopard.

I found that the previous Portfile I submitted did not work with the wizard variant.
The attached Portfile fixes this (at least on Loepard).
I could not get the creation of Doxywizard.app to work however.

Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: Portfile2.diff added

Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: qfiledefs_p.patch added

comment:4 Changed 17 years ago by cssdev

Could you attach the errors you received trying to build Doxywizard.app? I'll see what happens on Tiger. Thanks for the additional patches!

comment:5 Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Doxywizard.app builds with no problem.
When I double click on the application, it does not start.
When I use the command line (open Doxywizard.app), I get the error:
LSOpenFromURLSpec() failed with error -10810

comment:6 Changed 17 years ago by cssdev

It looks like that's an error from qmake improperly creating the Info.plist. It quotes the application name.

comment:7 Changed 17 years ago by cssdev

Rather than doing all the search-and-replace within the wizard makefiles, wouldn't it be easier to run the MacPorts version of qmake to pick up everything automatically? I have to admit that I never use the wizard or Qt, as I usually just run everything from the command line.

comment:8 Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

doxygen, for whatever reason, does not seem to use qmake.
In the source directory addon/doxywizard, there is a Makefile which builds doxywizard.
To use qmake, I believe it would be necessary to generate and maintain the qmake file ourselves.
I have not invested any effort into this option, but it seems to me that it would make maintaining the port more difficult.
Although it is a bit annoying to have to manually add -I and -L flags using reinplace, I would argue that it is the easier solution.

I have never used doxywizard either (and I am no expert in Qt) so I am just offering my best guess.

comment:9 Changed 17 years ago by cssdev

Building the wizard fails on tiger ... the inconsistency in the use of iconv is infuriating! I might just update the port sans wizard for the time being.

comment:10 Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Removing the wizard variant is certainly an option.
Since I do not have a Tiger machine to work with, I can not test anything.
I made a best guess as to the problem if you want to try one last patch.

Who knew that a minor version bump would be so much work?

Changed 17 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: Portfile3.diff added

Changed 17 years ago by cssdev

Builds the 1.5.5 command line tool

comment:11 Changed 17 years ago by cssdev

Could someone with Leopard check building the command line tool with this latest patch? I want to go ahead and get the command line update pushed out rather than waiting until the wizard is completely resolved.

comment:12 Changed 17 years ago by raimue (Rainer Müller)

Builds fine for me here on Leopard.

$ port installed doxygen
The following ports are currently installed:
  doxygen @1.5.5_0 (active)

I had to rename files/qfiledefs_p.patch -> files/patch-qfiledefs_p.h.

comment:13 Changed 17 years ago by cssdev

I committed the 1.5.5 command line update in r36012, but I'm still trying to resolve using the wizard on both Tiger and Leopard.

comment:14 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Port: doxygen added

If all that's left here is to fix the wizard issue, shouldn't this ticket be closed? The wizard issue is #14368.

comment:15 Changed 16 years ago by cssdev

Resolution: fixed
Status: assignedclosed

Yes, it can be closed. Thanks for the bump! :)

comment:16 Changed 16 years ago by jmroot (Joshua Root)

Type: enhancementupdate

comment:17 Changed 16 years ago by (none)

Milestone: Port Updates

Milestone Port Updates deleted

Note: See TracTickets for help on using tickets.