Skip to content

Deploy to Cachix even for PR from forks.#86

Merged
palmskog merged 1 commit intomasterfrom
pull-request-target
Sep 3, 2021
Merged

Deploy to Cachix even for PR from forks.#86
palmskog merged 1 commit intomasterfrom
pull-request-target

Conversation

@Zimmi48
Copy link
Copy Markdown
Member

@Zimmi48 Zimmi48 commented Aug 31, 2021

Apply the changes from rocq-community/templates#100.

@Zimmi48
Copy link
Copy Markdown
Member Author

Zimmi48 commented Aug 31, 2021

Tested at #84.

Copy link
Copy Markdown
Member

@palmskog palmskog left a comment

Choose a reason for hiding this comment

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

Looks good to me in the CI logs.

However, I'm always a bit mystified that all Nix builds seem to have this log message multiple times:

Error: Cannot find plugins directory

It seems harmless and to never affect the build, though.

@palmskog palmskog merged commit b717c88 into master Sep 3, 2021
@palmskog palmskog deleted the pull-request-target branch September 3, 2021 16:55
@Zimmi48
Copy link
Copy Markdown
Member Author

Zimmi48 commented Sep 3, 2021

Thanks for reporting, I'll look into it.

@palmskog
Copy link
Copy Markdown
Member

palmskog commented Sep 3, 2021

If it helps, here's a fresh example from a non-plugin project: https://github.com/coq-community/huffman/pull/30/checks?check_run_id=3508651515#step:8:384

@Zimmi48
Copy link
Copy Markdown
Member Author

Zimmi48 commented Sep 6, 2021

OK, the reason was that some version comparison logic mistake made it so that we were using the COQLIB variable instead of the COQMF_COQLIB variable to tell Coq where to install the library. This is fixed in NixOS/nixpkgs#136908.

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.

2 participants