From: Benjamin Drung <benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
To: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: Benjamin Drung <benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
Subject: [PATCH rdma-core] debian: Only run udevadm if the binary is available
Date: Wed, 27 Dec 2017 11:41:35 +0100 [thread overview]
Message-ID: <20171227104135.24391-1-benjamin.drung@profitbricks.com> (raw)
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(-)
diff --git a/debian/rdma-core.postinst b/debian/rdma-core.postinst
index 6486f62e..26aec6d1 100644
--- a/debian/rdma-core.postinst
+++ b/debian/rdma-core.postinst
@@ -6,7 +6,9 @@ set -e
if [ "$1" = "configure" ]; then
# we ship udev rules, so trigger an update. This has to be done after
# DEBHELPER restarts systemd to get our new service files loaded.
- udevadm trigger --subsystem-match=infiniband --action=change || true
- udevadm trigger --subsystem-match=net --action=change || true
- udevadm trigger --subsystem-match=infiniband_mad --action=change || true
+ if which udevadm >/dev/null 2>&1; then
+ udevadm trigger --subsystem-match=infiniband --action=change
+ udevadm trigger --subsystem-match=net --action=change
+ udevadm trigger --subsystem-match=infiniband_mad --action=change
+ fi
fi
diff --git a/debian/srptools.postinst b/debian/srptools.postinst
index 398534db..9cdce914 100644
--- a/debian/srptools.postinst
+++ b/debian/srptools.postinst
@@ -6,5 +6,7 @@ set -e
if [ "$1" = "configure" ]; then
# we ship udev rules, so trigger an update. This has to be done after
# DEBHELPER restarts systemd to get our new service files loaded.
- udevadm trigger --subsystem-match=infiniband_mad --action=change || true
+ if which udevadm >/dev/null 2>&1; then
+ udevadm trigger --subsystem-match=infiniband_mad --action=change
+ fi
fi
--
2.14.1
--
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 reply other threads:[~2017-12-27 10:41 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-27 10:41 Benjamin Drung [this message]
[not found] ` <20171227104135.24391-1-benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2017-12-27 20:26 ` [PATCH rdma-core] debian: Only run udevadm if the binary is available Jason Gunthorpe
[not found] ` <20171227202627.GH25436-uk2M96/98Pc@public.gmane.org>
2017-12-28 9:18 ` Benjamin Drung
[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=20171227104135.24391-1-benjamin.drung@profitbricks.com \
--to=benjamin.drung-eikl63zcoxah+58jc4qpia@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