mingw-cross-env-list
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Mingw-cross-env-list] [PATCH] Add OpenSceneGraph


From: Volker Grabsch
Subject: Re: [Mingw-cross-env-list] [PATCH] Add OpenSceneGraph
Date: Tue, 12 Jan 2010 20:52:57 +0100
User-agent: Mutt/1.5.13 (2006-08-11)

Martin Lambers <address@hidden> schrieb:
> On Tue, Jan 12, 2010 at 04:36:38PM +0100, Volker Grabsch wrote:
> > Can anyone provide the correct Debian/FreeBSD/OpenSolaris/...
> > package names for CMake?
> 
> Debian: cmake
> FreeBSD: cmake (according to freshports.org)

Thanks!

CMake is now an official requirement for mingw-cross-env.
http://hg.savannah.gnu.org/hgweb/mingw-cross-env/rev/dbb6e9acf671

> > I think that your comments are great, although a bit verbose.
> > Can't the long compiler line be reduced to something like this?
> > 
> >     i686-pc-mingw32-g++ `pkg-config osc --cflags` `pkg-config osc --libs` 
> > ...
> 
> Unfortunately not, for the following reasons:
> - 'i686-pc-mingw32-pkg-config --libs openscenegraph' lists all base OSG libs,
>   but in the wrong order, so that linking fails.

This is a bug in the openscenegraph package and should be fixed,
either via a patch (which should also be presented to upstream),
or via some $(SED) actions.

> - The plugin library directory (lib/osgPlugins-2.8.2) is not added with -L, 
> and
>   no plugin libraries are reported by pkg-config. This is because OSG is
>   designed to load dynamic plugins at runtime itself.

Nevertheless, the additional -L... option wouldn't harm. If
upstream refuses that addition, consider adding it conditionally,
i.e. only when openscenegraph was built for static linking.

If you need any help with that, please let me know.

> > Anyway, I think that it shouldn't go into the Makefile, because
> > it's somewhat hidden there. I think it would fit well in the
> > tutorial as:
> > 
> >     Step 5d: Cross compile your Project (OpenSceneGraph)
> > 
> > Would you mind providing a documentation patch for "doc/index.html"
> > of the current development version?
> 
> I have a attached a patch.

Thanks. Except for the overly long compile command, it looks great!


Greets,

    Volker

-- 
Volker Grabsch
---<<(())>>---
Administrator
NotJustHosting GbR




reply via email to

[Prev in Thread] Current Thread [Next in Thread]