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

glib 2.52.3- fix pkg-config for linking non-framework libraries #15693

Closed
wants to merge 1 commit into from
Closed

glib 2.52.3- fix pkg-config for linking non-framework libraries #15693

wants to merge 1 commit into from

Conversation

StephanSiemen
Copy link
Contributor

caused problems downstream when linking unix libraries which are not frameworks

See message: https://discourse.brew.sh/t/linking-libraries-problem-caused-by-glib-pkg-config-file/927

  • [ X] Have you followed the guidelines for contributing?
  • [ X] Have you checked that there aren't other open pull requests 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] Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

caused problems downstream when linking unix libraries which are not frameworks
@@ -89,6 +89,9 @@ def install
"Libs: -L${libdir} -lglib-2.0 -L#{gettext}/lib -lintl"
s.gsub! "Cflags: -I${includedir}/glib-2.0 -I${libdir}/glib-2.0/include",
"Cflags: -I${includedir}/glib-2.0 -I${libdir}/glib-2.0/include -I#{gettext}/include"
# correct linking of framework - caused broken links downstream
Copy link
Contributor

Choose a reason for hiding this comment

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

Please report this upstream and link to the report here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you mean with "upstream"?

Copy link
Contributor

Choose a reason for hiding this comment

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

@ilovezfs ilovezfs added the upstream issue An upstream issue report is needed label Jul 17, 2017
@stale stale bot added the stale No recent activity label Aug 12, 2017
@stale
Copy link

stale bot commented Aug 12, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@JCount
Copy link
Contributor

JCount commented Aug 16, 2017

@Stale[bot] This still should be reported upstream.

@stale stale bot removed the stale No recent activity label Aug 16, 2017
@ilovezfs
Copy link
Contributor

@StephanSiemen what's the status here?

@fxcoudert
Copy link
Member

Closing, because:

  • -Wl,-framework -Wl,CoreFoundation is correct as far as I can see: it passes down to the linker the options -framework CoreFoundation, which is exactly what is wanted. It is functionally identical to -Wl,-framework,CoreFoundation. I am not sure why other software doesn't like it, but it is not incorrect.
  • This needs to be reported to upstream, even if it were a problem, it is not Homebrew's but glib's.

@fxcoudert fxcoudert closed this Sep 10, 2017
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
upstream issue An upstream issue report is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants