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

Add borgbackup v0.26.1 #15102

Closed
wants to merge 1 commit into from
Closed

Add borgbackup v0.26.1 #15102

wants to merge 1 commit into from

Conversation

rmoriz
Copy link
Contributor

@rmoriz rmoriz commented Nov 11, 2015

@rmoriz rmoriz mentioned this pull request Nov 11, 2015
7 tasks

gpg "#{url}.asc", :key_id => '243ACFA951F78E01'

container :type => :naked
Copy link
Member

Choose a reason for hiding this comment

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

Are you sure this is 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.

It's just the binary without any container (tar, dmg) and naked is not auto-detected https://github.com/caskroom/homebrew-cask/blob/master/lib/hbc/container.rb

$ wget https://github.com/borgbackup/borg/releases/download/0.26.1/borg-macosx64 
…
$ file borg-macosx64 
borg-macosx64: Mach-O 64-bit executable x86_64

@anarcat
Copy link

anarcat commented Nov 11, 2015

this is outdated: the latest borg release is 0.28.1.

@rmoriz
Copy link
Contributor Author

rmoriz commented Nov 11, 2015

@anarcat can you provide binaries for 0.28.1? Maintainer and gpg signee has not released binaries yet: https://github.com/borgbackup/borg/releases


url "https://github.com/borgbackup/borg/releases/download/#{version}/borg-macosx64"
gpg "#{url}.asc", :key_id => '243ACFA951F78E01'

Copy link
Member

Choose a reason for hiding this comment

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

This blank line shouldn’t be here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I used the gpg-example from the docs which references https://github.com/caskroom/homebrew-cask/blob/master/Casks/libreoffice.rb including the blank line between the gpg and name part?

Also neither https://github.com/caskroom/homebrew-cask/blob/master/CONTRIBUTING.md#style-guide nor Rubocop have problems with the blank line, you may want to document this better for future contributors…

@anarcat
Copy link

anarcat commented Nov 11, 2015

@rmoriz i asked the main dev for an update - @ThomasWaldmann: where are the binaries for 0.28.1?

@adidalal adidalal added upstream Issue which needs to be resolved by the upstream project. on hold awaiting user reply Issue needs response from a user. and removed awaiting user reply Issue needs response from a user. enhancement cask labels Nov 11, 2015
@anarcat
Copy link

anarcat commented Nov 11, 2015

@rmoriz in the meantime, there are binaries for 0.27...

@adidalal
Copy link
Contributor

Note for maintainers: This is on hold until upstream binaries for v0.28.1 are created and the Cask updated accordingly.

Alternatively, if @rmoriz wants to update the cask to v0.27, this PR can be merged (and a new one opened when binaries for the latest version become available)

@anarcat
Copy link

anarcat commented Nov 16, 2015

0.28.2 was published with binaries.

@adidalal
Copy link
Contributor

Thank you for the contribution. It needed some fixes, so they were made in 47b865e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

@anarcat I forgot to change the commit message, but 0.28.2 is the version included in homebrew cask now.

@adidalal adidalal closed this Nov 16, 2015
@adidalal adidalal removed the awaiting user reply Issue needs response from a user. label Jan 3, 2016
@adidalal adidalal removed the on hold label Feb 23, 2016
@miccal miccal removed the upstream Issue which needs to be resolved by the upstream project. label Dec 23, 2016
@raxod502
Copy link

Given that Borg is an open-source command-line tool, why is it in Cask instead of Homebrew proper?

@commitay
Copy link
Contributor

@raxod502 Please don't post comments on PRs that are over two years old.

If this can be added to homebrew/core submit a PR and it will be removed from cask.

@Homebrew Homebrew locked as off-topic and limited conversation to collaborators Apr 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants