Fixed a template typo.

Variable interpolation needs <%= not just <%. :)

Also, while I was in there, I replaced default with "oneiric",
because I don't actually know that the value is a good default value,
and I removed a couple of comments about moving to MySQL and Apache
modules from upstream.

Change-Id: Iec5b10cee2cbd0e0a2573fefa707d34d2a363cb4

modules/gerrit/manifests/init.pp

@@ -45,8 +45,6 @@
 #   testmode:
 #     Set this to true to disable cron jobs and replication,
 #     which can interfere with testing.
-# TODO: move apache configuration to another module
-# TODO: move mysql configuration to another module
 # TODO: make more gerrit options configurable here
 
 class gerrit($virtual_hostname=$fqdn,
@@ -83,7 +81,7 @@ class gerrit($virtual_hostname=$fqdn,
 
   $java_home = $lsbdistcodename ? {
       "precise" => "/usr/lib/jvm/java-6-openjdk-amd64/jre",
-      default => "/usr/lib/jvm/java-6-openjdk/jre",
+      "oneiric" => "/usr/lib/jvm/java-6-openjdk/jre",
     }
 
   user { "gerrit2":

modules/gerrit/templates/gerrit.config.erb

@@ -22,7 +22,7 @@
 	smtpServer = localhost
 [container]
 	user = gerrit2
-	javaHome = <% java_home %>
+	javaHome = <%= java_home %>
 <% if container_heaplimit != "" -%>
 	heapLimit = <%= container_heaplimit %>
 <% end -%>