RFE: support greater than/less than versions for packages in treefile

[miabbott]

RHCOS would like to be able to use syntax like podman > 1.4 in the treefile to enforce that higher NVRs always win out. (This is not always the case when dealing with modules)

Speaking with @cgwalters on Slack, he noted that rpm-ostree install 'cowsay >= 3.00' works just fine. But we split the entries in the package: section on whitespace, so that wouldn't work as is.

The suggestion was a new section in the treefile such as packages-literal that would allow the use of the greater/less than syntax

[miabbott]

Granted, if RHCOS used lockfiles like FCOS does this wouldn't be a problem. Still might be a nice feature for the broader community.

[jlebon]

We can also just skip splitting whitespace if the entry starts with a quote (and unquote it). I.e., require that such entries be their own array element, quoted.

[cgwalters]

In a quick look at the code, I think we have some potential drift between the rpm-ostree install paths and the compose path. It looks like the former uses

GPtrArray*
rpmostree_get_matching_packages (DnfSack *sack,
                                 const char *pattern)
{
  /* mimic dnf_context_install() */

But the compose path is using dnf_context_install()...they may differ now.

[kelvinfan001]

does this mean we need to update compose path to look like the one in rpm-ostree install, i.e. /* mimic dnf_context_install() */ instead of simply dnf_context_install()?

[cgwalters]

Honestly I'm not sure what's going wrong; I'd probably add debugging prints (or use gdb) and verify that e.g. rpm-ostree install 'cowsay > 1' on the client side is passing the same thing to dnf_context_install() that you are with the server side.

I'd also triple check the quoting is the same in both cases, i.e. you're correctly stripping quotes before it ends up being passed to dnf_context_install().

[miabbott]

Closed via #2151