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

pkgs/test/nixos-functions: use dummy versioning #50774

Merged
merged 1 commit into from Nov 22, 2018

Conversation

oxij
Copy link
Member

@oxij oxij commented Nov 19, 2018

With this OfBorg and similar tools won't report this derivation as changed every single time.

/cc @Mic92 who can now unblacklist it in nix-review :)

With this OfBorg and similar tools won't report this derivation as
changed every single time.
@grahamc
Copy link
Member

grahamc commented Nov 19, 2018

OfBorg looks at derivation paths, not versions.

@oxij
Copy link
Member Author

oxij commented Nov 19, 2018

But those variables influence drvpath and outpath of the NixOS top-level system derivation, which influences the derivation paths of those tests.

If you look at what OfBorg reports lately, you'll notice that it reports 1-10 linux changes for noop PRs. This will fix that.

@7c6f434c
Copy link
Member

Let's see; would be funny if it affects NixOS/ofborg#273 too…

@7c6f434c 7c6f434c merged commit 7b1d232 into NixOS:master Nov 22, 2018
@oxij
Copy link
Member Author

oxij commented Nov 22, 2018

I think that's unlikely.

Thanks.

@oxij oxij deleted the pkgs/test-versioning branch August 12, 2023 09:06
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

4 participants