mypaint / libmypaint

libmypaint, a.k.a. "brushlib", is a library for making brushstrokes which is used by MyPaint and other projects.
http://mypaint.org
Other
307 stars 87 forks source link

Meson #170

Closed Salamandar closed 2 years ago

Salamandar commented 4 years ago

I continued the work of @QuLogic , refactored a bit.

jplloyd commented 4 years ago

Switching the dist to xenial or bionic should fix the meson dependency, though the xenial one is probably too old.

Salamandar commented 4 years ago

@jplloyd unfortunately Travis only supports up to Bionic, and Meson 0.49.0 is only provided since Disco :'( https://packages.ubuntu.com/search?keywords=meson

jplloyd commented 4 years ago

@jplloyd unfortunately Travis only supports up to Bionic, and Meson 0.49.0 is only provided since Disco :'( https://packages.ubuntu.com/search?keywords=meson

If meson >= 0.49 is essential, we can run the meson build on a docker image instead.

Here's an example of a travis configuration using docker for the actual build: https://github.com/mypaint/mypaint-appimage/blob/master/.travis.yml

Salamandar commented 4 years ago

I'll try that.

Salamandar commented 4 years ago

OK, I think I had a hold of it. I'll squash the commits to have a clean history.

jplloyd commented 4 years ago

Since the commit history will have to be cleaned up before merge anyway, I suggest you add [skip appveyor] to any further commit messages (first line), as there are currently 4 identical appveyor builds queued up.

Salamandar commented 4 years ago

Yeah, travis succeedes, I just need to add doxygen inside the docker :)

Salamandar commented 4 years ago

Weirdly enough, installing sphinx requires timezone configuration… So I removed the docs test.

jplloyd commented 4 years ago

... and when you're only changing the appveyor setup you can add [skip travis] (no underscores) to the first line of the commit message (I was a bit unclear last time - as long as it's on the first line travis/appveyor will respect it).

Salamandar commented 4 years ago

The Gegl test on Window fails a weird way : it does not find internal Gegl libraries. Maybe that's why gegl was not installed on appveyor ?

Salamandar commented 4 years ago

Weird, I fixed the Travis build, why did it not run again ?

jplloyd commented 4 years ago

I'll review the changes on Saturday and then we can work on speeding up the CI.

nielsdg commented 2 years ago

Any possibility of this making any progress @jplloyd or @Salamandar ? :)