Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Dockerfile to Bullseye and updated libvips #384

Merged
merged 1 commit into from
Feb 20, 2022

Conversation

ItalyPaleAle
Copy link
Contributor

I made this change in my fork and I thought it would help upstream too.

The core of the change was updating the base image from Buster to Bullseye. By doing that, libjemalloc is now available in the official Debian repos in the version that was used before, so it is possible to install the package without having to build it from source: this should keep the container much smaller.

Also, I've updated libvips to the latest version.

1. Changed base image to bullseye
2. The updated base image contains an updated version of libjemalloc too, so building from source is no longer necessary
3. Updated libvips version too
libgsf-1-114 fftw3 liborc-0.4-0 librsvg2-2 libcfitsio7 libimagequant0 libheif1 && \
libmagickwand-6.q16-6 libpango1.0-0 libmatio11 libopenslide0 libjemalloc2 \
libgsf-1-114 fftw3 liborc-0.4-0 librsvg2-2 libcfitsio9 libimagequant0 libheif1 && \
ln -s /usr/lib/$(uname -m)-linux-gnu/libjemalloc.so.2 /usr/local/lib/libjemalloc.so && \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

About this: the library that is installed from the DEB package goes in a folder whose name is dependent on the architecture of the system (e.g. aarch64). This made it more complicated to set an ENV var in the container. So I thought it'd be easier to just keep a symlink to the same path where the library was installed (from source) earlier.

@ItalyPaleAle ItalyPaleAle mentioned this pull request Feb 20, 2022
@h2non h2non merged commit 9489834 into h2non:master Feb 20, 2022
@ItalyPaleAle ItalyPaleAle deleted the update-dockerfile branch July 28, 2022 04:24
wslaghekke added a commit to recognizegroup/imaginary that referenced this pull request Feb 1, 2023
* Fixed Timeout in readme, 30 ~> 60 (h2non#340)

* Fix invalid parameters "-path-prefix" (h2non#344)

* fix: small errors in docs (h2non#346)

* fix: use proper formatter for usage template (h2non#347)

Co-authored-by: Mads Moeller <madsmm@gmail.com>

* Add Cloud Run Button (h2non#362)

* Delete app.json

* Update README.md

* Update README.md

* updated docker builder OS to go version 1.17 (h2non#371)

* fix(readme): remove gocard obsolete badge

* feat(readme): update placeholder description

* fix(readme): update fly deploy tutorial

* fix(docs): allowed-origins examples h2non#333

* memory leak issue fixed with jemalloc (h2non#381)

* exposed palette from GET endpoints (h2non#380)

* Updated Dockerfile (h2non#384)

1. Changed base image to bullseye
2. The updated base image contains an updated version of libjemalloc too, so building from source is no longer necessary
3. Updated libvips version too

* Added dev container (h2non#385)

* Added dev container

* Removed irrelevant lines

* allow speed from get (h2non#383)

* allow speed from get

* updating the version to use effor param in libvips

* Return with and heigh of the generated images (h2non#382)

* Return with and heigh of the generated images

Use case:

When using the fit image transformation, it is helpful to know the size
of the resulting image without having to either read the image locally
or do another request to the info endpoint.

Used in: nextcloud/server#24166

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

* Make mimetype support always return true

* Add command line option to enable this feature

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

* refactor: remove deprecated X- prefix in response headers

Co-authored-by: Tom <tomas@aparicio.me>

* Decompression exploit check (h2non#404)

* Bump bimg version to 1.1.7

* Add decompression bomb exploit check

* Update README with new flag

* Fix tests

* Fix typos (h2non#405)

Found via `codespell -S .git`.

---------

Co-authored-by: Julian <2564520+judomu@users.noreply.github.com>
Co-authored-by: liuxu <i@liuxu.me>
Co-authored-by: 0xflotus <0xflotus@gmail.com>
Co-authored-by: Mads Moeller <mmoeller@users.noreply.github.com>
Co-authored-by: Mads Moeller <madsmm@gmail.com>
Co-authored-by: James Ward <james@jamesward.com>
Co-authored-by: Angelo Girardi <angelo.girardi@goat.com>
Co-authored-by: Tom <tomas@aparicio.me>
Co-authored-by: Vaibhav Sharma <vaib.sharma44@gmail.com>
Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
Co-authored-by: Vaibhav Sharma <vaibhavsharma.v@udaan.com>
Co-authored-by: Carl Schwan <carl@carlschwan.eu>
Co-authored-by: SeaaaaaSharp <98841125+SeaaaaaSharp@users.noreply.github.com>
Co-authored-by: Kian-Meng Ang <kianmeng.ang@gmail.com>
suntong pushed a commit to suntong/imaginary that referenced this pull request Nov 28, 2023
1. Changed base image to bullseye
2. The updated base image contains an updated version of libjemalloc too, so building from source is no longer necessary
3. Updated libvips version too
suntong pushed a commit to suntong/imaginary that referenced this pull request Nov 29, 2023
1. Changed base image to bullseye
2. The updated base image contains an updated version of libjemalloc too, so building from source is no longer necessary
3. Updated libvips version too
gkmw pushed a commit to gkmw/imaginary that referenced this pull request Jun 19, 2024
1. Changed base image to bullseye
2. The updated base image contains an updated version of libjemalloc too, so building from source is no longer necessary
3. Updated libvips version too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants