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

mint: 0.5.0 -> 0.7.1 #80391

Merged
merged 1 commit into from Feb 18, 2020
Merged

mint: 0.5.0 -> 0.7.1 #80391

merged 1 commit into from Feb 18, 2020

Conversation

infinisil
Copy link
Member

Motivation for this change

This also fixes the build with the latest crystal version

Don't backport this, 20.03 can be fixed without a version update

Things done
  • Built it successfully
  • Created a project and tested it

This also fixes the build with the latest crystal version
Copy link
Member

@Br1ght0ne Br1ght0ne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nixpkgs-review doesn't pass

[0 built (1 failed)]
error: build of '/nix/store/w80af30mcg32a4lrbxykwr3kisb87nyc-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/80391
1 package failed to build:
mint

This bug is fixed in #80390

impure path `/homeless-shelter/.cache/crystal/build-source-lib-baked_file_system-src-loader.cr/macro_run' used in link
collect2: error: ld returned 1 exit status

So this should work with Crystal 0.33.0

@infinisil
Copy link
Member Author

@filalex77 Huh weird, both me and ofborg seem to be able to build it successfully on Linux. Or are you on Darwin perhaps?

@Br1ght0ne
Copy link
Member

Br1ght0ne commented Feb 18, 2020

@infinisil This ran on Ubuntu 18.04 with single-user Nix installed. I'll try to run it on NixOS.

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

diff LGTM
fixes broken build
shows usage

https://github.com/NixOS/nixpkgs/pull/80391
1 package built:
mint

@jonringer jonringer merged commit 9408c29 into NixOS:master Feb 18, 2020
@infinisil infinisil deleted the update/mint branch February 18, 2020 14:55
@Br1ght0ne Br1ght0ne mentioned this pull request Feb 26, 2020
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants