[Toaster] Question about Build Dashboard page

[Barros Pena, Belen]

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?

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