Skip to content

Remove backwards compatibility with V1 code #710

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

Merged
merged 1 commit into from
Jun 19, 2020
Merged

Conversation

hamishmack
Copy link
Collaborator

@hamishmack hamishmack commented Jun 19, 2020

This was a nice idea, but in practice it did not work unless you
were explicitly extracting just config, overlays and system
(see #542).

~/iohk/haskell.nix$ nix-build -E 'import <nixpkgs> (import ./.)'
error: anonymous function at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/default.nix:20:1 called with unexpected argument '__functor', at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/impure.nix:84:1

This change will replace #542 errors with #709 errors.

@hamishmack hamishmack requested a review from angerman June 19, 2020 04:05
@hamishmack hamishmack mentioned this pull request Jun 19, 2020
This was a nice idea, but in practice it did not work unless you
were explicitly extracting just `config`, `overlays` and `system`
(see #542).

```
~/iohk/haskell.nix$ nix-build -E 'import <nixpkgs> (import ./.)'
error: anonymous function at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/default.nix:20:1 called with unexpected argument '__functor', at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/impure.nix:84:1
```

This change will replace #542 errors with #709 errors.
@hamishmack hamishmack merged commit da44734 into master Jun 19, 2020
@iohk-bors iohk-bors bot deleted the hkm/remove-v1 branch June 19, 2020 04:11
@rvl
Copy link
Contributor

rvl commented Jun 20, 2020

How do I update cardano-wallet for this change? I never saw any warning trace. And I think changes like this should be put in the changelog.

@hamishmack
Copy link
Collaborator Author

https://github.com/input-output-hk/cardano-wallet/blob/47651ad7a8a66314c9e31f54aea167734e70f921/nix/default.nix#L16

Change:

haskellNix = import sources."haskell.nix"

to

haskellNix = (import sources."haskell.nix" {}).nixpkgsArgs;

hamishmack added a commit that referenced this pull request Jun 20, 2020
hamishmack added a commit that referenced this pull request Jun 20, 2020
* Revert "Remove backwards compatibility with V1 code (#710)"
* Fix issue with warning not being displayed
booniepepper pushed a commit to booniepepper/haskell.nix that referenced this pull request Feb 4, 2022
This was a nice idea, but in practice it did not work unless you
were explicitly extracting just `config`, `overlays` and `system`
(see input-output-hk#542).

```
~/iohk/haskell.nix$ nix-build -E 'import <nixpkgs> (import ./.)'
error: anonymous function at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/default.nix:20:1 called with unexpected argument '__functor', at /nix/store/f7mwvx7gl8lwjiqmpv60xapc3crvc474-nixpkgs-20.03pre194824.1f56d679f40/nixpkgs/pkgs/top-level/impure.nix:84:1
```

This change will replace input-output-hk#542 errors with input-output-hk#709 errors.
booniepepper pushed a commit to booniepepper/haskell.nix that referenced this pull request Feb 4, 2022
* Revert "Remove backwards compatibility with V1 code (input-output-hk#710)"
* Fix issue with warning not being displayed
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.

3 participants