From: Pascal Stumpf <pascal@stumpf.co>
Subject: Re: [new] devel/collada-dom
To: Kirill Bychkov <kirby@linklevel.net>, ports@openbsd.org
Date: Fri, 14 Nov 2025 13:30:29 +0100

On Fri, 14 Nov 2025 12:13:55 +0000, Stuart Henderson wrote:
> On 2025/11/14 11:17, Pascal Stumpf wrote:
> > On Wed, 12 Nov 2025 11:15:53 +0300, "Kirill Bychkov" wrote:
> > > On Sat, November 8, 2025 21:17, Pascal Stumpf wrote:
> > > > This is required for OpenMW 0.50.0.
> > > >
> > > > OK?
> > > >
> > > >
> > > > COLLADA defines an XML-based schema to make it easy to transport
> > > > 3D assets between applications - enabling diverse 3D authoring and
> > > > content processing tools to be combined into a production pipeline.
> > > > The intermediate language provides comprehensive encoding of visual
> > > > scenes including: geometry, shaders and effects, physics, animation,
> > > > kinematics, and even multiple version representations of the same
> > > > asset.  COLLADA FX enables leading 3D authoring tools to work
> > > > effectively together to create shader and effects applications and
> > > > assets to be authored and packaged using OpenGL Shading Language,
> > > > Cg, CgFX, and DirectX FX.
> > > >
> > > 
> > > Looks good for me but do we need COLLADA_DOM_SOVERSION in lib name and
> > > other places?
> > > Do we plan to use different versions of collada simultaneously?
> > 
> > Not really, but this is apparently the naming scheme upstream wants, and
> > what is detected by cmake.  
> 
> changing that probably means you'll need to patch consumers, there's no
> point changing that. (same in many other ports).
> 
> graphics would seem a better category than devel.

Alright, I'll change that.

> what are all the files in 1.4/1.5 dirs in include about?

There are different versions of the specification, with both being
supported by the same library, but different header versions.  There are
also three different versions of the pkgconfig file for this reason.

> > > It would be nice to comment/remove
> > > #set (COLLADA_DOM_SOVERSION
> > > ${COLLADA_DOM_VERSION_MAJOR}.${COLLADA_DOM_VERSION_MINOR}) in CMakeListst.txt
> > > and regen PLIST.
> > > 
> > > With this change or not, ok kirby@
> > > 
> >