All my dev machines are fedora, so this is the most tested distro.

This is a squashed commit, resolving two issues:

- A typo in the yaml stage tag
- Trying to update rather than just install deps

d#on't bother update, just install deps

This was previously 7 attempts at fixing the build.
However, none worked. The commits were all just altering
the build deps.

Joseph Walton-Rivers authored 3 years ago and Joseph Walton-Rivers committed 3 years ago

This is a merged commit of a bunch of typos and whatnot.

This is only a first approxmiation of what I would like for CI. Ideally,
it'd be it's own step that can pass/fail regardless of the build status.
With this setup, it's integrated into the build (and also running tidy
over our vendored dependencies, which are outside our scope).

Something like this: http://www.stablecoder.ca/2018/10/30/full-cmake-helper-suite.html
might be better for the future...

Mesh now has a (fairly primative) method for removing unused and
duplicated verticies. This means its possible to use conbined
geometry without duplicate verticies.

In my test scene, this halved the number of verticies in the combined
mesh.

There are plenty of other optimisations that could be implemented. Some
that would be nice to have but I don't have time for:

* Removal of internal faces in combined meshes
* Optimise triangle order so GL_TRIANGLE_STRIP or the like can be used

The lighting bug in the previous commit was caused by the fact that the
combined (averaged) normals don't work if there should be a clear seam
this commit duplicates each vertex.