From: maurosr@linux.vnet.ibm.com (Mauro Rodrigues)
Subject: [PATCH 3/3] nvmetcli: Install nvmetcli executable in $PATH
Date: Wed, 5 Apr 2017 20:56:11 -0300 [thread overview]
Message-ID: <20170405235611.GA8885@korriban> (raw)
In-Reply-To: <20170330082855.GC11540@lst.de>
On Thu, Mar 30, 2017@10:28:55AM +0200, Christoph Hellwig wrote:
> On Mon, Mar 27, 2017@02:46:45PM -0300, Mauro S. M. Rodrigues wrote:
> > According to the README, one can install nvmetcli through setup.py.
> > When we try this, though, the nvmetcli wasn't configured in system's
> > $PATH.
> > Adding it to script list in setup.py solved the issue.
>
> Andy sent the same patch a while ago, but it needs a fixup to
>
> a) install into /usr/sbin
> b) fixups to the deb and rpm packaging to take this into account
>
> Do you want to fix this up? Otherwise I'll try to get to it myself,
> I should be able to access a Fedora test box again now.
>
Hi Cristoph!
I just sent a new version of the patch. I've also checked the contents
of both rpm and deb packages and everything looks correct:
~/dev/nvmetcli$ dpkg -c dist/nvmetcli_0.3.3.g08963d2_all.deb |grep /usr/sbin/nvmetcli
-rwxr-xr-x root/root 17050 2017-04-05 15:47 ./usr/sbin/nvmetcli
and for the rpm:
# rpm -ql nvmetcli |grep /usr/sbin/nvmetcli
/usr/sbin/nvmetcli
Please let me know if I missed anything here.
Thanks in advanced!
--
Mauro Rodrigues
next prev parent reply other threads:[~2017-04-05 23:56 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-27 17:46 [PATCH 1/3] nvmetcli: Fix Port and Host operations in shell Mauro S. M. Rodrigues
2017-03-27 17:46 ` [PATCH 2/3] nvmetcli: Fix Referral operation " Mauro S. M. Rodrigues
2017-03-28 10:41 ` Sagi Grimberg
2017-03-30 8:27 ` Christoph Hellwig
2017-03-27 17:46 ` [PATCH 3/3] nvmetcli: Install nvmetcli executable in $PATH Mauro S. M. Rodrigues
2017-03-28 10:41 ` Sagi Grimberg
2017-03-30 8:28 ` Christoph Hellwig
2017-04-05 23:56 ` Mauro Rodrigues [this message]
2017-04-06 7:20 ` Christoph Hellwig
2017-03-28 10:40 ` [PATCH 1/3] nvmetcli: Fix Port and Host operations in shell Sagi Grimberg
2017-03-30 8:27 ` Christoph Hellwig
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=20170405235611.GA8885@korriban \
--to=maurosr@linux.vnet.ibm.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.