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

Changes allowing audit plugin to work under Ubuntu 18 (master) #47

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

d-w-moore
Copy link
Contributor

No description provided.

@d-w-moore d-w-moore marked this pull request as draft May 8, 2020 18:02
@d-w-moore
Copy link
Contributor Author

This will have to wait a little bit longer til the jansson is replaced by nlohmann::json

@d-w-moore d-w-moore marked this pull request as ready for review May 11, 2020 18:04
@trel
Copy link
Member

trel commented May 14, 2020

awaiting #48

@trel
Copy link
Member

trel commented Oct 12, 2021

needs a rebase here too.

@alanking
Copy link
Contributor

alanking commented Apr 6, 2022

I think this is already in main at some level. I will pull it into my repo to see what happens

@alanking
Copy link
Contributor

alanking commented Apr 6, 2022

The CMakeLists changes have been superceded by recent overhauls to the CMake system in this and other repositories. The version bump in the build hook is actually older than the one currently in there. I think we can close this PR without merging as the changes herein are already in main at some capacity.

@trel
Copy link
Member

trel commented Apr 6, 2022

okie doke. power through.

@korydraughn
Copy link
Contributor

We can leave this open until we confirm the plugin works for 4.3.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants