On Mon, Aug 14, 2023 at 09:20:38PM +0300, Leon Romanovsky wrote: > On Mon, Aug 14, 2023 at 12:32:24PM -0300, Jason Gunthorpe wrote: > > On Sun, Aug 13, 2023 at 01:33:05PM +0300, Leon Romanovsky wrote: > > > On Sat, Aug 12, 2023 at 04:40:57PM +0000, St Savage, Shane wrote: > > > > Hi all, > > > > > > > > Just wanted to report that infiniband-diags cannot currently be installed in Fedora CoreOS because the perl dependency is explicitly forbidden. > > > > > > > > https://github.com/coreos/fedora-coreos-config/blob/testing-devel/manifests/fedora-coreos.yaml#L170 > > > > > > > > This is a bit unfortunate because it also prevents usage of all the non-perl utilities (ibstat, etc) included in infiniband-diags. > > > > > > > > Would it make sense to split the perl utilities to a separate package infiniband-diags-perl so that the C and shell utilities in infiniband-diags can be installed without the perl dependency? > > > > > > I suggest to remove perl dependency from rdma--core.spec and install > > > perl-dependant scripts only if perl is found on the system. > > > > That is not how packaging is supposed to work > > > > Everything should be installed always > > Isn't how we use some packages in pyverbs? No, just the test suite which we don't really package as an actual binary Jason
