On Wed, Apr 3, 2013 at 10:12 AM, Klaus Aehlig <[email protected]> wrote:
>> +
>> +#Install the base system
>> +debootstrap --arch amd64 squeeze $CHDIR
>> +
>
> For the record: here you're hard-coding archicteture and release; this
> will have to be changed in the future. But it is probably best addressed
> in a follow-up commit when we move on from squeeze amd64.
>
> So, together with the interdiff in the other mail: LGTM.

I would suggest making at least "squeeze" configurable as an env var. :)

Thanks,

Guido

Reply via email to