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

packaging: update Macports portfile #65

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

Conversation

barracuda156
Copy link

Updates Macports packaging.

I am not sure if the current portfile dated back to 2014 was in Macports before, but I have restored libbtbb into Macports now.
macports/macports-ports#16538

Build is confirmed on macOS 12, macOS 11 (CI checks in the PR), 10.8.5, 10.6.8 Rosetta and 10.6 PPC.

P. S. There is no activity from @dominicgs for past 3 years, but I will be happy to add him as a co-maintainer, if he happens to be interested.

@straithe
Copy link
Member

straithe commented Nov 1, 2022

Dominic no longer works at GSG. I'll see who else on the team is available to review.

@barracuda156
Copy link
Author

Dominic no longer works at GSG. I'll see who else on the team is available to review.

@straithe Thanks!

@barracuda156
Copy link
Author

@straithe I have rebased now with the portfile merged into Macports master (we decided to have two variants for the port): https://github.com/macports/macports-ports/blob/master/devel/libbtbb/Portfile
Builds across the board (not seen in buildbots, but including ppc): https://ports.macports.org/port/libbtbb/details

@straithe
Copy link
Member

straithe commented Dec 6, 2022

Thank you! My reviewer on the GSG team is unavailable and will continue to be for a while. I'm working on finding community members to review this PR.

@barracuda156
Copy link
Author

@straithe Thanks! No hurry.

@mossmann
Copy link
Member

mossmann commented Dec 7, 2023

@barracuda156 should we perhaps remove this Portfile from libbtbb since it lives in Macports now?

@barracuda156
Copy link
Author

@mossmann This is up to you. I do not know the background how it originally came up to be. Perhaps either removing or updating is fine, just do not carry on the old one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants