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
Tremor location fix #90464
Tremor location fix #90464
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your first PR! 🎉
As the repository for Tremor
is now hosted on gitlab, I think it would be preferable to use fetchFromGitLab
instead of the generic fetchgit
. Do you feel up for changing that?
Also, when you're done: Please squash your commits into one, with a commit message matching CONTRIBUTING.md.
f691e1e
to
ca6bbdf
Compare
I think, it's ready |
This PR causes a lot of rebuilds, see tags added by @GrahamcOfBorg. It should be made against the staging branch (see manual). Ideally, you can just change the base branch in the github interface for this PR. |
ah, indeed, should have gone to staging. Oh well, it's not too bad in this case. |
@FRidh What number of rebuilds would you consider a rough threshold for targeting staging instead of master? The manual speaks of "mass rebuilds", but isn't clear what that is xD. Maybe when the tag goes red, i.e. |
Yes, we draw the line at 500. |
Tremor is not available any longer via 'https://git.xiph.org/tremor.git'.
Motivation for this change
Tremor is not accessible via https://git.xiph.org/tremor.git
Things done
Outdated git location replaced with actual
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)