-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
osquery: remove #74081
osquery: remove #74081
Conversation
osquery was marked as broken since April. If somebody steps up to fix it, we can always revive it from the histroy, but there's not much value in shipping completely broken things in current master. cc @Ma27
Ping @Ma27 who indicated in #58834 (comment) to look into fixing the build (Edit: Oh you already pinged them, duh) |
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.
I don't use osquery
anymore. I tried to look into this several times, but the build system is pretty complex (also due to our cmake-related patches for osquery).
I don't have the time and motivation atm to fix this, so if noone else steps up, I'd be fine with removing it.
So let's remove it. If somebody wants to get it going before the next release, we can add it back and remove it from the changelog. |
The package has been removed in #74081.
osquery was marked as broken since April.
If somebody steps up to fix it, we can always revive it from the
histroy, but there's not much value in shipping completely broken things
in current master.
cc @Ma27
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)