Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

Update homebrew tap for Java#6

Merged
Brantone merged 2 commits into
Roblox:masterfrom
zirkov:feature/homebrew-cask-versions-update
Jan 15, 2020
Merged

Update homebrew tap for Java#6
Brantone merged 2 commits into
Roblox:masterfrom
zirkov:feature/homebrew-cask-versions-update

Conversation

@zirkov

@zirkov zirkov commented Jan 15, 2020

Copy link
Copy Markdown

Summary

The continuation of #5 (can be closed when this PR is merged)

sous-chefs#566: caskroom/versions was moved. Tap homebrew/cask-versions instead

Description

Issues Resolved

Fixes <sous-chefs#566>:
`caskroom/versions` was moved. Tap `homebrew/cask-versions` instead

* Based on proposed changes from <sous-chefs#567>
* Switch homebrew tap to homebrew/cask-versions
* Make Homebrew Tap name an attribute to allow for other options
Comment thread CHANGELOG.md

This file is used to list changes made in each version of the Java cookbook.

## Unreleased

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Apprehensive about removing the Unreleased tag, because it's technically not released in the upstream and this was method by way of denoting that (and I think was a convention a year ago).
Might be worth keeping it or denoting Roblox Release or something

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Oh, didn't realize that
Fixing

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@Brantone Updated

@Brantone Brantone left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Minor tweak/thoughts on Changelog

Comment thread attributes/default.rb
default['java']['windows']['returns'] = 0
when 'mac_os_x'
default['java']['install_flavor'] = 'homebrew'
default['java']['homebrew']['tap'] = 'homebrew/cask-versions'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

@Brantone Brantone merged commit 052862b into Roblox:master Jan 15, 2020
@zirkov zirkov deleted the feature/homebrew-cask-versions-update branch January 15, 2020 22:14
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.

2 participants