/var/log/app_name permissions fixed

sbt · Dec 18, 2013 · 474c98f · 474c98f
1 parent 72e38aa
commit 474c98f
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/src/main/scala/com/typesafe/sbt/packager/GenericPackageSettings.scala b/src/main/scala/com/typesafe/sbt/packager/GenericPackageSettings.scala
@@ -63,6 +63,7 @@ trait GenericPackageSettings
 
     // Default place to install code.
     defaultLinuxInstallLocation := "/usr/share",
+    defaultLinuxLogsLocation := "/var/log",
 
     // First we look at the src/linux files
     linuxPackageMappings <++= (sourceDirectory in Linux) map { dir =>

diff --git a/src/main/scala/com/typesafe/sbt/packager/Keys.scala b/src/main/scala/com/typesafe/sbt/packager/Keys.scala
@@ -29,4 +29,5 @@ object Keys extends linux.Keys
          |  APP_DEFINES - the defines to go into the app
          |  """.stripMargin)
   val defaultLinuxInstallLocation = SettingKey[String]("defaultLinuxInstallLocation", "The location where we will install generic linux packages.")
+  val defaultLinuxLogsLocation = SettingKey[String]("defaultLinuxLogsLocation", "The location where application logs will be stored.")
 }
diff --git a/src/main/scala/com/typesafe/sbt/packager/archetypes/JavaServerApplication.scala b/src/main/scala/com/typesafe/sbt/packager/archetypes/JavaServerApplication.scala
@@ -62,9 +62,13 @@ object JavaServerAppPackaging {
             s <- script.toSeq
           } yield LinuxPackageMapping(Seq(s -> path)).withPerms(permissions)
         },
-      // TODO the /var/log should be generalized like defaultLinuxInstallLocation
-      linuxPackageMappings in Debian <+= (normalizedName) map {
-        name => packageTemplateMapping("/var/log/" + name)()
+      // TODO should we specify daemonGroup in configs?
+      linuxPackageMappings in Debian <+= (normalizedName, defaultLinuxLogsLocation, target in Debian, daemonUser in Debian) map {
+        (name, logsDir, target, user) =>
+          // create empty var/log directory
+          val d = target / logsDir
+          d.mkdirs()
+          LinuxPackageMapping(Seq(d -> s"$logsDir/$name"), LinuxFileMetaData(user, user))
       },
       linuxPackageSymlinks in Debian <+= (normalizedName, defaultLinuxInstallLocation) map {
         (name, install) => LinuxSymlink(install + "/" + name + "/logs", "/var/log/" + name)