libvisio-0.1 @0.1.1 build fails on Leopard/PPC

[khepler]

Building libvisio-0.1 fails on Leopard/PPC (PowerPC G5) with the error:

...snip...
/opt/local/include/boost/spirit/home/classic/symbols/symbols.hpp:105: warning: declaration of 'result' shadows a member of 'this'  CXX      VSDGeometryList.lo
  CXX      VSDOutputElementList.lo  CXX      VSDPages.lo
  CXX      VSDParagraphList.lo
  CXX      VSDParser.lo
  CXX      VSDMetaData.lo
VSDMetaData.cpp:229: error: integer constant is too large for 'long' type
make[3]: *** [VSDMetaData.lo] Error 1
make[3]: *** Waiting for unfinished jobs....
...snip...

The offending line in VSDMetaData.cpp is:

static const uint64_t epoch = 11644473600;

Is this considered an upstream bug or should it be patched in Macports because the compiler it chose assumes a non-suffixed literal integer is a long?

Secondly, after applying the supplied patch (that appends ULL to the literal), the build fails if doxygen is not installed.

[larryv (Lawrence Velázquez)]

In future tickets, please Cc relevant port maintainers (port info --maintainers libvisio-0.1).

[zzanderr]

Replying to khepler@…:

Thanks for the patch. I have been having the same problem. After patching, libvisio-0.1 built and installed fine for me. I will see if I can get the latest inkscape to work with it and let you know how it goes on this post.

I have doxygen installed, so I am unable to give you any info on whether libvisio-0.1 would fail if this were not the case.

(PPC PowerBook G4 1.67 GHz with Leopard 10.5.8)

[dbevans (David B. Evans)]

Fix committed in r133352 which

• adds patch provided to fix build on 10.5.8, appears to build without problem on later platforms.
• add +docs variant to enable build of documentation using doxygen, disabled by default.

Please confirm whether this fixes your Leopard issues or not.

Thanks for the report and the solution. I don't have a working Leopard system any more so I have to rely on those who do for solutions to problems like this.

[khepler]

Builds fine now without +docs variant.

Having trouble testing +docs because doxygen is erroring. But +docs properly sets doxygen as a dependency at least.

edit: Looks like there's some trouble when passing +docs down to doxygen: #43894

libvisio-0.1 +docs builds fine when doxygen is built beforehand without +docs.

[dbevans (David B. Evans)]

Replying to khepler@…:

Builds fine now without +docs variant.

Good!

Having trouble testing +docs because doxygen is erroring. But +docs properly sets doxygen as a dependency at least.

edit: Looks like there's some trouble when passing +docs down to doxygen: #43894

libvisio-0.1 +docs builds fine when doxygen is built beforehand without +docs.

Yes, that's why I added the +docs variant as the doxygen problem is/has been a known issue on this platform.

Thanks for verifying the fix.