All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
@ 2017-07-06 16:06 Mahesh Radhakrishnan
  2017-07-06 17:08 ` Pang, Ivan
  0 siblings, 1 reply; 4+ messages in thread
From: Mahesh Radhakrishnan @ 2017-07-06 16:06 UTC (permalink / raw)
  To: meta-ti

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



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
  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
  0 siblings, 1 reply; 4+ messages in thread
From: Pang, Ivan @ 2017-07-06 17:08 UTC (permalink / raw)
  To: Radhakrishnan, Mahesh, meta-ti@yoctoproject.org

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
  2017-07-06 17:08 ` Pang, Ivan
@ 2017-07-06 17:26   ` Radhakrishnan, Mahesh
  2017-07-10 19:23     ` Denys Dmytriyenko
  0 siblings, 1 reply; 4+ messages in thread
From: Radhakrishnan, Mahesh @ 2017-07-06 17:26 UTC (permalink / raw)
  To: Pang, Ivan, meta-ti@yoctoproject.org

The board module failed without this patch.
So, the references to ti-staging.inc needs to be removed from ALL the recipes?

-----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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ti-staging: Inhibiting sysroot strip & package debug split
  2017-07-06 17:26   ` Radhakrishnan, Mahesh
@ 2017-07-10 19:23     ` Denys Dmytriyenko
  0 siblings, 0 replies; 4+ messages in thread
From: Denys Dmytriyenko @ 2017-07-10 19:23 UTC (permalink / raw)
  To: Radhakrishnan, Mahesh; +Cc: meta-ti@yoctoproject.org

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-07-10 19:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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 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.