Fix UFW autostart issue #2453
Merged
Fix UFW autostart issue #2453
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update Matte Black ghostty.conf with more color settings * Rename theme from 'MatteBlack' to 'Matte Black'
Closes basecamp#1815
… on the ISO" This reverts commit ad01082.
Added: MacBook 8,1 MacBook 9,1 MacBook 8,1 uses slightly different moduels from what I can tell, so I’ve added a case statment. Please tweak as needed.
The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation.
- $TERMINAL should remain variable
slamotte
added a commit
to slamotte/omarchy
that referenced
this pull request
Oct 16, 2025
ahartness
pushed a commit
to ahartness/omarchy-hrtns
that referenced
this pull request
Oct 19, 2025
* Improve theme-set-vscode to support jsonc and respect symlinks (basecamp#1756) * Ghostty needs a very different scrollfactor than Alacritty and Kitty cc @mitchellh * Hide hyperland cursor while typing (basecamp#1812) * Fix Matte Black theme ghostty and vscode configs (basecamp#1801) * Update Matte Black ghostty.conf with more color settings * Rename theme from 'MatteBlack' to 'Matte Black' * Catch obsidian vault dialog too * Add Update > Hardware > Audio for restarting pipewire service * Give output on what's intended to happen * Ensure libyaml is available for installs made before that was on the ISO * Limit pattern to prevent incorrect matches * fix: ghostty config typos * No longer used Closes basecamp#1815 * Revert "Ensure libyaml is available for installs made before that was on the ISO" This reverts commit ad01082. * Add Helium browser to supported browsers list in omarchy-launch-webapp * Fix Helium browser detection in launch script * feat: Added MacBook10,1 to the fix-apple-spi-keyboard.sh * Suggest how to have a smaller-than-max-width single window aspect ratio * Add more Mac models to SPI keyboard support Added: MacBook 8,1 MacBook 9,1 MacBook 8,1 uses slightly different moduels from what I can tell, so I’ve added a case statment. Please tweak as needed. * Fix conflict + rearrange * Added fix for ealy 2013 macbook pro's using an older chipset, BCM4331 * Tried with a different approach for cleaner code * Clean up Tested the logic on the MacBook. It worked and the install of the drivers are reenabled. * Combine ifs * Update install script name * Fix for '(null)' display on AirPod source selection * Use output display or region geometry to record The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation. * Fix TUI install script - $TERMINAL should remain variable * Add migration to fix Docker and Disk Usage TUIs * Replace "--class" with "--class=" * Revert "Fix for '(null)' display on AirPod source selection" * Typo * Fix comment * Change up firewall setup * Enable ufw * Disable ufw-docker (isolation test) * Add some reboot on done debugging * Debug firewall install/config * Add first-run logging * Don't suppress output from apparently-failing systemctl reboot command * Fix logging * Use old reboot. Still chrooted so probably won't work. * Allow systemctl on first run * Revert logging and other changes --------- Co-authored-by: Omar Skalli <[email protected]> Co-authored-by: David Heinemeier Hansson <[email protected]> Co-authored-by: Johan Sørensen <[email protected]> Co-authored-by: Taha <[email protected]> Co-authored-by: Ryan Hughes <[email protected]> Co-authored-by: Che-Wei Lin <[email protected]> Co-authored-by: Patrick Byrne <[email protected]> Co-authored-by: Eduardo Mínguez <[email protected]> Co-authored-by: Joel Gaff, Jr <[email protected]> Co-authored-by: Michael Larsen <[email protected]> Co-authored-by: Shawn Yeager <[email protected]> Co-authored-by: observable <[email protected]> Co-authored-by: Djordje Kovacevic <[email protected]>
dhcrzf
pushed a commit
to dhcrzf/omarchy
that referenced
this pull request
Oct 21, 2025
* Improve theme-set-vscode to support jsonc and respect symlinks (basecamp#1756) * Ghostty needs a very different scrollfactor than Alacritty and Kitty cc @mitchellh * Hide hyperland cursor while typing (basecamp#1812) * Fix Matte Black theme ghostty and vscode configs (basecamp#1801) * Update Matte Black ghostty.conf with more color settings * Rename theme from 'MatteBlack' to 'Matte Black' * Catch obsidian vault dialog too * Add Update > Hardware > Audio for restarting pipewire service * Give output on what's intended to happen * Ensure libyaml is available for installs made before that was on the ISO * Limit pattern to prevent incorrect matches * fix: ghostty config typos * No longer used Closes basecamp#1815 * Revert "Ensure libyaml is available for installs made before that was on the ISO" This reverts commit ad01082. * Add Helium browser to supported browsers list in omarchy-launch-webapp * Fix Helium browser detection in launch script * feat: Added MacBook10,1 to the fix-apple-spi-keyboard.sh * Suggest how to have a smaller-than-max-width single window aspect ratio * Add more Mac models to SPI keyboard support Added: MacBook 8,1 MacBook 9,1 MacBook 8,1 uses slightly different moduels from what I can tell, so I’ve added a case statment. Please tweak as needed. * Fix conflict + rearrange * Added fix for ealy 2013 macbook pro's using an older chipset, BCM4331 * Tried with a different approach for cleaner code * Clean up Tested the logic on the MacBook. It worked and the install of the drivers are reenabled. * Combine ifs * Update install script name * Fix for '(null)' display on AirPod source selection * Use output display or region geometry to record The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation. * Fix TUI install script - $TERMINAL should remain variable * Add migration to fix Docker and Disk Usage TUIs * Replace "--class" with "--class=" * Revert "Fix for '(null)' display on AirPod source selection" * Typo * Fix comment * Change up firewall setup * Enable ufw * Disable ufw-docker (isolation test) * Add some reboot on done debugging * Debug firewall install/config * Add first-run logging * Don't suppress output from apparently-failing systemctl reboot command * Fix logging * Use old reboot. Still chrooted so probably won't work. * Allow systemctl on first run * Revert logging and other changes --------- Co-authored-by: Omar Skalli <[email protected]> Co-authored-by: David Heinemeier Hansson <[email protected]> Co-authored-by: Johan Sørensen <[email protected]> Co-authored-by: Taha <[email protected]> Co-authored-by: Ryan Hughes <[email protected]> Co-authored-by: Che-Wei Lin <[email protected]> Co-authored-by: Patrick Byrne <[email protected]> Co-authored-by: Eduardo Mínguez <[email protected]> Co-authored-by: Joel Gaff, Jr <[email protected]> Co-authored-by: Michael Larsen <[email protected]> Co-authored-by: Shawn Yeager <[email protected]> Co-authored-by: observable <[email protected]> Co-authored-by: Djordje Kovacevic <[email protected]>
zachasme
pushed a commit
to zachasme/omarchy
that referenced
this pull request
Oct 27, 2025
* Improve theme-set-vscode to support jsonc and respect symlinks (basecamp#1756) * Ghostty needs a very different scrollfactor than Alacritty and Kitty cc @mitchellh * Hide hyperland cursor while typing (basecamp#1812) * Fix Matte Black theme ghostty and vscode configs (basecamp#1801) * Update Matte Black ghostty.conf with more color settings * Rename theme from 'MatteBlack' to 'Matte Black' * Catch obsidian vault dialog too * Add Update > Hardware > Audio for restarting pipewire service * Give output on what's intended to happen * Ensure libyaml is available for installs made before that was on the ISO * Limit pattern to prevent incorrect matches * fix: ghostty config typos * No longer used Closes basecamp#1815 * Revert "Ensure libyaml is available for installs made before that was on the ISO" This reverts commit ad01082. * Add Helium browser to supported browsers list in omarchy-launch-webapp * Fix Helium browser detection in launch script * feat: Added MacBook10,1 to the fix-apple-spi-keyboard.sh * Suggest how to have a smaller-than-max-width single window aspect ratio * Add more Mac models to SPI keyboard support Added: MacBook 8,1 MacBook 9,1 MacBook 8,1 uses slightly different moduels from what I can tell, so I’ve added a case statment. Please tweak as needed. * Fix conflict + rearrange * Added fix for ealy 2013 macbook pro's using an older chipset, BCM4331 * Tried with a different approach for cleaner code * Clean up Tested the logic on the MacBook. It worked and the install of the drivers are reenabled. * Combine ifs * Update install script name * Fix for '(null)' display on AirPod source selection * Use output display or region geometry to record The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation. * Fix TUI install script - $TERMINAL should remain variable * Add migration to fix Docker and Disk Usage TUIs * Replace "--class" with "--class=" * Revert "Fix for '(null)' display on AirPod source selection" * Typo * Fix comment * Change up firewall setup * Enable ufw * Disable ufw-docker (isolation test) * Add some reboot on done debugging * Debug firewall install/config * Add first-run logging * Don't suppress output from apparently-failing systemctl reboot command * Fix logging * Use old reboot. Still chrooted so probably won't work. * Allow systemctl on first run * Revert logging and other changes --------- Co-authored-by: Omar Skalli <[email protected]> Co-authored-by: David Heinemeier Hansson <[email protected]> Co-authored-by: Johan Sørensen <[email protected]> Co-authored-by: Taha <[email protected]> Co-authored-by: Ryan Hughes <[email protected]> Co-authored-by: Che-Wei Lin <[email protected]> Co-authored-by: Patrick Byrne <[email protected]> Co-authored-by: Eduardo Mínguez <[email protected]> Co-authored-by: Joel Gaff, Jr <[email protected]> Co-authored-by: Michael Larsen <[email protected]> Co-authored-by: Shawn Yeager <[email protected]> Co-authored-by: observable <[email protected]> Co-authored-by: Djordje Kovacevic <[email protected]>
guirossibrum
pushed a commit
to guirossibrum/ArchGui
that referenced
this pull request
Oct 30, 2025
* Improve theme-set-vscode to support jsonc and respect symlinks (basecamp#1756) * Ghostty needs a very different scrollfactor than Alacritty and Kitty cc @mitchellh * Hide hyperland cursor while typing (basecamp#1812) * Fix Matte Black theme ghostty and vscode configs (basecamp#1801) * Update Matte Black ghostty.conf with more color settings * Rename theme from 'MatteBlack' to 'Matte Black' * Catch obsidian vault dialog too * Add Update > Hardware > Audio for restarting pipewire service * Give output on what's intended to happen * Ensure libyaml is available for installs made before that was on the ISO * Limit pattern to prevent incorrect matches * fix: ghostty config typos * No longer used Closes basecamp#1815 * Revert "Ensure libyaml is available for installs made before that was on the ISO" This reverts commit ad01082. * Add Helium browser to supported browsers list in omarchy-launch-webapp * Fix Helium browser detection in launch script * feat: Added MacBook10,1 to the fix-apple-spi-keyboard.sh * Suggest how to have a smaller-than-max-width single window aspect ratio * Add more Mac models to SPI keyboard support Added: MacBook 8,1 MacBook 9,1 MacBook 8,1 uses slightly different moduels from what I can tell, so I’ve added a case statment. Please tweak as needed. * Fix conflict + rearrange * Added fix for ealy 2013 macbook pro's using an older chipset, BCM4331 * Tried with a different approach for cleaner code * Clean up Tested the logic on the MacBook. It worked and the install of the drivers are reenabled. * Combine ifs * Update install script name * Fix for '(null)' display on AirPod source selection * Use output display or region geometry to record The previous code did not identify the output display correctly, leading to wf-recorder asking the user to select the region when the user requested the display output to be selected. With this change the bash script will use slurp with the -o flag to identify the display output. This has not been tested with wl-screenrec as no change has been made to its call signature in omarchy-cmd-screenrecord and the wl-screenrec binary is not included in the default omarchy installation. * Fix TUI install script - $TERMINAL should remain variable * Add migration to fix Docker and Disk Usage TUIs * Replace "--class" with "--class=" * Revert "Fix for '(null)' display on AirPod source selection" * Typo * Fix comment * Change up firewall setup * Enable ufw * Disable ufw-docker (isolation test) * Add some reboot on done debugging * Debug firewall install/config * Add first-run logging * Don't suppress output from apparently-failing systemctl reboot command * Fix logging * Use old reboot. Still chrooted so probably won't work. * Allow systemctl on first run * Revert logging and other changes --------- Co-authored-by: Omar Skalli <[email protected]> Co-authored-by: David Heinemeier Hansson <[email protected]> Co-authored-by: Johan Sørensen <[email protected]> Co-authored-by: Taha <[email protected]> Co-authored-by: Ryan Hughes <[email protected]> Co-authored-by: Che-Wei Lin <[email protected]> Co-authored-by: Patrick Byrne <[email protected]> Co-authored-by: Eduardo Mínguez <[email protected]> Co-authored-by: Joel Gaff, Jr <[email protected]> Co-authored-by: Michael Larsen <[email protected]> Co-authored-by: Shawn Yeager <[email protected]> Co-authored-by: observable <[email protected]> Co-authored-by: Djordje Kovacevic <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
13 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes were added in #1429 to ensure that
ufwstarts automatically. However, the command to enable this fails silently because thesudoersfile created byinstall/preflight/first-run-mode.shdoes not addsystemctlto the allowed commands.This PR adds that which allows fresh installs to have
ufwstart automatically.