Skip to content

better CI for json plugin #572

@Yushan-Wang

Description

@Yushan-Wang

Originally posted by @jbigot in #571 (comment)

Forcing -DUSE_JSON=EMBEDDED is not ideal. We should do like for all other libraries, include it in some test images (the all images) and not others (the mini images) so that we test both the SYSTEM and EMBEDDED situations.

If we want to switch the plugin from OFF by default to "${BUILD_UNSTABLE}", there are a few things we should do:

  • We should create new test images with these new dependencies (v4). And since Ubuntu Focal is becoming unsupported, I would also switch to Jammy as the oldest version we support.
  • This also means this is a new dependency that should be documented.
  • We should make a Checklist of what to do when introducing a new plugin.
  • We'll have to package this new plugin in all our packaging (binary, spack, etc.)

I propose to make an issue about this and enable the plugin as an unstable one in the next feature release, but keep it as a disabled preview for this release so that we don't delay it too much.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions