snapcrafters / gimp

Unofficial GNU Image Manipulation Program (GIMP) installer for Linux
https://snapcraft.io/gimp
MIT License
51 stars 25 forks source link

bump gimp #242

Closed popey closed 8 months ago

popey commented 8 months ago

This updates gimp to the latest release. Had to update git URLs for gmic as it's moved. They also stopped supporting cmake, so switched to qmake.

popey commented 8 months ago

Built and briefly tested locally here.

Screenshot from 2023-10-08 01-41-08

merlijn-sebrechts commented 8 months ago

The build seems to be failing on arm. @popey or @lucyllewy, do you want to continue to get the latest version at least on amd64, or do you want to fix the arm64 issues first?

popey commented 8 months ago

I'm stumped. I don't have an arm64 machine in which to test this. @lucyllewy any ideas?

Building libaom 
+ EXTRA=
+ '[' aarch64-linux-gnu = arm-linux-gnueabihf ']'
+ export DESTDIR=/data/parts/libaom/install
+ DESTDIR=/data/parts/libaom/install
+ cmake /data/parts/libaom/src -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr -DENABLE_DOCS=OFF -DENABLE_EXAMPLES=OFF -DENABLE_TESTDATA=OFF -DENABLE_TESTS=OFF -DCONFIG_PIC=1 -DCONFIG_AV1_ENCODER=1 -DCONFIG_AV1_DECODER=1 -DCONFIG_MULTITHREAD=1
error: unknown command "/data/parts/libaom/src", see 'snap help'.
Failed to run 'override-build': Exit code was 64.
Error: The process '/usr/bin/docker' failed with exit code 2

I can't fathom where that error: unknown command "/data/parts/libaom/src", see 'snap help'. is coming from.

Also, a datapoint: according to the store stats, around 0.2% of users are on the arm64 build. 99.4% are on amd64.

soumyaDghosh commented 8 months ago

I'm stumped. I don't have an arm64 machine in which to test this. @lucyllewy any ideas?

Building libaom 
+ EXTRA=
+ '[' aarch64-linux-gnu = arm-linux-gnueabihf ']'
+ export DESTDIR=/data/parts/libaom/install
+ DESTDIR=/data/parts/libaom/install
+ cmake /data/parts/libaom/src -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr -DENABLE_DOCS=OFF -DENABLE_EXAMPLES=OFF -DENABLE_TESTDATA=OFF -DENABLE_TESTS=OFF -DCONFIG_PIC=1 -DCONFIG_AV1_ENCODER=1 -DCONFIG_AV1_DECODER=1 -DCONFIG_MULTITHREAD=1
error: unknown command "/data/parts/libaom/src", see 'snap help'.
Failed to run 'override-build': Exit code was 64.
Error: The process '/usr/bin/docker' failed with exit code 2

I can't fathom where that error: unknown command "/data/parts/libaom/src", see 'snap help'. is coming from.

Also, a datapoint: according to the store stats, around 0.2% of users are on the arm64 build. 99.4% are on amd64.

I can fix this. We can simply use core22 repo package libaom3, also the ffmpeg-sdk I said about in #243 have aom support, so, no need of it here too.

soumyaDghosh commented 8 months ago

@popey if you help me a little more, I guess my current manifest is in the shape of working, and it's also in core22. We can make it work.

popey commented 8 months ago

@soumyaDghosh, let's continue the conversation about your work on your PR, as it's quite a comprehensive change.

For this PR, which I'm doing minor changes to get 2.10.34 out the door, I have rolled back the version of libaom to a version I know used to build on core20, so it builds on arm64.

I have test-built this in Launchpad. It gets past libaom, so I am somewhat confident this will build in the GitHub action. So I have pushed this one-line change to get it building. Let's see if this works, and if it does, get this merged so we get 2.10.34 built in edge, and can test it.

popey commented 8 months ago

This failed build is a lie and we should not trust it. This PR built perfectly fine in Launchpad, which matters because that's how the published builds in the store are made. Please can this PR be merged @merlijn-sebrechts ? We can investigate the errant GH action separately - which I have filed #244 for.