All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phillip Susi <psusi@ubuntu.com>
To: christophe.varoqui@opensvc.com
Cc: device-mapper development <dm-devel@redhat.com>,
	Christophe Varoqui <christophe.varoqui@gmail.com>
Subject: Re: [PATCH] Don't add 'p' delimiter when you shouldn't
Date: Tue, 08 Jan 2013 09:20:06 -0500	[thread overview]
Message-ID: <50EC2B16.9080007@ubuntu.com> (raw)
In-Reply-To: <1357628699.3819.4.camel@lapoo.opensvc.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 1/8/2013 2:04 AM, Christophe Varoqui wrote:
> On lun., 2013-01-07 at 14:56 -0500, Phillip Susi wrote:
>> The 'p' delimiter is supposed to be added when the base disk name
>> ends in a digit.  This decision was based on the name given on
>> the command line, not the canonical device name, so giving
>> /dev/dm-0 instead of /dev/mapper/foo triggered the digit test and
>> added the 'p'.  Changed test to use the canonical name rather
>> than the given name.
>> 
> Isn't that already merged ?
> 
> commit 02ce0e8f40f10c0d0c6ad15b3abd02cb43653879 Author: Phillip
> Susi <psusi@ubuntu.com> Date:   Thu Feb 9 21:16:21 2012 +0100

Sorry, for some reason I was using
git://git.kernel.org/pub/scm/linux/storage/multipath/hare/multipath-tools.git,
and it's out of date.




-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (MingW32)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQEcBAEBAgAGBQJQ7CsWAAoJEJrBOlT6nu75jeUH/3sVO84wu+C2ua+ysziBES94
MWDb+SfCfYkvgQvob+W/PXaWL7/dkjTXPDj0ohCLsufYHw1kI58PZKnWSEZWI2Xa
we/C49PZcdOrGI0EOi0PPdIgoX2tEtv5THk9Yo0M2H6I0CzvZ0R6s4+xd4aOMtvF
LZR1C0jbtdcbF2gyd6mTqGD4jzH5z3wATYRbQUJahOTYS2t8Q1ROrekKst0bbB5c
iXSTvJPZd9/f/sCsZa+rS1jKbI3NBMr51YvAr+i/i4akac9f7I//EC8iiOO3COBv
k8IPspXYB0Myy0H3XG4vTPki+Juh3zK605BIoFGdCKnUAuQ5WSLcDV6X8bHvlWQ=
=9Ozx
-----END PGP SIGNATURE-----

  reply	other threads:[~2013-01-08 14:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-07 19:56 [PATCH] Don't add 'p' delimiter when you shouldn't Phillip Susi
2013-01-08  7:04 ` Christophe Varoqui
2013-01-08 14:20   ` Phillip Susi [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-01-09  3:08 Phillip Susi
2012-02-09 19:33 ` Phillip Susi
2012-02-09 20:17 ` Christophe Varoqui

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=50EC2B16.9080007@ubuntu.com \
    --to=psusi@ubuntu.com \
    --cc=christophe.varoqui@gmail.com \
    --cc=christophe.varoqui@opensvc.com \
    --cc=dm-devel@redhat.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.