pdf2image: migrate to brewed X11 #64738
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tracking issue: #64166 - [x] Have you followed the [guidelines for contributing](https://github.com/Homebrew/homebrew-core/blob/HEAD/CONTRIBUTING.md)? - [x] Have you checked that there aren't other open [pull requests](https://github.com/Homebrew/homebrew-core/pulls) for the same formula update/change? - [x] Have you built your formula locally with `brew install --build-from-source <formula>`, where `<formula>` is the name of the formula you're submitting? - [x] Is your test running fine `brew test <formula>`, where `<formula>` is the name of the formula you're submitting? - [x] Does your build pass `brew audit --strict <formula>` (after doing `brew install <formula>`)? ----- - [x] `brew install FORMULA` on this list (should be poured from a bottle, not built from source) - [x] Run `brew linkage FORMULA` and observe which X11 libraries it links against - [x] Modify the formula to remove `depends_on :x11` and replace it with e.g. `depends_on "libx11"` and any additional X libraries - [x] Update `/opt/X11` or `MacOS::XQuartz` references to instead point them to the brewed formula paths - [x] `revision` bump - [x] `brew install -s FORMULA` and `brew linkage FORMULA` to check that it's now linking against brewed X and not XQuartz ----- `libx11` appears to be an optional build dependency, so not 100% certain I did this right. Would appreciate a second opinion.
jonchang
approved these changes
Nov 14, 2020
|
🤖 A scheduled task has triggered a merge. |
45 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
License found here.
Tracking issue: #64166
brew install --build-from-source <formula>, where<formula>is the name of the formula you're submitting?brew test <formula>, where<formula>is the name of the formula you're submitting?brew audit --strict <formula>(after doingbrew install <formula>)?brew install FORMULAon this list (should be poured from a bottle, not built from source)brew linkage FORMULAand observe which X11 libraries it links againstdepends_on :x11and replace it with e.g.depends_on "libx11"and any additional X libraries/opt/X11orMacOS::XQuartzreferences to instead point them to the brewed formula pathsrevisionbumpbrew install -s FORMULAandbrew linkage FORMULAto check that it's now linking against brewed X and not XQuartzlibx11appears to be an optional build dependency, so not 100% certain I did this right. Would appreciate a second opinion.