Skip to content

dist/tools: remove desvirt - #22724

Open
crasbe wants to merge 1 commit into
RIOT-OS:masterfrom
crasbe:pr/remove_desvirt
Open

crasbe wants to merge 1 commit into
RIOT-OS:masterfrom
crasbe:pr/remove_desvirt

Conversation

@crasbe

@crasbe crasbe commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Contribution description

The desvirt integration does not work and does not appear to have been changed or maintained since it was added in 2016.

Well, in theory it might still work, but it is based on Python 2 and the support for that has ended uuuuuuuuh 2020.
I think the last Ubuntu that even shipped Python 2 was Ubuntu 22.04 LTS and the support for that ends in April 2029.

The Python 2 support was dropped from the build docker container in RIOT-OS/riotdocker#281 because 26.04 LTS does not provide an installation candidate for it.

Testing procedure

No (relevant) references left:

buechse@skyleaf:~/RIOTstuff/riot-upstream/RIOT$ grep -RnwIi desvirt
release-notes.txt:20984:+ desvirt integration into the RIOT build system

Issues/PRs references

None.

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • none

The desvirt integration does not work and does not appear
to have been changed or maintained since it was added
in 2016.
@crasbe
crasbe requested a review from OlegHahm September 16, 2026 17:27
@crasbe
crasbe requested a review from AnnsAnns as a code owner September 16, 2026 17:27
@crasbe crasbe added Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: removal Integration Process: The PR is removing a deprecated feature or API AI: Not Used AI was stated to not be used in this PR/Issue labels Sep 16, 2026
@github-actions github-actions Bot added Area: doc Area: Documentation Area: build system Area: Build system Area: tools Area: Supplementary tools labels Sep 16, 2026
@crasbe

crasbe commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

I asked in the RIOT Maintainer Matrix channel and @OlegHahm, the last person who modified the desvirt repository four years ago, said that it can still be useful.

Installing Python 2 seems to be possible at least on Arch Linux through an AUR package.

Therefore this is up for discussion.

@AnnsAnns AnnsAnns left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When I see Python 2 in 2026 I approve no questions asked

@crasbe

crasbe commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

I'd still like to leave this open for discussion a bit longer :D

@riot-ci

riot-ci commented Sep 16, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

32e873b dist/tools: remove desvirt

Success Failures Total Runtime
11279 0 11283 11m:53s

Artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Not Used AI was stated to not be used in this PR/Issue Area: build system Area: Build system Area: doc Area: Documentation Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Discussion: RFC The issue/PR is used as a discussion starting point about the item of the issue/PR Process: removal Integration Process: The PR is removing a deprecated feature or API Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants