Hi Cirilo,

that should be no issue because since the main glm.hpp include file takes care 
of the locations of those files.

I checked the current (main trunk) usage of glm on kicad:
https://github.com/KiCad/kicad-source-mirror/search?p=1&q=include+%3Cgal%2Fopengl%2Fglm%2F&utf8=%E2%9C%93
and checked with the two versions, the included files used in kicad's source 
code are in the same place.

I found that in my branch I am using this include: matrix_decompose.hpp
that is not present in 0.9.4 (was introduced in 0.9.6) so latest official 
release (0.9.7) have it already.

Mario
________________________________________
From: Cirilo Bernardo [cirilo.berna...@gmail.com]
Sent: 04 December 2015 20:51
To: Mário Luzeiro
Cc: Wayne Stambaugh; kicad-developers@lists.launchpad.net
Subject: Re: [Kicad-developers] GLM

GLM seems to have changed the names/locations of some header files somewhere
between v0.94 and 0.97. For example 0.94 has core/type_mat4x4.hpp  while 0.97
has mat4x4.hpp + detail/type_mat4x4.hpp. The 'mat4x4.hpp' seems to be a new
file as well. I would prefer to code for the newer release; if we code using 
the older
headers we're only adding unnecessary work in the future if we happen to use the
part of the API which changed.

I thought the GLM code may have been included because of problems on some
platforms; on Linux the headers are available for installation via the package
managers and it would be easy to implement FindGLM.cmake if one doesn't
already exist.

- Cirilo


On Sat, Dec 5, 2015 at 4:52 AM, Mário Luzeiro 
<mrluze...@ua.pt<mailto:mrluze...@ua.pt>> wrote:
Hi Wayne,

GLM library is already used by GAL on pcbnew so I adopted it also for 3d-viewer.
The current version of the lib on kicad is 0.9.4, the official GLM release is 
at moment on 0.9.7.1

I don't remember why I updated (0.9.8) it in my branch but since I am using 
just some basic functions, my code should work with the stable GLM release and 
also with the (old) 0.9.4 version on Kicad.
I will check if my code works with the current one used in kicad and let you 
and Orson decide if it useful to updated it or not.

btw, GLM is provide as an header file only library, so only need to have 
included the proper path to it (that kicad already have it working)

Mario

________________________________________
From: Kicad-developers 
[kicad-developers-bounces+mrluzeiro=ua...@lists.launchpad.net<mailto:ua...@lists.launchpad.net>]
 on behalf of Wayne Stambaugh 
[stambau...@gmail.com<mailto:stambau...@gmail.com>]
Sent: 04 December 2015 17:08
To: 
kicad-developers@lists.launchpad.net<mailto:kicad-developers@lists.launchpad.net>
Subject: Re: [Kicad-developers] GLM

Is the GLM library a new dependency for the 3D viewer?  If so, why are
you not using the current released version?  I do not want to be on a
continuous cycle of using unreleased libraries.  In the past this has
caused us a lot of grief.  This also creates maintenance headaches and
issues for package developers.  Please consider this carefully.  I will
reject any change that requires KiCad to build any dependencies.  If we
cannot use FindSomeLibrary.cmake, then it's a no go as far as I'm
concerned.  I'm not terribly thrilled with the libraries as source we
include to get KiCad to build but that is tolerable.  I'm not going to
take the draconian route and say I will never accept it but we need to
think any new dependency carefully.

On 12/4/2015 3:49 AM, Mário Luzeiro wrote:
> Hi Orson,
>
> Cirilo got the version from my branch and I am using it since I started my 
> branch.
> I guess there are no issues, so it maybe now a good idea (after the release) 
> to update some libraries.
> The 0.9.8 is the version of the trunk of GLM (the official in website still 
> at 0.9.7 )
> https://github.com/g-truc/glm/tree/master/glm
>
> Mario
> ________________________________________
> From: Kicad-developers 
> [kicad-developers-bounces+mrluzeiro=ua...@lists.launchpad.net<mailto:ua...@lists.launchpad.net>]
>  on behalf of Maciej Sumiński 
> [maciej.sumin...@cern.ch<mailto:maciej.sumin...@cern.ch>]
> Sent: 04 December 2015 08:37
> To: Cirilo Bernardo; KiCad Developers
> Subject: Re: [Kicad-developers] GLM
>
> Hi Cirilo,
>
> I do not see any obstacles. Are you already testing the new version? If
> everything seems fine, then I will update it in the product branch.
>
> Regards,
> Orson
>
> On 12/03/2015 10:10 PM, Cirilo Bernardo wrote:
>> While looking into preparing an initial merge of the 3D refactor branch I
>> noticed that kicad has the GLM v0.94.  The newer rendering code makes
>> use of GLM v0.98.  Are there any objections to replacing GLM with the
>> later version?
>>
>> - Cirilo
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : 
> kicad-developers@lists.launchpad.net<mailto:kicad-developers@lists.launchpad.net>
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>


_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : 
kicad-developers@lists.launchpad.net<mailto:kicad-developers@lists.launchpad.net>
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : 
kicad-developers@lists.launchpad.net<mailto:kicad-developers@lists.launchpad.net>
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to