Ok, then I'll do it before submitting and then send an interdiff.

Thanks,
Michele


On Wed, Apr 3, 2013 at 11:03 AM, Guido Trotter <[email protected]> wrote:

> 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