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

libmd: fix build with gcc #25950

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

barracuda156
Copy link
Contributor

Closes: https://trac.macports.org/ticket/70706

Description

Fix build with gcc

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.6
Xcode 3.2

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@macportsbot
Copy link

Notifying maintainers:
@raimue for port libmd.

@macportsbot macportsbot added type: bugfix maintainer: open Affects an openmaintainer port labels Sep 26, 2024
@raimue
Copy link
Member

raimue commented Sep 26, 2024

The port builds by default on all recent macOS versions. I do not feel comfortable to maintain such a patch locally in MacPorts without understanding the configuration under which it would be necessary.

Such a patch should also be submitted upstream first and be discussed there. A fix for the problem can be committed to the upstream source for the next release. Then this can be backported as a patch in MacPorts to fix the immediate build problem of the software in its current release.

@barracuda156
Copy link
Contributor Author

@raimue I have attached log in the trac ticket.

For comparison, see: https://github.com/jpouellet/signify-osx/blob/master/hashaliases.c
This works fine, but the source of libmd uses something else instead, because it defines libmd_strong_alias.

@reneeotten
Copy link
Contributor

so what dis upstream say about your proposed patch? Clearly @raimue doesn't want this merged without a upstream discussion/resolution. If there is none yet, please close the PR until that has happened.

@raimue
Copy link
Member

raimue commented Oct 19, 2024

The trouble I see is that the patch implies that libmd_strong_alias only works with clang.

But then this should be detected at configure time and libmd_strong_alias should only be defined for clang on macOS. Maybe there is another attribute for (older) gcc on macOS that would work, but then the definition of libmd_strong_alias needs to be changed and not where it is used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer: open Affects an openmaintainer port type: bugfix
Development

Successfully merging this pull request may close these issues.

4 participants