[review-request] bbarrosp/add-available

[review-request] bbarrosp/add-available

[Barros Pena, Belen]

http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=bbarrosp/add-
available

Adds the word 'available' to the 'all' pages, as suggested by QA.

Thanks!

Belén

Re: [review-request] bbarrosp/add-available

[Michael Wood]

On 05/02/15 14:46, Barros Pena, Belen wrote:
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=bbarrosp/add-
> available
>
> Adds the word 'available' to the 'all' pages, as suggested by QA.
>
> Thanks!
>
> Belén
>
Patch is fine.

Though I think "all available n" is really "all compatible n", as it 
won't show you all available, it only shows you all which are compatible 
and come from your preferred layer source as defined by the project, 
e.g. you might know that a layer is available because you've seen it on 
the layer index but you can't find it in your "all available layers" 
because it doesn't have a release for the release your project is using.
Not sure how likely it is, but might be something to consider.

Michael

Re: [review-request] bbarrosp/add-available

[Barros Pena, Belen]

On 06/02/2015 16:18, "Michael Wood" <michael.g.wood@intel.com> wrote:

>On 05/02/15 14:46, Barros Pena, Belen wrote:
>> 
>>http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=bbarrosp/ad
>>d-
>> available
>>
>> Adds the word 'available' to the 'all' pages, as suggested by QA.
>>
>> Thanks!
>>
>> Belén
>>
>Patch is fine.
>
>Though I think "all available n" is really "all compatible n", as it
>won't show you all available, it only shows you all which are compatible
>and come from your preferred layer source as defined by the project,
>e.g. you might know that a layer is available because you've seen it on
>the layer index but you can't find it in your "all available layers"
>because it doesn't have a release for the release your project is using.
>Not sure how likely it is, but might be something to consider.

This is a good point. I will change to 'compatible' and resubmit.

Thanks!

Belén

>
>Michael
>-- 
>_______________________________________________
>toaster mailing list
>toaster@yoctoproject.org
>https://lists.yoctoproject.org/listinfo/toaster

Re: [review-request] bbarrosp/add-available

[Damian, Alexandru]

[-- Attachment #1: Type: text/plain, Size: 1545 bytes --]

Taken for submission,

Thank you,
Alex

On Fri, Feb 6, 2015 at 4:23 PM, Barros Pena, Belen <
belen.barros.pena@intel.com> wrote:

>
>
> On 06/02/2015 16:18, "Michael Wood" <michael.g.wood@intel.com> wrote:
>
> >On 05/02/15 14:46, Barros Pena, Belen wrote:
> >>
> >>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=bbarrosp/ad
> >>d-
> >> available
> >>
> >> Adds the word 'available' to the 'all' pages, as suggested by QA.
> >>
> >> Thanks!
> >>
> >> Belén
> >>
> >Patch is fine.
> >
> >Though I think "all available n" is really "all compatible n", as it
> >won't show you all available, it only shows you all which are compatible
> >and come from your preferred layer source as defined by the project,
> >e.g. you might know that a layer is available because you've seen it on
> >the layer index but you can't find it in your "all available layers"
> >because it doesn't have a release for the release your project is using.
> >Not sure how likely it is, but might be something to consider.
>
> This is a good point. I will change to 'compatible' and resubmit.
>
> Thanks!
>
> Belén
>
> >
> >Michael
> >--
> >_______________________________________________
> >toaster mailing list
> >toaster@yoctoproject.org
> >https://lists.yoctoproject.org/listinfo/toaster
>
> --
> _______________________________________________
> toaster mailing list
> toaster@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/toaster
>



-- 
Alex Damian
Yocto Project
SSG / OTC

[-- Attachment #2: Type: text/html, Size: 2992 bytes --]