I was simply following the example from lower down where the puppet labs repo is configured. I can remove the conditional if you'd like.

-Jeremy

--
Insert PGP signature here.

On Jul 14, 2013, at 11:29 PM, Marek Hulán <notifications@github.com> wrote:

> In app/views/unattended/preseed.erb:
> 
> > @@ -24,7 +24,7 @@ d-i hw-detect/load_firmware boolean true
> >  d-i mirror/country string manual
> >  d-i mirror/http/hostname string <%= @preseed_server %>
> >  d-i mirror/http/directory string <%= @preseed_path %>
> > -d-i mirror/http/proxy string
> > +d-i mirror/http/proxy string <%= @host.parameters['preseed-proxy'] ? @host.parameters['preseed-proxy'] : '' %>
> Why do you have condition there? nil and '' are both evaluated to empty string in erb. Only case that you solve by this condition is a case when @host.parameters['preseed-proxy'] is false. Was this the intention? Otherwise I think that's cool.
> 
> —
> Reply to this email directly or view it on GitHub.