Skip to content

Conversation

@kartlee
Copy link
Contributor

@kartlee kartlee commented Jan 5, 2026

Ubuntu 24.04 has made changes to its system libraries to address 2038 time transition problem, and as a result some of the packages we need as canonical requirement needs the t64 suffix. Falling in this category are these two packages which needs to be changed in the canonical requirement for ubuntu 24.04. Without this change using the original package name is installable, but is no longer queryable via dpkg.

  • libxt6 needs to be changed to libxt6t64
  • libtirpc3 needs to be changed to libtirpc3t64

If this separation looks good, I will be posting a change in mmshare updating installation_check to use ubuntu24 for 24 and higher versions of ubuntu.

And a PR for core-stu-container making the below changes for the package name change.

NOTE: I made the above change from 24-4 and higher as we list ubuntu 24.04 as supported distro.

Testing:
- With the change to core-stu-container to install above packages, below works

(ubuntu24.04) rajagopa@desk-lu554:~$ $SCHRODINGER/run check_deps.py
Success: All required Linux packages are present.
- Running maestro from ubuntu2404 docker, works.

…uffix from older INST-2211

- Starting ubuntu24 some packages were named with t64 suffix and expect them to be used
instead of non-t64 suffix. These are not queryable by dpkg otherwise.
@kartlee kartlee requested review from hajizar, lennax and mbeachy January 5, 2026 21:59
Copy link
Member

@mbeachy mbeachy left a comment

Choose a reason for hiding this comment

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

Doesn't the script also need to be adjusted to detect ubuntu24 vs previous ubuntu versions?

It looks like the script gives a completely different set of dependencies if a platform argument is provided. I don't understand this. (And, the script is kind of a disaster for readability with the functions and calls to those functions interspersed.

@kartlee
Copy link
Contributor Author

kartlee commented Jan 6, 2026

Doesn't the script also need to be adjusted to detect ubuntu24 vs previous ubuntu versions?

It looks like the script gives a completely different set of dependencies if a platform argument is provided. I don't understand this. (And, the script is kind of a disaster for readability with the functions and calls to those functions interspersed.

The script expects the package requirement to be fed, and this is documented here - https://github.com/schrodinger/linux_package_requirement?tab=readme-ov-file#how-to-use

We can probably change this to auto detect and expect the requirements to reside side-by side to the script, but its not supported now.

Copy link
Contributor

@hajizar hajizar left a comment

Choose a reason for hiding this comment

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

LGTM!

@kartlee
Copy link
Contributor Author

kartlee commented Jan 7, 2026

Doesn't the script also need to be adjusted to detect ubuntu24 vs previous ubuntu versions?
It looks like the script gives a completely different set of dependencies if a platform argument is provided. I don't understand this. (And, the script is kind of a disaster for readability with the functions and calls to those functions interspersed.

The script expects the package requirement to be fed, and this is documented here - https://github.com/schrodinger/linux_package_requirement?tab=readme-ov-file#how-to-use

We can probably change this to auto detect and expect the requirements to reside side-by side to the script, but its not supported now.

I organized the script moving functions and the caller into separate section, and added docs to it.

@kartlee kartlee merged commit 35a2071 into main Jan 7, 2026
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.

4 participants