Code review comment for lp://qastaging/~dooferlad/linaro-image-tools/fetch_image_cli

On Thu, Jun 16, 2011, James Tunnicliffe wrote:
> >> +    if config.args['platform'] == "snapshot":
> >> +        config.args['release_or_snapshot'] = "snapshot"
> >> +    else:
> >> +        config.args['release_or_snapshot'] = "release"
> >
> >  Instead of release_or_snapshot, "archive"?  or "source"?
>
> Release points to official Linaro releases, as currently hosted on
> releases.linaro.org and shapshot the daily builds on
> snapshots.linaro.org.

 Yup; I understood the reference, but in a way it seemed to hardcode
 that there would only be these two sources and their names in the
 variable name. It seemed like we could possibly change this over time,
 perhaps dropping one, or adding one like a ci.linaro.org download
 archive and using "release_or_snapshot" seemed a bit heavily biased
 towards exactly 2 sources of downloads.

--
Loïc Minier