Code review comment for lp:~dooferlad/linaro-license-protection/manifest_generation_fix

Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

I told it would be easier to do on android-build side ;-).

Are you sure this works and as expected? We actually have 2 transfers, 1st one transfer main bunch of files, 2nd one only lava-job-info. So, I'd expect to see >>MANIFEST, or otherwise expect it to skip updating manifest if it exists, or if the file in lava-job-info (no big point to include it in artifact list imho).

review: Needs Information

« Back to merge proposal