Closed Bug 832202 Opened 11 years ago Closed 11 years ago

buildconfig.py fails when config.status contains empty lists

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla21

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

On comm-central, the headers list is empty, and this makes buildconfig.py fail.
Note the patch probably doesn't apply on m-c because of the context which depends on bug 780561, but the modified part doesn't depend on bug 780561.
Attachment #703814 - Flags: review?(ted)
Blocks: new-packager
Comment on attachment 703814 [details] [diff] [review]
Handle empty lists from config.status in buildconfig.py

Review of attachment 703814 [details] [diff] [review]:
-----------------------------------------------------------------

::: build/buildconfig.py
@@ +20,5 @@
>  # This effectively imports topsrcdir, topobjdir, defines, substs, files,
>  # headers and non_global_defines
>  for var in config.__all__:
>      value = getattr(config, var)
> +    if isinstance(value, list) and len(value) and isinstance(value[0], tuple):

I'd prefer if the middle test was either just "value" or "len(value) > 0".
Attachment #703814 - Flags: review?(ted) → review+
https://hg.mozilla.org/mozilla-central/rev/6b280e155484
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.