On 06/04/12 18:28, Tim Foster wrote:
On 06/ 5/12 12:14 PM, Brock Pytlik wrote:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/7126934-v1

I wonder could you indent the content.write(..) at line 443 a little? I admit that indentation of that mako file is a bit hit/miss, but it improves readability slightly.

   441  % for uri in reversed(sorted(uri_pub_map.keys())):
   442          % if uri.startswith("http"):
   443 <%                context.write("ProxyPass / %s" % uri) %>
   444          % endif
   445  % endfor uri

    cheers,
            tim
That's what I had originally, but when I looked at the html file that was produced, the ProxyPass lines seemed to have been indented, which seemed bad. (Maybe I had the <% indented too and that was the cause of the problem ...) I'll double check that that happens (which seemed bizarre to me at the time anyway). If it doesn't cause that problem again, I'll indent it, otherwise I'll leave it as is.

Brock
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to