From: Bart Van Assche <bart.vanassche@sandisk.com>
To: Xose Vazquez Perez <xose.vazquez@gmail.com>
Cc: device-mapper development <dm-devel@redhat.com>
Subject: Re: [PATCH] multipath-tools: use internal drd.h file
Date: Mon, 10 Oct 2016 13:01:23 -0700 [thread overview]
Message-ID: <b40d2cee-ea80-24c1-54d8-58c109a044b2@sandisk.com> (raw)
In-Reply-To: <20161008202212.11963-1-xose.vazquez@gmail.com>
On 10/08/2016 01:22 PM, Xose Vazquez Perez wrote:
> Cc: Bart Van Assche <bart.vanassche@sandisk.com>
> Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>
> Cc: device-mapper development <dm-devel@redhat.com>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
> libmultipath/debug.c | 2 +-
> multipathd/main.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libmultipath/debug.c b/libmultipath/debug.c
> index 295136c..fbe171a 100644
> --- a/libmultipath/debug.c
> +++ b/libmultipath/debug.c
> @@ -8,7 +8,7 @@
> #include "log_pthread.h"
> #include <sys/types.h>
> #include <time.h>
> -#include <../third-party/valgrind/drd.h>
> +#include "../third-party/valgrind/drd.h"
> #include "vector.h"
> #include "config.h"
>
> diff --git a/multipathd/main.c b/multipathd/main.c
> index cdfafe8..0998085 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -23,7 +23,6 @@
> #endif
> #include <semaphore.h>
> #include <time.h>
> -#include <valgrind/drd.h>
>
> /*
> * libmultipath
> @@ -85,6 +84,7 @@ int uxsock_timeout;
> #include "lock.h"
> #include "waiter.h"
> #include "wwids.h"
> +#include "../third-party/valgrind/drd.h"
>
> #define FILE_NAME_SIZE 256
> #define CMDSIZE 160
Hello Xose,
The patch description is slightly misleading. I don't think anyone has a
/usr/third-party directory on her or his system so #include
<../third-party/valgrind/drd.h> will select the internal drd.h header
file. However, that first change improves consistency:
Reviewed-by: Bart Van Assche <bart.vanassche@sandisk.com>
next prev parent reply other threads:[~2016-10-10 20:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-08 20:22 [PATCH] multipath-tools: use internal drd.h file Xose Vazquez Perez
2016-10-10 20:01 ` Bart Van Assche [this message]
2016-10-16 10:26 ` Xose Vazquez Perez
2016-10-16 15:54 ` Christophe Varoqui
2016-10-16 16:59 ` Hannes Reinecke
2016-10-16 17:34 ` 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=b40d2cee-ea80-24c1-54d8-58c109a044b2@sandisk.com \
--to=bart.vanassche@sandisk.com \
--cc=dm-devel@redhat.com \
--cc=xose.vazquez@gmail.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.