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

mapnik: revision for gdal #65895

Closed
wants to merge 1 commit into from
Closed

mapnik: revision for gdal #65895

wants to merge 1 commit into from

Conversation

iMichka
Copy link
Member

@iMichka iMichka commented Nov 29, 2020

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Fixes:

2020-11-29T15:40:57.4198860Z �[34m==>�[0m �[1mbrew linkage --test mapnik�[0m
2020-11-29T15:40:59.6497430Z �[31m==>�[0m �[1m�[31mFAILED�[0m�[0m
2020-11-29T15:40:59.6498070Z Missing libraries:
2020-11-29T15:40:59.6498620Z   unexpected (/usr/lib/libpq.5.dylib)
2020-11-29T15:40:59.6499170Z Broken dependencies:
2020-11-29T15:40:59.6499900Z   /usr/local/opt/gdal/lib/libgdal.27.dylib (gdal)

@BrewTestBot BrewTestBot added the deprecated license Formula uses a deprecated SPDX license which should be updated label Nov 29, 2020
@fxcoudert
Copy link
Member

I don't understand where the /usr/lib/libpq.5.dylib is coming from

@iMichka
Copy link
Member Author

iMichka commented Nov 29, 2020

gdal depends on libpq. But I am unsure what happened here.

I saw this while rebuilding poppler on CI.

@fxcoudert
Copy link
Member

There are some upstream fixes for it, but I don't know which commit is the right one (if any): mapnik/mapnik#4196

@fxcoudert fxcoudert mentioned this pull request Nov 30, 2020
@Rylan12 Rylan12 mentioned this pull request Dec 12, 2020
5 tasks
@Rylan12
Copy link
Member

Rylan12 commented Dec 13, 2020

FYI using @redneb's patch that they linked in mapnik/mapnik#4196 worked locally.

@carlocab carlocab mentioned this pull request Dec 13, 2020
@iMichka
Copy link
Member Author

iMichka commented Dec 13, 2020

I asked if they can tag a new release. If not, I'll add the patch tomorrow.

@carlocab carlocab mentioned this pull request Dec 14, 2020
5 tasks
Add patch to compile with boost:
mapnik/mapnik#4196
@BrewTestBot
Copy link
Member

:shipit: @iMichka has triggered a merge.

@iMichka iMichka deleted the mapnik branch December 15, 2020 06:38
@iMichka
Copy link
Member Author

iMichka commented Dec 15, 2020

I went for the patch as the new release should come before end of the year, but we did not get a clear date.

@chenrui333 chenrui333 mentioned this pull request Jan 3, 2021
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 15, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
deprecated license Formula uses a deprecated SPDX license which should be updated outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants