Reinstate BUILD-INFO.txt requirement on publishing

Bug #1146220 reported by Данило Шеган
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linaro-license-protection
Fix Released
High
Georgy Redkozubov

Bug Description

Around release time for 2013.02 we did a rollout of unrolled changes for lp:linaro-license-protection. That included changes to require BUILD-INFO.txt which were never rolled out before and that broke a bunch of builds, including some that actually included BUILD-INFO.txt files and that worked properly before that.

As an urgent measure, we've reverted these in r170:

170 Danilo Šegan 2013-02-26
      Revert r155-r157 introducing checks for BUILD-INFO.txt.

To re-introduce these, we need to make sure we do it early in the cycle to ensure we can fix all the problems these cause. And not forget to roll out with these changes.

Vishal might still have links to builds that failed with these.

summary: - Reverted BUILD-INFO.txt requirement
+ Reinstate BUILD-INFO.txt requirement on publishing
Revision history for this message
Данило Шеган (danilo) wrote :
Revision history for this message
Данило Шеган (danilo) wrote :

Gesha and I looked at this yesterday and it seems there are several problems:
 - we don't look for BUILD-INFO recursively (before we reshuffle the files for android)
 - we have transactions which push only a single file like lava-job-info without the BUILD-INFO files

The first one is easier, the latter is trickier. Ideally, we'd require that to include BUILD-INFO file as well, however, then we'd need smart BUILD-INFO combining tools. So, for now, let's go with something simpler like just special casing lava-job-info if that's the only thing we really need.

Changed in linaro-license-protection:
status: Triaged → Fix Committed
Milo Casagrande (milo)
Changed in linaro-license-protection:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.