DPDK patches and discussions
 help / color / mirror / Atom feed
From: Panu Matilainen <pmatilai@redhat.com>
To: Thomas Monjalon <thomas.monjalon@6wind.com>,
	Bruce Richardson <bruce.richardson@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] reserve 'make install' for future use
Date: Mon, 30 Nov 2015 14:26:20 +0200	[thread overview]
Message-ID: <565C406C.8020005@redhat.com> (raw)
In-Reply-To: <3475702.r5OlE1Gpee@xps13>

On 11/27/2015 07:33 PM, Thomas Monjalon wrote:
> 2015-11-25 10:48, Panu Matilainen:
>> On 11/24/2015 06:54 PM, Bruce Richardson wrote:
>>> On Fri, Nov 06, 2015 at 02:04:54PM +0100, Thomas Monjalon wrote:
>>>> 2015-11-06 12:57, Bruce Richardson:
>>>>> So, any thoughts or comments on this? There has been lots of discussion in this
>>>>> general area but nothing yet going into the release to try and improve the situation.
>>>>>
>>>>> Are we just going to kick the problem down the road to the 2.3 release?
>>>>
>>>> I plan to check these patches in the coming days for an integration in 2.2.
>>>>
>>> Anything further on this?
>>> Any thoughts from anyone else about this whole area of a saner build/install
>>> system for DPDK and the various patches floating around.
>>
>> Well, it seems we wont have a sane "make install" in 2.2 yet, but this
>> is at least a step in the right direction so +1 from me.
>
> Why is it a step in the right direction?

Right direction as in, everybody seems to agree we want "make install" 
ultimately do the right thing. If we can't have it in 2.2 yet then a 
preparatory step is better than not having it.

> We just need to install the files in a different hierarchy and adapt
> the makefiles to be able to compile an application while keeping the
> RTE_SDK variable to specify the root directory (previously built thanks
> to DESTDIR).
> As the hierarchy could be tuned, we need more variables, e.g.:
> 	DPDK_INC_DIR (default = RTE_SDK/include/dpdk)
> 	DPDK_LIB_DIR (default = RTE_SDK/lib)
>
> While doing it, we can have a specific handling of T= to keep compatibility
> with the current (old) syntax.
>
> What have I missed?

Perhaps its me who has missed the fact that 2.2 is still open for these 
kind of things.

If it were up to me, I think I'd just apply Marios latest patch series 
(perhaps condence it somewhat) to get it over with, fine-tune later 
if/as necessary. This is veering to the side of bikeshedding real fast.

BTW, one noteworthy point is that in all of these related threads, 
nobody absolutely nobody has spoken up for the current behavior of "make 
install". Which makes me wonder if anybody is actually using it, and 
whether all this is just worrying for nothing.

	- Panu -

  parent reply	other threads:[~2015-11-30 12:26 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-06 10:24 Bruce Richardson
2015-11-06 10:24 ` [dpdk-dev] [PATCH] mk: rename 'make install' to 'make sdk' Bruce Richardson
2015-11-06 12:57 ` [dpdk-dev] [PATCH] reserve 'make install' for future use Bruce Richardson
2015-11-06 13:04   ` Thomas Monjalon
2015-11-24 16:54     ` Bruce Richardson
2015-11-25  8:48       ` Panu Matilainen
2015-11-27 17:33         ` Thomas Monjalon
2015-11-30 11:08           ` Richardson, Bruce
2015-11-30 11:27             ` Thomas Monjalon
2015-11-30 11:41               ` Richardson, Bruce
2015-11-30 11:49                 ` Bruce Richardson
2015-11-30 14:26                   ` Thomas Monjalon
2015-12-04 15:57                     ` Thomas Monjalon
2015-12-04 16:21                       ` Richardson, Bruce
2015-11-30 12:26           ` Panu Matilainen [this message]
2015-11-30 14:19             ` Thomas Monjalon

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=565C406C.8020005@redhat.com \
    --to=pmatilai@redhat.com \
    --cc=bruce.richardson@intel.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).