Skip to content

Conversation

@Imdabossou
Copy link

@Imdabossou Imdabossou commented Jan 15, 2026

nomodeset fallback graphics mode required for booting in certain enviroments such as vSphere

Summary by CodeRabbit

  • New Features
    • Added a new boot entry for ArchBang Live ISO (x86_64, UEFI) in basic graphics mode, enabling system startup without advanced graphics module support for systems with graphics compatibility issues.

✏️ Tip: You can customize this high-level summary in your review settings.

nomodeset fallback graphics mode required for booting in certain enviroments such as vSphere
@coderabbitai
Copy link

coderabbitai bot commented Jan 15, 2026

Walkthrough

Adds a new systemd-boot loader entry configuration file for ArchBang Live ISO with x86_64 UEFI support in basic graphics mode. The entry specifies kernel parameters including no module set, installation directory, UUID search, and COW space allocation.

Changes

Cohort / File(s) Summary
ArchBang Boot Entry Configuration
efiboot/loader/entries/02-archbang-x86_64-linux-nomodeset.conf
New systemd-boot loader entry for ArchBang Live ISO (x86_64, UEFI) with nomodeset option. Specifies kernel, initramfs paths, and boot options including archisobasedir, archisosearchuuid, cow_spacesize=4G, and nomodeset graphics mode.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A new boot path hops into place,
Where ArchBang finds its grafical grace,
With nomodeset's gentle care,
The UEFI world is now more fair! 🥾✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the main change: adding a boot entry configuration file for ArchBang Live ISO, which matches the file addition and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


🧹 Recent nitpick comments
efiboot/loader/entries/02-archbang-x86_64-linux-nomodeset.conf (1)

1-5: Configuration looks correct for the stated purpose.

The boot entry configuration is well-structured and the nomodeset kernel parameter is appropriate for a basic graphics fallback mode. The placeholders and options align with standard archiso conventions.

Two optional style suggestions:

  1. Line 1: Consider capitalizing "Iso" → "ISO" for consistency (it's an acronym).
  2. Add a trailing newline at the end of the file (common convention for text files).
🔧 Optional style fixes
-title    ArchBang Live Iso [labwc] (x86_64, UEFI) Basic Graphics Mode
+title    ArchBang Live ISO [labwc] (x86_64, UEFI) Basic Graphics Mode
 sort-key 02
 linux    /%INSTALL_DIR%/boot/x86_64/vmlinuz-linux
 initrd   /%INSTALL_DIR%/boot/x86_64/initramfs-linux.img
 options  archisobasedir=%INSTALL_DIR% archisosearchuuid=%ARCHISO_UUID% cow_spacesize=4G nomodeset
+

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e170143 and 7168717.

📒 Files selected for processing (1)
  • efiboot/loader/entries/02-archbang-x86_64-linux-nomodeset.conf

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mrgreen3
Copy link
Owner

Will have to think about this one off feature request. As I understand it only for VMWare use. It is not a big deal to add the option to kernel line live or even add post install. Do not think it warrants another boot option.

A user such as your self could easily build image and add whatever feature they so wish.

Out of over 3000 downloads this week no one else has asked for this.

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.

2 participants