-
Notifications
You must be signed in to change notification settings - Fork 25
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
Tarsnap.app isn't in /Applications #198
Comments
I wrote a brew cask formula that should fix this: https://github.com/shawwn/homebrew-cask/blob/tarsnap/Casks/tarsnap.rb (Be sure to change It puts |
Thanks, this is a great start! I'm a little bit leery about the number of new features in the cask and your 1.0.3:
In many (or all) of these cases, adding 1-3 sentences of comments (either to the files themselves, or the git commit messages) would clear things up. For now, I'm going to go ahead and announce a beta-test of 1.0.2 on the |
That's fine. There wasn't any reason for the name change other than convention. And it might be simpler to migrate to Cask with identical names; I'm not sure.
For consistency, the UI elements related to scheduling should probably be disabled if the user hasn't enabled it yet. But I don't think active users would mind having it enabled. One way it could work: If a user tries to change a job's schedule from Disabled to something else, it could run
That's a good point, but consider that each Tarsnap.app will be signed with the tarsnap-public-keys. That doesn't prevent
The reason for the shell script was to support the The other benefit of this model is that it simplifies log collection, since the script can automatically pipe stdout and stderr someplace on disk. But the downside is as you say: extra complexity, which may be worth avoiding. I don't have any strong feelings either way.
Solely convenience. I think a Cask formula needs to ship either a .zip or a .dmg, so there would have to be an extra step added to the build process anyway. And since I was testing many iterations of the .app, I eventually just automated it. It doesn't need to be merged into master (though FWIW it does work on Ubuntu as well). |
(Oh, and I didn't mean to imply that the cask was production-ready. Apologies if that was unclear. It was only intended as an experiment to explore some possible solutions to existing problems, and a starting point for what a production-ready cask formula might look like.) |
Yikes:
Ok, that's a strong reason in favor of it! (BTW, that's exactly the kind of text we like to see in commit messages.) I'd like to investigate the loading error a bit more, and try to dig up a link to the Qt or OSX docs to show why those errors are expected (i.e. the desired behaviour isn't supported) or a link to a bug report about it. But other than that bit of housekeeping, I'm totally on board with the shell script. |
More info about this in Tarsnap/tarsnap#316. |
Just tried it multiple times on Sonoma and tarnsap-gui doesn't show up in /Applications. It's from 2018 but I see the issue is still open. Is there any other way to install the GUI on mac Sonoma so that I can use it on my new laptop. |
It looks like
brew linkapps
is deprecated. I think we need to have a brew "cask" formula to get Tarsnap.app in /Applications instead of hidden away in@ilovezfs, does that sound right?
(I'll work on this after I've fixed the "reload launchd after fixing the binary path" problem.)
The text was updated successfully, but these errors were encountered: