From: Neil Horman <nhorman@tuxdriver.com>
To: Thomas Monjalon <thomas.monjalon@6wind.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 01/15 v2] makefiles: Fixed -share command line option error
Date: Fri, 2 May 2014 07:09:27 -0400 [thread overview]
Message-ID: <20140502110927.GA15335@hmsreliant.think-freely.org> (raw)
In-Reply-To: <4041301.VqGVgh9vVV@xps13>
On Wed, Apr 30, 2014 at 01:42:12AM +0200, Thomas Monjalon wrote:
> 2014-04-16 09:51, Neil Horman:
> > The shared libraries built with the current makefile set produce static
> > libraries rather than actual shared objects. This is due to several missing
> > options that are required to correctly build shared objects using ld, as
> > well as a mis-specified -share option (which should be -shared). Switching
> > to the use of CC rather than LD and fixing the -shared option corrects
> > these problems and builds the DSOs correctly.
> >
> > Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
>
> Applied for version 1.6.0r2.
>
> Thanks
> --
> Thomas
>
So, I just went and looked at 1.6.0r2 and noted that you applied this patch, but
the rest of the series is still missing. This is what I was talking about
earlier when I said you weren't applying patch series atomically. It makes it
impossible to have any clue what the upstream development head is going to look
like. On top of that, since you're clearly integrating other changes ahead of
this, theres every likelyhood the rest of my v5 series won't apply.
the v5 series has sat out here for a few weeks now without comment. If there
aren't any objections to it, apply it. Whats the problem here? I'm not going
to package the DPDK until this series (or the functionality it offers) is in
place.
Neil
next prev parent reply other threads:[~2014-05-02 11:09 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-15 18:05 [dpdk-dev] [PATCH 0/15] dpdk: Separate compile time linkage between eal lib and pmd's Neil Horman
2014-04-15 18:05 ` [dpdk-dev] [PATCH 01/15] makefiles: Fixed -share command line option error Neil Horman
2014-04-16 9:22 ` Thomas Monjalon
2014-04-16 11:00 ` Neil Horman
2014-04-16 11:37 ` Thomas Monjalon
2014-04-16 13:51 ` [dpdk-dev] [PATCH 01/15 v2] " Neil Horman
2014-04-18 11:23 ` Thomas Monjalon
2014-04-18 13:18 ` Neil Horman
2014-04-18 13:29 ` Thomas Monjalon
2014-04-18 17:36 ` Neil Horman
2014-04-21 14:41 ` Neil Horman
2014-04-29 23:42 ` Thomas Monjalon
2014-05-02 11:09 ` Neil Horman [this message]
2014-05-02 12:22 ` Thomas Monjalon
2014-05-02 13:01 ` Neil Horman
2014-05-02 13:18 ` Thomas Monjalon
2014-04-15 18:05 ` [dpdk-dev] [PATCH 02/15] make: include whole archive on static link Neil Horman
2014-04-16 9:26 ` Thomas Monjalon
2014-04-16 11:02 ` Neil Horman
2014-04-16 11:40 ` Thomas Monjalon
2014-04-16 13:02 ` Neil Horman
2014-04-16 13:33 ` Neil Horman
2014-04-15 18:05 ` [dpdk-dev] [PATCH 03/15] pmd: Add PMD_REGISTER_DRIVER macro Neil Horman
2014-04-16 11:52 ` Thomas Monjalon
2014-04-16 12:59 ` John W. Linville
2014-04-16 13:08 ` Neil Horman
2014-04-16 16:11 ` Olivier MATZ
2014-04-16 17:15 ` John W. Linville
2014-04-16 17:29 ` Neil Horman
2014-04-17 8:08 ` Olivier MATZ
2014-04-17 10:59 ` Neil Horman
2014-04-18 11:42 ` Thomas Monjalon
2014-04-18 12:04 ` Neil Horman
2014-04-18 12:08 ` Thomas Monjalon
2014-04-18 13:20 ` Neil Horman
2014-04-18 13:32 ` Thomas Monjalon
2014-04-18 17:42 ` Neil Horman
2014-04-15 18:05 ` [dpdk-dev] [PATCH 04/15] pcap: Convert to use of PMD_REGISTER_DRIVER and fix linking Neil Horman
2014-04-15 18:05 ` [dpdk-dev] [PATCH 05/15] ring: " Neil Horman
2014-04-16 13:53 ` [dpdk-dev] [PATCH 05/15 v2] " Neil Horman
2014-04-17 9:50 ` [dpdk-dev] [PATCH 05/15] " Ananyev, Konstantin
2014-04-17 11:06 ` Neil Horman
2014-04-17 15:16 ` [dpdk-dev] [PATCH 05/15 v3] " Neil Horman
2014-06-13 13:28 ` De Lara Guarch, Pablo
2014-04-15 18:06 ` [dpdk-dev] [PATCH 06/15] xenvirt: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 07/15] eal: Make vdev init path generic for both virtual and physcial devices Neil Horman
2014-04-18 12:02 ` Thomas Monjalon
2014-04-15 18:06 ` [dpdk-dev] [PATCH 08/15] igb: Convert to use of PMD_REGISTER_DRIVER and fix linking Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 09/15] igbvf: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 10/15] e1000: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 11/15] ixgbe: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 12/15] ixgbevf: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 13/15] virtio: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 14/15] vmxnet3: " Neil Horman
2014-04-15 18:06 ` [dpdk-dev] [PATCH 15/15] pmd: Remove rte_pmd_init_all Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH v5 00/14] dpdk: Separate compile time linkage between eal lib and pmd's Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 01/14] makefiles: Fixed -share command line option error Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 02/14] pmd: Add PMD_REGISTER_DRIVER macro Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 03/14] pcap: Convert to use of PMD_REGISTER_DRIVER and fix linking Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 04/14] ring: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 05/14] xenvirt: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 06/14] eal: Make vdev init path generic for both virtual and physcial devices Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 07/14] igb: Convert to use of PMD_REGISTER_DRIVER and fix linking Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 08/14] igbvf: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 09/14] e1000: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 10/14] ixgbe: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 11/14] ixgbevf: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 12/14] virtio: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 13/14] vmxnet3: " Neil Horman
2014-04-21 14:59 ` [dpdk-dev] [PATCH 0/X v5 14/14] pmd: Remove rte_pmd_init_all Neil Horman
2014-04-21 17:05 ` [dpdk-dev] [PATCH v5 00/14] dpdk: Separate compile time linkage between eal lib and pmd's Neil Horman
2014-04-21 20:10 ` Stephen Hemminger
2014-04-21 20:36 ` Neil Horman
2014-05-16 15:28 ` Neil Horman
2014-05-16 15:39 ` Thomas Monjalon
2014-05-20 12:45 ` Thomas Monjalon
2014-05-20 14:13 ` Neil Horman
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=20140502110927.GA15335@hmsreliant.think-freely.org \
--to=nhorman@tuxdriver.com \
--cc=dev@dpdk.org \
--cc=thomas.monjalon@6wind.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).