-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Update paths, and remove deprecated Facebook names #7210
Update paths, and remove deprecated Facebook names #7210
Conversation
If you rename the launchctl related files then you'll also need a change in |
Ok this should work if you:
https://github.com/osquery/osquery/blob/master/.github/workflows/build_aarch64.yml#L32 |
514e3d2
to
8916cdc
Compare
Looks like the three executable files need a |
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.
(see comment above)
Hey @sharvilshah and @directionless, where do the |
Hey @theopolis they are from #7184 |
Awesome, what do we need to do to merge that PR? |
Changes to update the various facebook names to the osquery ones. I tried to get the paths as well. I might have missed a couple
79fdcbd
to
01541c7
Compare
I think the only missing part is to update osquery-packaging, then include the new submodule commit here so that we don't break the packages generated by the CI from master; currently, the .pkg archive does not install correctly |
Huh? I thought we did that in #7184 |
I am not sure if I am doing something wrong, but I have attempted to download and install the .pkg file generated by the CI, and it installed empty folders:
|
Changes to update the various facebook names to the osquery ones.
I tried to get the paths as well. I might have missed a couple
Closes: #7159