From: Thomas Monjalon <thomas.monjalon@6wind.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>,
"Mrzyglod, DanielX T" <danielx.t.mrzyglod@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] examples/bond: add header to support freebsd compilation
Date: Mon, 07 Dec 2015 02:40:53 +0100 [thread overview]
Message-ID: <1668763.pkmebeYeZt@xps13> (raw)
In-Reply-To: <20151202105358.GA25180@sivlogin002.ir.intel.com>
2015-12-02 10:53, Ferruh Yigit:
> On Thu, Nov 26, 2015 at 09:55:17AM +0000, Mrzyglod, DanielX T wrote:
> >
> >
> > >-----Original Message-----
> > >From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> > >Sent: Wednesday, November 25, 2015 7:08 PM
> > >
> > >2015-11-25 19:03, Daniel Mrzyglod:
> > >> definition of 'AF_INET' enum was missing - is available in <sys/socket.h>
> > >>
> > >> Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@intel.com>
> > >
> > >It is definitely the right fix as Ferruh submitted the same one
> > >less than one hour ago.
> > >Should we understand it is an ack?
> > >
> > >It seems Ferruh was hesitating about the line where inserting the
> > >include. I would say I prefer your choice :)
> >
> > I acked Ferruh patch. I didn't notice he send patch first.
> > You can combine both patches :> He has much better description :>
>
> Thomas said he prefer this one, let's use this one.
>
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
Applied, thanks
prev parent reply other threads:[~2015-12-07 1:42 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-25 18:03 Daniel Mrzyglod
2015-11-25 18:08 ` Thomas Monjalon
2015-11-26 9:55 ` Mrzyglod, DanielX T
2015-12-02 10:53 ` Ferruh Yigit
2015-12-07 1:40 ` Thomas Monjalon [this message]
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=1668763.pkmebeYeZt@xps13 \
--to=thomas.monjalon@6wind.com \
--cc=danielx.t.mrzyglod@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).