Opened 4 years ago

Closed 4 years ago

#48231 closed update (fixed)

doxygen @1.8.9.1: update to 1.8.10

Reported by: Schamschula (Marius Schamschula) Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version: 2.3.3
Keywords: haspatch Cc: ryandesign (Ryan Schmidt)
Port: doxygen

Description

doxygen has been updated to version 1.8.10.

Instead of tmake the new version uses cmake. The means a number of adjustments. My attached patches have only been tested with the default variant.

I tried to build the docs variant, and found that the build stage attempted to copy files into /opt/local. This should only occur in the destroot phase.

Attachments (5)

patch-libiconv.diff (3.0 KB) - added by Schamschula (Marius Schamschula) 4 years ago.
patch-src-portable_c.c.diff (641 bytes) - added by Schamschula (Marius Schamschula) 4 years ago.
Portfile-doxygen.diff (6.4 KB) - added by Schamschula (Marius Schamschula) 4 years ago.
patch-doc-CMakeLists.txt.diff (578 bytes) - added by Schamschula (Marius Schamschula) 4 years ago.
patch-examples-CMakeLists.txt.diff (638 bytes) - added by Schamschula (Marius Schamschula) 4 years ago.

Download all attachments as: .zip

Change History (9)

Changed 4 years ago by Schamschula (Marius Schamschula)

Attachment: patch-libiconv.diff added

comment:1 Changed 4 years ago by ryandesign (Ryan Schmidt)

Cc: ryandesign@… added; css@… removed
Owner: changed from macports-tickets@… to css@…

There are numerous issues to be resolved here before this could be committed, but a big one is the libiconv patch you attached to prevent the use of MacPorts libiconv. That won't work, because the patch includes details specific to your MacPorts installation, including the MacPorts prefix, the location of Xcode, the build architecture, the C++ std library, the deployment target, the location of your ports tree, maybe more.

We want to build with MacPorts libiconv. The missing symbols error you saw when building with MacPorts libiconv should be reported to the developers of doxygen so they can fix it.

The missing symbols error often indicates that the headers of MacPorts libiconv are being used while the library of OS X libiconv is being used, or vice versa.

Looks like there are cmake variables that one should be able to set to tell the build system which libiconv to use (ICONV_INCLUDE_DIR:PATH and ICONV_LIBRARY:FILEPATH) but setting these has no effect. I suspect this is another symptom of the same problem.

Changed 4 years ago by Schamschula (Marius Schamschula)

Attachment: patch-src-portable_c.c.diff added

comment:2 Changed 4 years ago by Schamschula (Marius Schamschula)

The updated Portfile (and accompanying patch file) fixes the libiconv issue: we now use MacPorts iconv.

Changed 4 years ago by Schamschula (Marius Schamschula)

Attachment: Portfile-doxygen.diff added

Changed 4 years ago by Schamschula (Marius Schamschula)

Changed 4 years ago by Schamschula (Marius Schamschula)

comment:3 Changed 4 years ago by Schamschula (Marius Schamschula)

The latest update fixes the docs and wizard variants.

comment:4 Changed 4 years ago by neverpanic (Clemens Lang)

Resolution: fixed
Status: newclosed

Committed in r143539 with the following changes:

  • patch doc/CMakeLists.txt to install manpages in share/man
  • add explicit cmake.out_of_source no, because out of source build fails
  • backport compatibility fixes for flex 2.6, see #49881
Note: See TracTickets for help on using tickets.