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

feat: automatically remove Apples quarantine flag #155

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

MarvinJWendt
Copy link
Contributor

Apple usually puts a quarantine flag on downloaded binaries, that are not signed. This prevents execution of the file. This PR adds a command to the install.sh file, to automatically remove the quarantine flag of the installed binary.

tested on macOS 14.x

Apple usually puts a quarantine flag on downloaded binaries, that are not signed. This prevents execution of the file.
This PR adds a command to the `install.sh` file, to automatically remove the quarantine flag of the installed binary.

*tested on macOS 14.x*
@@ -74,6 +74,7 @@ if [ "$OS" = "Darwin" ]; then
install_unix
create_launchd_plist
enable_launchd_plist
sudo xattr -r -d com.apple.quarantine /usr/local/bin/backrest # remove quarantine flag
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: let's pull off -r since it's operating directly on the binary! Shouldn't be needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, I'll fix it when I am back home :)

Copy link
Owner

@garethgeorge garethgeorge Mar 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! As a side note I see you maintain https://github.com/installer/instl, how maintained is it / would it be a good candidate for download metrics for backrest? Getting some usage data to display on the repo is in my backlog and I like the idea of using an OSS platform for transparency.

Copy link
Contributor Author

@MarvinJWendt MarvinJWendt Mar 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instl is usable, but I don't actively work on it at the moment. I want to revive it tho, as I still love the idea.
However, GitHub features a download count API, which can be utilized via shields.io do make a badge:

https://img.shields.io/github/downloads/garethgeorge/backrest/total

![GitHub Downloads (all assets, all releases)](https://img.shields.io/github/downloads/garethgeorge/backrest/total)

GitHub Downloads (all assets, all releases)

You can configure the badge with additional settings here:
https://shields.io/badges/git-hub-downloads-all-assets-all-releases

@garethgeorge
Copy link
Owner

This looks great, thanks for the PR!

@garethgeorge
Copy link
Owner

garethgeorge commented Sep 4, 2024

Hi, I'm evaluating relicensing Backrest as AGPL-3.0 . Thanks for contributing this improvement, I suspect it has very much streamlined MacOS installs for a lot of users in the last 4 or so months.

I'm wondering if you'd be willing to retroactively sign a CLA? Or grant more specific permission to retain the copyright for your PRs but relicense your changes in the codebase under AGPL-3.0 .

I've become aware of a few SAAS looking at selling Backrest which goes philosophically against what I created it for (well they're welcome to -- but their platforms should be OSS and self-hostable re: the same freedoms Backrest provides!).

@MarvinJWendt
Copy link
Contributor Author

Signed the CLA, feel free to do anything you want with my one line contribution 😆

@garethgeorge
Copy link
Owner

Haha, much appreciated :)

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