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
[20.03] haskellPackages.geojson: unmark as broken #90677
[20.03] haskellPackages.geojson: unmark as broken #90677
Conversation
Sorry for mass ping -.- I wrongly chose |
It's mainly for end users, so they don't have to compile a broken package etc. It can also easily be bypassed in several ways. |
It’s mostly for the hydra tbh; otherwise it would have to try to build every broken package every time somebody pushes a commit to nixpkgs. |
But I agree with @worldofpeace, it’s super easy to wrap It should be better documented how to mark packages as unbroken at the same time however. |
@worldofpeace @Profpatsch I see your points here, especially the one about Hydra resource usage. Still I want to emphasize that by fixing the problem on my side (e.g. by |
I don't think we can conceptually work around this. We have over 6000 packages from hackage marked as broken. Since there is no way to depublish anything on hackage, a lot of them are probably really of no relevance to anyone. What we should think about is better tooling. I had the idea about a database exposed via a webpage, that shows all haskellPackages and which of them are marked broken. Then in that list we could (if we had the build power) display if it builds despite the broken flag. (Maybe we could make this pull based? Offer a button to trigger a built?) And we could actually offer a button which creates a PR to remove the broken flag. Just toying around here, not sure when I would have the time to implement something like that. |
This seems to build the whole time.
I've said it in the past but let this be one more datapoint in the discussion: Having to unmark packages annoys me (as a contributor) more than seeing a broken package fail every time I want to use it.
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)