Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use xdg-open instead of assuming Nautilus for Linux #239

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

aarjaneiro
Copy link
Contributor

@aarjaneiro aarjaneiro commented Aug 28, 2024

Also could instead add a call to set a constant outside of this function to determine the status of which xdg-open rather than having this be retested every time open_folder is called.

@badabing2005 badabing2005 merged commit 299c358 into badabing2005:main Aug 28, 2024
@badabing2005
Copy link
Owner

Thanks.
For now I leave it without remember the value, I don't think this feature is used that often.

badabing2005 added a commit that referenced this pull request Aug 29, 2024
… APatch app for patching, manual APatch patching is recommended. PixelFlasher automates this manual process—just select `Patch with APatch Alternate`. You’ll be prompted to choose between the latest release or the latest pre-release of Kernel Patch tools.

- **APatch Validation**: Added validation for APatch patching to ensure the kernel is a supported version and that `CONFIG_KALLSYMS` is enabled.
- **Homebrew Cask Support**: Added a Homebrew cask install option for macOS, ensuring automatic updates. Use the command: `brew install badabing2005/pixelflasher/pixelflasher`.
- **PR Merge #239**: On Linux, PixelFlasher now uses `xdg-open` (if available) instead of Nautilus, thanks to @aarjaneiro.
- **16K Page Size Preparation**: In preparation for the upcoming 16K page size feature in Android, PixelFlasher now reports the current page size.
- **Miscellaneous**: Various minor improvements.
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