Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

git-credential-manager 1.5.0 #50293

Closed
wants to merge 2 commits into from

Conversation

olivierdagenais
Copy link
Contributor

Notice the dependency has changed for Mac OS X 10.11+

To see the testing that was performed, refer to microsoft/Git-Credential-Manager-for-Mac-and-Linux#32

Notice the dependency has changed for Mac OS X 10.11+
@olivierdagenais olivierdagenais changed the title git-credential-manager: update to version 1.5.0 git-credential-manager 1.5.0 Mar 21, 2016

bottle :unneeded

depends_on :java => "1.7+"
depends_on :java => "1.8+" if MacOS.version >= :el_capitan
Copy link
Member

Choose a reason for hiding this comment

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

You can make this a single block conditional rather than two:

if MacOS.version >= :el_capitan
  depends_on :java => "1.8+"
else
  depends_on :java => "1.7+"
end

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can make this a single block conditional

Done in 346d83d.

@DomT4
Copy link
Member

DomT4 commented Mar 22, 2016

Merged in cd9b382. Thank you for another contribution to Homebrew @olivierdagenais! 😺

@DomT4 DomT4 closed this in cd9b382 Mar 22, 2016
xu-cheng pushed a commit to Homebrew/homebrew-core that referenced this pull request Mar 22, 2016
Notice the dependency has changed for Mac OS X 10.11+

Closes Homebrew/legacy-homebrew#50293.

Signed-off-by: Dominyk Tiller <dominyktiller@gmail.com>
@olivierdagenais olivierdagenais deleted the gcm4ml-1.5.0 branch March 22, 2016 12:44
benjaminfrank pushed a commit to benjaminfrank/homebrew that referenced this pull request Apr 9, 2016
Notice the dependency has changed for Mac OS X 10.11+

Closes Homebrew#50293.

Signed-off-by: Dominyk Tiller <dominyktiller@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
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.

None yet

2 participants