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

Removed Alpine repository for vips #135

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

AranGarcia
Copy link

Why is this change necessary?

Apparently, there's been an issue with inconsistency between shared object libraries in the Alpine distribution and other dependencies required for this project. It's difficult to pinpoint the exact dependency that causes this, but previous Docker builds throw an error similar to this one:

...[omitted logs]
#7 24.98 Executing gdk-pixbuf-2.42.10-r6.trigger
#7 24.98 Error relocating /usr/lib/libgio-2.0.so.0: statx: symbol not found
#7 24.98 Error relocating /lib/libmount.so.1: statx: symbol not found
#7 24.98 ERROR: gdk-pixbuf-2.42.10-r6.trigger: script exited with error 127
...[omitted logs]

This seems to be a recurring issue with other repositories that use Alpine. Examples:

How does this solve the issue?

Since vips appears to be included in the specified Docker image distribution, we will use the default repository instead of http://dl-3.alpinelinux.org/alpine/edge/main.

@AranGarcia AranGarcia self-assigned this Apr 4, 2024
Copy link

sonarqubecloud bot commented Apr 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@AranGarcia AranGarcia merged commit 1ae7780 into develop Apr 4, 2024
6 checks passed
@AranGarcia AranGarcia deleted the fix/dockerfile/alpine-vips branch April 4, 2024 16:03
@Arjona99 Arjona99 mentioned this pull request Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants