Skip to content

packaging: update Macports portfile#65

Open
barracuda156 wants to merge 1 commit intogreatscottgadgets:masterfrom
barracuda156:macports
Open

packaging: update Macports portfile#65
barracuda156 wants to merge 1 commit intogreatscottgadgets:masterfrom
barracuda156:macports

Conversation

@barracuda156
Copy link
Copy Markdown

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
Copy Markdown
Contributor

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
Copy Markdown
Author

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

@straithe Thanks!

@barracuda156
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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
Copy Markdown
Author

@straithe Thanks! No hurry.

@mossmann
Copy link
Copy Markdown
Member

mossmann commented Dec 7, 2023

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

@barracuda156
Copy link
Copy Markdown
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