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
sup: remove #68416
sup: remove #68416
Conversation
Still needs to be dropped from |
Thanks, I accidentally pushed this hunk to another branch. |
@@ -37,7 +37,5 @@ let | |||
}; | |||
}; | |||
in { | |||
# xapian-ruby needs 1.2.22 as of 2017-05-06 | |||
xapian_1_2_22 = generic "1.2.22" "0zsji22n0s7cdnbgj0kpil05a6bgm5cfv0mvx12d8ydg7z58g6r6"; |
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.
does this need an alias also?
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.
The gem was the last consumer for this library.
I considered adding 1.4.9
, since that's the last version of the xapian-ruby
gem, but until there's anything using it I'd say we can wait for that.
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.
nix-review
passes on NixOS
diff and intent LGTM
Nothing changed
https://github.com/NixOS/nixpkgs/pull/68416
No packages were successfully build, skip nix-shell
$ git worktree prune
Thanks. Cherry-picked as 1fed83d. |
Motivation for this change
Abandoned by upstream, and has dependencies with quite some issues, it's about time we get rid of it.
ZHF: #68361
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @cstrahan @lovek323 @nicknovitski