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
Mint-lang: Init at 2018.05.27 #41148
Conversation
homepage = https://mint-lang.com/; | ||
license = stdenv.lib.licenses.bsd3; | ||
maintainers = with stdenv.lib.maintainers; [ manveru ]; | ||
platforms = [ "x86_64-linux" "i686-linux" "x86_64-darwin" ]; |
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.
Why just these platforms? IIRC crystal supports others too, e.g. aarch64.
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.
Crystal only has releases for these platforms: https://github.com/crystal-lang/crystal/releases/tag/0.24.2 and since you need Crystal to build Crystal, i don't think we'll have other platforms soon. Please correct me if I'm wrong, but those platforms are also set in the meta of the crystal package itself.
stdenv.mkDerivation rec { | ||
version = "2018-05-27"; | ||
name = "mint-${version}"; | ||
src = fetchgit { |
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.
Shouldn't this be fetchFromGitHub, for bandwidth/CPU savings?
installPhase = ''true''; | ||
|
||
meta = { | ||
description = "A refreshing language for the front-end web."; |
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.
Package descriptions should not end in a period
@@ -0,0 +1,33 @@ | |||
require "yaml" |
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.
Should this tool really live in a mint-specific directory?
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.
That's a good question, and eventually I want to make it a standalone tool, but until I package more Crystal applications, I think it can live here for a bit.
I can remove it if you don't want to have it here, just thought it makes the work for future packages a bit easier.
Looks good now! |
Thanks for the review! |
@GrahamcOfBorg build mint |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: mint Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: mint Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: mint Partial log (click to expand)
|
Motivation for this change
Found this the other day, page is at https://www.mint-lang.com/ and it's an example of how to package crystal applications.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)