irods / irods_rule_engine_plugin_audit_amqp

BSD 3-Clause "New" or "Revised" License
2 stars 14 forks source link

[irods/irods#7531] bump irods version requirement and remove exact requirement (main) #135

Closed SwooshyCueb closed 9 months ago

SwooshyCueb commented 9 months ago

In service of irods/irods#7531

Merge irods/irods#7535 before merging this.

SwooshyCueb commented 9 months ago

#'d

alanking commented 9 months ago

Before I merge this, I just want to ask...

The minimum required CMake version on 4-3-stable is going to be higher than it is on main due to the use of the version ranges in that branch. I think that's okay, but just wanted to say it out loud before merging. Are we okay with that situation?

SwooshyCueb commented 9 months ago

I'm fine with it. We have 3.21 in our externals.

korydraughn commented 9 months ago

That sounds fine. If we run into issues, we can always bump the cmake minimum version.

alanking commented 9 months ago

Yep, sounds good. It is written. Mergin.