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

barrier: fix build with gcc #26964

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

barracuda156
Copy link
Contributor

Description

Fix the build on systems using 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?
  • 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:
@ryanakca for port barrier.

Copy link
Contributor

@reneeotten reneeotten left a comment

Choose a reason for hiding this comment

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

did you submit any of these patches upstream? Please add a link or do so

@barracuda156
Copy link
Contributor Author

did you submit any of these patches upstream? Please add a link or do so

@reneeotten Those are not needed with clang builds (which is why they are applied conditionally on gcc in use) and partially incompatible with the current way upstream builds this. And it will be pointless to apply them partially, since that is not sufficient to make gcc build work.

@reneeotten
Copy link
Contributor

most of the patches seem like they would be okay regardless if the compiler used and could/should thus be send upstream.

@barracuda156
Copy link
Contributor Author

barracuda156 commented Dec 11, 2024

@reneeotten Ok, I recalled now, I did not submit cstdint fix because there is already an existing PR for it: debauchee/barrier#1886

I think everything else is just tweaking the build to work for gcc, which does not support Apple ObjC/ObjC++ syntax fully.
I can try opening PR for some of that, but do not really expect it to get merged.

UPD. debauchee/barrier#2112

@barracuda156
Copy link
Contributor Author

@reneeotten @herbygillot Could we merge this?

As you can see from the fact that an absolutely credible and minimal fix for modern gcc is still pending in upstream, they are not quick with merging anything, it is pointless to wait. There were folks wanting specifically this port on powerpc, let them have it, since I doubt they are using my fork of MacPorts, and also they were on Leopard, not 10.6 ppc.

@@ -46,15 +44,33 @@ if { ${os.major} < 16 } {
configure.args-append \
Copy link
Contributor

Choose a reason for hiding this comment

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

this needs the os.platform check as well

aqua/barrier/Portfile Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants