Skip to content

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

Open
d-w-moore wants to merge 2 commits into
irods:mainfrom
d-w-moore:44.master-dev
Open

Changes allowing audit plugin to work under Ubuntu 18 (master)#47
d-w-moore wants to merge 2 commits into
irods:mainfrom
d-w-moore:44.master-dev

Conversation

@d-w-moore

Copy link
Copy Markdown
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
Copy Markdown
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

trel commented May 14, 2020

Copy link
Copy Markdown
Member

awaiting #48

@trel

trel commented Oct 12, 2021

Copy link
Copy Markdown
Member

needs a rebase here too.

@alanking

alanking commented Apr 6, 2022

Copy link
Copy Markdown
Contributor

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

@alanking

alanking commented Apr 6, 2022

Copy link
Copy Markdown
Contributor

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

trel commented Apr 6, 2022

Copy link
Copy Markdown
Member

okie doke. power through.

@korydraughn

Copy link
Copy Markdown
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