From: Benjamin Drung <benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
To: Jason Gunthorpe <jgg-uk2M96/98Pc@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH rdma-core] debian: Only run udevadm if the binary is available
Date: Thu, 28 Dec 2017 10:18:46 +0100 [thread overview]
Message-ID: <1514452726.4412.20.camel@profitbricks.com> (raw)
In-Reply-To: <20171227202627.GH25436-uk2M96/98Pc@public.gmane.org>
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.
--
Benjamin Drung
System Developer
Debian & Ubuntu Developer
ProfitBricks GmbH
Greifswalder Str. 207
D - 10405 Berlin
Email: benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org
URL: https://www.profitbricks.de
Sitz der Gesellschaft: Berlin
Registergericht: Amtsgericht Charlottenburg, HRB 125506 B
Geschäftsführer: Achim Weiss, Matthias Steinberg
--
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
next prev parent reply other threads:[~2017-12-28 9:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-27 10:41 [PATCH rdma-core] debian: Only run udevadm if the binary is available Benjamin Drung
[not found] ` <20171227104135.24391-1-benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2017-12-27 20:26 ` Jason Gunthorpe
[not found] ` <20171227202627.GH25436-uk2M96/98Pc@public.gmane.org>
2017-12-28 9:18 ` Benjamin Drung [this message]
[not found] ` <1514452726.4412.20.camel-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2017-12-28 21:01 ` Jason Gunthorpe
[not found] ` <20171228210124.GI25436-uk2M96/98Pc@public.gmane.org>
2018-01-03 10:22 ` [PATCH rdma-core v2] debian: Add udev dependency to rdma-core and srptools Benjamin Drung
[not found] ` <20180103102211.28307-1-benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2018-01-03 21:24 ` Jason Gunthorpe
2018-01-09 11:20 ` Leon Romanovsky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1514452726.4412.20.camel@profitbricks.com \
--to=benjamin.drung-eikl63zcoxah+58jc4qpia@public.gmane.org \
--cc=jgg-uk2M96/98Pc@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox