Page Comparison

Note (because it’s not obvious): I thought pkg:checksig's args might need an update to take multiple paths in rpm_dir (in order to accommodate JMG’s multiple OS/external repos work). However, the task internally glob-expands rpm_dir (into rpm_dirs), so it’s already possible to pass in multiple directories—although the string might end up looking extra weird.

Two drawbacks are worth mentioning:

1. If the paths are very different and don’t wildcard well, it will be necessary to use literal brace expansion (ex: {p,q}) instead.

2. But since it’s an argument for a Rake task, the commas in a brace expansion won’t work from the command line without some additional quote/escape ugliness.

From the command line (needs the quotes and escaped comma):

pkg:checksig['/path/to/{dir1/sub1\,dir2/sub2}']

rake-helpers style Ruby code just needs quotes:

Rake::Task["pkg:checksig"].invoke('/path/to/{dir1/sub1,dir2/sub2}')

If we split out the repos into the different directories JMG’s been working on, the brace expansion might look something like this:
Rake::Task["pkg:checksig"].invoke('/path/to/rpm_dirs/{SIMP,puppet,postgresql96,epel8}','/path/to/key_dir')

EL8 repoclosure CLI is missing args from EL7 repoclosure!

EL8 repoclosure CLI is missing args from EL7 repoclosure!

dnf repoclosure \
  --arch=x86_64 --arch=i686 --arch=noarch \
  --repofrompath=target-repo,file:///repos/compose/os \
  --repo=target-repo \
  --check=target-repo \
  --repofrompath=lookaside-repo-0,file:///repos/lookaside/BaseOS \
  --repo=lookaside-repo-0 \
  --repofrompath=lookaside-repo-1,file://repos/lookaside/AppStream \
  --repo=lookaside-repo-1

EL7 repoclosure still won’t work with this syntax

EL7 repoclosure understands --repofrompath=, but not --check , so this alternative won’t work in EL7 unless I can find something else. 🤦‍♀️

Or we just need to build a different check string between EL7 or EL8