Skip to content

Add debian trixie/Drop debian bullseye #515

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Earlopain
Copy link
Contributor

@Earlopain Earlopain commented Aug 7, 2025

Same as #514 but fixed for ruby 3.2. Took some stuff from docker-library/python#1042 as well.

For Ruby 3.2 libreadline-dev is being installed which causes unwanted output during dpkg-query --search:

dpkg-query --search *lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8.2
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.2.usr-is-merged
libreadline8t64:amd64: /usr/lib/x86_64-linux-gnu/libreadline.so.8.2
libreadline8t64:amd64: /usr/lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8.2
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.2.usr-is-merged

I don't believe there is any way to control this output, so just grep it away. Doesn't hurt to have this run on other versions, even though there aren't any other relevant packages that are affected.

Closes #514

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

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

Nice, thanks for picking this up! Just two tiny nits 👍

versions.sh Outdated
Comment on lines 86 to 87
"trixie",
"bookworm",
Copy link
Member

Choose a reason for hiding this comment

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

Indentation here is wrong: 👀

Suggested change
"trixie",
"bookworm",
"trixie",
"bookworm",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Editor insisted on using spaces here , oops. Should be ok now

Comment on lines 236 to 240
| xargs -r dpkg-query --search \
# diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
# diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
| grep -v "^diversion by" \
| cut -d: -f1 \
Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer to copy the implementation from docker-library/python#1042 instead:

Suggested change
| xargs -r dpkg-query --search \
# diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
# diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
| grep -v "^diversion by" \
| cut -d: -f1 \
| xargs -rt dpkg-query --search \
# https://manpages.debian.org/bookworm/dpkg/dpkg-query.1.en.html#S (we ignore diversions and it'll be really unusual for more than one package to provide any given .so file)
| awk 'sub(":$", "", $1) { print $1 }' \

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Switched over. I didn't have a clue what this was doing, so didn't want to copy it over but after a bit of reading up it makes sense now.

For Ruby 3.2 `libreadline-dev` is being installed which causes
unwanted output during `dpkg-query --search`:
```sh
dpkg-query --search *lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8.2
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.2.usr-is-merged
libreadline8t64:amd64: /usr/lib/x86_64-linux-gnu/libreadline.so.8.2
libreadline8t64:amd64: /usr/lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.usr-is-merged
diversion by libreadline8t64 from: /lib/x86_64-linux-gnu/libreadline.so.8.2
diversion by libreadline8t64 to: /lib/x86_64-linux-gnu/libreadline.so.8.2.usr-is-merged
```

I don't believe there is any way to control this output, so ignore those lines.
Doesn't hurt to have this run on other versions, even though there aren't
any other relevant packages that are affected.
@orgads
Copy link

orgads commented Aug 7, 2025

Thank you for picking this up! I was on vacation this week :)

@orgads orgads mentioned this pull request Aug 7, 2025
@orgads
Copy link

orgads commented Aug 11, 2025

Released! 🎉

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.

3 participants