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

Ignore text-icu tests on Darwin #2254

Closed
wants to merge 1 commit into from
Closed

Conversation

jwiegley
Copy link
Contributor

The normalization test fails, but this doesn't stop the library for being useful. I will open another issue against text-icu itself for the normalization failure.

The normalization test fails, but this doesn't stop the library for being
useful.  I will open another issue against text-icu itself for the
normalization failure.
@jwiegley
Copy link
Contributor Author

This was already opened by ocharles in haskell/text-icu#6

@peti
Copy link
Member

peti commented Apr 14, 2014

Manual edits to cabal2nix-generated files are lost when the package is updated. It would be best to add the appropriate know-how to cabal2nix's post-processing function. See line 17 in that file for an example of how that's done.

Once cabal2nix know about this setting, an automatic re-generation process gets that update into the expression in Nixpkgs. (The same process that drops the change if it's not registered in cabal2nix.)

@shlevy
Copy link
Member

shlevy commented Apr 14, 2014

Can we add a comment to the top of autogenerated haskell packages with the warning that they would be overwritten and a pointer to instructions on patching cabal2nix?

@peti
Copy link
Member

peti commented Apr 15, 2014

Yes, that's certainly a good idea.

@@ -7,6 +7,7 @@ cabal.mkDerivation (self: {
version = "0.6.3.7";
sha256 = "0mzzjamcxfrld9xwgdww3890zzkgmi1qb83v5z9zrixmdgwkvhf4";
buildDepends = [ text ];
doCheck = !stdenv.isDarwin;
Copy link
Member

Choose a reason for hiding this comment

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

Note that stdenv can be accessed as self.stdenv in Cabal derivations.

@peti
Copy link
Member

peti commented Apr 15, 2014

I've adopted cabal2nix in NixOS/cabal2nix@66b710d and merged the commit.

@peti peti closed this Apr 15, 2014
@jwiegley jwiegley deleted the text-icu branch April 15, 2014 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants