From: Denys Dmytriyenko <denys@ti.com>
To: "Radhakrishnan, Mahesh" <m-radhakrishnan2@ti.com>
Cc: "meta-ti@yoctoproject.org" <meta-ti@yoctoproject.org>
Subject: Re: [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
Date: Mon, 10 Jul 2017 15:23:53 -0400 [thread overview]
Message-ID: <20170710192353.GD14665@edge> (raw)
In-Reply-To: <5130E41A1BA9D145BA0F3B18D5CFB0A74379C51F@DFLE12.ent.ti.com>
On Thu, Jul 06, 2017 at 05:26:54PM +0000, Radhakrishnan, Mahesh wrote:
> The board module failed without this patch.
> So, the references to ti-staging.inc needs to be removed from ALL the recipes?
In many existing cases including this file is not even required at all! I
started cleaning this up recently:
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/commit/?h=morty&id=7b51cf71bf4c51cc0aeff42c2d8a8522e37828ce
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/commit/?h=morty&id=71f6beaa00039bf3add48fe0b42e100b01d81e45
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/commit/?h=morty&id=d3d5c5c5130ca2ac61e81dd39ccba8fe87e05a1c
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/commit/?h=morty&id=f2cd25abd2f5912069f907d9ab7385be3935258b
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/commit/?h=morty&id=d4e2ecba1a8c9445b31c9b68aeb5a6ca1ccc9580
In some cases that do need to set INHIBIT vars, those should be set in
corresponding recipes on a case-by-case basis, i.e. not in bulk!
--
Denys
> -----Original Message-----
> From: Pang, Ivan
> Sent: Thursday, July 06, 2017 1:08 PM
> To: Radhakrishnan, Mahesh; meta-ti@yoctoproject.org
> Subject: RE: [meta-ti] [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
>
> We had a previous discussion on this ti-staging.inc file and the conclusion is that removing this file is the better way going forward.
> Are there specific recipes that needed this patch?
>
> -----Original Message-----
> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-bounces@yoctoproject.org] On Behalf Of Radhakrishnan, Mahesh
> Sent: Thursday, July 06, 2017 11:06 AM
> To: meta-ti@yoctoproject.org
> Subject: [meta-ti] [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
>
> Signed-off-by: Mahesh Radhakrishnan <m-radhakrishnan2@ti.com>
> ---
> recipes-ti/includes/ti-staging.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/recipes-ti/includes/ti-staging.inc b/recipes-ti/includes/ti-staging.inc
> index b588805..ae523ce 100644
> --- a/recipes-ti/includes/ti-staging.inc
> +++ b/recipes-ti/includes/ti-staging.inc
> @@ -1,4 +1,6 @@
> INHIBIT_PACKAGE_STRIP = "1"
> +INHIBIT_SYSROOT_STRIP = "1"
> +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>
> ALLOW_EMPTY_${PN} = "1"
> ALLOW_EMPTY_${PN}-dev = "1"
> --
> 1.9.1
>
> --
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
> --
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
prev parent reply other threads:[~2017-07-10 19:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-06 16:06 [PATCH] ti-staging: Inhibiting sysroot strip & package debug split Mahesh Radhakrishnan
2017-07-06 17:08 ` Pang, Ivan
2017-07-06 17:26 ` Radhakrishnan, Mahesh
2017-07-10 19:23 ` Denys Dmytriyenko [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=20170710192353.GD14665@edge \
--to=denys@ti.com \
--cc=m-radhakrishnan2@ti.com \
--cc=meta-ti@yoctoproject.org \
/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.