Skip to content

apps/NXDoom: Initial port of DOOM to NuttX#19239

Merged
xiaoxiang781216 merged 3 commits into
apache:masterfrom
linguini1:nxdoom
Jul 3, 2026
Merged

apps/NXDoom: Initial port of DOOM to NuttX#19239
xiaoxiang781216 merged 3 commits into
apache:masterfrom
linguini1:nxdoom

Conversation

@linguini1

@linguini1 linguini1 commented Jun 29, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

This is the twin PR of the source port: apache/nuttx-apps#3573

The nuttx-apps needs to be merged first.

It includes a configuration for playing NXDoom on the NuttX simulator with X11 graphics and keyboard input, as well as documentation about the configuration and NXDoom itself.

This is part of my GSoC proposal, last milestone of making some cool user demos: #18507.

Impact

Users can play DOOM on NuttX!

Testing

Tested by playing NXDoom on the NuttX simulator in a Linux environment with the included configuration:

doom-on-sim

@linguini1 linguini1 mentioned this pull request Jun 29, 2026
Merged
@github-actions github-actions Bot added Area: Documentation Improvements or additions to documentation Size: M The size of the change in this PR is medium Board: simulator labels Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026 •
edited
Loading

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@linguini1 linguini1 added this to the GSoC milestone Jun 29, 2026
cederom
cederom previously approved these changes Jun 29, 2026
View reviewed changes

@cederom cederom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WHOAH!! CONGRATZ @linguini1 =)

@linguini1 linguini1 dismissed stale reviews from cederom and xiaoxiang781216 via 8f02eb1 June 29, 2026 20:35
@acassis

acassis commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@linguini1 please normalize sim:nxdoom

@linguini1

linguini1 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

Looks like some failures due to warnings that didn't cause a failing build during local testing. I will try to resolve them.

@acassis

acassis commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Looks like some failures due to warnings that didn't cause a failing build during local testing. I will try to resolve them.

@linguini1 you can test the CI locally too: https://nuttx.apache.org/docs/latest/testing/citests.html

@linguini1

linguini1 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

@linguini1 you can test the CI locally too: https://nuttx.apache.org/docs/latest/testing/citests.html

Yeah, I am set up for that, but I usually just run the checkpatch and a regular build because it's easier and catches most of my problems haha :)

@linguini1 linguini1 mentioned this pull request Jul 1, 2026
Merged
@linguini1

linguini1 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

Okay, CI errors resolved by apache/nuttx-apps#3577 @acassis !

Once merged, I'll re-run this PR's CI against the latest apps and it should be good to go.

linguini1 added 3 commits July 2, 2026 19:06
@linguini1
docs/games/NXDoom: Documentation for DOOM port
3388544 
Includes documentation about the current state of the DOOM port, where
it came from, how to play and what the requirements are. Includes all of
the warnings about experimental/unfinished features.

Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
@linguini1
boards/sim: Configuration for NXDoom on simulator
b505598 
Includes a defconfig to play NXDoom on the simulator. Keyboard input is
available through X11, as well as graphics rendering. The `/data` folder
is used as the default home for game files.

Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
@linguini1
docs/sim: Include documentation about NXDoom configuration
7f7beb7 
Include docs about new NXDoom configurations for playing NXDoom.

Signed-off-by: Matteo Golin <matteo.golin@gmail.com>
@linguini1

linguini1 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

@acassis @cederom Ready for merge!

@xiaoxiang781216 xiaoxiang781216 merged commit 51d0848 into apache:master Jul 3, 2026
28 checks passed
@jerpelea jerpelea mentioned this pull request Jul 3, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@acassis acassis acassis approved these changes
@cederom cederom cederom approved these changes
@jerpelea jerpelea jerpelea approved these changes
@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 left review comments
@raiden00pl raiden00pl Awaiting requested review from raiden00pl
@michallenc michallenc Awaiting requested review from michallenc
@yamt yamt Awaiting requested review from yamt yamt is a code owner
@tmedicci tmedicci Awaiting requested review from tmedicci tmedicci is a code owner
@btashton btashton Awaiting requested review from btashton btashton is a code owner

Assignees

No one assigned

Labels

Area: Documentation Improvements or additions to documentation Board: simulator Size: M The size of the change in this PR is medium

Projects

None yet

Milestone

GSoC

Development

Successfully merging this pull request may close these issues.

5 participants

@linguini1 @acassis @cederom @jerpelea @xiaoxiang781216