Bug 5649

Summary: Add the path to the license.manifest.txt file to the build information
Product: [Build System, Metadata & Runtime] Toaster Reporter: Belen Barros Pena <belen.barros.pena>
Component: toasterAssignee: Cristiana Voicu <cristiana.voicu>
Status: VERIFIED FIXED QA Contact:
Severity: enhancement    
Priority: Medium CC: alexandru.damian, belen.barros.pena, cristiana.voicu
Version: unspecified   
Target Milestone: 1.6 M4   
Hardware: All   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)
Bug Depends on:    
Bug Blocks: 5936    

Description Belen Barros Pena 2013-12-09 13:48:03 UTC
The license.manifest file is in:

/home/user/poky/build/tmp/deploy/licenses/<image_name>/
Comment 2 Belen Barros Pena 2014-03-18 13:19:29 UTC
The license_manifest_path in orm_target is empty. Checked in

master:e58089b9d708ba176594a758580cc52c23725a5b
Comment 3 Cristiana Voicu 2014-03-18 15:12:13 UTC
(In reply to comment #2)
> The license_manifest_path in orm_target is empty. Checked in
> 
> master:e58089b9d708ba176594a758580cc52c23725a5b

Hi Belen,
The license_manifest_path is populated only if a license.manifest is created, and this means only if do_rootfs task is running. Can you please confirm that the license_manifest_path was not populated in this case?
Thanks,
Cristiana
Comment 4 Cristiana Voicu 2014-03-18 16:04:54 UTC
There are 2 patches needed to populate license.manifest file, but only one is on master. I hope that the other one( http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=cvoicu/toaster-work&id=b0397121ab1f529ba9e52e96f57e328663852483) will get soon in the master branch. I will mark this bug in review.
Comment 5 Belen Barros Pena 2014-03-18 16:16:26 UTC
(In reply to comment #4)
> There are 2 patches needed to populate license.manifest file, but only one
> is on master. I hope that the other one(
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/commit/?h=cvoicu/
> toaster-work&id=b0397121ab1f529ba9e52e96f57e328663852483) will get soon in
> the master branch. I will mark this bug in review.

Thanks! 

FWIW, I did double-check, and a build with 

master:e58089b9d708ba176594a758580cc52c23725a5b 

where do_rootfs did execute, has a blank license_manifest_path
Comment 6 Cristiana Voicu 2014-03-21 12:56:26 UTC
The second patch was merged to master.
Comment 7 Belen Barros Pena 2014-03-21 17:53:05 UTC
verified in master:99c08eebe5c99c4e8b9ac070193373ba08d2e03f