batt 0.2.1 (new formula)

[Jerry1144]

• Have you followed the guidelines for contributing?
• Have you ensured that your commits follow the commit style guide?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

---

Add batt, a utility to manually control charging of Apple Silicon Macbooks, instead of hoping macOS limit charging to 80% for us. I tried to use brew services to handle installing the relevant launchd.plist, although the tool itself has a built-in routine to do so.

I wonder if, instead of hardcoding them, there's a more elegant way of fetching VERSION and GIT_COMMIT variables from the URL. Having Ruby variables holding these two means we don't forget about those in the test do block below.

I cannot decide whether to use system "plutil" or inreplace to modify the plist file, too. The former won't work on Linux, but it handles insertion better than the inflexible replacement syntax.

The uses_from_macos "curl" documents the peculiar fact that IPC is handled using http. Should I remove it instead?

[Jerry1144]

I think that's about it, a Formula with a sane test. Sort of. No idea why the batt daemon test does not generate the config file that I can test for existence at a later stage. There's nothing in testpath, it can find the config if there's one leftover at HomebrewPrefix/etc, but it cannot write one anew or that it got deleted right after the first test exited.

TODO: learn more about launchd.plist to see whether the SocketPath key makes no sense in this case.

[charlie0129]

The uses_from_macos "curl" documents the peculiar fact that IPC is handled using http. Should I remove it instead?

I think it can be removed because the ipc http client is contained in the binary itself.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[Jerry1144]

The failing test should have been resolved by Homebrew/brew#16914, a relatively new change. Now the tests should not be failing anymore.

[chenrui333]

@Jerry1144 can you rebase your pr to the latest master and give it a rerun? Thanks!

[MikeMcQuaid]

Looks good, nice work @Jerry1144!

[github-actions]

🤖 An automated task has requested bottles to be published to this PR.