Skip to content
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

Separate preferences from plist and add user-concept #3455

Merged
merged 4 commits into from
Jul 3, 2017

Conversation

theopolis
Copy link
Member

This separates the overloaded use of the preferences table.

preferences acts like defaults on macOS. It is limited in that it cannot join users or report the system's defaults. In the current form it can also take any path input and report the flattened plist items.

This separates the defaults, keeping the functionality in preferences from the plist (a new table) functions.

@facebook-github-bot facebook-github-bot added the cla signed Automated label: Pull Request author has signed the osquery CLA label Jul 2, 2017
@osqueryer
Copy link

👎 The commit 52748a2 (Job results: 2120) failed one or more tests (Windows).

@osqueryer
Copy link

👎 The commit 52748a2 (Job results: 1337) failed one or more tests (macOS/OS X).

@osqueryer
Copy link

👎 The commit 52748a2 (Job results: 5220) failed one or more tests (Linux).

@osqueryer
Copy link

👎 The commit 52748a2 (Job results: 1338) failed one or more tests (macOS/OS X).

@facebook-github-bot
Copy link

@theopolis updated the pull request - view changes

@osqueryer
Copy link

🙅 The commit cdc2220 (Job results: 1971) failed the code audit or documentation test.

@osqueryer
Copy link

👎 The commit cdc2220 (Job results: 230) failed one or more tests (FreeBSD).

@osqueryer
Copy link

👎 The commit cdc2220 (Job results: 231) failed one or more tests (FreeBSD).

@facebook-github-bot
Copy link

@theopolis updated the pull request - view changes

@osqueryer
Copy link

🙅 The commit f49a185 (Job results: 1972) failed the code audit or documentation test.

@theopolis
Copy link
Member Author

Fixes: #3083

@theopolis
Copy link
Member Author

ok to test

@facebook-github-bot
Copy link

@theopolis updated the pull request - view changes

@facebook-github-bot
Copy link

@theopolis updated the pull request - view changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API change cla signed Automated label: Pull Request author has signed the osquery CLA macOS virtual tables
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants