Skip to content

Add flake support to make this easy to consume for nixos and nix enabled devices#153

Merged
mrrfv merged 3 commits intomrrfv:masterfrom
f0rdprefect:master
Feb 24, 2026
Merged

Add flake support to make this easy to consume for nixos and nix enabled devices#153
mrrfv merged 3 commits intomrrfv:masterfrom
f0rdprefect:master

Conversation

@f0rdprefect
Copy link
Copy Markdown
Contributor

I needed this to run on my NixOS machine. So I vibed this flake which works fine for me.

Use nix develop to get a development shell which can then be used to do the backup and restore

Use `nix develop` in flake enabled nix installation to get everything to use this repo
@mrrfv
Copy link
Copy Markdown
Owner

mrrfv commented Nov 4, 2025

Looks good at first glance, thank you. I'll try to test the flake next weekend.

I see a few minor issues, though:

  • The shellHook sets a variable called OAB_COMPRESSION_LEVEL, which isn't used anywhere in the project. I presume this was a hallucination. TMPDIR may not be required either.
  • It's probably unnecessary to check if adb is available in the environment.
  • I'd remove the "Environment variables for automation" part and refer to the README instead, to avoid redundancy.

Beyond that, I think this will be a nice enhancement!

- Remove unused OAB_COMPRESSION_LEVEL and TMPDIR exports
- Remove redundant ADB checks and env var documentation
- Add Nix installation method to README
- Simplify shellHook to essential information only
@mrrfv mrrfv merged commit 0943c93 into mrrfv:master Feb 24, 2026
@mrrfv
Copy link
Copy Markdown
Owner

mrrfv commented Feb 24, 2026

Thanks again :)

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