Skip to content

Comments

python3-setuptools integrated in manual.md#110

Open
mattih11 wants to merge 1 commit intoilbers:masterfrom
mattih11:patch-1
Open

python3-setuptools integrated in manual.md#110
mattih11 wants to merge 1 commit intoilbers:masterfrom
mattih11:patch-1

Conversation

@mattih11
Copy link

While testing on Ubuntu 24.10 I realized that python3-distutils can not be installed. Using apt install python3-setuptools is working. I added this to the manual, to help other users circumvent this problem

While testing on Ubuntu 24.10 I realized that python3-distutils can not be installed. Using apt install python3-setuptools is working. I added this to the manual, to help other users circumvent this problem
@ismagulb
Copy link
Contributor

Thanks much @mattih11 for improving the docs!
Could you git send-email to isar-users@googlegroups.com, as described in https://github.com/ilbers/isar/blob/next/CONTRIBUTING.md#formatting-patches? That is, including Signed-off-by, etc. We don't use PRs but can't disable this feature on github.
As for the contents, I'd prefer to see specific instructions for specific Debian distros, meaning the default document flow is valid for Debian stable (bookworm ATM), with notes for specific releases (like bullseye, trixie, etc.) instead of the "if... try" clause.
Also, I personally prefer apt-get in code and docs due to its guaranteed interface (I know we haven't been 100% consistent on this in the past).

@amikan
Copy link
Collaborator

amikan commented Dec 18, 2024

As python3-setuptools already depends on python3-distutils in distros where it is present we can just always install python3-setuptools.
Moreover, because Debian Stretch is no more supported, I think we can remove even if >= buster condition and add python3-setuptools into generic Isar build dependency list.

@mattih11
Copy link
Author

mattih11 commented Dec 18, 2024 via email

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