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

Proposal: explicit (temporary) name for darwin SDK check #175862

Closed
ylh opened this issue Jun 1, 2022 · 1 comment
Closed

Proposal: explicit (temporary) name for darwin SDK check #175862

ylh opened this issue Jun 1, 2022 · 1 comment
Labels
6.topic: darwin Running or building packages on Darwin 6.topic: developer experience

Comments

@ylh
Copy link
Contributor

ylh commented Jun 1, 2022

#175653 (comment) was written somewhat tongue-in-cheek, but I stand by the core idea:

While the SDK situation on x86_64-darwin gets figured out, we could do with an interim solution for expressing brokenness as a result of the 10.12 SDK getting long in the tooth, and it could do with a name. The value would be along the lines of:

stdenv.isDarwin && lib.versionOlder stdenv.hostPlatform.darwinSdkVersion "10.13"

(can't be just the latter as darwinSdkVersion still evaluates to something on Linux)

Currently, others and I have been using broken = stdenv.isDarwin && stdenv.isx86_64 in those situations, especially to correct spurious broken = stdenv.isDarwin, but that's not really what we're trying to express here, and it unfortunately won't cause rebuilds when the x86_64 SDK does get bumped.

The idea behind giving it a name is that:

  • it's concise,
  • it encourages people to actually use it, and
  • its references and definition can be easily grepped and edited out of existence once it's no longer necessary.

The questions I have for the maintainers are:

  • Is this a good idea?
  • What would be a good name? isOldDarwinSdk doesn't exactly roll off the fingers.
  • Should it be a function in lib that takes a platform (if so, where in lib), or would it be acceptable to directly apply it and pollute stdenv's namespace for the sake of concision, bearing in mind that it's temporary?

Once I get some feedback, I'll put in a proper PR accordingly.

@FliegendeWurst FliegendeWurst added 6.topic: darwin Running or building packages on Darwin 6.topic: developer experience labels Jun 2, 2022
@reckenrode
Copy link
Contributor

Is this still needed? The SDK situation is being improved by #346043. After that lands, you can use a newer SDK by adding it to your build inputs.

@SuperSandro2000 SuperSandro2000 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: darwin Running or building packages on Darwin 6.topic: developer experience
Projects
None yet
Development

No branches or pull requests

4 participants