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

fheroes2: use legacysupport.use_mp_libcxx on Mojave and older #18690

Conversation

aeiouaeiouaeiouaeiouaeiouaeiou
Copy link
Contributor

Description

icn2img uses std::filesystem since version 1.0.1: ihhub/fheroes2@275ccfd
Failing build: https://build.macports.org/builders/ports-10.9_x86_64-builder/builds/231789/steps/install-port/logs/stdio

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

macOS 10.13.6 17G14042 x86_64
Xcode 10.1 10B61

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?
  • 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:
@oleg-derevenetz for port fheroes2.

@macportsbot macportsbot added the maintainer: open Affects an openmaintainer port label May 15, 2023
@herbygillot herbygillot merged commit f079f02 into macports:master May 15, 2023
3 checks passed
@oleg-derevenetz
Copy link
Contributor

Hi @aeiouaeiouaeiouaeiouaeiouaeiou @herbygillot thanks! I looked at legacysupport port group at the time, but but in its README there is no mention of std::filesystem support :(

@aeiouaeiouaeiouaeiouaeiouaeiou aeiouaeiouaeiouaeiouaeiouaeiou deleted the fheroes2-filesystem branch May 15, 2023 09:59
@kencu
Copy link
Contributor

kencu commented May 18, 2023

the legacysupport portgroup already takes into account whether you're using libc++, so you don't need to test for clang being the compiler.

@kencu
Copy link
Contributor

kencu commented May 18, 2023

like most of macports' portgroups, that change fairly quickly, the documentation for the support for a newer libcxx is in the first paragraphs of the portgroup code (legacysupport-1.1.tcl).

MacPorts support for a newer libc++ has nothing to do with the macports-legacy-support headers or library, which is why it is not documented there.

You could say the support for a newer libc++ should never have been added to the legacysupport PortGroup then, if it doesn't use the macports-legacy-support library and there is some logic to that statement, however it was decided to add it there as the concepts are similar (supporting older systems).

If you are using the newer libc++, then the legacysupport-1.1.tcl PortGroup file will automatically add the proper dependency on macports-libcxx for you, as the code to do that has been added there.

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
5 participants