All of lore.kernel.org
 help / color / mirror / Atom feed
* [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a
@ 2020-02-26  1:09 Denys Dmytriyenko
  2020-02-26  1:23 ` [meta-ti] " Khem Raj
  0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2020-02-26  1:09 UTC (permalink / raw)
  To: meta-ti; +Cc: Denys Dmytriyenko

dwarfsrcfiles: .../work/armv7at2hf-neon-linux-gnueabi/ti-cgt6x/1_8.3.2-r0/package/usr/share/ti/cgt-c6x/lib/libc.a: not a valid ELF file

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 recipes-ti/devtools/ti-cgt6x_8.3.2.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
index c63c3f6..1188b4e 100644
--- a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
+++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
@@ -100,6 +100,7 @@ FILES_${PN} += "${datadir}/ti/*"
 FILES_${PN}-dbg += "${datadir}/ti/cgt-c6x/bin/.debug"
 
 INSANE_SKIP_${PN} += "staticdev"
+INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
 
 BBCLASSEXTEND = "native nativesdk"
 
-- 
2.7.4


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

* Re: [meta-ti] [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a
  2020-02-26  1:09 [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a Denys Dmytriyenko
@ 2020-02-26  1:23 ` Khem Raj
  2020-02-26  1:34   ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2020-02-26  1:23 UTC (permalink / raw)
  To: denys, meta-ti; +Cc: meta-ti



On 2/25/20 5:09 PM, Denys Dmytriyenko via Lists.Yoctoproject.Org wrote:
> dwarfsrcfiles: .../work/armv7at2hf-neon-linux-gnueabi/ti-cgt6x/1_8.3.2-r0/package/usr/share/ti/cgt-c6x/lib/libc.a: not a valid ELF file
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  recipes-ti/devtools/ti-cgt6x_8.3.2.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> index c63c3f6..1188b4e 100644
> --- a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> +++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> @@ -100,6 +100,7 @@ FILES_${PN} += "${datadir}/ti/*"
>  FILES_${PN}-dbg += "${datadir}/ti/cgt-c6x/bin/.debug"
>  
>  INSANE_SKIP_${PN} += "staticdev"
> +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>  

Thanks for taking care of this issue, does it have any downsides, like
not stripping binaries which otherwise should have been stripped besides
libc.a

>  BBCLASSEXTEND = "native nativesdk"
>  
> 
> 
> 
> 

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

* Re: [meta-ti] [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a
  2020-02-26  1:23 ` [meta-ti] " Khem Raj
@ 2020-02-26  1:34   ` Denys Dmytriyenko
  2020-02-26  1:55     ` Khem Raj
  0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2020-02-26  1:34 UTC (permalink / raw)
  To: Khem Raj; +Cc: meta-ti

On Tue, Feb 25, 2020 at 05:23:48PM -0800, Khem Raj wrote:
> 
> 
> On 2/25/20 5:09 PM, Denys Dmytriyenko via Lists.Yoctoproject.Org wrote:
> > dwarfsrcfiles: .../work/armv7at2hf-neon-linux-gnueabi/ti-cgt6x/1_8.3.2-r0/package/usr/share/ti/cgt-c6x/lib/libc.a: not a valid ELF file
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  recipes-ti/devtools/ti-cgt6x_8.3.2.bb | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> > index c63c3f6..1188b4e 100644
> > --- a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> > +++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> > @@ -100,6 +100,7 @@ FILES_${PN} += "${datadir}/ti/*"
> >  FILES_${PN}-dbg += "${datadir}/ti/cgt-c6x/bin/.debug"
> >  
> >  INSANE_SKIP_${PN} += "staticdev"
> > +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> >  
> 
> Thanks for taking care of this issue, does it have any downsides, like
> not stripping binaries which otherwise should have been stripped besides
> libc.a

This is C6000 DSP code and is not the main architecture for the platform being 
built, so standard ARM tools won't be able to handle that anyway. But all the 
other ARM binaries in that package are not stripped now, unfortunately.

On the other hand, I'm getting the same error from dwarfsrcfiles on another 
static .a library from another package, which contains the proper ARM code, 
which is rather strange. The only suspicious thing there is an empty metafile 
called ".created", but for now I have to disable stripping there as well...


> >  BBCLASSEXTEND = "native nativesdk"
> >  
> > 
> > 
> > 
> > 

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

* Re: [meta-ti] [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a
  2020-02-26  1:34   ` Denys Dmytriyenko
@ 2020-02-26  1:55     ` Khem Raj
  2020-03-02  2:47       ` Denys Dmytriyenko
  0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2020-02-26  1:55 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: meta-ti



On 2/25/20 5:34 PM, Denys Dmytriyenko wrote:
> On Tue, Feb 25, 2020 at 05:23:48PM -0800, Khem Raj wrote:
>>
>>
>> On 2/25/20 5:09 PM, Denys Dmytriyenko via Lists.Yoctoproject.Org wrote:
>>> dwarfsrcfiles: .../work/armv7at2hf-neon-linux-gnueabi/ti-cgt6x/1_8.3.2-r0/package/usr/share/ti/cgt-c6x/lib/libc.a: not a valid ELF file
>>>
>>> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
>>> ---
>>>  recipes-ti/devtools/ti-cgt6x_8.3.2.bb | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
>>> index c63c3f6..1188b4e 100644
>>> --- a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
>>> +++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
>>> @@ -100,6 +100,7 @@ FILES_${PN} += "${datadir}/ti/*"
>>>  FILES_${PN}-dbg += "${datadir}/ti/cgt-c6x/bin/.debug"
>>>  
>>>  INSANE_SKIP_${PN} += "staticdev"
>>> +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
>>>  
>>
>> Thanks for taking care of this issue, does it have any downsides, like
>> not stripping binaries which otherwise should have been stripped besides
>> libc.a
> 
> This is C6000 DSP code and is not the main architecture for the platform being 
> built, so standard ARM tools won't be able to handle that anyway. But all the 
> other ARM binaries in that package are not stripped now, unfortunately.
> 

yeah I thought so, and dwarfsrcfiles is quite simplistic, perhaps you
might want to file a defect so we could define a WHITELIST of files in
recipes which should be spared from
stripping

> On the other hand, I'm getting the same error from dwarfsrcfiles on another 
> static .a library from another package, which contains the proper ARM code, 
> which is rather strange. The only suspicious thing there is an empty metafile 
> called ".created", but for now I have to disable stripping there as well...
> 
> 
>>>  BBCLASSEXTEND = "native nativesdk"
>>>  
>>>
>>>
>>> 
>>>

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

* Re: [meta-ti] [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a
  2020-02-26  1:55     ` Khem Raj
@ 2020-03-02  2:47       ` Denys Dmytriyenko
  0 siblings, 0 replies; 5+ messages in thread
From: Denys Dmytriyenko @ 2020-03-02  2:47 UTC (permalink / raw)
  To: Khem Raj; +Cc: meta-ti

On Tue, Feb 25, 2020 at 05:55:38PM -0800, Khem Raj wrote:
> 
> 
> On 2/25/20 5:34 PM, Denys Dmytriyenko wrote:
> > On Tue, Feb 25, 2020 at 05:23:48PM -0800, Khem Raj wrote:
> >>
> >>
> >> On 2/25/20 5:09 PM, Denys Dmytriyenko via Lists.Yoctoproject.Org wrote:
> >>> dwarfsrcfiles: .../work/armv7at2hf-neon-linux-gnueabi/ti-cgt6x/1_8.3.2-r0/package/usr/share/ti/cgt-c6x/lib/libc.a: not a valid ELF file
> >>>
> >>> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >>> ---
> >>>  recipes-ti/devtools/ti-cgt6x_8.3.2.bb | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> >>> index c63c3f6..1188b4e 100644
> >>> --- a/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> >>> +++ b/recipes-ti/devtools/ti-cgt6x_8.3.2.bb
> >>> @@ -100,6 +100,7 @@ FILES_${PN} += "${datadir}/ti/*"
> >>>  FILES_${PN}-dbg += "${datadir}/ti/cgt-c6x/bin/.debug"
> >>>  
> >>>  INSANE_SKIP_${PN} += "staticdev"
> >>> +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> >>>  
> >>
> >> Thanks for taking care of this issue, does it have any downsides, like
> >> not stripping binaries which otherwise should have been stripped besides
> >> libc.a
> > 
> > This is C6000 DSP code and is not the main architecture for the platform being 
> > built, so standard ARM tools won't be able to handle that anyway. But all the 
> > other ARM binaries in that package are not stripped now, unfortunately.
> 
> yeah I thought so, and dwarfsrcfiles is quite simplistic, perhaps you
> might want to file a defect so we could define a WHITELIST of files in
> recipes which should be spared from
> stripping

Sent v2 to use INHIBIT_PACKAGE_STRIP_FILES for whitelisting.


> > On the other hand, I'm getting the same error from dwarfsrcfiles on another 
> > static .a library from another package, which contains the proper ARM code, 
> > which is rather strange. The only suspicious thing there is an empty metafile 
> > called ".created", but for now I have to disable stripping there as well...
> > 
> > 
> >>>  BBCLASSEXTEND = "native nativesdk"
> >>>  
> >>>
> >>>
> >>> 
> >>>

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

end of thread, other threads:[~2020-03-02  2:48 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-02-26  1:09 [master][PATCH] ti-cgt6x: fix packaging issue due to libc.a Denys Dmytriyenko
2020-02-26  1:23 ` [meta-ti] " Khem Raj
2020-02-26  1:34   ` Denys Dmytriyenko
2020-02-26  1:55     ` Khem Raj
2020-03-02  2:47       ` 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.