FS#63829 - [glm] CMake discovery broken in 0.9.9.6-1

Attached to Project: Community Packages
Opened by Paul Melis (paulmelis) - Wednesday, 18 September 2019, 19:31 GMT
Last edited by Sven-Hendrik Haase (Svenstaro) - Friday, 27 September 2019, 12:36 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sven-Hendrik Haase (Svenstaro)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

CMake can no longer find 0.9.9.6-1 as files in `/usr/lib/cmake/glm/` are no longer included in the package.

```
paulm@cmstorm 21:30:~$ pacman -Qi glm
Name : glm
Version : 0.9.9.5-1
Description : C++ mathematics library for 3D software based on the OpenGL Shading Language (GLSL) specification
Architecture : any
URL : http://glm.g-truc.net
Licenses : MIT
Groups : None
Provides : None
Depends On : None
Optional Deps : None
Required By : None
Optional For : None
Conflicts With : None
Replaces : None
Installed Size : 19.71 MiB
Packager : Sven-Hendrik Haase <svenstaro@gmail.com>
Build Date : di 09 apr 2019 16:34:06 CEST
Install Date : do 11 apr 2019 16:32:39 CEST
Install Reason : Explicitly installed
Install Script : No
Validated By : Signature

paulm@cmstorm 21:30:~$ pacman -Ql glm |grep cmake
glm /usr/lib/cmake/
glm /usr/lib/cmake/glm/
glm /usr/lib/cmake/glm/glmConfig.cmake
glm /usr/lib/cmake/glm/glmConfigVersion.cmake
glm /usr/lib/cmake/glm/glmTargets.cmake

melis@juggle 21:31:~$ pacman -Qi glm
Name : glm
Version : 0.9.9.6-1
Description : C++ mathematics library for 3D software based on the OpenGL Shading Language (GLSL) specification
Architecture : any
URL : http://glm.g-truc.net
Licenses : MIT
Groups : None
Provides : None
Depends On : None
Optional Deps : None
Required By : pymol
Optional For : None
Conflicts With : None
Replaces : None
Installed Size : 17.83 MiB
Packager : Sven-Hendrik Haase <svenstaro@gmail.com>
Build Date : Tue 10 Sep 2019 01:35:17 AM CEST
Install Date : Wed 18 Sep 2019 06:22:38 PM CEST
Install Reason : Explicitly installed
Install Script : No
Validated By : Signature

melis@juggle 21:31:~$ pacman -Ql glm|grep cmake
melis@juggle 21:31:~$
```
This task depends upon

Closed by  Sven-Hendrik Haase (Svenstaro)
Friday, 27 September 2019, 12:36 GMT
Reason for closing:  Fixed
Comment by Antonio Rojas (arojas) - Wednesday, 18 September 2019, 19:54 GMT
This looks like an intentional upstream change
https://github.com/g-truc/glm/commit/5f352ecce21bb1ab37fa56fac0f383c779b351a3
Comment by Eli Schwartz (eschwartz) - Wednesday, 18 September 2019, 20:22 GMT
So, how many of the packages that depend on this are totally broken by that change?

EDIT: It seems unwise to update to this version in the first place. New versions of stuff is good, but not when it renders actual usage of the package impossible and breaks all its reverse dependencies.
Comment by Eli Schwartz (eschwartz) - Wednesday, 18 September 2019, 20:34 GMT
There is an upstream issue opened at https://github.com/g-truc/glm/issues/947 where a number of people are asking why this change occurred. The upstream developer is, sadly, being evasive and giving non-answers -- it's unclear what is going on.

I encourage you to lend your voice there.
Comment by Sven-Hendrik Haase (Svenstaro) - Wednesday, 18 September 2019, 22:31 GMT
I was unsure about this change myself but I really thought upstream had just kind of figured it out and the removal was a triviality. The latter might have been, I thought, because upstream knew that every software agreed about the canonical location of glm and thus didn't need files to locate it.

What I didn't expect was that this was an entirely unnecessary, breaking, and unprovoked change. Frankly, I might just roll the package back or provide those CMake files as a fixed part of the downstream packaging (which I despise, but if this is the way forward, then so be it).
Comment by Sven-Hendrik Haase (Svenstaro) - Sunday, 22 September 2019, 01:57 GMT
Please test rel -2.

Loading...