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

Homebrew 3.0.0 deprecations/disables #10397

Merged
merged 1 commit into from
Feb 1, 2021
Merged

Homebrew 3.0.0 deprecations/disables #10397

merged 1 commit into from
Feb 1, 2021

Conversation

MikeMcQuaid
Copy link
Member

@MikeMcQuaid MikeMcQuaid commented Jan 22, 2021

Do the usual deprecate/disable dance for Homebrew 3.0.0.

@BrewTestBot
Copy link
Member

Review period will end on 2021-01-25 at 15:00:10 UTC.

@BrewTestBot BrewTestBot added the waiting for feedback Merging is blocked until sufficient time has passed for review label Jan 22, 2021
@Rylan12
Copy link
Member

Rylan12 commented Jan 23, 2021

Screen Shot 2021-01-23 at 3 39 45 PM

What a beauty!


I just merged a new uncommented deprecation in #10370 to deprecate the brew release-notes command. I'm happy to push the change but it requires a rebase and I don't want to force-push over your stuff here as I'm not totally sure what your workflow is for these.

@jonchang
Copy link
Contributor

We probably don't want to delete brew pr-pull given that our bottle publishing workflow relies on it.

@EricFromCanada
Copy link
Member

Can also drop the special reformatting commands in parser.rb lines 196 & 346.

@MikeMcQuaid
Copy link
Member Author

I just merged a new uncommented deprecation in #10370 to deprecate the brew release-notes command. I'm happy to push the change but it requires a rebase and I don't want to force-push over your stuff here as I'm not totally sure what your workflow is for these.

@Rylan12 Done!

Can also drop the special reformatting commands in parser.rb lines 196 & 346.

@EricFromCanada Done!

We probably don't want to delete brew pr-pull given that our bottle publishing workflow relies on it.

@jonchang Come on, live a little (fixed 😅)!

@MikeMcQuaid MikeMcQuaid marked this pull request as ready for review January 25, 2021 10:41
@MikeMcQuaid
Copy link
Member Author

Note to self: after this is merged I/we should create an issue to delete everything remaining in Library/Homebrew/cask/cmd (and corresponding tests) and fix the code reliant on it.

Copy link
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

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

I took a glance through everything here and it looks good to me. I didn't check the cask command stuff really because I'm much less familiar with that, but all the things I was involved in (i.e. release-notes, Homebrew::CLI::Parser, Completions, etc.) look good.

Any estimate for when this will be merged? I'm asking because I'm thinking about the completion PRs that are coming. It doesn't really matter whether this is merged before or after those PRs, but just keep in mind that whichever is merged second will need to rebase after the first is merged to not break the brew man output.

@BrewTestBot BrewTestBot removed the waiting for feedback Merging is blocked until sufficient time has passed for review label Jan 25, 2021
@BrewTestBot
Copy link
Member

Review period ended.

@MikeMcQuaid
Copy link
Member Author

Any estimate for when this will be merged?

"Beginning of February" most likely.

It doesn't really matter whether this is merged before or after those PRs, but just keep in mind that whichever is merged second will need to rebase after the first is merged to not break the brew man output.

Merged yours, I'm happy to rebase (but thanks for thinking about it!).

I assume this is an editor thing. I'm not opposed to it but it does make the diffs a little harder to read (can't easily tell if lines were added/removed). Not a huge deal, but I'd rather we be consistent about this in the future. I feel like I've seen it flip-flop several times.

Yeh, I'll extract this into another PR.

@MikeMcQuaid
Copy link
Member Author

Yeh, I'll extract this into another PR.

#10429

@MikeMcQuaid
Copy link
Member Author

Any estimate for when this will be merged?

Next week.

@MikeMcQuaid MikeMcQuaid merged commit b8d55e5 into Homebrew:master Feb 1, 2021
@MikeMcQuaid MikeMcQuaid deleted the deprecations-disables branch February 1, 2021 08:37
@jonchang jonchang mentioned this pull request Feb 2, 2021
8 tasks
@md5
Copy link
Contributor

md5 commented Feb 23, 2021

Apologies if I should have had an easier time finding this, but is there any documentation about the removal of the :java requirement type?

I was just trying to see if it was possible to install the gradle formula without installing openjdk since I want it to use the Oracle JDK I already have installed, but brew audit didn't like when I tried to make "openjdk" a :recommended formula since it's in homebrew-core.

I then saw that a JavaRequirement existed at some point that looked like just what I wanted, but it looks like this commit removed it 😞

Any advice on how one can successfully use the Oracle JDK and avoid installing openjdk when it isn't needed would also be greatly appreciated.

@jonchang
Copy link
Contributor

Please use our discussions forum for general questions, thanks! https://github.com/Homebrew/discussions/discussions

@Homebrew Homebrew locked and limited conversation to collaborators Feb 24, 2021
@MikeMcQuaid
Copy link
Member Author

Apologies if I should have had an easier time finding this, but is there any documentation about the removal of the :java requirement type?

The release notes.

I was just trying to see if it was possible to install the gradle formula without installing openjdk since I want it to use the Oracle JDK I already have installed

It is not. We don't use the system Java any more.

Any advice on how one can successfully use the Oracle JDK and avoid installing openjdk when it isn't needed would also be greatly appreciated.

You'll need to create your own formula.

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.

6 participants