Re: [PATCH rdma-core] debian: Only run udevadm if the binary is available

[Jason Gunthorpe <jgg-uk2M96/98Pc@public.gmane.org>]

On Thu, Dec 28, 2017 at 10:18:46AM +0100, Benjamin Drung wrote:
> Am Mittwoch, den 27.12.2017, 13:26 -0700 schrieb Jason Gunthorpe:
> > On Wed, Dec 27, 2017 at 11:41:35AM +0100, Benjamin Drung wrote:
> > > When udevadm is not available (e.g. in a minimal chroot
> > > environment),
> > > the rdma-core package installation will print these warnings:
> > > 
> > > /var/lib/dpkg/info/rdma-core.postinst: 33:
> > > /var/lib/dpkg/info/rdma-core.postinst: udevadm: not found
> > > /var/lib/dpkg/info/rdma-core.postinst: 34:
> > > /var/lib/dpkg/info/rdma-core.postinst: udevadm: not found
> > > /var/lib/dpkg/info/rdma-core.postinst: 35:
> > > /var/lib/dpkg/info/rdma-core.postinst: udevadm: not found
> > > 
> > > To silence these warnings, only run udevadm if this binary is
> > > available.
> > > 
> > > Signed-off-by: Benjamin Drung <benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
> > >  debian/rdma-core.postinst | 8 +++++---
> > >  debian/srptools.postinst  | 4 +++-
> > >  2 files changed, 8 insertions(+), 4 deletions(-)
> > 
> > Hurm. Some packages have this which test, others do not..
> > 
> > I'm confused how we get to a state where rdma-core can be installed
> > without udev? Virtually the entire purpose of that package revolves
> > around udev and systemd ... So are we missing a dependency?
> 
> Adding a dependency on udev is a solution too and is probably the
> better solution.

Okay, can you send that?

Thanks,
Jason

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html