we don't want snapshots. fixes #61

softprops · Jun 12, 2012 · 63a298a · 63a298a
1 parent de76bcb
commit 63a298a
Showing 1 changed file with 18 additions and 9 deletions.
diff --git a/plugin/src/main/scala/ls.scala b/plugin/src/main/scala/ls.scala
@@ -72,6 +72,7 @@ object Plugin extends sbt.Plugin
           parseJson[Library](json)
         }
         out.log.debug("Valid version info")
+        if (snapshot(vinfo.version)) out.log.warn(SnapshotWarning)
         true
       } catch {
         case e =>
@@ -88,15 +89,19 @@ object Plugin extends sbt.Plugin
       (out, maybeUser, maybeRepo, branch, vers, host, vfile, lint) =>
         (maybeUser, maybeRepo) match {
           case (Some(user), Some(repo)) =>
-            if(lint) {
-              out.log.info("lsyncing project %s/%s@%s..." format(user, repo, vers))
-              try {
-                // todo: should this by an async server request?
-                http(Client(host).lsync(user, repo, branch.getOrElse("master"), vers) as_str)
-                out.log.info("Project was synchronized")
-              } catch {
-                case e =>
-                  out.log.warn("Error synchronizing project libraries %s" format e.getMessage)
+            if (lint) {
+              if (snapshot(vers)) {
+                out.log.warn(SnapshotWarning)
+              } else {
+                out.log.info("lsyncing project %s/%s@%s..." format(user, repo, vers))
+                try {
+                  // todo: should this by an async server request?
+                  http(Client(host).lsync(user, repo, branch.getOrElse("master"), vers) as_str)
+                  out.log.info("Project was synchronized")
+                } catch {
+                  case e =>
+                    out.log.warn("Error synchronizing project libraries %s" format e.getMessage)
+                }
               }
             } else sys.error("Your version descriptor was invalid. %s" format(
               IO.read(vfile)
@@ -105,6 +110,10 @@ object Plugin extends sbt.Plugin
         }
     }
 
+  private def snapshot(vstr: String) = vstr.toUpperCase.endsWith("SNAPSHOT")
+
+  private val SnapshotWarning = "ls only supports release versions not likely to change. This excludes snapshot versions."
+
   private def writeVersionTask: Initialize[Task[Unit]] = 
      (streams, versionFile, versionInfo, skipWrite) map {
       (out, f, info, skip) =>