[meta-freescale] [meta-fsl-arm][PATCH v2 1/5] gpu-viv-bin-mx6q: Rework package for a single backend use

[Otavio Salvador]

On Fri, Apr 12, 2013 at 12:09 PM, Thomas Senyk
<thomas.senyk at pelagicore.com> wrote:
> On Fri, April 12, 2013 08:44:05 AM Otavio Salvador wrote:
>> On Fri, Apr 12, 2013 at 6:13 AM, Thomas Senyk
>>
>> <thomas.senyk at pelagicore.com> wrote:
>> > On Thu, April 04, 2013 10:42:16 PM Otavio Salvador wrote:
>> .>> +            renamed=$backend
>>
>> >> +            mv $f $(echo $f | sed "s,-$backend\.so,.so,g")
>> >
>> > I'd personally rather vote for a symlink then a mv.
>> > It's nearly the same cost, but it let's people identify which
>> > implementation is installed (besides other things)
>>
>> The other uses of libraries with the backend suffix where already
>> fixed and as we cannot have both backends at same image, working, I
>> see no gain in duplicate this information. Can you elaborate it a
>> little more?
>
> No strong vote! :)
> And it's not duplicate information ... it's just keeping the original
> information (file name) rather then destroying the information (by renaming
> it)
>
> It's just that during development it's nice to see that the 'right' version of
> you egl implementation got deployed.
> ... otherwise some people (me once in a while) will double-check if the gpu.bb
> toke the right one.
>
> 'ls' looks nicer with:
> libEGL.so -> libEGL-wl.so
> libEGL-wl.so
>
> then just:
> libEGL.so
>
> ;)
>
> ... but again: no strong vote, I can happily live with the current approach ;)

This patch has been already merged so if we'd like to change this, it
will need to be another patch.

>From Yocto point of view it should not be confusing as it is dependent
of the distro features. Is it possible for you to give current master
(and preferrable with my current patchset) a go and check if all works
for you?

--
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br