Re: [orbitcpp-list] Build issues and questions

[Sam C. <sa...@to...>]

Braden McDaniel <br...@en...> wrote:
>         1. The current build overwrites the user variables CFLAGS and
>            CXXFLAGS. [ ... ] I have fixed
>            this and I can provide a patch.

Yes please. Use the Patch Manager at SourceForge.

>         2. The build system winds up redundantly defining some
>            variables, and automake 1.5 spews warnings as a result. I
>            have fixed this and I can provide a patch.

Yes please. Use the Patch Manager at SourceForge.

I'm currently finishing the final year of my university degree, so I
won't be able to look at these patches for a few weeks, but after that
I'll have all the time in the world. :)

>         3. The project does the exact opposite of what the libtool
>            documentation recommends, and ties it's package version
>            number to the library version number. How come?

No idea. That's how it was before I got involved. It should probably be
fixed.

>         4. A number of the tests build with "-static" in CXXFLAGS.
>            AFAICT, this is a linker option, so it should go in LDFLAGS.
>            But why is it being used?

No idea, and it probably shouldn't be used.

>         5. The build currently creates and installs a static library for
>            the IDL compiler. Unless I'm missing something, this is
>            completely useless. I think I can fix this, but I think it
>            would involved creating an additional configure script just
>            for the compiler. This script would be called by the main
>            configure script (see AC_CONFIG_SUBDIRS). I'd be happy to
>            work on this, but first I wanted to make sure there's
>            interest in such a change.

It is useless, as it's a dynamically loaded plugin for orbit-idl. It's
never linked against directly.

But it doesn't cost much to build the static version, so don't put too
much effort into making it not build the static version. I think making
an additional configure script just for the compiler is a bad idea.
--=20
Sam Couter          |   Internet Engineer   |   http://www.topic.com.au/
sa...@to...    |   tSA Consulting      |
OpenPGP key ID:       DE89C75C,  available on key servers
OpenPGP fingerprint:  A46B 9BB5 3148 7BEA 1F05  5BD5 8530 03AE DE89 C75C