[Toaster] Question about Build Dashboard page

[Barros Pena, Belen]

On 04/03/2014 15:21, "Barros Pena, Belen" <belen.barros.pena at intel.com>
wrote:

>On 04/03/2014 14:56, "Damian, Alexandru" <alexandru.damian at intel.com>
>wrote:
>
>>Hello,
>>
>>After revieweing the license.manifest content, it appears that we already
>>have this information (i.e package name, version, recipe name and
>>license) already stored in the database.
>>
>>
>>Therefore this seems rather a info display problem, and the file may not
>>be needed. I'm postponing any modifications until Belen takes a decision
>>either way about this bug.
>
>Right, I still think is worth displaying the path to the file, at least
>for the current release where the main use case is still local. But the
>information in it we already store and we can show it. The easiest thing
>for the moment is turning the "License manifest" label into a link. I
>think turning the path itself into a link will be a bit misleading: you
>would probably expect that to download the real file, not to bring you
>somewhere else in Toaster.
> 
>
>When you click the link, we bring you to the list of installed packages
>(/gui/build/<build_id>/target/<target_id>/) with the following columns
>shown:
>
>Package
>Package version
>License
>Recipe
>
>That is exactly the same information you get in the license manifest file.
>
>Thoughts?

Since there are no thoughts, I have updated the design document to include
the above behaviour.

See page 12 of the document attached to
https://bugzilla.yoctoproject.org/show_bug.cgi?id=4258

Cheers

Belén

>
>Belén
>
>
>>
>>I'm pushing the wip-15022014 patchset ast it is to toaster/master, and
>>I'm focusing on collecting correct data for the new database structure.
>>
>>Cheers,
>>Alex
>>
>>
>>
>>
>>On Tue, Mar 4, 2014 at 11:58 AM, Damian, Alexandru
>><alexandru.damian at intel.com> wrote:
>>
>>Hi,
>>
>>I have not seen any other reply to the review requested patchset for the
>>DB changes.
>>
>>
>>So I assume that this is the review, I will add the License Manifest
>>field to Target_Image_File, and merge everything into toaster/master.
>>
>>
>>There is not yet code for populating the data in there, I'm working on
>>it.
>>
>>Cheers,
>>
>>Alex
>>
>>
>>
>>
>>On Thu, Feb 27, 2014 at 9:54 PM, Reyna, David
>><david.reyna at windriver.com> wrote:
>>
>>Hi Alex and Belen,
>> 
>>I am having Farrell complete the Build Dashboard page, merging in Amit¹s
>>Error/Warning work when it is ready. We have reviewed your patch set that
>>adds the ³Target_Image_File² table. It applied to Farrell¹s installation,
>>and he is updating his code to
>> use it.
>> 
>>We do have two questions:
>>  * We do not yet have a table member for the License Manifest.
>>  * We did not see anywhere in the patch that populates the
>>³Target_Image_File² table (Farrell is pushing simulated data in to the db
>>for now).
>> 
>>I think that we should accept your patch and push it so that I can also
>>update the ³build.html² with the To-Do for the ³output² column.
>> 
>>- David
>> 
>>
>>
>>
>>
>>
>>
>>
>>
>>
>>-- 
>>Alex Damian
>>Yocto Project
>>
>>SSG / OTC 
>>
>>
>>
>>
>>
>>
>>
>>
>>-- 
>>Alex Damian
>>Yocto Project
>>
>>SSG / OTC 
>>
>>
>