#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)
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: | new → assigned |
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
Attachment: | Portfile-1.5.5_command_line.diff added |
---|
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: | assigned → closed |
Yes, it can be closed. Thanks for the bump! :)
comment:16 Changed 16 years ago by jmroot (Joshua Root)
Type: | enhancement → update |
---|
Well that certainly makes the port much cleaner!
Unfortunately I get a compile error on Tiger: