-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
tuist: init at 4.38.2 #369050
base: master
Are you sure you want to change the base?
tuist: init at 4.38.2 #369050
Conversation
67fff2f
to
cdebc85
Compare
version = "4.38.2"; | ||
|
||
src = fetchurl { | ||
url = "https://github.com/tuist/tuist/releases/download/${finalAttrs.version}/tuist.zip"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As the source is available, please download that and compile from source instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It didn't work because we have an old version of Swift
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could I ask what version of swift it needs?
It shouldn't be a problem to update the version of swift in nixpkgs
unstable, then build from source using that. It needs doing anyhow if it is out of date, so it is preferable to importing a binary blob
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tuist needs Swift 5.10, nixpkgs is at 5.8
Seems like Swift update isn't going to be simple: #297655 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough, not much else you can do!
I'll see if I can lend a hand with a PR for Swift 6, it seems crazy to be importing binary blobs due to that derivation being stuck
cdebc85
to
e8b3181
Compare
Things done
Closes #166545
Tested basic functionality
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.