Skip to content

Commit

Permalink
Remove FUSE installation since we're now on a binary
Browse files Browse the repository at this point in the history
FUSE was needed for ImageMagick's AppImage to work. Since we're now
back to a self-built binary as of #341, this is no longer needed.
  • Loading branch information
brandur committed Sep 26, 2023
1 parent 65eadb0 commit 5cc661e
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,16 +44,14 @@ jobs:
# It'd be nice to take apt-get back out of the flow since it's so slow,
# but we seem to need it for a few things:
#
# * FUSE is needed to run AppImages, which are needed for ImageMagick.
#
# * pngquant has no particularly easy alternative install options.
#
# * WebPs are run through `cbwebp` (which comes from the webp package)
# by ImageMagick. This may have changed when we went back to the
# AppImage version of ImageMagick, but I haven't tested it.
#
- name: Install FUSE + PNGQuant + WebP
run: sudo apt-get install fuse pngquant webp
- name: Install PNGQuant + WebP
run: sudo apt-get install pngquant webp

# The copy of ImageMagick we could get from apt-get is ancient and
# doesn't handle niceties like `.heic` files, so here we get a binary
Expand Down Expand Up @@ -156,8 +154,8 @@ jobs:

steps:
# See notes in check_and_test.
- name: Install FUSE + PNGQuant + WebP
run: sudo apt-get install fuse pngquant webp
- name: Install PNGQuant + WebP
run: sudo apt-get install pngquant webp

# See notes in check_and_test.
- name: Install ImageMagick
Expand Down Expand Up @@ -221,8 +219,8 @@ jobs:

steps:
# See notes in check_and_test.
- name: Install FUSE + PNGQuant + WebP
run: sudo apt-get install fuse pngquant webp
- name: Install PNGQuant + WebP
run: sudo apt-get install pngquant webp

# See notes in check_and_test.
- name: Install ImageMagick
Expand Down

0 comments on commit 5cc661e

Please sign in to comment.