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

use nix-build as build command #48

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

nrdxp
Copy link

@nrdxp nrdxp commented Oct 26, 2022

So that users don't have to enable experimental features, and to work around NixOS/nix#7220, use nix-build instead of nix build.

@Mic92
Copy link
Owner

Mic92 commented Oct 27, 2022

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 27, 2022

Configuration problem:
bors.toml: not found

@Mic92
Copy link
Owner

Mic92 commented Oct 27, 2022

bors merge

bors bot added a commit that referenced this pull request Oct 27, 2022
48: use nix-build as build command r=Mic92 a=nrdxp



Co-authored-by: Timothy DeHerrera <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 27, 2022

Timed out.

So that users don't have to enable experimental features, and to work
around NixOS/nix#7220, use `nix-build`
instead of `nix build`.
@nrdxp
Copy link
Author

nrdxp commented Oct 27, 2022

@Mic92, I see you just added bors support, so perhaps you are not yet aware, but bors takes the toml from the PR branch so that is mostly likely why it failed. Seems I don't have the priviledge to do a bors try to test it again (I just force pushed to incude the toml). If you wanna give it another shot though 🙏

@Mic92
Copy link
Owner

Mic92 commented Oct 28, 2022

bors merge

bors bot added a commit that referenced this pull request Oct 28, 2022
48: use nix-build as build command r=Mic92 a=nrdxp



Co-authored-by: Timothy DeHerrera <[email protected]>
@bors
Copy link
Contributor

bors bot commented Oct 28, 2022

Timed out.

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

2 participants