diff --git a/.gitignore b/.gitignore
index 7e740ce2..a680191b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,15 +1,15 @@
.gradle
annotations
.idea/*
-!.idea/modules.xml
-!.idea/libraries
-!.idea/misc.xml
-buildScript
+*.iml
+nonBuildScript
kobaltBuild
local.properties
classes
libs
.kobalt/
-./build/
out
.DS_Store
+lib/kotlin-*
+build
+.history
diff --git a/.idea/libraries/KotlinJavaRuntime.xml b/.idea/libraries/KotlinJavaRuntime.xml
deleted file mode 100644
index 207fc74d..00000000
--- a/.idea/libraries/KotlinJavaRuntime.xml
+++ /dev/null
@@ -1,12 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/.idea/libraries/kobalt_jar.xml b/.idea/libraries/kobalt_jar.xml
deleted file mode 100644
index d70d1a6e..00000000
--- a/.idea/libraries/kobalt_jar.xml
+++ /dev/null
@@ -1,16 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/.idea/misc.xml b/.idea/misc.xml
deleted file mode 100644
index df7e8eb0..00000000
--- a/.idea/misc.xml
+++ /dev/null
@@ -1,9 +0,0 @@
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/.idea/modules.xml b/.idea/modules.xml
deleted file mode 100644
index a41ffd7c..00000000
--- a/.idea/modules.xml
+++ /dev/null
@@ -1,11 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/.travis.yml b/.travis.yml
deleted file mode 100644
index 0c1d3b1d..00000000
--- a/.travis.yml
+++ /dev/null
@@ -1,13 +0,0 @@
-cache:
- directories:
- - $HOME/.m2
- - $HOME/.kobalt
-
-language: java
-
-jdk:
- - oraclejdk8
-
-install: true
-
-script: ./build-travis.sh
diff --git a/README.md b/README.md
index e8db3ef6..d5d7cbe0 100644
--- a/README.md
+++ b/README.md
@@ -1,11 +1,14 @@
-# Kobalt [](https://travis-ci.org/cbeust/kobalt)
+# Kobalt
+
+[
](https://teamcity.jetbrains.com/project.html?projectId=OpenSourceProjects_Kobalt&tab=projectOverview)
+
Kobalt is a universal build system.
To build it:
```
-./kobaltw assemble
+$ ./kobaltw assemble
```
Please see [the web site](http://beust.com/kobalt/) for the full documentation.
diff --git a/build.gradle b/build.gradle
new file mode 100644
index 00000000..3f0053cc
--- /dev/null
+++ b/build.gradle
@@ -0,0 +1,58 @@
+allprojects {
+ group = 'com.beust'
+ version = '1.1.0'
+}
+
+subprojects {
+ apply plugin: 'java'
+ apply plugin: 'maven-publish'
+
+ ext {
+ bndlib = '3.5.0'
+ findbugs = '3.0.2'
+ groovy = '2.4.12'
+ gson = '2.8.2'
+ guice = '4.2.2'
+ inject = '1'
+ jaxb = '2.3.0'
+ jcommander = '1.72'
+ kotlin = '1.2.71'
+ maven = '3.5.2'
+ mavenResolver = '1.1.0'
+ okhttp = '3.9.1'
+ okio = '1.13.0'
+ retrofit = '2.3.0'
+ slf4j = '1.7.3'
+ spark = '2.6.0'
+ testng = '6.12'
+
+ junit = '4.12'
+ junitJupiter = '5.1.0'
+ junitPlatform = '1.1.0'
+ }
+
+ repositories {
+ mavenCentral()
+ mavenLocal()
+ jcenter()
+ maven {
+ url = 'https://dl.bintray.com/cbeust/maven'
+ }
+
+ maven {
+ url = 'https://repo.maven.apache.org/maven2'
+ }
+ }
+
+ sourceCompatibility = '1.7'
+
+ task sourcesJar(type: Jar) {
+ from sourceSets.main.allJava
+ archiveClassifier = 'sources'
+ }
+
+ task javadocJar(type: Jar) {
+ from javadoc
+ archiveClassifier = 'javadoc'
+ }
+}
diff --git a/dist/kobaltw b/dist/kobaltw
index a8df4bb9..333738df 100755
--- a/dist/kobaltw
+++ b/dist/kobaltw
@@ -1,2 +1,11 @@
#!/usr/bin/env sh
-java -jar "`dirname "$0"`/../kobalt/wrapper/kobalt-wrapper.jar" $*
\ No newline at end of file
+
+case "$(uname)" in
+ CYGWIN*) DIRNAME=$(cygpath -d "$(dirname "$(readlink -f "$0")")");;
+ Darwin*) DIRNAME=$(dirname "$(readlink "$0")");;
+ *) DIRNAME=$(dirname "$(readlink -f "$0")");;
+esac
+if [ "$DIRNAME" = "." ]; then
+ DIRNAME="$(dirname "$0")"
+fi
+java -jar "${DIRNAME}/../kobalt/wrapper/kobalt-wrapper.jar" $*
\ No newline at end of file
diff --git a/gradle/wrapper/gradle-wrapper.jar b/gradle/wrapper/gradle-wrapper.jar
new file mode 100644
index 00000000..5c2d1cf0
Binary files /dev/null and b/gradle/wrapper/gradle-wrapper.jar differ
diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
new file mode 100644
index 00000000..838e6bc8
--- /dev/null
+++ b/gradle/wrapper/gradle-wrapper.properties
@@ -0,0 +1,5 @@
+distributionBase=GRADLE_USER_HOME
+distributionPath=wrapper/dists
+distributionUrl=https\://services.gradle.org/distributions/gradle-5.3-bin.zip
+zipStoreBase=GRADLE_USER_HOME
+zipStorePath=wrapper/dists
diff --git a/gradlew b/gradlew
new file mode 100755
index 00000000..b0d6d0ab
--- /dev/null
+++ b/gradlew
@@ -0,0 +1,188 @@
+#!/usr/bin/env sh
+
+#
+# Copyright 2015 the original author or authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+##############################################################################
+##
+## Gradle start up script for UN*X
+##
+##############################################################################
+
+# Attempt to set APP_HOME
+# Resolve links: $0 may be a link
+PRG="$0"
+# Need this for relative symlinks.
+while [ -h "$PRG" ] ; do
+ ls=`ls -ld "$PRG"`
+ link=`expr "$ls" : '.*-> \(.*\)$'`
+ if expr "$link" : '/.*' > /dev/null; then
+ PRG="$link"
+ else
+ PRG=`dirname "$PRG"`"/$link"
+ fi
+done
+SAVED="`pwd`"
+cd "`dirname \"$PRG\"`/" >/dev/null
+APP_HOME="`pwd -P`"
+cd "$SAVED" >/dev/null
+
+APP_NAME="Gradle"
+APP_BASE_NAME=`basename "$0"`
+
+# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
+DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'
+
+# Use the maximum available, or set MAX_FD != -1 to use that value.
+MAX_FD="maximum"
+
+warn () {
+ echo "$*"
+}
+
+die () {
+ echo
+ echo "$*"
+ echo
+ exit 1
+}
+
+# OS specific support (must be 'true' or 'false').
+cygwin=false
+msys=false
+darwin=false
+nonstop=false
+case "`uname`" in
+ CYGWIN* )
+ cygwin=true
+ ;;
+ Darwin* )
+ darwin=true
+ ;;
+ MINGW* )
+ msys=true
+ ;;
+ NONSTOP* )
+ nonstop=true
+ ;;
+esac
+
+CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar
+
+# Determine the Java command to use to start the JVM.
+if [ -n "$JAVA_HOME" ] ; then
+ if [ -x "$JAVA_HOME/jre/sh/java" ] ; then
+ # IBM's JDK on AIX uses strange locations for the executables
+ JAVACMD="$JAVA_HOME/jre/sh/java"
+ else
+ JAVACMD="$JAVA_HOME/bin/java"
+ fi
+ if [ ! -x "$JAVACMD" ] ; then
+ die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME
+
+Please set the JAVA_HOME variable in your environment to match the
+location of your Java installation."
+ fi
+else
+ JAVACMD="java"
+ which java >/dev/null 2>&1 || die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
+
+Please set the JAVA_HOME variable in your environment to match the
+location of your Java installation."
+fi
+
+# Increase the maximum file descriptors if we can.
+if [ "$cygwin" = "false" -a "$darwin" = "false" -a "$nonstop" = "false" ] ; then
+ MAX_FD_LIMIT=`ulimit -H -n`
+ if [ $? -eq 0 ] ; then
+ if [ "$MAX_FD" = "maximum" -o "$MAX_FD" = "max" ] ; then
+ MAX_FD="$MAX_FD_LIMIT"
+ fi
+ ulimit -n $MAX_FD
+ if [ $? -ne 0 ] ; then
+ warn "Could not set maximum file descriptor limit: $MAX_FD"
+ fi
+ else
+ warn "Could not query maximum file descriptor limit: $MAX_FD_LIMIT"
+ fi
+fi
+
+# For Darwin, add options to specify how the application appears in the dock
+if $darwin; then
+ GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\""
+fi
+
+# For Cygwin, switch paths to Windows format before running java
+if $cygwin ; then
+ APP_HOME=`cygpath --path --mixed "$APP_HOME"`
+ CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`
+ JAVACMD=`cygpath --unix "$JAVACMD"`
+
+ # We build the pattern for arguments to be converted via cygpath
+ ROOTDIRSRAW=`find -L / -maxdepth 1 -mindepth 1 -type d 2>/dev/null`
+ SEP=""
+ for dir in $ROOTDIRSRAW ; do
+ ROOTDIRS="$ROOTDIRS$SEP$dir"
+ SEP="|"
+ done
+ OURCYGPATTERN="(^($ROOTDIRS))"
+ # Add a user-defined pattern to the cygpath arguments
+ if [ "$GRADLE_CYGPATTERN" != "" ] ; then
+ OURCYGPATTERN="$OURCYGPATTERN|($GRADLE_CYGPATTERN)"
+ fi
+ # Now convert the arguments - kludge to limit ourselves to /bin/sh
+ i=0
+ for arg in "$@" ; do
+ CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -`
+ CHECK2=`echo "$arg"|egrep -c "^-"` ### Determine if an option
+
+ if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then ### Added a condition
+ eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"`
+ else
+ eval `echo args$i`="\"$arg\""
+ fi
+ i=$((i+1))
+ done
+ case $i in
+ (0) set -- ;;
+ (1) set -- "$args0" ;;
+ (2) set -- "$args0" "$args1" ;;
+ (3) set -- "$args0" "$args1" "$args2" ;;
+ (4) set -- "$args0" "$args1" "$args2" "$args3" ;;
+ (5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;;
+ (6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;;
+ (7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;;
+ (8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;;
+ (9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;;
+ esac
+fi
+
+# Escape application args
+save () {
+ for i do printf %s\\n "$i" | sed "s/'/'\\\\''/g;1s/^/'/;\$s/\$/' \\\\/" ; done
+ echo " "
+}
+APP_ARGS=$(save "$@")
+
+# Collect all arguments for the java command, following the shell quoting and substitution rules
+eval set -- $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS "\"-Dorg.gradle.appname=$APP_BASE_NAME\"" -classpath "\"$CLASSPATH\"" org.gradle.wrapper.GradleWrapperMain "$APP_ARGS"
+
+# by default we should be in the correct project dir, but when run from Finder on Mac, the cwd is wrong
+if [ "$(uname)" = "Darwin" ] && [ "$HOME" = "$PWD" ]; then
+ cd "$(dirname "$0")"
+fi
+
+exec "$JAVACMD" "$@"
diff --git a/gradlew.bat b/gradlew.bat
new file mode 100644
index 00000000..9991c503
--- /dev/null
+++ b/gradlew.bat
@@ -0,0 +1,100 @@
+@rem
+@rem Copyright 2015 the original author or authors.
+@rem
+@rem Licensed under the Apache License, Version 2.0 (the "License");
+@rem you may not use this file except in compliance with the License.
+@rem You may obtain a copy of the License at
+@rem
+@rem http://www.apache.org/licenses/LICENSE-2.0
+@rem
+@rem Unless required by applicable law or agreed to in writing, software
+@rem distributed under the License is distributed on an "AS IS" BASIS,
+@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+@rem See the License for the specific language governing permissions and
+@rem limitations under the License.
+@rem
+
+@if "%DEBUG%" == "" @echo off
+@rem ##########################################################################
+@rem
+@rem Gradle startup script for Windows
+@rem
+@rem ##########################################################################
+
+@rem Set local scope for the variables with windows NT shell
+if "%OS%"=="Windows_NT" setlocal
+
+set DIRNAME=%~dp0
+if "%DIRNAME%" == "" set DIRNAME=.
+set APP_BASE_NAME=%~n0
+set APP_HOME=%DIRNAME%
+
+@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
+set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
+
+@rem Find java.exe
+if defined JAVA_HOME goto findJavaFromJavaHome
+
+set JAVA_EXE=java.exe
+%JAVA_EXE% -version >NUL 2>&1
+if "%ERRORLEVEL%" == "0" goto init
+
+echo.
+echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
+echo.
+echo Please set the JAVA_HOME variable in your environment to match the
+echo location of your Java installation.
+
+goto fail
+
+:findJavaFromJavaHome
+set JAVA_HOME=%JAVA_HOME:"=%
+set JAVA_EXE=%JAVA_HOME%/bin/java.exe
+
+if exist "%JAVA_EXE%" goto init
+
+echo.
+echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
+echo.
+echo Please set the JAVA_HOME variable in your environment to match the
+echo location of your Java installation.
+
+goto fail
+
+:init
+@rem Get command-line arguments, handling Windows variants
+
+if not "%OS%" == "Windows_NT" goto win9xME_args
+
+:win9xME_args
+@rem Slurp the command line arguments.
+set CMD_LINE_ARGS=
+set _SKIP=2
+
+:win9xME_args_slurp
+if "x%~1" == "x" goto execute
+
+set CMD_LINE_ARGS=%*
+
+:execute
+@rem Setup the command line
+
+set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
+
+@rem Execute Gradle
+"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %CMD_LINE_ARGS%
+
+:end
+@rem End local scope for the variables with windows NT shell
+if "%ERRORLEVEL%"=="0" goto mainEnd
+
+:fail
+rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
+rem the _cmd.exe /c_ return code!
+if not "" == "%GRADLE_EXIT_CONSOLE%" exit 1
+exit /b 1
+
+:mainEnd
+if "%OS%"=="Windows_NT" endlocal
+
+:omega
diff --git a/kobalt.iml b/kobalt.iml
deleted file mode 100644
index 65950402..00000000
--- a/kobalt.iml
+++ /dev/null
@@ -1,433 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/kobalt/Build.kt.iml b/kobalt/Build.kt.iml
deleted file mode 100644
index 00b83dc6..00000000
--- a/kobalt/Build.kt.iml
+++ /dev/null
@@ -1,13 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/kobalt/src/Build.kt b/kobalt/src/Build.kt
index b8d2d8d9..0d09844a 100644
--- a/kobalt/src/Build.kt
+++ b/kobalt/src/Build.kt
@@ -1,17 +1,14 @@
-
-import com.beust.kobalt.TaskResult
+import com.beust.kobalt.*
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.annotation.Task
-import com.beust.kobalt.homeDir
import com.beust.kobalt.plugin.application.application
import com.beust.kobalt.plugin.java.javaCompiler
import com.beust.kobalt.plugin.kotlin.kotlinCompiler
import com.beust.kobalt.plugin.packaging.assemble
+import com.beust.kobalt.plugin.publish.autoGitTag
import com.beust.kobalt.plugin.publish.bintray
import com.beust.kobalt.plugin.publish.github
-import com.beust.kobalt.project
-import com.beust.kobalt.test
import org.apache.maven.model.Developer
import org.apache.maven.model.License
import org.apache.maven.model.Model
@@ -20,17 +17,41 @@ import java.io.File
import java.nio.file.Files
import java.nio.file.Paths
import java.nio.file.StandardCopyOption
+import java.util.zip.ZipEntry
+import java.util.zip.ZipOutputStream
+
+val bs = buildScript {
+ repos("https://dl.bintray.com/cbeust/maven")
+}
object Versions {
- val okhttp = "3.2.0"
- val okio = "1.6.0"
- val retrofit = "2.1.0"
- val gson = "2.6.2"
- val aether = "1.1.0"
- val sonatypeAether = "1.13.1"
- val maven = "3.3.9"
+ val kotlin = "1.2.71"
+ val okhttp = "3.9.1"
+ val okio = "1.13.0"
+ val retrofit = "2.3.0"
+ val gson = "2.8.2"
+ val guice = "4.2.2"
+ val maven = "3.5.2"
+ val mavenResolver = "1.1.0"
+ val slf4j = "1.7.3"
+ val aether = "1.0.2.v20150114"
+ val testng = "6.12"
+ val jcommander = "1.72"
+
+ // JUnit 5
+ val junit = "4.12"
+ val junitPlatform = "1.1.0"
+ val junitJupiter = "5.1.0"
}
+fun mavenResolver(vararg m: String)
+ = m.map { "org.apache.maven.resolver:maven-resolver-$it:${Versions.mavenResolver}" }
+ .toTypedArray()
+
+fun aether(vararg m: String)
+ = m.map { "org.eclipse.aether:aether-$it:${Versions.aether}" }
+ .toTypedArray()
+
val wrapper = project {
name = "kobalt-wrapper"
group = "com.beust"
@@ -43,6 +64,7 @@ val wrapper = project {
}
assemble {
+ jar { }
jar {
name = projectName + ".jar"
manifest {
@@ -54,6 +76,13 @@ val wrapper = project {
application {
mainClass = "com.beust.kobalt.wrapper.Main"
}
+
+ bintray {
+ publish = true
+ sign = true
+ }
+
+ pom = createPom(name, "Wrapper for Kobalt")
}
val kobaltPluginApi = project {
@@ -63,53 +92,44 @@ val kobaltPluginApi = project {
version = readVersion()
directory = "modules/kobalt-plugin-api"
description = "A build system in Kotlin"
- url = "http://beust.com/kobalt"
+ url = "https://beust.com/kobalt"
- pom = Model().apply {
- name = project.name
- description = "A build system in Kotlin"
- url = "http://beust.com/kobalt"
- licenses = listOf(License().apply {
- name = "Apache 2.0"
- url = "http://www.apache .org/licenses/LICENSE-2.0"
- })
- scm = Scm().apply {
- url = "http://github.com/cbeust/kobalt"
- connection = "https://github.com/cbeust/kobalt.git"
- developerConnection = "git@github.com:cbeust/kobalt.git"
- }
- developers = listOf(Developer().apply {
- name = "Cedric Beust"
- email = "cedric@beust.com"
- })
- }
+ pom = createPom(name, "A build system in Kotlin")
dependencies {
compile(
- "com.google.inject:guice:4.0",
- "com.google.inject.extensions:guice-assistedinject:4.0",
+ "org.jetbrains.kotlin:kotlin-stdlib:${Versions.kotlin}",
+ "com.google.inject:guice:${Versions.guice}",
+ "com.google.inject.extensions:guice-assistedinject:4.1.0",
"javax.inject:javax.inject:1",
- "com.google.guava:guava:19.0",
+ "com.google.guava:guava:27.0.1-jre",
"org.apache.maven:maven-model:${Versions.maven}",
- "io.reactivex:rxjava:1.1.5",
+ "io.reactivex:rxjava:1.3.3",
"com.squareup.okio:okio:${Versions.okio}",
"com.google.code.gson:gson:${Versions.gson}",
"com.squareup.okhttp3:okhttp:${Versions.okhttp}",
"com.squareup.retrofit2:retrofit:${Versions.retrofit}",
"com.squareup.retrofit2:converter-gson:${Versions.retrofit}",
- "com.beust:jcommander:1.48",
- "org.eclipse.jgit:org.eclipse.jgit:4.5.0.201609210915-r",
+ "com.beust:jcommander:${Versions.jcommander}",
+ "org.eclipse.jgit:org.eclipse.jgit:4.9.0.201710071750-r",
+ "org.slf4j:slf4j-simple:${Versions.slf4j}",
+ *mavenResolver("api", "spi", "util", "impl", "connector-basic", "transport-http", "transport-file"),
+ "org.apache.maven:maven-aether-provider:3.3.9",
+ "org.testng.testng-remote:testng-remote:1.3.2",
+ "org.testng:testng:${Versions.testng}",
+ "org.junit.platform:junit-platform-surefire-provider:${Versions.junitPlatform}",
+ "org.junit.platform:junit-platform-runner:${Versions.junitPlatform}",
+ "org.junit.platform:junit-platform-engine:${Versions.junitPlatform}",
+ "org.junit.platform:junit-platform-console:${Versions.junitPlatform}",
+ "org.junit.jupiter:junit-jupiter-engine:${Versions.junitJupiter}",
+ "org.junit.vintage:junit-vintage-engine:${Versions.junitJupiter}",
+ "org.apache.commons:commons-compress:1.15",
+ "commons-io:commons-io:2.6",
- "org.slf4j:slf4j-nop:1.6.0",
- "org.eclipse.aether:aether-spi:${Versions.aether}",
- "org.eclipse.aether:aether-impl:${Versions.aether}",
- "org.eclipse.aether:aether-connector-basic:${Versions.aether}",
- "org.eclipse.aether:aether-transport-file:${Versions.aether}",
- "org.eclipse.aether:aether-transport-http:${Versions.aether}",
- "org.sonatype.aether:aether-api:${Versions.sonatypeAether}",
- "org.sonatype.aether:aether-connector-wagon:1.13.1",
- "org.apache.maven:maven-aether-provider:${Versions.maven}"
+ // Java 9
+ "javax.xml.bind:jaxb-api:2.3.0"
)
+ exclude(*aether("impl", "spi", "util", "api"))
}
@@ -122,12 +142,8 @@ val kobaltPluginApi = project {
}
}
-// install {
-// libDir = "lib-test"
-// }
-
kotlinCompiler {
- args("-nowarn")
+ args("nowarn")
}
bintray {
@@ -143,25 +159,35 @@ val kobaltApp = project(kobaltPluginApi, wrapper) {
dependencies {
// Used by the plugins
- compile("org.jetbrains.kotlin:kotlin-compiler-embeddable:1.1.0-beta-17")
+ compile("org.jetbrains.kotlin:kotlin-compiler-embeddable:${Versions.kotlin}")
// Used by the main app
- compile("com.github.spullara.mustache.java:compiler:0.9.1",
+ compile(
+ "org.jetbrains.kotlin:kotlin-stdlib:${Versions.kotlin}",
+ "com.github.spullara.mustache.java:compiler:0.9.5",
"javax.inject:javax.inject:1",
- "com.google.inject:guice:4.0",
- "com.google.inject.extensions:guice-assistedinject:4.0",
- "com.beust:jcommander:1.48",
+ "com.google.inject:guice:${Versions.guice}",
+ "com.google.inject.extensions:guice-assistedinject:${Versions.guice}",
+ "com.beust:jcommander:${Versions.jcommander}",
"org.apache.maven:maven-model:${Versions.maven}",
- "com.google.code.findbugs:jsr305:3.0.1",
+ "com.google.code.findbugs:jsr305:3.0.2",
"com.google.code.gson:gson:${Versions.gson}",
"com.squareup.retrofit2:retrofit:${Versions.retrofit}",
"com.squareup.retrofit2:converter-gson:${Versions.retrofit}",
- "com.squareup.okhttp3:okhttp-ws:${Versions.okhttp}",
- "biz.aQute.bnd:bndlib:2.4.0",
+// "com.squareup.okhttp3:okhttp-ws:3.4.2",
+ "biz.aQute.bnd:biz.aQute.bndlib:3.5.0",
+ *mavenResolver("spi"),
- "com.squareup.okhttp3:logging-interceptor:3.2.0",
+ "com.squareup.okhttp3:logging-interceptor:3.9.0",
- "com.sparkjava:spark-core:2.5"
+ "com.sparkjava:spark-core:2.6.0",
+ "org.codehaus.groovy:groovy:2.4.12",
+
+ // Java 9
+ "javax.xml.bind:jaxb-api:2.3.0",
+ "com.sun.xml.bind:jaxb-impl:2.3.0",
+ "com.sun.xml.bind:jaxb-core:2.3.0",
+ "com.sun.activation:javax.activation:1.2.0"
// "org.eclipse.jetty:jetty-server:${Versions.jetty}",
// "org.eclipse.jetty:jetty-servlet:${Versions.jetty}",
@@ -175,8 +201,11 @@ val kobaltApp = project(kobaltPluginApi, wrapper) {
}
dependenciesTest {
- compile("org.testng:testng:6.9.11",
- "org.assertj:assertj-core:3.4.1")
+ compile("org.jetbrains.kotlin:kotlin-test:${Versions.kotlin}",
+ "org.testng:testng:${Versions.testng}",
+ "org.assertj:assertj-core:3.8.0",
+ *mavenResolver("util")
+ )
}
assemble {
@@ -188,17 +217,27 @@ val kobaltApp = project(kobaltPluginApi, wrapper) {
}
zip {
val dir = "kobalt-$version"
- include(from("dist"), to("$dir/bin"), "kobaltw")
- include(from("dist"), to("$dir/bin"), "kobaltw.bat")
- include(from("$buildDirectory/libs"), to("$dir/kobalt/wrapper"),
- "$projectName-$version.jar")
- include(from("modules/wrapper/$buildDirectory/libs"), to("$dir/kobalt/wrapper"),
- "$projectName-wrapper.jar")
+ val files = listOf(
+ "dist", "$dir/bin", "kobaltw",
+ "dist", "$dir/bin", "kobaltw.bat",
+ "$buildDirectory/libs", "$dir/kobalt/wrapper", "$projectName-$version.jar",
+ "modules/wrapper/$buildDirectory/libs", "$dir/kobalt/wrapper", "$projectName-wrapper.jar")
+
+ (0 .. files.size - 1 step 3).forEach { i ->
+ include(from(files[i]), to(files[i + 1]), files[i + 2])
+ }
+
+ // Package the sources
+ val currentDir = Paths.get(".").toAbsolutePath().normalize().toString()
+ zipFolders("$currentDir/$buildDirectory/libs/all-sources/$projectName-$version-sources.jar",
+ "$currentDir/$directory/src/main/kotlin",
+ "$currentDir/${kobaltPluginApi.directory}/src/main/kotlin")
+ include(from("$buildDirectory/libs/all-sources"), to("$dir/kobalt/wrapper"), "$projectName-$version-sources.jar")
}
}
kotlinCompiler {
- args("-nowarn")
+ args("nowarn")
}
bintray {
@@ -207,12 +246,37 @@ val kobaltApp = project(kobaltPluginApi, wrapper) {
github {
file("$buildDirectory/libs/$name-$version.zip", "$name/$version/$name-$version.zip")
- autoGitTag = true
}
test {
args("-log", "2", "src/test/resources/testng.xml")
}
+
+ autoGitTag {
+ enabled = true
+ }
+}
+
+fun zipFolders(zipFilePath: String, vararg foldersPath: String) {
+ val zip = Paths.get(zipFilePath)
+ Files.deleteIfExists(zip)
+ Files.createDirectories(zip.parent)
+ val zipPath = Files.createFile(zip)
+ ZipOutputStream(Files.newOutputStream(zipPath)).use {
+ foldersPath.map {Paths.get(it)}.forEach { folderPath ->
+ Files.walk(folderPath)
+ .filter { path -> !Files.isDirectory(path) }
+ .forEach { path ->
+ val zipEntry = ZipEntry(folderPath.relativize(path).toString())
+ try {
+ it.putNextEntry(zipEntry)
+ Files.copy(path, it)
+ it.closeEntry()
+ } catch (e: Exception) {
+ }
+ }
+ }
+ }
}
fun readVersion() : String {
@@ -232,9 +296,31 @@ fun taskCopyVersionForWrapper(project: Project) : TaskResult {
File(toString).mkdirs()
val from = Paths.get("src/main/resources/kobalt.properties")
val to = Paths.get("$toString/kobalt.properties")
- Files.copy(from,
- to,
- StandardCopyOption.REPLACE_EXISTING)
+ // Only copy if necessary so we don't break incremental compilation
+ if (! to.toFile().exists() || (from.toFile().readLines() != to.toFile().readLines())) {
+ Files.copy(from,
+ to,
+ StandardCopyOption.REPLACE_EXISTING)
+ }
}
return TaskResult()
}
+
+fun createPom(projectName: String, projectDescription: String) = Model().apply {
+ name = projectName
+ description = projectDescription
+ url = "https://beust.com/kobalt"
+ licenses = listOf(License().apply {
+ name = "Apache-2.0"
+ url = "https://www.apache.org/licenses/LICENSE-2.0"
+ })
+ scm = Scm().apply {
+ url = "https://github.com/cbeust/kobalt"
+ connection = "https://github.com/cbeust/kobalt.git"
+ developerConnection = "git@github.com:cbeust/kobalt.git"
+ }
+ developers = listOf(Developer().apply {
+ name = "Cedric Beust"
+ email = "cedric@beust.com"
+ })
+}
diff --git a/kobalt/wrapper/kobalt-wrapper.jar b/kobalt/wrapper/kobalt-wrapper.jar
index 9ef336dc..848fb463 100644
Binary files a/kobalt/wrapper/kobalt-wrapper.jar and b/kobalt/wrapper/kobalt-wrapper.jar differ
diff --git a/kobalt/wrapper/kobalt-wrapper.properties b/kobalt/wrapper/kobalt-wrapper.properties
index 9fca6a4b..0ca8045f 100644
--- a/kobalt/wrapper/kobalt-wrapper.properties
+++ b/kobalt/wrapper/kobalt-wrapper.properties
@@ -1 +1 @@
-kobalt.version=0.916
+kobalt.version=1.0.122
\ No newline at end of file
diff --git a/kobaltw-test b/kobaltw-test
new file mode 100755
index 00000000..2693c3aa
--- /dev/null
+++ b/kobaltw-test
@@ -0,0 +1,8 @@
+#!/usr/bin/env sh
+JAR=$(ls -1 -t kobaltBuild/libs/*.jar | grep -Ev "(sources|javadoc)" | head -1)
+TEMPDIR=$(mktemp -d)
+cp -pf "$JAR" "$TEMPDIR"
+TEMPJAR=$TEMPDIR/$(basename "$JAR")
+export KOBALT_JAR=$TEMPJAR
+java -jar "$TEMPJAR" "$@"
+rm -rf "$TEMPDIR"
\ No newline at end of file
diff --git a/lib/kotlin-reflect.jar b/lib/kotlin-reflect.jar
deleted file mode 100644
index 575b230f..00000000
Binary files a/lib/kotlin-reflect.jar and /dev/null differ
diff --git a/lib/kotlin-runtime-sources.jar b/lib/kotlin-runtime-sources.jar
deleted file mode 100644
index c0232531..00000000
Binary files a/lib/kotlin-runtime-sources.jar and /dev/null differ
diff --git a/lib/kotlin-runtime.jar b/lib/kotlin-runtime.jar
deleted file mode 100644
index b89a8866..00000000
Binary files a/lib/kotlin-runtime.jar and /dev/null differ
diff --git a/modules/kobalt-plugin-api/build.gradle b/modules/kobalt-plugin-api/build.gradle
new file mode 100644
index 00000000..56085220
--- /dev/null
+++ b/modules/kobalt-plugin-api/build.gradle
@@ -0,0 +1,85 @@
+plugins {
+ id 'org.jetbrains.kotlin.jvm' version '1.2.71'
+ id 'com.github.johnrengelman.shadow' version '5.0.0'
+}
+
+dependencies {
+ implementation "biz.aQute.bnd:biz.aQute.bndlib:$bndlib"
+ implementation "com.google.code.findbugs:jsr305:$findbugs"
+ implementation "com.sparkjava:spark-core:$spark"
+ implementation "com.squareup.okhttp3:logging-interceptor:$okhttp"
+ implementation 'commons-io:commons-io:2.6'
+ implementation 'io.reactivex:rxjava:1.3.3'
+ implementation "javax.inject:javax.inject:$inject"
+ implementation "javax.xml.bind:jaxb-api:$jaxb"
+ implementation 'org.apache.commons:commons-compress:1.15'
+ implementation 'org.apache.maven:maven-aether-provider:3.3.9'
+ implementation "org.apache.maven.resolver:maven-resolver-api:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-connector-basic:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-impl:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-spi:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-transport-file:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-transport-http:$mavenResolver"
+ implementation "org.apache.maven.resolver:maven-resolver-util:$mavenResolver"
+ implementation "org.codehaus.groovy:groovy:$groovy"
+ implementation 'org.eclipse.jgit:org.eclipse.jgit:4.9.0.201710071750-r'
+ implementation "org.junit.jupiter:junit-jupiter-engine:$junitJupiter"
+ implementation "org.junit.platform:junit-platform-console:$junitPlatform"
+ implementation "org.junit.platform:junit-platform-engine:$junitPlatform"
+ implementation "org.junit.platform:junit-platform-runner:$junitPlatform"
+ implementation "org.junit.platform:junit-platform-surefire-provider:$junitPlatform"
+ implementation "org.junit.vintage:junit-vintage-engine:$junitJupiter"
+ implementation "org.slf4j:slf4j-simple:$slf4j"
+ implementation "org.testng:testng:$testng"
+ implementation 'org.testng.testng-remote:testng-remote:1.3.2'
+ implementation "com.beust:jcommander:$jcommander"
+ implementation "com.google.code.gson:gson:$gson"
+ implementation "com.google.inject:guice:$guice"
+ implementation "com.google.inject.extensions:guice-assistedinject:$guice"
+ implementation "com.squareup.okio:okio:$okio"
+ implementation "com.squareup.retrofit2:converter-gson:$retrofit"
+ implementation "com.squareup.retrofit2:retrofit:$retrofit"
+ implementation "org.apache.maven:maven-model:$maven"
+ implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin"
+}
+
+shadowJar {
+ classifier = null
+}
+
+test {
+ useTestNG()
+}
+
+publishing {
+ publications {
+ shadow(MavenPublication) { publication ->
+ project.shadow.component(publication)
+ artifact sourcesJar
+ artifact javadocJar
+
+ pom {
+ name = project.name
+ description = 'A build system in Kotlin'
+ url = 'https://beust.com/kobalt'
+ licenses {
+ license {
+ name = 'Apache-2.0'
+ url = 'https://www.apache.org/licenses/LICENSE-2.0'
+ }
+ }
+ developers {
+ developer {
+ name = 'Cedric Beust'
+ email = 'cedric@beust.com'
+ }
+ }
+ scm {
+ connection = 'scm:https://github.com/cbeust/kobalt.git'
+ developerConnection = 'scm:git@github.com:cbeust/kobalt.git'
+ url = 'https://github.com/cbeust/kobalt'
+ }
+ }
+ }
+ }
+}
diff --git a/modules/kobalt-plugin-api/kobalt-plugin-api.iml b/modules/kobalt-plugin-api/kobalt-plugin-api.iml
deleted file mode 100644
index 4abe3e8a..00000000
--- a/modules/kobalt-plugin-api/kobalt-plugin-api.iml
+++ /dev/null
@@ -1,549 +0,0 @@
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/pom.xml b/modules/kobalt-plugin-api/pom.xml
new file mode 100644
index 00000000..f9026387
--- /dev/null
+++ b/modules/kobalt-plugin-api/pom.xml
@@ -0,0 +1,279 @@
+
+ 4.0.0
+
+ com.beust
+ kobalt-pom
+ 1.1.0
+ ../..
+
+
+ kobalt-plugin-api
+ jar
+ 1.1.0
+
+
+
+ org.jetbrains.kotlin
+ kotlin-stdlib
+ ${kotlin.version}
+
+
+ org.apache.maven
+ maven-aether-provider
+ 3.3.9
+
+
+ org.eclipse.aether
+ impl
+
+
+ org.eclipse.aether
+ spi
+
+
+ org.eclipse.aether
+ util
+
+
+ org.eclipse.aether
+ api
+
+
+
+
+ org.apache.maven.resolver
+ maven-resolver-api
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-spi
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-util
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-impl
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-connector-basic
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-transport-http
+ ${mavenresolver.version}
+
+
+ org.apache.maven.resolver
+ maven-resolver-transport-file
+ ${mavenresolver.version}
+
+
+ io.reactivex
+ rxjava
+ 1.3.3
+
+
+ com.squareup.okio
+ okio
+ ${okio.version}
+
+
+ javax.inject
+ javax.inject
+ 1
+ compile
+
+
+ com.google.inject
+ guice
+ 4.2.2
+
+
+ com.google.inject.extensions
+ guice-assistedinject
+ 4.2.2
+
+
+ com.beust
+ jcommander
+ 1.72
+
+
+ org.apache.maven
+ maven-model
+ 3.5.2
+
+
+ com.google.code.findbugs
+ jsr305
+ 3.0.2
+
+
+ com.google.code.gson
+ gson
+ 2.8.2
+
+
+ com.squareup.retrofit2
+ retrofit
+ 2.3.0
+
+
+ com.squareup.retrofit2
+ converter-gson
+ 2.3.0
+
+
+ biz.aQute.bnd
+ biz.aQute.bndlib
+ 3.5.0
+
+
+ com.squareup.okhttp3
+ logging-interceptor
+ ${okhttp3.version}
+
+
+ com.sparkjava
+ spark-core
+ 2.6.0
+
+
+ org.codehaus.groovy
+ groovy
+ 2.4.12
+
+
+ org.apache.commons
+ commons-compress
+ 1.15
+
+
+ commons-io
+ commons-io
+ 2.6
+
+
+ org.junit.platform
+ junit-platform-surefire-provider
+ ${junit.version}
+
+
+ org.junit.platform
+ junit-platform-runner
+ ${junit.version}
+
+
+ org.junit.platform
+ junit-platform-engine
+ ${junit.version}
+
+
+ org.junit.platform
+ junit-platform-console
+ ${junit.version}
+
+
+ org.junit.jupiter
+ junit-jupiter-engine
+ ${junitJupiter.version}
+
+
+ org.junit.vintage
+ junit-vintage-engine
+ ${junitJupiter.version}
+
+
+ org.testng.testng-remote
+ testng-remote
+ 1.3.2
+
+
+ org.testng
+ testng
+ ${testng.version}
+
+
+ org.eclipse.jgit
+ org.eclipse.jgit
+ 4.9.0.201710071750-r
+
+
+ org.slf4j
+ slf4j-simple
+ ${slf4j.version}
+
+
+
+ javax.xml.bind
+ jaxb-api
+ 2.3.0
+
+
+
+
+
+
+ org.jetbrains.kotlin
+ kotlin-maven-plugin
+ ${kotlin.version}
+
+
+ compile
+ compile
+
+
+ ${project.basedir}/src/main/kotlin
+
+
+
+
+ test-compile
+ test-compile
+
+
+ ${project.basedir}/src/test/kotlin
+
+
+
+
+
+
+ org.apache.maven.plugins
+ maven-compiler-plugin
+ 3.5.1
+
+
+
+ default-compile
+ none
+
+
+
+ default-testCompile
+ none
+
+
+ java-compile
+ compile
+ compile
+
+
+ java-test-compile
+ test-compile
+ testCompile
+
+
+
+
+
+
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/ArchiveGenerator.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/ArchiveGenerator.kt
new file mode 100644
index 00000000..8158c642
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/ArchiveGenerator.kt
@@ -0,0 +1,21 @@
+package com.beust.kobalt
+
+import com.beust.kobalt.api.KobaltContext
+import com.beust.kobalt.api.Project
+import com.beust.kobalt.archive.Zip
+import com.beust.kobalt.misc.KFiles
+import java.io.File
+
+interface ArchiveGenerator {
+ fun findIncludedFiles(project: Project, context: KobaltContext, zip: Zip) : List
+ val suffix: String
+ fun generateArchive(project: Project, context: KobaltContext, zip: Zip, files: List) : File
+
+ fun fullArchiveName(project: Project, context: KobaltContext, archiveName: String?) : File {
+ val fullArchiveName = context.variant.archiveName(project, archiveName, suffix)
+ val archiveDir = File(KFiles.libsDir(project))
+ val result = File(archiveDir.path, fullArchiveName)
+ return result
+ }
+
+}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Args.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Args.kt
index b2b7ec97..372f1ba1 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Args.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Args.kt
@@ -7,7 +7,7 @@ class Args {
var targets: List = arrayListOf()
@Parameter(names = arrayOf("-bf", "--buildFile"), description = "The build file")
- var buildFile: String? = null
+ var buildFile: String? = "kobalt/src/Build.kt"
@Parameter(names = arrayOf("--checkVersions"), description = "Check if there are any newer versions of the " +
"dependencies")
@@ -22,6 +22,10 @@ class Args {
@Parameter(names = arrayOf("--download"), description = "Force a download from the downloadUrl in the wrapper")
var download: Boolean = false
+ @Parameter(names = arrayOf("--downloadSources"),
+ description = "Force a download of sources and javadocs when resolving dependencies")
+ var downloadSources: Boolean = false
+
@Parameter(names = arrayOf("--dryRun"), description = "Display all the tasks that will get run without " +
"actually running them")
var dryRun: Boolean = false
@@ -43,7 +47,7 @@ class Args {
var listTemplates: Boolean = false
@Parameter(names = arrayOf("--log"), description = "Define the log level " +
- "(${Constants.LOG_DEFAULT_LEVEL}-${Constants.LOG_MAX_LEVEL})")
+ "(${Constants.LOG_QUIET_LEVEL}-${Constants.LOG_MAX_LEVEL})")
var log: Int = Constants.LOG_DEFAULT_LEVEL
@Parameter(names = arrayOf("--logTags"),
@@ -57,8 +61,8 @@ class Args {
@Parameter(names = arrayOf("--noIncremental"), description = "Turn off incremental builds")
var noIncremental: Boolean = false
- @Parameter(names = arrayOf("--parallel"), description = "Build all the projects in parallel whenever possible")
- var parallel: Boolean = true
+ @Parameter(names = arrayOf("--offline"), description = "Don't try to download dependencies even if there is no cached version")
+ var offline: Boolean = false
@Parameter(names = arrayOf("--plugins"), description = "Comma-separated list of plug-in Maven id's")
var pluginIds: String? = null
@@ -82,7 +86,14 @@ class Args {
@Parameter(names = arrayOf("--projectInfo"), description = "Display information about the current projects")
var projectInfo: Boolean = false
- @Parameter(names = arrayOf("--sequential"), description = "Build all the projects in sequence")
+ @Parameter(names = arrayOf("--noIncrementalKotlin"), description = "Disable incremental Kotlin compilation")
+ var noIncrementalKotlin: Boolean = false
+
+ companion object {
+ const val SEQUENTIAL = "--sequential"
+ }
+
+ @Parameter(names = arrayOf(Args.SEQUENTIAL), description = "Build all the projects in sequence")
var sequential: Boolean = false
@Parameter(names = arrayOf("--server"), description = "Run in server mode")
@@ -93,5 +104,8 @@ class Args {
@Parameter(names = arrayOf("--update"), description = "Update to the latest version of Kobalt")
var update: Boolean = false
+
+ @Parameter(names = arrayOf("--version"), description = "Display the current version of Kobalt")
+ var version: Boolean = false
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/AsciiArt.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/AsciiArt.kt
index df506b9f..e138fabc 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/AsciiArt.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/AsciiArt.kt
@@ -17,16 +17,16 @@ class AsciiArt {
companion object {
private val BANNERS = arrayOf(
" __ __ __ __ __ \n" +
- " / //_/ ____ / /_ ____ _ / / / /_\n" +
- " / ,< / __ \\ / __ \\ / __ `/ / / / __/\n" +
- " / /| | / /_/ / / /_/ // /_/ / / / / /_ \n" +
- " /_/ |_| \\____/ /_.___/ \\__,_/ /_/ \\__/ ",
+ " / //_/ ____ / /_ ____ _ / / / /_\n" +
+ " / ,< / __ \\ / __ \\ / __ `/ / / / __/\n" +
+ " / /| | / /_/ / / /_/ // /_/ / / / / /_ \n" +
+ " /_/ |_| \\____/ /_.___/ \\__,_/ /_/ \\__/ ",
" _ __ _ _ _ \n" +
- " | |/ / ___ | |__ __ _ | | | |_ \n" +
- " | ' / / _ \\ | '_ \\ / _` | | | | __|\n" +
- " | . \\ | (_) | | |_) | | (_| | | | | |_ \n" +
- " |_|\\_\\ \\___/ |_.__/ \\__,_| |_| \\__| "
+ " | |/ / ___ | |__ __ _ | | | |_ \n" +
+ " | ' / / _ \\ | '_ \\ / _` | | | | __|\n" +
+ " | . \\ | (_) | | |_) | | (_| | | | | |_ \n" +
+ " |_|\\_\\ \\___/ |_.__/ \\__,_| |_| \\__| "
)
val banner : String get() = BANNERS[Random().nextInt(BANNERS.size)]
@@ -85,7 +85,7 @@ class AsciiArt {
}
fun logBox(s: String, bl: String = bottomLeft, br: String = bottomRight, indent: Int = 0)
- = logBox(listOf(s), bl, br, indent)
+ = logBox(listOf(s), bl, br, indent)
fun fill(n: Int) = buildString { repeat(n, { append(" ")})}.toString()
@@ -105,7 +105,7 @@ class AsciiArt {
const val CYAN = "\u001B[36m"
const val WHITE = "\u001B[37m"
- private fun wrap(s: CharSequence, color: String) = color + s + RESET
+ fun wrap(s: CharSequence, color: String) = color + s + RESET
private fun blue(s: CharSequence) = wrap(s, BLUE)
private fun red(s: CharSequence) = wrap(s, RED)
private fun yellow(s: CharSequence) = wrap(s, YELLOW)
@@ -141,18 +141,18 @@ class AsciiTable {
fun build() : String {
val formattedHeaders =
- headers.mapIndexed { index, s ->
- val s2 = col(widths[index], s)
- s2
- }.joinToString(vb)
+ headers.mapIndexed { index, s ->
+ val s2 = col(widths[index], s)
+ s2
+ }.joinToString(vb)
val result = StringBuffer().apply {
append(AsciiArt.logBox(formattedHeaders, AsciiArt.bottomLeft2, AsciiArt.bottomRight2))
append("\n")
}
var lineLength = 0
- rows.forEachIndexed { index, row ->
+ rows.forEachIndexed { _, row ->
val formattedRow = row.mapIndexed { i, s -> col(widths[i], s) }.joinToString(vb)
- val line = vb + " " + formattedRow + " " + vb
+ val line = "$vb $formattedRow $vb"
result.append(line).append("\n")
lineLength = line.length
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/BuildScript.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/BuildScript.kt
index a91e82e0..4c35b9ed 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/BuildScript.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/BuildScript.kt
@@ -16,26 +16,33 @@ var BUILD_SCRIPT_CONFIG : BuildScriptConfig? = null
class BuildScriptConfig {
/** The list of repos used to locate plug-ins. */
+ @Directive
fun repos(vararg r: String) = newRepos(*r)
/** The list of plug-ins to use for this build file. */
+ @Directive
fun plugins(vararg pl: String) = newPlugins(*pl)
/** The build file classpath. */
+ @Directive
fun buildFileClasspath(vararg bfc: String) = newBuildFileClasspath(*bfc)
- // The following settings modify the compiler used to compile the build file.
- // Projects should use kotlinCompiler { compilerVersion } to configure the Kotin compiler for their source files.
+ /** Options passed to Kobalt */
+ @Directive
+ fun kobaltOptions(vararg options: String) = Kobalt.addKobaltOptions(options)
+
+ /** Where to find additional build files */
+ @Directive
+ fun buildSourceDirs(vararg dirs: String) = Kobalt.addBuildSourceDirs(dirs)
+
+ // The following settings modify the compiler used to compile the build file, which regular users should
+ // probably never need to do. Projects should use kotlinCompiler { compilerVersion } to configure the
+ // Kotin compiler for their source files.
var kobaltCompilerVersion : String? = null
var kobaltCompilerRepo: String? = null
var kobaltCompilerFlags: String? = null
}
-@Directive
-fun buildScript(init: BuildScriptConfig.() -> Unit) {
- BUILD_SCRIPT_CONFIG = BuildScriptConfig().apply { init() }
-}
-
@Directive
fun homeDir(vararg dirs: String) : String = SystemProperties.homeDir +
File.separator + dirs.toMutableList().joinToString(File.separator)
@@ -67,7 +74,18 @@ data class ProxyConfig(val host: String = "", val port: Int = 0, val type: Strin
fun toAetherProxy() = Proxy(type, host, port) // TODO make support for proxy auth
}
-data class HostConfig(var url: String = "", var username: String? = null, var password: String? = null) {
+data class HostConfig(var url: String = "", var name: String = HostConfig.createRepoName(url),
+ var username: String? = null, var password: String? = null) {
+
+ companion object {
+ /**
+ * For repos specified in the build file (repos()) that don't have an associated unique name,
+ * create such a name from the URL. This is a requirement from Maven Resolver, and failing to do
+ * this leads to very weird resolution errors.
+ */
+ private fun createRepoName(url: String) = url.replace("/", "_").replace("\\", "_").replace(":", "_")
+ }
+
fun hasAuth() : Boolean {
return (! username.isNullOrBlank()) && (! password.isNullOrBlank())
}
@@ -98,6 +116,7 @@ fun buildFileClasspath(vararg deps: String) {
}
fun newBuildFileClasspath(vararg deps: String) {
+ //FIXME newBuildFileClasspath called twice
deps.forEach { Kobalt.addBuildFileClasspath(it) }
}
@@ -107,7 +126,7 @@ fun authRepos(vararg repos : HostConfig) {
}
@Directive
-fun authRepo(init: HostConfig.() -> Unit) = HostConfig().apply { init() }
+fun authRepo(init: HostConfig.() -> Unit) = HostConfig(name = "").apply { init() }
@Directive
fun glob(g: String) : IFileSpec.GlobSpec = IFileSpec.GlobSpec(g)
@@ -124,4 +143,3 @@ fun localMaven() : String {
}
return result.toURI().toString()
}
-
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Constants.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Constants.kt
index 98a23e75..8eb73c84 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Constants.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Constants.kt
@@ -3,20 +3,19 @@ package com.beust.kobalt
import com.beust.kobalt.misc.KFiles
object Constants {
+ const val LOG_QUIET_LEVEL = 0
const val LOG_DEFAULT_LEVEL = 1
const val LOG_MAX_LEVEL = 3
val BUILD_FILE_NAME = "Build.kt"
val BUILD_FILE_DIRECTORY = "kobalt/src"
val BUILD_FILE_PATH = KFiles.joinDir(BUILD_FILE_DIRECTORY, BUILD_FILE_NAME)
- val KOTLIN_COMPILER_VERSION = "1.1.0-beta-17"
+ val KOTLIN_COMPILER_VERSION = "1.2.70"
- internal val DEFAULT_REPOS = listOf(
+ internal val DEFAULT_REPOS = listOf(
// "https://maven-central.storage.googleapis.com/",
- "http://repo1.maven.org/maven2/",
- "https://jcenter.bintray.com/",
- "http://repository.jetbrains.com/all/",
- "https://dl.bintray.com/kotlin/kotlin-eap",
- "https://dl.bintray.com/kotlin/kotlin-eap-1.1"
+ HostConfig("https://repo1.maven.org/maven2/", "Maven"),
+ HostConfig("https://jcenter.bintray.com/", "JCenter")
+// "https://repository.jetbrains.com/all/", // <-- contains snapshots
// snapshots
// "https://oss.sonatype.org/content/repositories/snapshots/"
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Directives.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Directives.kt
index ef96a5f1..93d9434c 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Directives.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Directives.kt
@@ -4,6 +4,8 @@ import com.beust.kobalt.api.Kobalt
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.annotation.Directive
import com.beust.kobalt.internal.JvmCompilerPlugin
+import kotlin.properties.ReadWriteProperty
+import kotlin.reflect.KProperty
@Directive
fun project(vararg projects: Project, init: Project.() -> Unit): Project {
@@ -14,3 +16,24 @@ fun project(vararg projects: Project, init: Project.() -> Unit): Project {
}
}
+@Directive
+fun buildScript(init: BuildScriptConfig.() -> Unit): BuildScriptConfig {
+ val buildScriptConfig = BuildScriptConfig().apply { init() }
+ BUILD_SCRIPT_CONFIG = buildScriptConfig
+ return buildScriptConfig
+}
+
+@Directive
+fun profile(): ReadWriteProperty {
+ val result = object: ReadWriteProperty {
+ var value: Boolean = false
+ override operator fun getValue(thisRef: Nothing?, property: KProperty<*>): Boolean {
+ return value
+ }
+
+ override operator fun setValue(thisRef: Nothing?, property: KProperty<*>, value: Boolean) {
+ this.value = value
+ }
+ }
+ return result
+}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/FileSpec.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/FileSpec.kt
index 715c4221..1eb409f4 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/FileSpec.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/FileSpec.kt
@@ -28,15 +28,15 @@ sealed class IFileSpec {
private fun isIncluded(includeMatchers: Glob, excludes: List, rel: Path) : Boolean {
excludes.forEach {
if (it.matches(rel)) {
- kobaltLog(3, "Excluding ${rel.toFile()}")
+ kobaltLog(3, " Excluding ${rel.toFile()}")
return false
}
}
if (includeMatchers.matches(rel)) {
- kobaltLog(3, "Including ${rel.toFile().path}")
+ kobaltLog(3, " Including ${rel.toFile().path}")
return true
}
- kobaltLog(2, "Excluding ${rel.toFile()} (not matching any include pattern")
+ kobaltLog(2, " Excluding ${rel.toFile()} (not matching any include pattern")
return false
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludeFromTo.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludeFromTo.kt
new file mode 100644
index 00000000..ea189851
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludeFromTo.kt
@@ -0,0 +1,43 @@
+package com.beust.kobalt
+
+import com.beust.kobalt.api.annotation.Directive
+import java.io.File
+
+/**
+ * Base classes for directives that support install(from,to) (e.g. install{} or jar{}).
+ */
+open class IncludeFromTo {
+ /**
+ * Prefix path to be removed from the zip file. For example, if you add "build/lib/a.jar" to the zip
+ * file and the excludePrefix is "build/lib", then "a.jar" will be added at the root of the zip file.
+ */
+ val includedFiles = arrayListOf()
+
+ @Directive
+ fun from(s: String) = From(s)
+
+ @Directive
+ fun to(s: String) = To(s)
+
+ @Directive
+ fun copy(from: From, to: To) {
+ val dir = File(from.path).absoluteFile.parentFile
+ includedFiles.add(IncludedFile(from(dir.absolutePath), to, listOf(IFileSpec.FileSpec(from.path))))
+ }
+
+ @Directive
+ fun include(vararg files: String) {
+ includedFiles.add(IncludedFile(files.map { IFileSpec.FileSpec(it) }))
+ }
+
+ @Directive
+ fun include(from: From, to: To, vararg specs: String) {
+ includedFiles.add(IncludedFile(from, to, specs.map { IFileSpec.FileSpec(it) }))
+ }
+
+ @Directive
+ fun include(from: From, to: To, vararg specs: IFileSpec.GlobSpec) {
+ includedFiles.add(IncludedFile(from, to, listOf(*specs)))
+ }
+}
+
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludedFile.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludedFile.kt
new file mode 100644
index 00000000..46dea15e
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/IncludedFile.kt
@@ -0,0 +1,44 @@
+package com.beust.kobalt
+
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.toString
+import java.io.File
+import java.nio.file.Paths
+
+class IncludedFile(val fromOriginal: From, val toOriginal: To, val specs: List,
+ val expandJarFiles: Boolean = false) {
+ constructor(specs: List, expandJarFiles: Boolean = false) : this(From(""), To(""), specs, expandJarFiles)
+ fun from(s: String) = File(if (fromOriginal.isCurrentDir()) s else KFiles.joinDir(from, s))
+ val from: String get() = fromOriginal.path.replace("\\", "/")
+ fun to(s: String) = File(if (toOriginal.isCurrentDir()) s else KFiles.joinDir(to, s))
+ val to: String get() = toOriginal.path.replace("\\", "/")
+ override fun toString() = toString("IncludedFile",
+ "files - ", specs.map { it.toString() },
+ "from", from,
+ "to", to)
+
+ fun allFromFiles(directory: String? = null): List {
+ val result = arrayListOf()
+ specs.forEach { spec ->
+// val fullDir = if (directory == null) from else KFiles.joinDir(directory, from)
+ spec.toFiles(directory, from).forEach { source ->
+ result.add(if (source.isAbsolute) source else File(source.path))
+ }
+ }
+ return result.map { Paths.get(it.path).normalize().toFile()}
+ }
+}
+
+open class Direction(open val p: String) {
+ override fun toString() = path
+ fun isCurrentDir() = path == "./"
+
+ val path: String get() =
+ if (p.isEmpty()) "./"
+ else if (p.startsWith("/") || p.endsWith("/")) p
+ else p + "/"
+}
+
+class From(override val p: String) : Direction(p)
+
+class To(override val p: String) : Direction(p)
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/JarGenerator.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/JarGenerator.kt
index 0edc4ffc..19bb52c6 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/JarGenerator.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/JarGenerator.kt
@@ -3,21 +3,22 @@ package com.beust.kobalt
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
import com.beust.kobalt.archive.Archives
-import com.beust.kobalt.archive.Jar
+import com.beust.kobalt.archive.MetaArchive
+import com.beust.kobalt.archive.Zip
import com.beust.kobalt.maven.DependencyManager
import com.beust.kobalt.maven.aether.Scope
-import com.beust.kobalt.misc.*
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.kobaltLog
import com.google.inject.Inject
import java.io.File
import java.io.FileInputStream
-import java.io.OutputStream
import java.nio.file.Paths
-import java.util.jar.JarOutputStream
import java.util.jar.Manifest
-class JarGenerator @Inject constructor(val dependencyManager: DependencyManager) {
+class JarGenerator @Inject constructor(val dependencyManager: DependencyManager) : ArchiveGenerator {
companion object {
- fun findIncludedFiles(directory: String, files: List, excludes: List)
+ fun findIncludedFiles(directory: String, files: List, excludes: List,
+ throwOnError: Boolean = true)
: List {
val result = arrayListOf()
files.forEach { includedFile ->
@@ -27,7 +28,7 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
if (File(directory, fromPath).exists()) {
spec.toFiles(directory, fromPath).forEach { file ->
val fullFile = File(KFiles.joinDir(directory, fromPath, file.path))
- if (! fullFile.exists()) {
+ if (! fullFile.exists() && throwOnError) {
throw AssertionError("File should exist: $fullFile")
}
@@ -40,7 +41,7 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
}
} else {
- kobaltLog(2, "Directory $fromPath doesn't exist, not including it in the jar")
+ kobaltLog(2, " Directory $fromPath doesn't exist, not including it in the jar")
}
}
if (includedSpecs.size > 0) {
@@ -52,7 +53,9 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
}
}
- fun findIncludedFiles(project: Project, context: KobaltContext, jar: Jar) : List {
+ override val suffix = ".jar"
+
+ override fun findIncludedFiles(project: Project, context: KobaltContext, zip: Zip) : List {
//
// Add all the applicable files for the current project
//
@@ -60,7 +63,7 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
val result = arrayListOf()
val classesDir = KFiles.makeDir(buildDir.path, "classes")
- if (jar.includedFiles.isEmpty()) {
+ if (zip.includedFiles.isEmpty()) {
// If no includes were specified, assume the user wants a simple jar file made of the
// classes of the project, so we specify a From("build/classes/"), To("") and
// a list of files containing everything under it
@@ -70,7 +73,7 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
// Class files
val files = KFiles.findRecursively(classesDir).map { File(relClassesDir.toFile(), it) }
val filesNotExcluded : List = files.filter {
- ! KFiles.Companion.isExcluded(KFiles.joinDir(project.directory, it.path), jar.excludes)
+ ! KFiles.Companion.isExcluded(KFiles.joinDir(project.directory, it.path), zip.excludes)
}
val fileSpecs = arrayListOf()
filesNotExcluded.forEach {
@@ -86,16 +89,14 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
//
// The user specified an include, just use it verbatim
//
- val includedFiles = findIncludedFiles(project.directory, jar.includedFiles, jar.excludes)
+ val includedFiles = findIncludedFiles(project.directory, zip.includedFiles, zip.excludes, false)
result.addAll(includedFiles)
}
//
// If fatJar is true, add all the transitive dependencies as well: compile, runtime and dependent projects
//
- if (jar.fatJar) {
- context.logger.log(project.name, 2, "Finding included files for fat jar")
-
+ if (zip.fatJar) {
val seen = hashSetOf()
@Suppress("UNCHECKED_CAST")
val allDependencies = project.compileDependencies + project.compileRuntimeDependencies +
@@ -104,13 +105,13 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
context.variant.productFlavor.compileDependencies +
context.variant.productFlavor.compileRuntimeDependencies
val transitiveDependencies = dependencyManager.calculateDependencies(project, context,
- listOf(Scope.COMPILE), allDependencies)
+ scopes = listOf(Scope.COMPILE), passedDependencies = allDependencies)
transitiveDependencies.map {
it.jarFile.get()
}.forEach { file : File ->
if (! seen.contains(file.path)) {
seen.add(file.path)
- if (! KFiles.Companion.isExcluded(file, jar.excludes)) {
+ if (! KFiles.Companion.isExcluded(file, zip.excludes)) {
result.add(IncludedFile(specs = arrayListOf(IFileSpec.FileSpec(file.absolutePath)),
expandJarFiles = true))
}
@@ -121,19 +122,18 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
return result
}
- fun generateJar(project: Project, context: KobaltContext, jar: Jar) : File {
- val includedFiles = findIncludedFiles(project, context, jar)
-
+ override fun generateArchive(project: Project, context: KobaltContext, zip: Zip,
+ includedFiles: List) : File {
//
// Generate the manifest
// If manifest attributes were specified in the build file, use those to generateAndSave the manifest. Otherwise,
// try to find a META-INF/MANIFEST.MF and use that one if we find any. Otherwise, use the default manifest.
//
val manifest =
- if (jar.attributes.size > 1) {
- context.logger.log(project.name, 2, "Creating MANIFEST.MF from " + jar.attributes.size + " attributes")
+ if (zip.attributes.size > 1) {
+ context.logger.log(project.name, 2, "Creating MANIFEST.MF from " + zip.attributes.size + " attributes")
Manifest().apply {
- jar.attributes.forEach { attribute ->
+ zip.attributes.forEach { attribute ->
mainAttributes.putValue(attribute.first, attribute.second)
}
}
@@ -142,7 +142,7 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
val allFiles = includedFiles.flatMap { file ->
file.allFromFiles(project.directory).map { file.from(it.path) }
}
- val manifestFiles = allFiles.filter { it.path.contains("META-INF/MANIFEST.MF") }
+ val manifestFiles = allFiles.filter { it.path.contains(MetaArchive.MANIFEST_MF) }
return if (manifestFiles.any()) manifestFiles[0] else null
}
@@ -151,14 +151,12 @@ class JarGenerator @Inject constructor(val dependencyManager: DependencyManager)
context.logger.log(project.name, 2, "Including MANIFEST.MF file $manifestFile")
Manifest(FileInputStream(manifestFile))
} else {
- Manifest()
+ null
}
}
- val jarFactory = { os: OutputStream -> JarOutputStream(os, manifest) }
-
- return Archives.generateArchive(project, context, jar.name, ".jar", includedFiles,
- true /* expandJarFiles */, jarFactory)
+ return Archives.generateArchive(project, context, zip.name, ".jar", includedFiles,
+ true /* expandJarFiles */, manifest)
}
}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Jvm.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Jvm.kt
index 598ca401..14c55efd 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Jvm.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Jvm.kt
@@ -67,7 +67,7 @@ open class Jvm constructor(
return toolsJar
}
if (javaHome!!.name.equals("jre", true)) {
- javaHome = javaHome!!.parentFile
+ _javaHome = javaHome!!.parentFile
toolsJar = File(javaHome, "lib/tools.jar")
if (toolsJar.exists()) {
return toolsJar
@@ -78,7 +78,7 @@ open class Jvm constructor(
val version = SystemProperties.Companion.javaVersion
if (javaHome!!.name.toRegex().matches("jre\\d+")
|| javaHome!!.name == "jre$version") {
- javaHome = File(javaHome!!.parentFile, "jdk$version")
+ _javaHome = File(javaHome!!.parentFile, "jdk$version")
toolsJar = File(javaHome, "lib/tools.jar")
if (toolsJar.exists()) {
return toolsJar
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Plugins.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Plugins.kt
index 83c197cd..0102dd8b 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Plugins.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Plugins.kt
@@ -4,11 +4,14 @@ import com.beust.kobalt.api.*
import com.beust.kobalt.api.annotation.IncrementalTask
import com.beust.kobalt.api.annotation.Task
import com.beust.kobalt.internal.IncrementalManager
+import com.beust.kobalt.internal.KobaltSettings
import com.beust.kobalt.internal.PluginInfo
import com.beust.kobalt.internal.TaskManager
import com.beust.kobalt.maven.DependencyManager
-import com.beust.kobalt.maven.LocalRepo
-import com.beust.kobalt.misc.*
+import com.beust.kobalt.misc.JarUtils
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.KobaltExecutors
+import com.beust.kobalt.misc.kobaltLog
import com.google.inject.Provider
import java.io.File
import java.lang.reflect.Method
@@ -23,9 +26,8 @@ import javax.inject.Singleton
class Plugins @Inject constructor (val taskManagerProvider : Provider,
val files: KFiles,
val depManager: DependencyManager,
- val localRepo: LocalRepo,
+ val settings: KobaltSettings,
val executors: KobaltExecutors,
- val pluginInfo: PluginInfo,
val incrementalManagerFactory: IncrementalManager.IFactory,
val taskManager: TaskManager) {
@@ -168,6 +170,9 @@ class Plugins @Inject constructor (val taskManagerProvider : Provider()
+// @Inject
+// lateinit var pluginInfo: PluginInfo
+
fun installPlugins(dependencies: List, scriptClassLoader: ClassLoader) {
val executor = executors.newExecutor("Plugins", 5)
dependencies.forEach {
@@ -188,6 +193,8 @@ class Plugins @Inject constructor (val taskManagerProvider : Provider>) {
@@ -57,10 +81,12 @@ class ResolveDependency @Inject constructor(
else leftMiddle
val indent = level * increment
for(i in 0..indent - 2) {
- if (i % increment == 0) print(vertical)
- else print(" ")
+ if (!KobaltLogger.isQuiet) {
+ if (i == 0 || ((i + 1) % increment == 0)) print(vertical)
+ else print(" ")
+ }
}
- println(left + " " + dep.id)
+ kobaltLog(1, left + " " + dep.id + (if (dep.optional) " (optional)" else ""))
display(node.children)
}
}
@@ -76,7 +102,12 @@ class ResolveDependency @Inject constructor(
kobaltLog(2, "Found dependency ${dep.dep.id} level: ${dep.level}")
result.add(node)
seen.add(it.id)
- node.addChildren(findChildren(node, seen))
+ try {
+ node.addChildren(findChildren(node, seen))
+ } catch(ex: Exception) {
+ if (! it.optional) warn("Couldn't resolve " + node)
+ // else don't warn about missing optional dependencies
+ }
}
}
kobaltLog(2, "Children for ${root.value.dep.id}: ${result.size}")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/SystemProperties.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/SystemProperties.kt
index b7877264..d5507497 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/SystemProperties.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/SystemProperties.kt
@@ -1,9 +1,17 @@
package com.beust.kobalt
-public class SystemProperties {
+class SystemProperties {
companion object {
- val javaBase = System.getProperty("java.home") ?:
- (System.getenv("JAVA_HOME") ?: throw IllegalArgumentException("JAVA_HOME not defined"))
+ val javaBase : String
+ get() {
+ val jh = System.getenv("JAVA_HOME")
+ ?: System.getProperty("java.home")
+ ?: throw IllegalArgumentException("JAVA_HOME not defined")
+ val result =
+ if (jh.toLowerCase().endsWith("jre")) jh.substring(0, jh.length - 4)
+ else jh
+ return result
+ }
val javaVersion = System.getProperty("java.version")
val homeDir = System.getProperty("user.home")
val tmpDir = System.getProperty("java.io.tmpdir")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TaskResult.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TaskResult.kt
index 4d6a45f3..241bc045 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TaskResult.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TaskResult.kt
@@ -1,3 +1,8 @@
package com.beust.kobalt
-open public class TaskResult(val success: Boolean = true, val errorMessage: String? = null)
+class TestResult(val success: Boolean, val shortMessage: String? = null, val longMessage: String? = null)
+
+open class TaskResult(val success: Boolean = true,
+ val testResult: TestResult? = null,
+ val errorMessage: String? = null
+)
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestConfig.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestConfig.kt
new file mode 100644
index 00000000..f84b094d
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestConfig.kt
@@ -0,0 +1,40 @@
+package com.beust.kobalt
+
+import com.beust.kobalt.api.Project
+import com.beust.kobalt.api.annotation.Directive
+
+class TestConfig(val project: Project, val isDefault : Boolean = false) {
+ val testArgs = arrayListOf()
+ val jvmArgs = arrayListOf()
+ val testIncludes = arrayListOf("**/*Test.class")
+ val testExcludes = arrayListOf()
+
+ @Directive
+ var name: String = ""
+
+ @Directive
+ fun args(vararg arg: String) {
+ testArgs.addAll(arg)
+ }
+
+ @Directive
+ fun jvmArgs(vararg arg: String) {
+ jvmArgs.addAll(arg)
+ }
+
+ @Directive
+ fun include(vararg arg: String) {
+ testIncludes.apply {
+ clear()
+ addAll(arg)
+ }
+ }
+
+ @Directive
+ fun exclude(vararg arg: String) {
+ testExcludes.apply {
+ clear()
+ addAll(arg)
+ }
+ }
+}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestDirective.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestDirective.kt
index d7f6cb7e..ad026380 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestDirective.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/TestDirective.kt
@@ -3,43 +3,13 @@ package com.beust.kobalt
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.annotation.Directive
-class TestConfig(val project: Project, val isDefault : Boolean = false) {
- val testArgs = arrayListOf()
- val jvmArgs = arrayListOf()
- val testIncludes = arrayListOf("**/*Test.class")
- val testExcludes = arrayListOf()
-
- var name: String = ""
-
- fun args(vararg arg: String) {
- testArgs.addAll(arg)
- }
-
- fun jvmArgs(vararg arg: String) {
- jvmArgs.addAll(arg)
- }
-
- fun include(vararg arg: String) {
- testIncludes.apply {
- clear()
- addAll(arg)
- }
- }
-
- fun exclude(vararg arg: String) {
- testExcludes.apply {
- clear()
- addAll(arg)
- }
- }
-}
-
@Directive
-fun Project.test(init: TestConfig.() -> Unit) = let { project ->
+fun Project.test(init: TestConfig.() -> Unit): TestConfig = let { project ->
with(testConfigs) {
val tf = TestConfig(project).apply { init() }
if (! map { it.name }.contains(tf.name)) {
add(tf)
+ tf
} else {
throw KobaltException("Test configuration \"${tf.name}\" already exists, give it a different "
+ "name with test { name = ... }")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Variant.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Variant.kt
index 04b428a8..13120fa0 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Variant.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/Variant.kt
@@ -29,7 +29,7 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
/**
* for {internal, release}, return [internalRelease, internal, release]
*/
- fun allDirectories(project: Project): List {
+ fun allDirectories(): List {
return arrayListOf().apply {
add(toCamelcaseDir())
add(productFlavor.name)
@@ -70,7 +70,7 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
val result = arrayListOf()
val sourceDirectories = sourceSet.correctSourceSet(project)
.filter { File(project.directory, it).exists() }
- .map { File(it) }
+ .map(::File)
if (isDefault) {
result.addAll(sourceDirectories)
@@ -88,7 +88,7 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
result.add(dir)
}
- result.addAll(allDirectories(project)
+ result.addAll(allDirectories()
.map { File(KFiles.joinDir("src", it, suffix)) }
.filter(File::exists))
@@ -113,8 +113,8 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
if (isDefault) {
archiveName ?: project.name + "-" + project.version + suffix
} else {
- val base = if (archiveName != null) archiveName.substring(0, archiveName.length - suffix.length)
- else project.name + "-" + project.version
+ val base = archiveName?.substring(0, archiveName.length - suffix.length)
+ ?: project.name + "-" + project.version
val flavor = if (productFlavor.name.isEmpty()) "" else "-" + productFlavor.name
val type = if (buildType.name.isEmpty()) "" else "-" + buildType.name
val result: String = base + flavor + type + suffix
@@ -124,18 +124,16 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
return result
}
- val shortArchiveName = if (isDefault) "" else "-" + productFlavor.name + "-" + buildType.name
-
var generatedSourceDirectory: File? = null
private fun findBuildTypeBuildConfig(project: Project, variant: Variant?) : BuildConfig? {
val buildTypeName = variant?.buildType?.name
- return project.buildTypes[buildTypeName]?.buildConfig ?: null
+ return project.buildTypes[buildTypeName]?.buildConfig
}
private fun findProductFlavorBuildConfig(project: Project, variant: Variant?) : BuildConfig? {
val buildTypeName = variant?.productFlavor?.name
- return project.productFlavors[buildTypeName]?.buildConfig ?: null
+ return project.productFlavors[buildTypeName]?.buildConfig
}
/**
@@ -164,7 +162,8 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
?: throw KobaltException(
"packageName needs to be defined on the project in order to generateAndSave BuildConfig")
- val contributor = ActorUtils.selectAffinityActor(context.pluginInfo.buildConfigContributors, project)
+ val contributor = ActorUtils.selectAffinityActor(project, context,
+ context.pluginInfo.buildConfigContributors)
if (contributor != null) {
val code = contributor.generateBuildConfig(project, context, pkg, this, buildConfigs)
val result = KFiles.makeDir(KFiles.generatedSourceDir(project, this, "buildConfig"))
@@ -222,7 +221,7 @@ class Variant(val initialProductFlavor: ProductFlavorConfig? = null,
}
fun toCamelcaseDir() : String {
- fun lci(s : String) = if (s.length == 0 || s.length == 1) s else s[0].toLowerCase() + s.substring(1)
+ fun lci(s : String) = if (s.isEmpty() || s.length == 1) s else s[0].toLowerCase() + s.substring(1)
return lci(productFlavor.name) + buildType.name.capitalize()
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/CompilerActionInfo.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/CompilerActionInfo.kt
index f0be2eb5..e323e474 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/CompilerActionInfo.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/CompilerActionInfo.kt
@@ -11,4 +11,6 @@ data class CompilerActionInfo(val directory: String?,
val suffixesBeingCompiled: List,
val outputDir: File,
val compilerArgs: List,
- val friendPaths: List)
+ val friendPaths: List,
+ val forceRecompile: Boolean,
+ val compilerSeparateProcess: Boolean = false)
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/DependencyHolder.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/DependencyHolder.kt
index a561cd9f..e1195ca3 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/DependencyHolder.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/DependencyHolder.kt
@@ -13,6 +13,7 @@ interface IDependencyHolder {
val compileDependencies : ArrayList
val optionalDependencies : ArrayList
val compileProvidedDependencies : ArrayList
+ val compileOnlyDependencies : ArrayList
val compileRuntimeDependencies : ArrayList
val excludedDependencies : ArrayList
val nativeDependencies : ArrayList
@@ -29,6 +30,7 @@ open class DependencyHolder : IDependencyHolder {
override val compileDependencies : ArrayList = arrayListOf()
override val optionalDependencies : ArrayList = arrayListOf()
override val compileProvidedDependencies : ArrayList = arrayListOf()
+ override val compileOnlyDependencies : ArrayList = arrayListOf()
override val compileRuntimeDependencies : ArrayList = arrayListOf()
override val excludedDependencies : ArrayList = arrayListOf()
override val nativeDependencies : ArrayList = arrayListOf()
@@ -37,7 +39,7 @@ open class DependencyHolder : IDependencyHolder {
override fun dependencies(init: Dependencies.() -> Unit) : Dependencies {
dependencies = Dependencies(project, compileDependencies, optionalDependencies, compileProvidedDependencies,
- compileRuntimeDependencies, excludedDependencies, nativeDependencies)
+ compileOnlyDependencies, compileRuntimeDependencies, excludedDependencies, nativeDependencies)
dependencies!!.init()
return dependencies!!
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildConfigContributor.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildConfigContributor.kt
index 35553f74..ef9d3b4d 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildConfigContributor.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildConfigContributor.kt
@@ -5,7 +5,7 @@ import com.beust.kobalt.Variant
/**
* Plug-ins that can generate a BuildConfig file.
*/
-interface IBuildConfigContributor : ISimpleAffinity {
+interface IBuildConfigContributor : IProjectAffinity {
fun generateBuildConfig(project: Project, context: KobaltContext, packageName: String, variant: Variant,
buildConfigs: List) : String
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildListener.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildListener.kt
index 1e53cc49..2b0fdadb 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildListener.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IBuildListener.kt
@@ -4,8 +4,12 @@ package com.beust.kobalt.api
* Plug-ins that listen to build events.
*/
interface IBuildListener : IListener {
+
+ class TaskEndInfo(val success: Boolean, val shortMessage: String? = null,
+ val longMessage: String? = null)
+
fun taskStart(project: Project, context: KobaltContext, taskName: String) {}
- fun taskEnd(project: Project, context: KobaltContext, taskName: String, success: Boolean) {}
+ fun taskEnd(project: Project, context: KobaltContext, taskName: String, info: TaskEndInfo) {}
fun projectStart(project: Project, context: KobaltContext) {}
fun projectEnd(project: Project, context: KobaltContext, status: ProjectBuildStatus) {}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IClasspathDependency.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IClasspathDependency.kt
index e529d148..527e6f13 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IClasspathDependency.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IClasspathDependency.kt
@@ -22,7 +22,7 @@ interface IClasspathDependency {
/** @return true if this dependency represents a Maven coordinate */
val isMaven: Boolean
- /** @return whether this dependency is optional */
+ /** @return true if this dependency is optional */
val optional: Boolean
/** Absolute path to the jar file on the local file system */
@@ -36,4 +36,6 @@ interface IClasspathDependency {
/** Used to only keep the most recent version for an artifact if no version was specified */
val shortId: String
+
+ val excluded: ArrayList
}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ICompilerContributor.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ICompilerContributor.kt
index b4788370..b6ce8fd2 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ICompilerContributor.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ICompilerContributor.kt
@@ -1,7 +1,6 @@
package com.beust.kobalt.api
import com.beust.kobalt.TaskResult
-import com.beust.kobalt.misc.warn
interface ICompilerDescription : Comparable {
/**
@@ -55,13 +54,15 @@ class CompilerDescription(override val name: String, override val sourceDirecto
override val canCompileDirectories: Boolean = false) : ICompilerDescription {
override fun compile(project: Project, context: KobaltContext, info: CompilerActionInfo): TaskResult {
val result =
- if (info.sourceFiles.size > 0) {
+ if (info.sourceFiles.isNotEmpty()) {
compiler.compile(project, context, info)
} else {
- warn("Couldn't find any source files to compile")
+ context.logger.log(project.name, 2, "$name couldn't find any source files to compile")
TaskResult()
}
return result
}
+
+ override fun toString() = name + " compiler"
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IDependencyManager.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IDependencyManager.kt
index ab80cd3f..3a66f980 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IDependencyManager.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IDependencyManager.kt
@@ -1,6 +1,11 @@
package com.beust.kobalt.api
+import com.beust.kobalt.maven.aether.Filters.EXCLUDE_OPTIONAL_FILTER
+import com.beust.kobalt.maven.aether.KobaltMavenResolver
import com.beust.kobalt.maven.aether.Scope
+import com.beust.kobalt.misc.kobaltLog
+import org.eclipse.aether.graph.DependencyFilter
+import org.eclipse.aether.graph.DependencyNode
/**
* Manage the creation of dependencies and also provide dependencies for projects.
@@ -24,7 +29,7 @@ interface IDependencyManager {
/**
* @return the source dependencies for this project, including the contributors.
*/
- fun dependencies(project: Project, context: KobaltContext): List
+ fun dependencies(project: Project, context: KobaltContext, scopes: List): List
/**
* @return the test dependencies for this project, including the contributors.
@@ -36,6 +41,48 @@ interface IDependencyManager {
* allDependencies is typically either compileDependencies or testDependencies
*/
fun calculateDependencies(project: Project?, context: KobaltContext,
- scopeFilters: Collection = listOf(Scope.COMPILE),
+ dependencyFilter: DependencyFilter =
+ createDependencyFilter(project, project?.compileDependencies ?: emptyList()),
+ scopes: List = listOf(Scope.COMPILE),
vararg passedDependencies: List): List
+
+ /**
+ * Create an Aether dependency filter that uses the dependency configuration included in each
+ * IClasspathDependency.
+ */
+ fun createDependencyFilter(project: Project?, dependencies: List) : DependencyFilter {
+ return DependencyFilter { p0, p1 ->
+ fun isNodeExcluded(node: DependencyNode, passedDep: IClasspathDependency) : Boolean {
+ val dep = create(KobaltMavenResolver.artifactToId(node.artifact))
+ return passedDep.excluded.any { ex -> ex.isExcluded(dep)}
+ }
+ fun isDepExcluded(node: DependencyNode, excluded: List?) : Boolean {
+ val dep = create(KobaltMavenResolver.artifactToId(node.artifact))
+ return excluded?.map { it.id }?.contains(dep.id) ?: false
+ }
+
+ val accept = dependencies.isEmpty() || dependencies.any {
+ // Is this dependency excluded?
+ val isExcluded = isNodeExcluded(p0, it) || isDepExcluded(p0, project?.excludedDependencies)
+
+ // Is the parent dependency excluded?
+ val isParentExcluded =
+ if (p1.any()) {
+ isNodeExcluded(p1[0], it) || isDepExcluded(p1[0], project?.excludedDependencies)
+ } else {
+ false
+ }
+
+ // Only accept if no exclusions were found
+ ! isExcluded && ! isParentExcluded
+ }
+
+ if (! accept) {
+ kobaltLog(2, "Excluding $p0")
+ }
+
+ if (accept) EXCLUDE_OPTIONAL_FILTER.accept(p0, p1)
+ else accept
+ }
+ }
}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IRunnerContributor.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IRunnerContributor.kt
index 5ab88cb0..f8c28b52 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IRunnerContributor.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/IRunnerContributor.kt
@@ -1,10 +1,11 @@
package com.beust.kobalt.api
import com.beust.kobalt.TaskResult
-import com.beust.kobalt.api.IClasspathDependency
/**
* Plugins that can run a project (task "run" or "test") should implement this interface.
+ *
+ * Currently not used.
*/
interface IRunnerContributor : IContributor, IProjectAffinity {
/**
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ITaskContributor.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ITaskContributor.kt
index 83621451..c606d54f 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ITaskContributor.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/ITaskContributor.kt
@@ -23,9 +23,10 @@ class DynamicTask(override val plugin: IPlugin, override val name: String, overr
override fun call(): TaskResult2 {
val taskResult = closure.invoke(project)
- return TaskResult2(taskResult.success, taskResult.errorMessage, this)
+ return TaskResult2(taskResult.success, errorMessage = taskResult.errorMessage, value = this)
}
- override fun toString() = "[DynamicTask $name dependsOn=$dependsOn reverseDependsOn=$reverseDependsOn]"
+ override fun toString() =
+ "[DynamicTask ${project.name}:$name dependsOn=$dependsOn reverseDependsOn=$reverseDependsOn]"
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/InputStreamJarTemplate.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/InputStreamJarTemplate.kt
new file mode 100644
index 00000000..222a2829
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/InputStreamJarTemplate.kt
@@ -0,0 +1,55 @@
+package com.beust.kobalt.api
+
+import com.beust.kobalt.Args
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.kobaltLog
+import java.io.*
+import java.net.URL
+import java.util.jar.JarInputStream
+
+/**
+ * Base class for templates that decompress a jar file.
+ */
+interface InputStreamJarTemplate : ITemplate {
+ val inputStream: InputStream
+
+ override fun generateTemplate(args: Args, classLoader: ClassLoader) {
+ val destDir = File(".")
+ JarInputStream(inputStream).use { ins ->
+ var entry = ins.nextEntry
+ while (entry != null) {
+ val f = File(destDir.path + File.separator + entry.name)
+ if (entry.isDirectory) {
+ f.mkdir()
+ entry = ins.nextEntry
+ continue
+ }
+
+ kobaltLog(2, " Extracting: $entry to ${f.absolutePath}")
+ FileOutputStream(f).use { fos ->
+ KFiles.copy(ins, fos)
+ }
+ entry = ins.nextEntry
+ }
+ }
+ }
+}
+
+abstract class ResourceJarTemplate(jarName: String, val classLoader: ClassLoader) : InputStreamJarTemplate {
+ override val inputStream : InputStream = classLoader.getResource(jarName).openConnection().inputStream
+}
+
+abstract class FileJarTemplate(val fileName: String) : InputStreamJarTemplate {
+ override val inputStream = FileInputStream(File(fileName))
+}
+
+abstract class HttpJarTemplate(val url: String) : InputStreamJarTemplate {
+ override val inputStream : InputStream
+ get() {
+ try {
+ return URL(url).openConnection().inputStream
+ } catch(ex: IOException) {
+ throw IllegalArgumentException("Couldn't connect to $url")
+ }
+ }
+}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/JarTemplate.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/JarTemplate.kt
deleted file mode 100644
index 8c68b2b8..00000000
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/JarTemplate.kt
+++ /dev/null
@@ -1,41 +0,0 @@
-package com.beust.kobalt.api
-
-import com.beust.kobalt.Args
-import com.beust.kobalt.misc.KFiles
-import com.beust.kobalt.misc.kobaltLog
-import java.io.File
-import java.io.FileOutputStream
-import java.util.jar.JarInputStream
-
-/**
- * Base class for templates that simply decompress a jar file to generate their project.
- */
-abstract class JarTemplate(val jarName: String) : ITemplate {
- companion object {
- fun extractFile(ins: JarInputStream, destDir: File) {
- var entry = ins.nextEntry
- while (entry != null) {
- val f = File(destDir.path + File.separator + entry.name)
- if (entry.isDirectory) {
- f.mkdir()
- entry = ins.nextEntry
- continue
- }
-
- kobaltLog(2, "Extracting: $entry to ${f.absolutePath}")
- FileOutputStream(f).use { fos ->
- KFiles.copy(ins, fos)
- }
- entry = ins.nextEntry
- }
- }
- }
-
- override fun generateTemplate(args: Args, classLoader: ClassLoader) {
- kobaltLog(2, "Generating template with class loader $classLoader")
- val destDir = File(".")
- val ins = JarInputStream(classLoader.getResource(jarName).openConnection().inputStream)
- extractFile(ins, destDir)
- }
-
-}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Kobalt.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Kobalt.kt
index 49b82050..7d37a0b8 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Kobalt.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Kobalt.kt
@@ -5,6 +5,7 @@ import com.beust.kobalt.HostConfig
import com.beust.kobalt.Plugins
import com.beust.kobalt.internal.PluginInfo
import com.beust.kobalt.maven.DependencyManager
+import com.beust.kobalt.maven.aether.KobaltMavenResolver
import com.google.inject.Guice
import com.google.inject.Injector
import com.google.inject.Module
@@ -35,12 +36,12 @@ class Kobalt {
*/
val repos : Set
get() {
- val settingsRepos = Kobalt.context?.settings?.defaultRepos ?: emptyList()
+ val settingsRepos = Kobalt.context?.settings?.defaultRepos?.map { HostConfig(it) } ?: emptyList()
// Repos from in the settings
val result = ArrayList(
(if (settingsRepos.isEmpty()) Constants.DEFAULT_REPOS
else settingsRepos)
- .map { HostConfig(it) })
+ )
// Repo from in the settings
Kobalt.context?.settings?.kobaltCompilerRepo?.let {
@@ -55,6 +56,9 @@ class Kobalt {
// Repos from the build file
result.addAll(reposFromBuildFiles)
+ result.forEach {
+ KobaltMavenResolver.initAuthentication(it)
+ }
return result.toHashSet()
}
@@ -118,5 +122,20 @@ class Kobalt {
get() = Duration.parse( kobaltProperties.getProperty(PROPERTY_KOBALT_VERSION_CHECK_TIMEOUT) ?: "P1D")
fun findPlugin(name: String) = Plugins.findPlugin(name)
+
+ val optionsFromBuild = arrayListOf()
+ fun addKobaltOptions(options: Array) {
+ optionsFromBuild.addAll(options)
+ }
+
+ val buildSourceDirs = arrayListOf()
+ fun addBuildSourceDirs(dirs: Array) {
+ buildSourceDirs.addAll(dirs)
+ }
+
+ fun cleanUp() {
+ buildSourceDirs.clear()
+ buildFileClasspath.clear()
+ }
}
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/KobaltContext.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/KobaltContext.kt
index f8a111c2..b7e5ace8 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/KobaltContext.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/KobaltContext.kt
@@ -12,7 +12,7 @@ import com.beust.kobalt.maven.DependencyManager
import com.beust.kobalt.maven.MavenId
import com.beust.kobalt.maven.PomGenerator
import com.beust.kobalt.maven.SimpleDep
-import com.beust.kobalt.maven.aether.KobaltAether
+import com.beust.kobalt.maven.aether.KobaltMavenResolver
import com.beust.kobalt.misc.KobaltExecutors
import java.io.File
@@ -53,9 +53,9 @@ class KobaltContext(val args: Args) {
FileType.JAVADOC -> toQualifier(dep, "", "javadoc")
FileType.OTHER -> id
}
- val resolved = aether.resolveToArtifact(fullId)
+ val resolved = resolver.resolveToArtifact(fullId)
if (resolved != null) {
- return resolved.artifact.file
+ return resolved.file
} else {
throw KobaltException("Couldn't resolve $id")
}
@@ -81,7 +81,7 @@ class KobaltContext(val args: Args) {
lateinit var executors: KobaltExecutors
lateinit var settings: KobaltSettings
lateinit var incrementalManager: IncrementalManager
- lateinit var aether: KobaltAether
+ lateinit var resolver: KobaltMavenResolver
lateinit var pomGeneratorFactory: PomGenerator.IFactory
lateinit var logger: ILogger
}
@@ -105,4 +105,12 @@ class InternalContext {
* The absolute directory of the current project.
*/
var absoluteDir: File? = null
+
+ /**
+ * If true, will force a recompile of the files even if using the incremental compile
+ */
+ var forceRecompile: Boolean = false
+
+ var noIncrementalKotlin: Boolean = false
+
}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Project.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Project.kt
index 65dd5219..e54e30ec 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Project.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/Project.kt
@@ -3,7 +3,8 @@ package com.beust.kobalt.api
import com.beust.kobalt.TestConfig
import com.beust.kobalt.api.annotation.Directive
import com.beust.kobalt.maven.DependencyManager
-import com.beust.kobalt.maven.aether.KobaltAether
+import com.beust.kobalt.maven.aether.AetherDependency
+import com.beust.kobalt.maven.aether.KobaltMavenResolver
import com.beust.kobalt.misc.KFiles
import com.beust.kobalt.misc.kobaltLog
import org.apache.maven.model.Model
@@ -11,6 +12,7 @@ import java.io.File
import java.util.*
import java.util.concurrent.Future
import java.util.concurrent.FutureTask
+import java.util.regex.Pattern
open class Project(
@Directive open var name: String = "",
@@ -24,6 +26,7 @@ open class Project(
@Directive open var url: String? = null,
@Directive open var pom: Model? = null,
@Directive open var dependsOn: ArrayList = arrayListOf(),
+ @Directive open var testsDependOn: ArrayList = arrayListOf(),
@Directive open var packageName: String? = group)
: IBuildConfig, IDependencyHolder by DependencyHolder() {
@@ -31,13 +34,15 @@ open class Project(
this.project = this
}
+ fun allProjectDependedOn() = project.dependsOn + project.testsDependOn
+
class ProjectExtra(project: Project) {
var isDirty = false
/**
* @return true if any of the projects we depend on is dirty.
*/
- fun dependsOnDirtyProjects(project: Project) = project.dependsOn.any { it.projectExtra.isDirty }
+ fun dependsOnDirtyProjects(project: Project) = project.allProjectDependedOn().any { it.projectExtra.isDirty }
}
/**
@@ -86,7 +91,8 @@ open class Project(
@Directive
fun dependenciesTest(init: Dependencies.() -> Unit) : Dependencies {
dependencies = Dependencies(this, testDependencies, arrayListOf(),
- testProvidedDependencies, compileRuntimeDependencies, excludedDependencies, nativeDependencies)
+ testProvidedDependencies, compileOnlyDependencies, compileRuntimeDependencies,
+ excludedDependencies, nativeDependencies)
dependencies!!.init()
return dependencies!!
}
@@ -94,6 +100,9 @@ open class Project(
val testDependencies : ArrayList = arrayListOf()
val testProvidedDependencies : ArrayList = arrayListOf()
+ fun testsDependOn(vararg projects: Project) = testsDependOn.addAll(projects)
+ fun dependsOn(vararg projects: Project) = dependsOn.addAll(projects)
+
/** Used to disambiguate various name properties */
@Directive
val projectName: String get() = name
@@ -120,6 +129,18 @@ open class Project(
return result
}
+ class Dep(val file: File, val id: String)
+
+ /**
+ * @return a list of the transitive dependencies (absolute paths to jar files) for the given dependencies.
+ * Can be used for example as `collect(compileDependencies)`.
+ */
+ @Directive
+ fun collect(dependencies: List) : List {
+ return (Kobalt.context?.dependencyManager?.transitiveClosure(dependencies) ?: emptyList())
+ .map { Dep(it.jarFile.get(), it.id) }
+ }
+
override fun toString() = "[Project $name]"
}
@@ -134,6 +155,7 @@ class Dependencies(val project: Project,
val dependencies: ArrayList,
val optionalDependencies: ArrayList,
val providedDependencies: ArrayList,
+ val compileOnlyDependencies: ArrayList,
val runtimeDependencies: ArrayList,
val excludedDependencies: ArrayList,
val nativeDependencies: ArrayList) {
@@ -145,11 +167,13 @@ class Dependencies(val project: Project,
* future tasks receive a get(), the repos will be correct.
*/
private fun addToDependencies(project: Project, dependencies: ArrayList,
- dep: Array, optional: Boolean = false): List>
+ dep: Array, optional: Boolean = false, excludeConfig: ExcludeConfig? = null): List>
= with(dep.map {
val resolved =
- if (KobaltAether.isRangeVersion(it)) {
- val result = Kobalt.INJECTOR.getInstance(KobaltAether::class.java).resolve(it).dependency.id
+ if (KobaltMavenResolver.isRangeVersion(it)) {
+ // Range id
+ val node = Kobalt.INJECTOR.getInstance(KobaltMavenResolver::class.java).resolveToArtifact(it)
+ val result = KobaltMavenResolver.artifactToId(node)
kobaltLog(2, "Resolved range id $it to $result")
result
} else {
@@ -158,12 +182,73 @@ class Dependencies(val project: Project,
DependencyManager.create(resolved, optional, project.directory)
}) {
dependencies.addAll(this)
+ if (excludeConfig != null) {
+ this.forEach { it.excluded.add(excludeConfig) }
+ }
+
this.map { FutureTask { it.jarFile.get() } }
}
@Directive
fun compile(vararg dep: String) = addToDependencies(project, dependencies, dep)
+ class ExcludeConfig {
+ val ids = arrayListOf()
+
+ @Directive
+ fun exclude(vararg passedIds: String) = ids.addAll(passedIds)
+
+ class ArtifactConfig(
+ var groupId: String? = null,
+ var artifactId: String? = null,
+ var version: String? = null
+ )
+
+ val artifacts = arrayListOf()
+
+ @Directive
+ fun exclude(groupId: String? = null, artifactId: String? = null, version: String? = null)
+ = artifacts.add(ArtifactConfig(groupId, artifactId, version))
+
+ fun match(pattern: String?, id: String) : Boolean {
+ return pattern == null || Pattern.compile(pattern).matcher(id).matches()
+ }
+
+ /**
+ * @return true if the dependency is excluded with any of the exclude() directives. The matches
+ * are performed by a regular expression match against the dependency.
+ */
+ fun isExcluded(dep: IClasspathDependency) : Boolean {
+ // Straight id match
+ var result = ids.any { match(it, dep.id) }
+
+ // Match on any combination of (groupId, artifactId, version)
+ if (! result && dep.isMaven) {
+ val mavenDep = dep as AetherDependency
+ val artifact = mavenDep.artifact
+ result = artifacts.any {
+ val match1 = it.groupId == null || match(it.groupId, artifact.groupId)
+ val match2 = it.artifactId == null || match(it.artifactId, artifact.artifactId)
+ val match3 = it.version == null || match(it.version, artifact.version)
+ match1 && match2 && match3
+ }
+ }
+
+ return result
+ }
+ }
+
+ @Directive
+ fun compile(dep: String, init: ExcludeConfig.() -> Unit) {
+ val excludeConfig = ExcludeConfig().apply {
+ init()
+ }
+ addToDependencies(project, dependencies, arrayOf(dep), excludeConfig = excludeConfig)
+ }
+
+ @Directive
+ fun compileOnly(vararg dep: String) = addToDependencies(project, compileOnlyDependencies, dep)
+
@Directive
fun compileOptional(vararg dep: String) {
addToDependencies(project, optionalDependencies, dep, optional = true)
@@ -173,7 +258,6 @@ class Dependencies(val project: Project,
@Directive
fun provided(vararg dep: String) {
addToDependencies(project, providedDependencies, dep)
- addToDependencies(project, dependencies, dep)
}
@Directive
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/TaskContributor.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/TaskContributor.kt
index 65eb529d..8c68be94 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/TaskContributor.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/api/TaskContributor.kt
@@ -44,6 +44,25 @@ class TaskContributor @Inject constructor(val incrementalManagerFactory: Increme
}
}
+ fun addTask(plugin: IPlugin, project: Project, taskName: String, description: String,
+ group: String = AnnotationDefault.GROUP,
+ dependsOn: List = emptyList(),
+ reverseDependsOn : List = emptyList(),
+ runBefore : List = emptyList(),
+ runAfter : List = emptyList(),
+ alwaysRunAfter: List = emptyList(),
+ runTask: (Project) -> TaskResult) {
+ dynamicTasks.add(DynamicTask(plugin, taskName, description, group, project,
+ dependsOn = dependsOn,
+ reverseDependsOn = reverseDependsOn,
+ runBefore = runBefore,
+ runAfter = runAfter,
+ alwaysRunAfter = alwaysRunAfter,
+ closure = { p: Project ->
+ runTask(project)
+ }))
+ }
+
fun addIncrementalVariantTasks(plugin: IPlugin, project: Project, context: KobaltContext, taskName: String,
group: String = AnnotationDefault.GROUP,
dependsOn: List = emptyList(),
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Archives.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Archives.kt
index 2c72c7b6..5334e09f 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Archives.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Archives.kt
@@ -1,25 +1,24 @@
package com.beust.kobalt.archive
-import com.beust.kobalt.Features
-import com.beust.kobalt.IFileSpec
+import com.beust.kobalt.*
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.annotation.ExportedProjectProperty
-import com.beust.kobalt.misc.*
+import com.beust.kobalt.misc.JarUtils
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.kobaltLog
import java.io.File
-import java.io.FileOutputStream
-import java.io.OutputStream
import java.util.*
-import java.util.zip.ZipOutputStream
class Archives {
companion object {
@ExportedProjectProperty(doc = "The name of the jar file", type = "String")
const val JAR_NAME = "jarName"
+ @ExportedProjectProperty(doc = "The name of the a jar file with a main() method", type = "String")
+ const val JAR_NAME_WITH_MAIN_CLASS = "jarNameWithMainClass"
- private val DEFAULT_STREAM_FACTORY = { os : OutputStream -> ZipOutputStream(os) }
-
- fun defaultArchiveName(project: Project) = project.name + "-" + project.version
+ fun defaultArchiveName(project: Project) = project.name +
+ if (project.version.isNullOrBlank()) "" else "-${project.version}"
fun generateArchive(project: Project,
context: KobaltContext,
@@ -27,15 +26,15 @@ class Archives {
suffix: String,
includedFiles: List,
expandJarFiles : Boolean = false,
- outputStreamFactory: (OutputStream) -> ZipOutputStream = DEFAULT_STREAM_FACTORY) : File {
+ manifest: java.util.jar.Manifest? = null) : File {
val fullArchiveName = context.variant.archiveName(project, archiveName, suffix)
val archiveDir = File(KFiles.libsDir(project))
val result = File(archiveDir.path, fullArchiveName)
context.logger.log(project.name, 3, "Creating $result")
if (! Features.USE_TIMESTAMPS || isOutdated(project.directory, includedFiles, result)) {
try {
- outputStreamFactory(FileOutputStream(result)).use {
- JarUtils.addFiles(project.directory, includedFiles, it, expandJarFiles)
+ MetaArchive(result, manifest).use { metaArchive ->
+ JarUtils.addFiles(project.directory, includedFiles, metaArchive, expandJarFiles)
context.logger.log(project.name, 2, "Added ${includedFiles.size} files to $result")
context.logger.log(project.name, 1, " Created $result")
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Jar.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Jar.kt
index f8a47848..d5086cbd 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Jar.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Jar.kt
@@ -8,7 +8,7 @@ import com.beust.kobalt.api.annotation.Directive
*/
open class Jar(override val project: Project,
override var name : String = Archives.defaultArchiveName(project) + ".jar",
- var fatJar: Boolean = false) : Zip(project, name), AttributeHolder {
+ override var fatJar: Boolean = false) : Zip(project, name, fatJar), AttributeHolder {
@Directive
fun manifest(init: Manifest.(p: Manifest) -> Unit) : Manifest {
val m = Manifest(this)
@@ -18,7 +18,7 @@ open class Jar(override val project: Project,
// Need to specify the version or attributes will just be dropped
@Directive
- val attributes = arrayListOf(Pair("Manifest-Version", "1.0"))
+ override val attributes = arrayListOf(Pair("Manifest-Version", "1.0"))
override fun addAttribute(k: String, v: String) {
attributes.add(Pair(k, v))
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/MetaArchive.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/MetaArchive.kt
new file mode 100644
index 00000000..c217c83e
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/MetaArchive.kt
@@ -0,0 +1,125 @@
+package com.beust.kobalt.archive
+
+import com.beust.kobalt.Glob
+import com.beust.kobalt.misc.KFiles
+import org.apache.commons.compress.archivers.ArchiveEntry
+import org.apache.commons.compress.archivers.zip.ZipArchiveEntry
+import org.apache.commons.compress.archivers.zip.ZipArchiveOutputStream
+import java.io.Closeable
+import java.io.File
+import java.io.FileInputStream
+import java.io.FileOutputStream
+import java.util.jar.Manifest
+import org.apache.commons.compress.archivers.zip.ZipFile as ApacheZipFile
+
+/**
+ * Abstraction of a zip/jar/war archive that automatically manages the addition of expanded jar files.
+ * Uses ZipArchiveOutputStream for fast inclusion of expanded jar files.
+ */
+class MetaArchive(outputFile: File, val manifest: Manifest?) : Closeable {
+ companion object {
+ const val MANIFEST_MF = "META-INF/MANIFEST.MF"
+ }
+
+ private val zos= ZipArchiveOutputStream(outputFile).apply {
+ encoding = "UTF-8"
+ }
+
+ init {
+ // If no manifest was passed, create an empty one so it's the first one in the archive
+ val m = manifest ?: Manifest()
+ val manifestFile = File.createTempFile("kobalt", "tmpManifest")
+ addEntry(ZipArchiveEntry("META-INF/"), null)
+ if (manifest != null) {
+ FileOutputStream(manifestFile).use { fos ->
+ m.write(fos)
+ }
+ }
+ val entry = zos.createArchiveEntry(manifestFile, MetaArchive.MANIFEST_MF)
+ addEntry(entry, FileInputStream(manifestFile))
+ }
+
+
+ fun addFile(f: File, entryFile: File, path: String?) {
+ maybeCreateParentDirectories(f)
+ addFile2(f, entryFile, path)
+ }
+
+ private fun addFile2(f: File, entryFile: File, path: String?) {
+ val file = f.normalize()
+ FileInputStream(file).use { inputStream ->
+ val actualPath = KFiles.fixSlashes(if (path != null) path + entryFile.path else entryFile.path)
+ ZipArchiveEntry(actualPath).let { entry ->
+ maybeCreateParentDirectories(File(actualPath))
+ maybeAddEntry(entry) {
+ addEntry(entry, inputStream)
+ }
+ }
+ }
+ }
+
+ private val createdDirs = hashSetOf()
+
+ /**
+ * For an entry a/b/c/File, an entry needs to be created for each individual directory:
+ * a/
+ * a/b/
+ * a/b/c
+ * a/b/c/File
+ */
+ private fun maybeCreateParentDirectories(file: File) {
+ val toCreate = arrayListOf()
+ var current = file.parentFile
+ while (current != null && current.path != ".") {
+ if (!createdDirs.contains(current.path)) {
+ toCreate.add(0, KFiles.fixSlashes(current) + "/")
+ createdDirs.add(current.path)
+ }
+ current = current.parentFile
+ }
+ toCreate.forEach { dir ->
+ addEntry(ZipArchiveEntry(dir), null)
+ }
+ }
+
+ fun addArchive(jarFile: File) {
+ ApacheZipFile(jarFile).use { jar ->
+ val jarEntries = jar.entries
+ for (entry in jarEntries) {
+ maybeAddEntry(entry) {
+ zos.addRawArchiveEntry(entry, jar.getRawInputStream(entry))
+ }
+ }
+ }
+ }
+
+
+
+ private fun okToAdd(name: String) : Boolean {
+ val result = !KFiles.isExcluded(name,
+ Glob("META-INF/*.SF", "META-INF/*.DSA", "META-INF/*.RSA", MANIFEST_MF))
+// if (name.startsWith("META-INF")) println((if (result) "ADDING" else "NOT ADDING") + " $name")
+ return result
+ }
+
+ override fun close() = zos.close()
+
+ private fun addEntry(entry: ArchiveEntry, inputStream: FileInputStream?) {
+ zos.putArchiveEntry(entry)
+ inputStream?.use { ins ->
+ ins.copyTo(zos, 50 * 1024)
+ }
+ zos.closeArchiveEntry()
+ }
+
+ private val seen = hashSetOf()
+
+ private fun maybeAddEntry(entry: ArchiveEntry, action:() -> Unit) {
+ entry.name.let { name ->
+ if (!seen.contains(name) && okToAdd(name)) {
+ action()
+ }
+ seen.add(name)
+ }
+ }
+}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/War.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/War.kt
index 295a1987..978f21bf 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/War.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/War.kt
@@ -6,7 +6,7 @@ import com.beust.kobalt.glob
class War(override val project: Project, override var name: String = Archives.defaultArchiveName(project) + ".war")
: Jar(project, name), AttributeHolder {
init {
- include(from("src/main/webapp"),to(""), glob("**"))
+ include(from("src/main/webapp"), to(""), glob("**"))
include(from("kobaltBuild/classes"), to("WEB-INF/classes"), glob("**"))
}
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Zip.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Zip.kt
index 61237291..41957218 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Zip.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/archive/Zip.kt
@@ -1,22 +1,13 @@
package com.beust.kobalt.archive
-import com.beust.kobalt.Glob
-import com.beust.kobalt.IFileSpec
+import com.beust.kobalt.*
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.annotation.Directive
-import com.beust.kobalt.misc.From
-import com.beust.kobalt.misc.IncludedFile
-import com.beust.kobalt.misc.To
-open class Zip(open val project: Project, open var name: String = Archives.defaultArchiveName(project) + ".zip") {
+open class Zip(open val project: Project, open var name: String = Archives.defaultArchiveName(project) + ".zip",
+ open var fatJar: Boolean = false): AttributeHolder, IncludeFromTo() {
val excludes = arrayListOf()
- @Directive
- fun from(s: String) = From(s)
-
- @Directive
- fun to(s: String) = To(s)
-
@Directive
fun exclude(vararg files: String) {
files.forEach { excludes.add(Glob(it)) }
@@ -28,26 +19,9 @@ open class Zip(open val project: Project, open var name: String = Archives.defau
}
@Directive
- fun include(vararg files: String) {
- includedFiles.add(IncludedFile(files.map { IFileSpec.FileSpec(it) }))
+ open val attributes = arrayListOf(Pair("Manifest-Version", "1.0"))
+
+ override fun addAttribute(k: String, v: String) {
+ attributes.add(Pair(k, v))
}
-
- @Directive
- fun include(from: From, to: To, vararg specs: String) {
- includedFiles.add(IncludedFile(from, to, specs.map { IFileSpec.FileSpec(it) }))
- }
-
- @Directive
- fun include(from: From, to: To, vararg specs: IFileSpec.GlobSpec) {
- includedFiles.add(IncludedFile(from, to, listOf(*specs)))
- }
-
- /**
- * Prefix path to be removed from the zip file. For example, if you add "build/lib/a.jar" to the zip
- * file and the excludePrefix is "build/lib", then "a.jar" will be added at the root of the zip file.
- */
- val includedFiles = arrayListOf()
-
}
-
-
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ActorUtils.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ActorUtils.kt
index 42aac9e2..e9b315a5 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ActorUtils.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ActorUtils.kt
@@ -1,7 +1,6 @@
package com.beust.kobalt.internal
import com.beust.kobalt.api.IProjectAffinity
-import com.beust.kobalt.api.ISimpleAffinity
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
@@ -11,19 +10,13 @@ class ActorUtils {
* Return the plug-in actor with the highest affinity.
*/
fun selectAffinityActor(project: Project, context: KobaltContext, actors: List)
- = actors.maxBy { it.affinity(project, context) }
+ = actors.maxBy { it.affinity(project, context) }
/**
* Return all the plug-in actors with a non zero affinity sorted from the highest to the lowest.
*/
fun selectAffinityActors(project: Project, context: KobaltContext, actors: List)
= actors.filter { it.affinity(project, context) > 0 }
- .sortedByDescending { it.affinity(project, context) }
-
- /**
- * Return the plug-in actor with the highest affinity.
- */
- fun , A> selectAffinityActor(actors: List, arg: A) = actors.maxBy { it.affinity(arg) }
+ .sortedByDescending { it.affinity(project, context) }
}
-
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseJvmPlugin.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseJvmPlugin.kt
index f45b4565..e89fc9aa 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseJvmPlugin.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseJvmPlugin.kt
@@ -24,11 +24,14 @@ abstract class BaseJvmPlugin(open val configActor: ConfigActor) :
override fun accept(project: Project) = sourceFileCount(project) > 0
// IBuildConfigContributor
-
protected fun sourceFileCount(project: Project)
- = KFiles.findSourceFiles(project.directory, project.sourceDirectories, sourceSuffixes()).size
+ = KFiles.findSourceFiles(project.directory, project.sourceDirectories, sourceSuffixes()).size +
+ KFiles.findSourceFiles(project.directory, project.sourceDirectoriesTest, sourceSuffixes()).size
fun affinity(project: Project) = sourceFileCount(project)
+ // IDocContributor
+ open fun affinity(project: Project, context: KobaltContext) = sourceFileCount(project)
+
abstract fun sourceSuffixes() : List
}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseProjectRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseProjectRunner.kt
index 2dc77ca0..963255bd 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseProjectRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BaseProjectRunner.kt
@@ -1,5 +1,7 @@
package com.beust.kobalt.internal
+import com.beust.kobalt.TestResult
+import com.beust.kobalt.api.IBuildListener
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
import com.beust.kobalt.api.ProjectBuildStatus
@@ -25,9 +27,14 @@ abstract class BaseProjectRunner {
}
fun runBuildListenersForTask(project: Project, context: KobaltContext, taskName: String, start: Boolean,
- success: Boolean = false) {
+ success: Boolean = false, testResult: TestResult? = null) {
context.pluginInfo.buildListeners.forEach {
- if (start) it.taskStart(project, context, taskName) else it.taskEnd(project, context, taskName, success)
+ if (start) {
+ it.taskStart(project, context, taskName)
+ } else {
+ val info = IBuildListener.TaskEndInfo(success, testResult?.shortMessage, testResult?.longMessage)
+ it.taskEnd(project, context, taskName, info)
+ }
}
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BuildListeners.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BuildListeners.kt
index 1a8781a5..58d8eed8 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BuildListeners.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/BuildListeners.kt
@@ -11,7 +11,8 @@ import java.util.concurrent.ConcurrentHashMap
*/
class BuildListeners : IBuildListener, IBuildReportContributor {
class ProfilerInfo(val taskName: String, val durationMillis: Long)
- class ProjectInfo(val projectName: String, var durationMillis: Long = 0)
+ class ProjectInfo(val projectName: String, var durationMillis: Long = 0,
+ var shortMessage: String? = null, var longMessage: String? = null)
private val startTimes = ConcurrentHashMap()
private val timings = arrayListOf()
@@ -29,18 +30,21 @@ class BuildListeners : IBuildListener, IBuildReportContributor {
}
// IBuildListener
- override fun taskEnd(project: Project, context: KobaltContext, taskName: String, success: Boolean) {
+ override fun taskEnd(project: Project, context: KobaltContext, taskName: String, info: IBuildListener.TaskEndInfo) {
+ val success = info.success
if (! success) hasFailures = true
startTimes[taskName]?.let {
val taskTime = System.currentTimeMillis() - it
timings.add(ProfilerInfo(taskName, taskTime))
projectInfos[project.name]?.let {
- it.durationMillis += taskTime.toLong()
+ it.durationMillis += taskTime
+ if (info.shortMessage != null && it.shortMessage == null) it.shortMessage = info.shortMessage
+ if (info.longMessage != null && it.longMessage == null) it.longMessage = info.longMessage
}
}
}
- private val projectStatuses = arrayListOf>()
+ private val projectStatuses = arrayListOf>()
// IBuildListener
override fun projectStart(project: Project, context: KobaltContext) {
@@ -49,7 +53,9 @@ class BuildListeners : IBuildListener, IBuildReportContributor {
// IBuildListener
override fun projectEnd(project: Project, context: KobaltContext, status: ProjectBuildStatus) {
- projectStatuses.add(Pair(project, status))
+ val shortMessage = projectInfos[project.name]?.shortMessage
+ val statusText = status.toString() + (if (shortMessage != null) " ($shortMessage)" else "")
+ projectStatuses.add(Pair(project, statusText))
}
// IBuildReportContributor
@@ -70,10 +76,15 @@ class BuildListeners : IBuildListener, IBuildReportContributor {
}
+ // Calculate the longest short message so we can create a column long enough to contain it
+ val width = 12 + (projectInfos.values.map { it.shortMessage?.length ?: 0 }.maxBy { it } ?: 0)
+
fun col1(s: String) = String.format(" %1\$-30s", s)
- fun col2(s: String) = String.format(" %1\$-13s", s)
+ fun col2(s: String) = String.format(" %1\$-${width}s", s)
fun col3(s: String) = String.format(" %1\$-8s", s)
+
+
// Only print the build report if there is more than one project and at least one of them failed
if (timings.any()) {
// if (timings.size > 1 && hasFailures) {
@@ -83,7 +94,7 @@ class BuildListeners : IBuildListener, IBuildReportContributor {
table.append(AsciiArt.logBox(listOf(line), AsciiArt.bottomLeft2, AsciiArt.bottomRight2, indent = 10) + "\n")
projectStatuses.forEach { pair ->
val projectName = pair.first.name
- val cl = listOf(col1(projectName), col2(pair.second.toString()),
+ val cl = listOf(col1(projectName), col2(pair.second),
col3(formatMillisLeft(projectInfos[projectName]!!.durationMillis, 8)))
.joinToString(AsciiArt.verticalBar)
table.append(" " + AsciiArt.verticalBar + " " + cl + " " + AsciiArt.verticalBar + "\n")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/CompilerUtils.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/CompilerUtils.kt
index 04e19d86..758a10e9 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/CompilerUtils.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/CompilerUtils.kt
@@ -3,7 +3,6 @@ package com.beust.kobalt.internal
import com.beust.kobalt.TaskResult
import com.beust.kobalt.api.*
import com.beust.kobalt.maven.DependencyManager
-import com.beust.kobalt.maven.DependencyManager2
import com.beust.kobalt.maven.aether.Scope
import com.beust.kobalt.maven.dependency.FileDependency
import com.beust.kobalt.misc.KFiles
@@ -15,9 +14,7 @@ import java.util.*
/**
* Central place to compile files, used by plug-ins and non plug-ins.
*/
-class CompilerUtils @Inject constructor(val files: KFiles,
- val dependencyManager: DependencyManager,
- val dependencyManager2: DependencyManager2) {
+class CompilerUtils @Inject constructor(val files: KFiles, val dependencyManager: DependencyManager) {
class CompilerResult(val successResults: List, val failedResult: TaskResult?)
@@ -33,7 +30,7 @@ class CompilerUtils @Inject constructor(val files: KFiles,
}
val sourceFiles = KFiles.findSourceFiles(project.directory,
contributedSourceDirs.map { it.path }, compiler.sourceSuffixes)
- if (sourceFiles.size > 0) {
+ if (sourceFiles.isNotEmpty()) {
// TODO: createCompilerActionInfo recalculates the source files, only compute them
// once and pass them
val info = createCompilerActionInfo(project, context, compiler, isTest,
@@ -72,24 +69,18 @@ class CompilerUtils @Inject constructor(val files: KFiles,
: CompilerActionInfo {
copyResources(project, context, SourceSet.of(isTest))
- val fullClasspath = dependencyManager2.resolve(project, context, isTest, listOf(Scope.COMPILE, Scope.TEST))
+ val fullClasspath = dependencyManager.calculateDependencies(project, context,
+ scopes = if (isTest) {
+ listOf(Scope.COMPILE, Scope.COMPILEONLY, Scope.TEST)
+ } else {
+ listOf(Scope.COMPILE, Scope.COMPILEONLY)
+ })
-// if (project.name == "ktor-core" && isTest) {
-// val contains = fullClasspath.filter{it.id.contains("json")}
-// val fc1 =
-// if (isTest) dependencyManager.testDependencies(project, context)
-// else dependencyManager.dependencies(project, context)
-//
-// println("DONOTCOMMIT")
-// val d2 = dependencyManager2.resolve(project, context, isTest, listOf(Scope.COMPILE, Scope.TEST))
-// }
File(project.directory, buildDirectory.path).mkdirs()
// Remove all the excluded dependencies from the classpath
- var classpath = fullClasspath.filter {
- ! isDependencyExcluded(it, project.excludedDependencies)
- }
+ var classpath = fullClasspath
// The classpath needs to contain $buildDirectory/classes as well so that projects that contain
// multiple languages can use classes compiled by the compiler run before them.
@@ -133,7 +124,8 @@ class CompilerUtils @Inject constructor(val files: KFiles,
// depending on the compiler's ability, sourceFiles can actually contain a list of directories
// instead of individual source files.
val projectDirectory = File(project.directory)
- val sourceFiles = if (compiler.canCompileDirectories) {
+ val sourceFiles =
+ if (compiler.canCompileDirectories) {
allSourceDirectories.map { File(projectDirectory, it.path).path }
} else {
files.findRecursively(projectDirectory, allSourceDirectories,
@@ -181,7 +173,7 @@ class CompilerUtils @Inject constructor(val files: KFiles,
// Finally, alter the info with the compiler interceptors before returning it
val initialActionInfo = CompilerActionInfo(projectDirectory.path, classpath, allSources,
sourceSuffixes, buildDirectory, emptyList() /* the flags will be provided by flag contributors */,
- emptyList())
+ emptyList(), context.internalContext.forceRecompile)
val result = context.pluginInfo.compilerInterceptors.fold(initialActionInfo, { ai, interceptor ->
interceptor.intercept(project, context, ai)
})
@@ -201,30 +193,21 @@ class CompilerUtils @Inject constructor(val files: KFiles,
val outputDir = sourceSet.outputDir
val variantSourceDirs = context.variant.resourceDirectories(project, sourceSet)
- if (variantSourceDirs.size > 0) {
+ if (variantSourceDirs.isNotEmpty()) {
context.logger.log(project.name, 2, "Copying $sourceSet resources")
val absOutputDir = File(KFiles.joinDir(project.directory, project.buildDirectory, outputDir))
variantSourceDirs
.map { File(project.directory, it.path) }
.filter(File::exists)
.forEach {
- context.logger.log(project.name, 2, "Copying from $it to $absOutputDir")
- KFiles.copyRecursively(it, absOutputDir, deleteFirst = false)
+ context.logger.log(project.name, 2, "Copying from $it to $absOutputDir")
+ KFiles.copyRecursively(it, absOutputDir, replaceExisting = true)
}
} else {
context.logger.log(project.name, 2, "No resources to copy for $sourceSet")
}
}
-
- /**
- * Naïve implementation: just exclude all dependencies that start with one of the excluded dependencies.
- * Should probably make exclusion more generic (full on string) or allow exclusion to be specified
- * formally by groupId or artifactId.
- */
- private fun isDependencyExcluded(id: IClasspathDependency, excluded: List)
- = excluded.any { id.id.startsWith(it.id) }
-
fun sourceCompilerFlags(project: Project?, context: KobaltContext, info: CompilerActionInfo) : List {
val adapters = context.pluginInfo.compilerFlagContributors.map {
val closure = { project: Project, context: KobaltContext, currentFlags: List,
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DocUrl.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DocUrl.kt
index 8acab0ab..93010294 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DocUrl.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DocUrl.kt
@@ -2,7 +2,7 @@ package com.beust.kobalt.internal
class DocUrl {
companion object {
- private const val HOST = "http://beust.com/kobalt/"
+ private const val HOST = "https://beust.com/kobalt/"
private fun url(path: String) = HOST + path
val PUBLISH_PLUGIN_URL = url("plug-ins/index.html#publishing")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DynamicGraph.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DynamicGraph.kt
index 75f8403e..a3e26afd 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DynamicGraph.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/DynamicGraph.kt
@@ -1,18 +1,14 @@
package com.beust.kobalt.internal
-import com.beust.kobalt.AsciiTable
-import com.beust.kobalt.KobaltException
-import com.beust.kobalt.TaskResult
-import com.beust.kobalt.misc.NamedThreadFactory
-import com.beust.kobalt.misc.error
-import com.beust.kobalt.misc.kobaltLog
-import com.beust.kobalt.misc.warn
+import com.beust.kobalt.*
+import com.beust.kobalt.misc.*
import com.google.common.collect.HashMultimap
import java.lang.reflect.InvocationTargetException
import java.util.*
import java.util.concurrent.*
-open class TaskResult2(success: Boolean, errorMessage: String?, val value: T) : TaskResult(success, errorMessage) {
+open class TaskResult2(success: Boolean, testResult: TestResult? = null,
+ errorMessage: String? = null, val value: T) : TaskResult(success, testResult, errorMessage) {
override fun toString() = com.beust.kobalt.misc.toString("TaskResult", "value", value, "success", success)
}
@@ -65,15 +61,18 @@ class DynamicGraph {
}
}
- fun transitiveClosureGraph(roots: List, childrenFor: (T) -> List) : List>
- = roots.map { transitiveClosureGraph(it, childrenFor) }
+ fun transitiveClosureGraph(roots: List, childrenFor: (T) -> List,
+ filter: (T) -> Boolean): List>
+ = roots.map { transitiveClosureGraph(it, childrenFor, filter) }
- fun transitiveClosureGraph(root: T, childrenFor: (T) -> List, seen: HashSet = hashSetOf()) : Node {
+ fun transitiveClosureGraph(root: T, childrenFor: (T) -> List,
+ filter: (T) -> Boolean = { t: T -> true },
+ seen: HashSet = hashSetOf()) : Node {
val children = arrayListOf>()
- childrenFor(root).forEach { child ->
+ childrenFor(root).filter(filter).forEach { child ->
if (! seen.contains(child)) {
seen.add(child)
- val c = transitiveClosureGraph(child, childrenFor, seen)
+ val c = transitiveClosureGraph(child, childrenFor, filter, seen)
children.add(c)
}
}
@@ -300,14 +299,14 @@ class DynamicGraphExecutor(val graph : DynamicGraph, val factory: IThreadW
duration = " (" + ((hl.timestamp - start) / 1000)
.toInt().toString() + ")"
} else {
- println("DONOTCOMMIT")
+ kobaltLog(1, "DONOTCOMMIT")
}
}
return hl.name + duration
}
historyLog.forEach { hl ->
- println("CURRENT LOG: " + currentLog + " HISTORY LINE: " + hl)
+ kobaltLog(1, "CURRENT LOG: " + currentLog + " HISTORY LINE: " + hl)
if (hl.start) {
projectStart[hl.name] = hl.timestamp
}
@@ -315,10 +314,10 @@ class DynamicGraphExecutor(val graph : DynamicGraph, val factory: IThreadW
currentLog = CompressedLog(hl.timestamp, hashMapOf(hl.threadId to hl.name))
} else currentLog?.let { cl ->
if (! hl.start || hl.timestamp - cl.timestamp < 1000) {
- println(" CURRENT LOG IS WITHING ONE SECOND: $hl")
+ kobaltLog(1, " CURRENT LOG IS WITHING ONE SECOND: $hl")
cl.threadMap[hl.threadId] = toName(hl)
} else {
- println(" ADDING COMPRESSED LINE $cl")
+ kobaltLog(1, " ADDING COMPRESSED LINE $cl")
compressed.add(cl)
currentLog = CompressedLog(hl.timestamp, hashMapOf(hl.threadId to toName(hl)))
}
@@ -341,37 +340,41 @@ class DynamicGraphExecutor(val graph : DynamicGraph, val factory: IThreadW
fun displayRegularLog(table: AsciiTable.Builder) : AsciiTable.Builder {
if (historyLog.any()) {
- val start = historyLog[0].timestamp
- val projectStart = ConcurrentHashMap()
- historyLog.forEach { line ->
- val row = arrayListOf()
- row.add(toSeconds(line.timestamp - start))
- threadIds.keys.forEach {
- if (line.threadId == it) {
- var duration = ""
- if (line.start) {
- projectStart[line.name] = line.timestamp
- } else {
- val projectStart = projectStart[line.name]
- if (projectStart != null) {
- duration = " (" + ((line.timestamp - projectStart) / 1000)
- .toInt().toString() + ")"
+ if (historyLog[0] != null) {
+ val start = historyLog[0].timestamp
+ val projectStart = ConcurrentHashMap()
+ historyLog.forEach { line ->
+ val row = arrayListOf()
+ row.add(toSeconds(line.timestamp - start))
+ threadIds.keys.forEach {
+ if (line.threadId == it) {
+ var duration = ""
+ if (line.start) {
+ projectStart[line.name] = line.timestamp
} else {
- warn("Couldn't determine project start: " + line.name)
+ val projectStart = projectStart[line.name]
+ if (projectStart != null) {
+ duration = " (" + ((line.timestamp - projectStart) / 1000)
+ .toInt().toString() + ")"
+ } else {
+ warn("Couldn't determine project start: " + line.name)
+ }
}
+ row.add((line.name + duration))
+ } else {
+ row.add("")
}
- row.add((line.name + duration))
- } else {
- row.add("")
}
+ table.addRow(row)
}
- table.addRow(row)
+ } else {
+ warn("Couldn't find historyLog")
}
}
return table
}
- println(displayRegularLog(table).build())
+ kobaltLog(1, displayRegularLog(table).build())
}
}
@@ -391,7 +394,7 @@ fun main(argv: Array) {
object: IWorker {
override fun call(): TaskResult2? {
kobaltLog(1, " Running worker $it")
- return TaskResult2(true, null, it)
+ return TaskResult2(true, value = it)
}
override val priority: Int get() = 0
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GenericRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GenericRunner.kt
index f35b6e26..995dba53 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GenericRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GenericRunner.kt
@@ -4,6 +4,7 @@ import com.beust.kobalt.*
import com.beust.kobalt.api.*
import com.beust.kobalt.misc.KFiles
import com.google.common.annotations.VisibleForTesting
+import com.google.inject.Inject
import java.io.File
import java.util.*
@@ -15,14 +16,27 @@ abstract class GenericTestRunner: ITestRunnerContributor {
abstract val dependencyName : String
abstract val mainClass: String
abstract val annotationPackage: String
+ abstract val runnerName: String
+ open var shortMessage: String? = null
+ open var longMessage: String? = null
+
+ @Inject
+ private lateinit var jvm: Jvm
+
abstract fun args(project: Project, context: KobaltContext, classpath: List,
testConfig: TestConfig) : List
- open fun filterTestClasses(classes: List) : List = classes
+ open fun onFinish(project: Project) {}
+
+ open val extraClasspath: List = emptyList()
+
+ open fun filterTestClasses(project: Project, context: KobaltContext, classes: List) : List = classes
override fun run(project: Project, context: KobaltContext, configName: String,
- classpath: List)
- = TaskResult(runTests(project, context, classpath, configName))
+ classpath: List) : TaskResult {
+ val tr = runTests(project, context, classpath, configName)
+ return TaskResult(tr.success, testResult = tr)
+ }
override fun affinity(project: Project, context: KobaltContext) : Int {
val result =
@@ -55,7 +69,7 @@ abstract class GenericTestRunner: ITestRunnerContributor {
// }
context.logger.log(project.name, 2, "Found ${result.size} test classes")
- return filterTestClasses(result.map { it.second })
+ return filterTestClasses(project, context, result.map { it.second })
}
/**
@@ -94,17 +108,20 @@ abstract class GenericTestRunner: ITestRunnerContributor {
/**
* @return true if all the tests passed
*/
- fun runTests(project: Project, context: KobaltContext, classpath: List,
- configName: String) : Boolean {
+ open fun runTests(project: Project, context: KobaltContext, classpath: List,
+ configName: String) : TestResult {
var result = false
+ context.logger.log(project.name, 1, "Running tests with $runnerName")
+
val testConfig = project.testConfigs.firstOrNull { it.name == configName }
+ var errorCode = -1
if (testConfig != null) {
val args = args(project, context, classpath, testConfig)
if (args.size > 0) {
- val java = JavaInfo.create(File(SystemProperties.javaBase)).javaExecutable
+ val java = jvm.javaExecutable
val jvmArgs = calculateAllJvmArgs(project, context, testConfig, classpath,
Kobalt.INJECTOR.getInstance (PluginInfo::class.java))
val allArgs = arrayListOf().apply {
@@ -120,12 +137,7 @@ abstract class GenericTestRunner: ITestRunnerContributor {
context.logger.log(project.name, 2, "Running tests with classpath size ${classpath.size}")
context.logger.log(project.name, 2, "Launching " + allArgs.joinToString(" "))
val process = pb.start()
- val errorCode = process.waitFor()
- if (errorCode == 0) {
- context.logger.log(project.name, 1, "All tests passed")
- } else {
- context.logger.log(project.name, 1, "Test failures")
- }
+ errorCode = process.waitFor()
result = result || errorCode == 0
} else {
context.logger.log(project.name, 1, " No tests to run")
@@ -134,7 +146,16 @@ abstract class GenericTestRunner: ITestRunnerContributor {
} else {
throw KobaltException("Couldn't find a test configuration named \"$configName\"")
}
- return result
+
+ onFinish(project)
+
+ if (errorCode == 0) {
+ context.logger.log(project.name, 1, "All tests passed")
+ } else {
+ context.logger.log(project.name, 1, longMessage!!)
+ }
+
+ return TestResult(result, shortMessage, longMessage)
}
/*
@@ -143,12 +164,13 @@ abstract class GenericTestRunner: ITestRunnerContributor {
@VisibleForTesting
fun calculateAllJvmArgs(project: Project, context: KobaltContext,
testConfig: TestConfig, classpath: List, pluginInfo: IPluginInfo) : List {
+ val fullClasspath = classpath.map { it.jarFile.get().absolutePath } + extraClasspath
// Default JVM args
val jvmFlags = arrayListOf().apply {
addAll(testConfig.jvmArgs)
add("-ea")
add("-classpath")
- add(classpath.map { it.jarFile.get().absolutePath }.joinToString(File.pathSeparator))
+ add(fullClasspath.joinToString(File.pathSeparator))
}
// JVM flags from the contributors
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GraphUtil.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GraphUtil.kt
new file mode 100644
index 00000000..6cef8ada
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/GraphUtil.kt
@@ -0,0 +1,35 @@
+package com.beust.kobalt.internal
+
+/**
+ * Generic operations on graph-like structures.
+ */
+object GraphUtil {
+ /**
+ * Apply the operation in `closure` to all the nodes in the tree.
+ */
+ fun map(roots: List, children: (T) -> List, closure: (T) -> Unit) {
+ roots.forEach {
+ closure(it)
+ map(children(it), children, closure)
+ }
+ }
+
+ /**
+ * Display each node in the roots by calling the `display` function on each of them.
+ */
+ fun displayGraph(roots: List,
+ children: (T) -> List,
+ display: (node: T, indent: String) -> Unit) {
+
+ fun pd(node: T, indent: String) {
+ display(node, indent)
+ children(node).forEach {
+ pd(it, indent + " ")
+ }
+ }
+ roots.forEach {
+ pd(it, "")
+ }
+ }
+}
+
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnit5Runner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnit5Runner.kt
new file mode 100644
index 00000000..2e9b534c
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnit5Runner.kt
@@ -0,0 +1,152 @@
+package com.beust.kobalt.internal
+
+import com.beust.jcommander.JCommander
+import com.beust.jcommander.Parameter
+import com.beust.kobalt.TestConfig
+import com.beust.kobalt.api.IAffinity
+import com.beust.kobalt.api.IClasspathDependency
+import com.beust.kobalt.api.KobaltContext
+import com.beust.kobalt.api.Project
+import com.beust.kobalt.misc.KFiles
+import com.beust.kobalt.misc.KobaltLogger
+import com.google.inject.Inject
+import org.junit.platform.engine.TestExecutionResult
+import org.junit.platform.engine.discovery.DiscoverySelectors
+import org.junit.platform.engine.reporting.ReportEntry
+import org.junit.platform.engine.support.descriptor.MethodSource
+import org.junit.platform.launcher.LauncherDiscoveryRequest
+import org.junit.platform.launcher.TestExecutionListener
+import org.junit.platform.launcher.TestIdentifier
+import org.junit.platform.launcher.TestPlan
+import org.junit.platform.launcher.core.LauncherDiscoveryRequestBuilder
+import org.junit.platform.launcher.core.LauncherFactory
+import java.io.File
+import java.nio.file.Paths
+
+/**
+ * Runner for JUnit 5 tests. This class also contains a main() entry point since JUnit 5 no longer supplies one.
+ */
+class JUnit5Runner @Inject constructor(kFiles: KFiles) : GenericTestRunner() {
+
+ override val dependencyName = "jupiter"
+ override val annotationPackage = "org.junit.jupiter.api"
+ override val mainClass = "com.beust.kobalt.internal.JUnit5RunnerKt"
+ override val runnerName = "JUnit 5"
+
+ override fun affinity(project: Project, context: KobaltContext) : Int {
+ val result =
+ if (project.testDependencies.any { it.id.contains("junit5") || it.id.contains("jupiter") })
+ IAffinity.DEFAULT_POSITIVE_AFFINITY + 100
+ else 0
+ return result
+
+ }
+
+ override fun args(project: Project, context: KobaltContext, classpath: List, testConfig: TestConfig): List {
+ val testClassDir = KFiles.joinDir(project.buildDirectory, KFiles.TEST_CLASSES_DIR)
+ val classDir = KFiles.joinDir(project.buildDirectory, KFiles.CLASSES_DIR)
+ val args = listOf("--testClassDir", testClassDir,
+ "--classDir", classDir,
+ "--log", KobaltLogger.LOG_LEVEL.toString())
+ return args
+ }
+
+ override val extraClasspath = kFiles.kobaltJar
+}
+
+private class Args {
+ @Parameter(names = arrayOf("--log"))
+ var log: Int = 1
+
+ @Parameter(names = arrayOf("--testClassDir"))
+ var testClassDir: String = "kobaltBuild/test-classes"
+
+ @Parameter(names = arrayOf("--classDir"))
+ var classDir: String = "kobaltBuild/classes"
+}
+
+fun main(argv: Array) {
+ val args = Args()
+ val jc = JCommander(args)
+ jc.parse(*argv)
+
+ val testClassDir = File(args.testClassDir).absolutePath
+ val classDir = File(args.classDir).absolutePath
+ val request : LauncherDiscoveryRequest = LauncherDiscoveryRequestBuilder()
+ .selectors(DiscoverySelectors.selectClasspathRoots(setOf(
+ Paths.get(testClassDir),
+ Paths.get(classDir)
+ )))
+ .selectors(DiscoverySelectors.selectDirectory(testClassDir))
+ .build()
+
+ fun testName(id: TestIdentifier) : String? {
+ val result =
+ if (id.source.isPresent) {
+ val source = id.source.get()
+ if (source is MethodSource) {
+ source.className + "." + source.methodName
+ } else {
+ null
+ }
+ } else {
+ null
+ }
+ return result
+ }
+
+ var passed = 0
+ var failed = 0
+ var skipped = 0
+ var aborted = 0
+
+ fun log(level: Int, s: String) {
+ if (level <= args.log) println(s)
+ }
+
+ val listener = object: TestExecutionListener {
+ override fun executionFinished(testIdentifier: TestIdentifier, testExecutionResult: TestExecutionResult) {
+ val testName = testName(testIdentifier)
+ if (testName != null) {
+ when(testExecutionResult.status) {
+ TestExecutionResult.Status.FAILED -> {
+ log(1, "FAILED: $testName, reason: " + testExecutionResult.throwable.get().toString())
+ failed++
+ }
+ TestExecutionResult.Status.ABORTED -> {
+ log(1, "ABORTED: $testName, reason: " + testExecutionResult.throwable.get().toString())
+ aborted++
+ }
+ TestExecutionResult.Status.SUCCESSFUL -> {
+ log(2, "PASSED: $testName")
+ passed++
+ } else -> {
+
+ }
+ }
+ }
+ }
+
+ override fun executionSkipped(testIdentifier: TestIdentifier, reason: String) {
+ testName(testIdentifier)?.let {
+ log(1, "Skipping $it because $reason")
+ skipped++
+ }
+ }
+
+ override fun executionStarted(testIdentifier: TestIdentifier) {
+ testName(testIdentifier)?.let {
+ log(2, "Starting $it")
+ }
+ }
+
+ override fun testPlanExecutionStarted(testPlan: TestPlan?) {}
+ override fun dynamicTestRegistered(testIdentifier: TestIdentifier?) {}
+ override fun reportingEntryPublished(testIdentifier: TestIdentifier?, entry: ReportEntry?) {}
+ override fun testPlanExecutionFinished(testPlan: TestPlan?) {}
+ }
+
+ LauncherFactory.create().execute(request, listener)
+
+ log(1, "TEST RESULTS: $passed PASSED, $failed FAILED, $skipped SKIPPED, $aborted ABORTED")
+}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnitRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnitRunner.kt
index eec22afd..c5b36997 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnitRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JUnitRunner.kt
@@ -4,16 +4,29 @@ import com.beust.kobalt.TestConfig
import com.beust.kobalt.api.IClasspathDependency
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
+import com.beust.kobalt.maven.DependencyManager
+import com.google.inject.Inject
+import java.lang.reflect.Modifier
+import java.net.URLClassLoader
open class JUnitRunner() : GenericTestRunner() {
override val mainClass = "org.junit.runner.JUnitCore"
-
override val annotationPackage = "org.junit"
-
override val dependencyName = "junit"
+ override val runnerName = "JUnit 4"
override fun args(project: Project, context: KobaltContext, classpath: List,
testConfig: TestConfig) = findTestClasses(project, context, testConfig)
+
+ @Inject
+ lateinit var dependencyManager: DependencyManager
+
+ override fun filterTestClasses(project: Project, context: KobaltContext, classes: List) : List {
+ val deps = dependencyManager.testDependencies(project, context)
+ val cl = URLClassLoader(deps.map { it.jarFile.get().toURI().toURL() }.toTypedArray())
+ return classes.filter { !Modifier.isAbstract(cl.loadClass(it).modifiers) }
+ }
+
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompiler.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompiler.kt
index 25294934..e7773737 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompiler.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompiler.kt
@@ -23,12 +23,11 @@ class JvmCompiler @Inject constructor(val dependencyManager: DependencyManager)
flags: List): TaskResult {
// Dependencies
- val allDependencies = (info.dependencies
- + dependencyManager.calculateDependencies(project, context!!, passedDependencies = info.dependencies))
+ val allDependencies = (info.dependencies + dependencyManager.calculateDependencies(project, context!!,
+ passedDependencies = info.dependencies))
.distinct()
// Plugins that add flags to the compiler
- val currentFlags = arrayListOf().apply { addAll(info.compilerArgs) }
val contributorFlags : List = if (project != null) flags else emptyList()
val addedFlags = contributorFlags + ArrayList(info.compilerArgs)
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompilerPlugin.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompilerPlugin.kt
index 6cf364e2..5e2a9354 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompilerPlugin.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/JvmCompilerPlugin.kt
@@ -9,9 +9,8 @@ import com.beust.kobalt.api.annotation.ExportedProjectProperty
import com.beust.kobalt.api.annotation.IncrementalTask
import com.beust.kobalt.api.annotation.Task
import com.beust.kobalt.maven.DependencyManager
-import com.beust.kobalt.maven.DependencyManager2
-import com.beust.kobalt.maven.LocalRepo
import com.beust.kobalt.maven.Md5
+import com.beust.kobalt.maven.aether.Scope
import com.beust.kobalt.misc.KFiles
import com.beust.kobalt.misc.KobaltExecutors
import com.beust.kobalt.misc.error
@@ -22,15 +21,13 @@ import javax.inject.Inject
import javax.inject.Singleton
/**
- * This plug-in takes care of compilation: it declares a bunch of tasks ("compile", "compileTest") and
+ * This plug-in takes care of compilation: it declares several common tasks ("compile", "compileTest")
* and picks up all the compiler contributors in order to run them whenever a compilation is requested.
*/
@Singleton
open class JvmCompilerPlugin @Inject constructor(
- open val localRepo: LocalRepo,
open val files: KFiles,
open val dependencyManager: DependencyManager,
- open val dependencyManager2: DependencyManager2,
open val executors: KobaltExecutors,
open val taskContributor : TaskContributor,
val compilerUtils: CompilerUtils)
@@ -88,9 +85,13 @@ open class JvmCompilerPlugin @Inject constructor(
context.pluginInfo.testRunnerContributors)
if (testContributor != null && testContributor.affinity(project, context) > 0) {
// val td1 = dependencyManager.testDependencies(project, context)
- val testDependencies = dependencyManager2.resolve(project, context, isTest = true)
- val compileDependencies = dependencyManager2.resolve(project, context, isTest = false)
- return testContributor.run(project, context, configName, testDependencies + compileDependencies)
+ val testDependencies = dependencyManager.calculateDependencies(project, context,
+ dependencyFilter = dependencyManager.createDependencyFilter(project, project.testDependencies),
+ scopes = listOf(Scope.TEST))
+ val compileDependencies = dependencyManager.calculateDependencies(project, context,
+ scopes = listOf(Scope.COMPILE, Scope.COMPILEONLY))
+ val allDependencies = (testDependencies + compileDependencies).distinct()
+ return testContributor.run(project, context, configName, allDependencies.toList())
} else {
context.logger.log(project.name, 2,
"Couldn't find a test runner for project ${project.name}, did you specify dependenciesTest{}?")
@@ -156,6 +157,10 @@ open class JvmCompilerPlugin @Inject constructor(
if (compilerContributors.isEmpty()) {
throw KobaltException("Couldn't find any compiler for project ${project.name}")
} else {
+
+ // Generate BuildConfig if applicable
+ context.variant.maybeGenerateBuildConfig(project, context)
+
val allCompilers = compilerContributors.flatMap { it.compilersFor(project, context)}.sorted()
/**
@@ -169,7 +174,10 @@ open class JvmCompilerPlugin @Inject constructor(
if (wi.value.sourceSuffixes.contains("java")) ij = wi.index
if (wi.value.sourceSuffixes.contains("kt")) ik = wi.index
}
- Collections.swap(result, ik, ij)
+
+ if (ik >= 0 && ij >= 0) {
+ Collections.swap(result, ik, ij)
+ }
return result
}
@@ -179,8 +187,8 @@ open class JvmCompilerPlugin @Inject constructor(
var done = false
// The directory where the classes get compiled
val buildDirectory =
- if (isTest) File(project.buildDirectory, KFiles.TEST_CLASSES_DIR)
- else File(project.classesDir(context))
+ if (isTest) File(KFiles.joinDir(project.buildDirectory, KFiles.TEST_CLASSES_DIR))
+ else File(KFiles.joinDir(project.classesDir(context)))
allCompilersSorted.doWhile({ ! done }) { compiler ->
val compilerResults = compilerUtils.invokeCompiler(project, context, compiler,
@@ -217,7 +225,8 @@ open class JvmCompilerPlugin @Inject constructor(
}
}
- @Task(name = "doc", description = "Generate the documentation for the project", group = GROUP_DOCUMENTATION)
+ @Task(name = "doc", description = "Generate the documentation for the project", group = GROUP_DOCUMENTATION,
+ runBefore = arrayOf("assemble"), runAfter = arrayOf("clean"))
fun taskJavadoc(project: Project): TaskResult {
val docGenerator = ActorUtils.selectAffinityActor(project, context, context.pluginInfo.docContributors)
if (docGenerator != null) {
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltPluginXml.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltPluginXml.kt
index 2b36f2e7..d8ca3555 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltPluginXml.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltPluginXml.kt
@@ -72,13 +72,13 @@ class PluginInfo(val xml: KobaltPluginXml, val pluginClassLoader: ClassLoader?,
val plugins = arrayListOf()
val projectContributors = arrayListOf()
val classpathContributors = arrayListOf()
- val initContributors = arrayListOf()
+ val templateContributors = arrayListOf()
val repoContributors = arrayListOf()
val compilerFlagContributors = arrayListOf()
val compilerInterceptors = arrayListOf()
val sourceDirectoriesInterceptors = arrayListOf()
val buildDirectoryInterceptors = arrayListOf()
- val runnerContributors = arrayListOf()
+// val runnerContributors = arrayListOf()
val testRunnerContributors = arrayListOf()
val classpathInterceptors = arrayListOf()
val compilerContributors = arrayListOf()
@@ -172,9 +172,8 @@ class PluginInfo(val xml: KobaltPluginXml, val pluginClassLoader: ClassLoader?,
}
val result = loadClass(className, classLoader)
- ?: Class.forName(className)
?: loadClass(className, pluginClassLoader)
- ?: throw ClassNotFoundException(className)
+ ?: Class.forName(className)
return result
}
@@ -194,11 +193,11 @@ class PluginInfo(val xml: KobaltPluginXml, val pluginClassLoader: ClassLoader?,
if (this is ICompilerFlagContributor) compilerFlagContributors.add(this)
if (this is ICompilerInterceptor) compilerInterceptors.add(this)
if (this is IDocContributor) docContributors.add(this)
- if (this is ITemplateContributor) initContributors.add(this)
+ if (this is ITemplateContributor) templateContributors.add(this)
if (this is IPlugin) plugins.add(this)
if (this is IProjectContributor) projectContributors.add(this)
if (this is IRepoContributor) repoContributors.add(this)
- if (this is IRunnerContributor) runnerContributors.add(this)
+// if (this is IRunnerContributor) runnerContributors.add(this)
if (this is ISourceDirectoryContributor) sourceDirContributors.add(this)
if (this is ISourceDirectoryInterceptor) sourceDirectoriesInterceptors.add(this)
if (this is ITaskContributor) taskContributors.add(this)
@@ -223,10 +222,10 @@ class PluginInfo(val xml: KobaltPluginXml, val pluginClassLoader: ClassLoader?,
}
fun cleanUp() {
- listOf(projectContributors, classpathContributors, initContributors,
+ listOf(projectContributors, classpathContributors, templateContributors,
repoContributors, compilerFlagContributors, compilerInterceptors,
sourceDirectoriesInterceptors, buildDirectoryInterceptors,
- runnerContributors, testRunnerContributors, classpathInterceptors,
+ /* runnerContributors, */ testRunnerContributors, classpathInterceptors,
compilerContributors, docContributors, sourceDirContributors,
testSourceDirContributors, buildConfigFieldContributors,
taskContributors, incrementalTaskContributors, assemblyContributors,
@@ -247,13 +246,13 @@ class PluginInfo(val xml: KobaltPluginXml, val pluginClassLoader: ClassLoader?,
plugins.addAll(pluginInfo.plugins)
classpathContributors.addAll(pluginInfo.classpathContributors)
projectContributors.addAll(pluginInfo.projectContributors)
- initContributors.addAll(pluginInfo.initContributors)
+ templateContributors.addAll(pluginInfo.templateContributors)
repoContributors.addAll(pluginInfo.repoContributors)
compilerFlagContributors.addAll(pluginInfo.compilerFlagContributors)
compilerInterceptors.addAll(pluginInfo.compilerInterceptors)
sourceDirectoriesInterceptors.addAll(pluginInfo.sourceDirectoriesInterceptors)
buildDirectoryInterceptors.addAll(pluginInfo.buildDirectoryInterceptors)
- runnerContributors.addAll(pluginInfo.runnerContributors)
+// runnerContributors.addAll(pluginInfo.runnerContributors)
testRunnerContributors.addAll(pluginInfo.testRunnerContributors)
classpathInterceptors.addAll(pluginInfo.classpathInterceptors)
compilerContributors.addAll(pluginInfo.compilerContributors)
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltSettingsXml.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltSettingsXml.kt
index 387c8a59..2eb5374c 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltSettingsXml.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KobaltSettingsXml.kt
@@ -25,7 +25,7 @@ class KobaltSettingsXml {
@XmlElement(name = "localMavenRepo") @JvmField
var localMavenRepo: String = homeDir(KFiles.KOBALT_DOT_DIR, "localMavenRepo")
- @XmlElement(name = "defaulRepos") @JvmField
+ @XmlElement(name = "defaultRepos") @JvmField
var defaultRepos: DefaultReposXml? = null
@XmlElement(name = "proxies") @JvmField
@@ -42,6 +42,9 @@ class KobaltSettingsXml {
@XmlElement(name = "kobaltCompilerSeparateProcess") @JvmField
var kobaltCompilerSeparateProcess: Boolean = false
+
+ @XmlElement(name = "autoUpdate") @JvmField
+ var autoUpdate: Boolean = false
}
class ProxiesXml {
@@ -85,6 +88,11 @@ class KobaltSettings @Inject constructor(val xmlFile: KobaltSettingsXml) {
*/
val localMavenRepo = KFiles.makeDir(xmlFile.localMavenRepo)
+ /**
+ * If true, Kobalt will automatically update itself if a new version is found.
+ */
+ val autoUpdate = xmlFile.autoUpdate
+
/**
* If true, the Kotlin compiler will always be launched in a separate JVM, even if the requested
* version is the same as the internal version.
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinJarFiles.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinJarFiles.kt
index 3f29427a..123e8b76 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinJarFiles.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinJarFiles.kt
@@ -16,6 +16,5 @@ class KotlinJarFiles @Inject constructor(val dependencyManager: DependencyManage
}
val stdlib: File get() = getKotlinCompilerJar("stdlib")
- val runtime: File get() = getKotlinCompilerJar("runtime")
val compiler: File get() = getKotlinCompilerJar("compiler-embeddable")
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinTestRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinTestRunner.kt
index 90c5d354..24e643d5 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinTestRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/KotlinTestRunner.kt
@@ -1,16 +1,21 @@
package com.beust.kobalt.internal
+import com.beust.kobalt.api.KobaltContext
+import com.beust.kobalt.api.Project
+
/**
* KotlinTestRunner triggers if it finds a dependency on io.kotlintest but other than that, it just
* uses the regular JUnitRunner.
*/
class KotlinTestRunner : JUnitRunner() {
override val dependencyName = "io.kotlintest"
+ override val runnerName = "Kotlin Test"
/**
* KotlinTestRunner runs tests in the init{} initializer, so ignore all the extra
* classes generated by the Kotlin compiler.
*/
- override fun filterTestClasses(classes: List) = classes.filter { ! it.contains("$") }
+ override fun filterTestClasses(projet: Project, context: KobaltContext, classes: List)
+ = classes.filter { !it.contains("$") }
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelLogger.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelLogger.kt
index afaa7e21..519a94c9 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelLogger.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelLogger.kt
@@ -2,9 +2,7 @@ package com.beust.kobalt.internal
import com.beust.kobalt.Args
import com.beust.kobalt.KobaltException
-import com.beust.kobalt.misc.kobaltError
-import com.beust.kobalt.misc.kobaltLog
-import com.beust.kobalt.misc.kobaltWarn
+import com.beust.kobalt.misc.*
import com.google.inject.Inject
import com.google.inject.Singleton
import java.util.*
@@ -67,9 +65,9 @@ class ParallelLogger @Inject constructor(val args: Args) : ILogger {
}
private fun debug(s: CharSequence) {
- if (args.log >= 2) {
+ if (args.log >= 3) {
val time = System.currentTimeMillis() - startTime!!
- println(" ### [$time] $s")
+ kobaltLog(1, " ### [$time] $s")
}
}
@@ -127,6 +125,6 @@ class ParallelLogger @Inject constructor(val args: Args) : ILogger {
runningProjects.forEach {
emptyProjectLog(it)
}
- println("")
+ kobaltLog(1, "")
}
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelProjectRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelProjectRunner.kt
index 7041c402..d98f0d8a 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelProjectRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/ParallelProjectRunner.kt
@@ -54,12 +54,12 @@ class ParallelProjectRunner(val tasksByNames: (Project) -> ListMultimap ListMultimap ListMultimap().apply {
projects.forEach { project ->
addNode(ProjectTask(project, args.dryRun))
- project.dependsOn.forEach {
+ project.allProjectDependedOn().forEach {
addEdge(ProjectTask(project, args.dryRun), ProjectTask(it, args.dryRun))
}
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SequentialProjectRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SequentialProjectRunner.kt
index 1f13626c..ec99b723 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SequentialProjectRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SequentialProjectRunner.kt
@@ -39,7 +39,7 @@ class SequentialProjectRunner(val tasksByNames: (Project) -> ListMultimap 0) {
klog(2, "Marking project $projectName as skipped")
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SpekRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SpekRunner.kt
index 2129a315..0c1ddad6 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SpekRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/SpekRunner.kt
@@ -6,5 +6,6 @@ package com.beust.kobalt.internal
*/
class SpekRunner : JUnitRunner() {
override val dependencyName = "org.jetbrains.spek"
+ override val runnerName = "Spek"
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TaskManager.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TaskManager.kt
index 2139326c..99aa7624 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TaskManager.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TaskManager.kt
@@ -17,7 +17,7 @@ import javax.inject.Singleton
@Singleton
class TaskManager @Inject constructor(val args: Args,
val incrementalManagerFactory: IncrementalManager.IFactory,
- val pluginInfo: PluginInfo, val kobaltLog: ParallelLogger) {
+ val kobaltLog: ParallelLogger) {
private val dependsOn = TreeMultimap.create()
private val reverseDependsOn = TreeMultimap.create()
private val runBefore = TreeMultimap.create()
@@ -80,6 +80,9 @@ class TaskManager @Inject constructor(val args: Args,
}
}
+// @Inject
+// lateinit var pluginInfo: PluginInfo
+
fun runTargets(passedTaskNames: List, allProjects: List): RunTargetResult {
// Check whether tasks passed at command line exist
passedTaskNames.forEach {
@@ -87,6 +90,7 @@ class TaskManager @Inject constructor(val args: Args,
throw KobaltException("Unknown task: $it")
}
+ val pluginInfo = Kobalt.INJECTOR.getInstance(PluginInfo::class.java)
var taskInfos = calculateDependentTaskNames(passedTaskNames, allProjects)
// Remove non existing tasks (e.g. dynamic task defined for a single project)
@@ -145,7 +149,7 @@ class TaskManager @Inject constructor(val args: Args,
val topological = Topological().apply {
projects.forEach { project ->
addNode(project)
- project.dependsOn.forEach {
+ project.allProjectDependedOn().forEach {
addEdge(project, it)
}
}
@@ -160,7 +164,7 @@ class TaskManager @Inject constructor(val args: Args,
return result
} else {
val rootProject = projects.find { it.name == ti.project }!!
- val allProjects = DynamicGraph.transitiveClosure(rootProject, { p -> p.dependsOn })
+ val allProjects = DynamicGraph.transitiveClosure(rootProject, Project::allProjectDependedOn)
val sortedProjects = sortProjectsTopologically(allProjects)
val sortedMaps = sortedProjects.map { TaskInfo(it.name, "compile")}
val result = sortedMaps.subList(0, sortedMaps.size - 1) + listOf(ti)
@@ -268,7 +272,8 @@ class TaskManager @Inject constructor(val args: Args,
object : BasePluginTask(plugin, name, description, group, project) {
override fun call(): TaskResult2 {
val taskResult = task(project)
- return TaskResult2(taskResult.success, taskResult.errorMessage, this)
+ return TaskResult2(taskResult.success, errorMessage = taskResult.errorMessage, value = this,
+ testResult = taskResult.testResult)
}
})
dependsOn.forEach { dependsOn(it, name) }
@@ -315,9 +320,11 @@ class TaskWorker(val tasks: List, val dryRun: Boolean, val pluginInfo: Pl
val tr = if (dryRun) TaskResult() else it.call()
BaseProjectRunner.runBuildListenersForTask(it.project, context, name, start = false, success = tr.success)
success = success and tr.success
- if (tr.errorMessage != null) errorMessages.add(tr.errorMessage)
+ tr.errorMessage?.let {
+ errorMessages.add(it)
+ }
}
- return TaskResult2(success, errorMessages.joinToString("\n"), tasks[0])
+ return TaskResult2(success, errorMessage = errorMessages.joinToString("\n"), value = tasks[0])
}
// override val timeOut : Long = 10000
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TestNgRunner.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TestNgRunner.kt
index 008f2291..f4ee96f8 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TestNgRunner.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/TestNgRunner.kt
@@ -1,28 +1,53 @@
package com.beust.kobalt.internal
+import com.beust.kobalt.AsciiArt
import com.beust.kobalt.TestConfig
+import com.beust.kobalt.TestResult
import com.beust.kobalt.api.IClasspathDependency
import com.beust.kobalt.api.KobaltContext
import com.beust.kobalt.api.Project
-import com.beust.kobalt.misc.KFiles
-import com.beust.kobalt.misc.warn
+import com.beust.kobalt.maven.aether.AetherDependency
+import com.beust.kobalt.misc.*
+import org.testng.remote.RemoteArgs
+import org.testng.remote.strprotocol.JsonMessageSender
+import org.testng.remote.strprotocol.MessageHelper
+import org.testng.remote.strprotocol.MessageHub
+import org.testng.remote.strprotocol.TestResultMessage
+import org.w3c.dom.Attr
+import org.w3c.dom.NodeList
+import org.xml.sax.InputSource
import java.io.File
+import java.io.FileReader
+import java.io.IOException
+import javax.xml.parsers.DocumentBuilderFactory
+import javax.xml.xpath.XPathConstants
+import javax.xml.xpath.XPathFactory
class TestNgRunner : GenericTestRunner() {
override val mainClass = "org.testng.TestNG"
-
override val dependencyName = "testng"
-
override val annotationPackage = "org.testng"
+ override val runnerName = "TestNG"
- fun defaultOutput(project: Project) = KFiles.joinDir(project.buildDirectory, "test-output")
+ private fun defaultOutputWithoutProjectDir(project: Project)
+ = KFiles.joinDir(project.buildDirectory, "test-output")
+ private fun defaultOutput(project: Project)
+ = KFiles.joinDir(project.directory, project.buildDirectory, "test-output")
override fun args(project: Project, context: KobaltContext, classpath: List,
testConfig: TestConfig) = arrayListOf().apply {
- var addOutput = true
- testConfig.testArgs.forEach { arg ->
- if (arg == "-d") addOutput = false
+
+ if (KobaltLogger.isQuiet) {
+ add("-log")
+ add("0")
+ }
+
+ if (testConfig.testArgs.none { it == "-d" }) {
+ add("-d")
+ // Don't include the project directory here since the generic runner will cd to that directory before
+ // running the tests
+ add(defaultOutputWithoutProjectDir(project))
}
if (testConfig.testArgs.size == 0) {
@@ -32,27 +57,212 @@ class TestNgRunner : GenericTestRunner() {
add(testngXml.absolutePath)
} else {
val testClasses = findTestClasses(project, context, testConfig)
- if (testClasses.size > 0) {
- if (addOutput) {
- add("-d")
- add(defaultOutput(project))
- }
+ if (testClasses.isNotEmpty()) {
addAll(testConfig.testArgs)
add("-testclass")
add(testClasses.joinToString(","))
} else {
- if (! testConfig.isDefault) warn("Couldn't find any test classes for ${project.name}")
+ if (!testConfig.isDefault) warn("Couldn't find any test classes for ${project.name}")
// else do nothing: since the user didn't specify an explicit test{} directive, not finding
// any test sources is not a problem
}
}
} else {
- if (addOutput) {
- add("-d")
- add(defaultOutput(project))
- }
addAll(testConfig.testArgs)
}
}
+
+ /**
+ * Extract test results from testng-results.xml and initialize shortMessage.
+ */
+ override fun onFinish(project: Project) {
+ File(defaultOutput(project), "testng-results.xml").let { file ->
+ val ins = InputSource(FileReader(file))
+ val doc = DocumentBuilderFactory.newInstance().newDocumentBuilder().parse(ins)
+
+ val root = doc.documentElement
+ var failed = 0
+ var skipped = 0
+ var passed = 0
+ val xp = XPathFactory.newInstance().newXPath()
+ val testMethods = xp.compile("/testng-results/suite/test/class/test-method[@status='FAIL']")
+ .evaluate(doc, XPathConstants.NODESET)
+ as NodeList
+ val failedMethods = arrayListOf()
+ repeat(testMethods.length) {
+ val tm = testMethods.item(it)
+ failedMethods.add(tm.attributes.getNamedItem("signature").textContent)
+ }
+ repeat(root.attributes.length) {
+ val attribute = root.attributes.item(it)
+ if (attribute is Attr) when (attribute.name) {
+ "failed" -> failed = Integer.parseInt(attribute.value)
+ "skipped" -> skipped = Integer.parseInt(attribute.value)
+ "passed" -> passed = Integer.parseInt(attribute.value)
+ }
+ }
+
+ if (failed == 0) {
+ shortMessage = "$passed tests"
+ } else if (failed > 0) {
+ shortMessage = "$failed failed" + (if (skipped > 0) ", $skipped skipped" else "") + " tests"
+ longMessage = "Failed tests:\n " + failedMethods.joinToString("\n ")
+ }
+ }
+ }
+
+ val VERSION_6_10 = StringVersion("6.10")
+
+ fun _runTests(project: Project, context: KobaltContext, classpath: List,
+// override fun runTests(project: Project, context: KobaltContext, classpath: List,
+ configName: String): TestResult {
+
+ val testConfig = project.testConfigs.firstOrNull { it.name == configName }
+
+ if (testConfig != null) {
+ context.logger.log(project.name, 1, "Running enhanced TestNG runner")
+
+ val testngDependency = (project.testDependencies.filter { it.id.contains("testng") }
+ .firstOrNull() as AetherDependency).version
+ val versions = findRemoteRunnerVersion(testngDependency)
+ val useOldRunner = System.getProperty("testng.oldRunner") != null
+ val result =
+ if (versions != null && ! useOldRunner) {
+ context.logger.log(project.name, 1, "Modern TestNG, displaying colors")
+ displayPrettyColors(project, context, classpath, testConfig, versions)
+ } else {
+ context.logger.log(project.name, 1, "Older TestNG ($testngDependency), using the old runner")
+ super.runTests(project, context, classpath, configName)
+ }
+ return result
+ } else {
+ return TestResult(true)
+ }
+ }
+
+ private fun findRemoteRunnerVersion(testngVersion: String) : Pair? {
+ val tng = StringVersion(testngVersion)
+ val result =
+ if (tng >= VERSION_6_10) Pair(testngVersion, "testng-remote6_10")
+ else if (tng >= StringVersion("6.9.10")) Pair("6.9.10", "testng-remote6_9_10")
+ else if (tng >= StringVersion("6.9.7")) Pair("6.9.7", "testng-remote6_9_7")
+ else if (tng >= StringVersion("6.5.1")) Pair("6.5.1", "testng-remote6_5_0")
+ else if (tng >= StringVersion("6.0")) Pair("6.0", "testng-remote6_0")
+ else null
+ return result
+ }
+
+ private fun displayPrettyColors(project: Project, context: KobaltContext,
+ classpath: List, testConfig: TestConfig, versions: Pair)
+ : TestResult {
+ val port = 2345
+// launchRemoteServer(project, context, classpath, testConfig, versions, port)
+
+ val mh = MessageHub(JsonMessageSender("localhost", port, true))
+ mh.setDebug(true)
+ mh.initReceiver()
+ val passed = arrayListOf()
+
+ data class FailedTest(val method: String, val cls: String, val stackTrace: String)
+
+ val failed = arrayListOf()
+ val skipped = arrayListOf()
+
+ fun d(n: Int, color: String)
+ = AsciiArt.wrap(String.format("%4d", n), color)
+
+ fun red(s: String) = AsciiArt.wrap(s, AsciiArt.RED)
+ fun green(s: String) = AsciiArt.wrap(s, AsciiArt.GREEN)
+ fun yellow(s: String) = AsciiArt.wrap(s, AsciiArt.YELLOW)
+
+ try {
+ var message = mh.receiveMessage()
+ kobaltLog(1, "")
+ kobaltLog(1, green("PASSED") + " | " + red("FAILED") + " | " + yellow("SKIPPED"))
+ while (message != null) {
+ message = mh.receiveMessage()
+ if (message is TestResultMessage) {
+ when (message.result) {
+ MessageHelper.PASSED_TEST -> passed.add(message.name)
+ MessageHelper.FAILED_TEST -> failed.add(FailedTest(message.testClass,
+ message.method, message.stackTrace))
+ MessageHelper.SKIPPED_TEST -> skipped.add(message.name)
+ }
+ }
+ if (!KobaltLogger.isQuiet) {
+ print("\r " + d(passed.size, AsciiArt.GREEN)
+ + " | " + d(failed.size, AsciiArt.RED)
+ + " | " + d(skipped.size, AsciiArt.YELLOW))
+ }
+ }
+ } catch(ex: IOException) {
+ kobaltLog(1, "Exception: ${ex.message}")
+ }
+ kobaltLog(1, "\nPassed: " + passed.size + ", Failed: " + failed.size + ", Skipped: " + skipped.size)
+ failed.forEach {
+ val top = it.stackTrace.substring(0, it.stackTrace.indexOf("\n"))
+ kobaltLog(1, " " + it.cls + "." + it.method + "\n " + top)
+ }
+ return TestResult(failed.isEmpty() && skipped.isEmpty())
+ }
+
+ fun launchRemoteServer(project: Project, context: KobaltContext, classpath: List,
+ testConfig: TestConfig, versions: Pair, port: Int) {
+ val testngVersion = versions.first
+ val remoteRunnerVersion = versions.second
+ val dep = with(context.dependencyManager) {
+ val jf = create("org.testng.testng-remote:testng-remote:1.3.0")
+ val tr = create("org.testng.testng-remote:$remoteRunnerVersion:1.3.0")
+ val testng = create("org.testng:testng:6.11")
+ transitiveClosure(kotlin.collections.listOf(jf, tr /*, testng */))
+ }
+
+ val cp = (classpath + dep).distinct().map { it.jarFile.get() }
+ .joinToString(File.pathSeparator)
+ val calculatedArgs = args(project, context, classpath, testConfig)
+
+ val jvmArgs = arrayListOf("-classpath", cp)
+ if (testConfig.jvmArgs.any()) {
+ jvmArgs.addAll(testConfig.jvmArgs)
+ }
+ val remoteArgs = listOf(
+ "org.testng.remote.RemoteTestNG",
+ "-serport", port.toString(),
+ "-version", testngVersion,
+ "-dontexit",
+ RemoteArgs.PROTOCOL,
+ "json")
+
+ val passedArgs = jvmArgs + remoteArgs + calculatedArgs
+
+ Thread {
+ runCommand {
+ command = "java"
+ directory = File(project.directory)
+ args = passedArgs
+ }
+ }.start()
+
+// Thread {
+// val args2 = arrayOf("-serport", port.toString(), "-dontexit", RemoteArgs.PROTOCOL, "json",
+// "-version", "6.10",
+// "src/test/resources/testng.xml")
+// RemoteTestNG.main(args2)
+// }.start()
+ }
+}
+
+fun main(args: Array) {
+ fun d(n: Int, color: String)
+ = AsciiArt.wrap(String.format("%4d", n), color)
+
+ if (!KobaltLogger.isQuiet) {
+ println("PASSED | FAILED | SKIPPED")
+ repeat(20) { i ->
+ print("\r " + d(i, AsciiArt.GREEN) + " | " + d(i * 2, AsciiArt.RED) + " | " + d(i, AsciiArt.YELLOW))
+ Thread.sleep(500)
+ }
+ println("")
+ }
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildFile.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildFile.kt
index b860f446..a2eb91e9 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildFile.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildFile.kt
@@ -1,10 +1,8 @@
package com.beust.kobalt.internal.build
-import com.beust.kobalt.misc.KFiles
import java.io.File
import java.nio.file.Files
import java.nio.file.Path
-import java.nio.file.attribute.BasicFileAttributes
/**
* Sometimes, build files are moved to temporary files, so we give them a specific name for clarity.
* @param path is the path where that file was moved, @param realPath is where the actual file is.
@@ -12,23 +10,5 @@ import java.nio.file.attribute.BasicFileAttributes
class BuildFile(val path: Path, val name: String, val realPath: Path = path) {
fun exists() : Boolean = Files.exists(path)
- val lastModified : Long
- get() = Files.readAttributes(realPath, BasicFileAttributes::class.java).lastModifiedTime().toMillis()
-
val directory : File get() = path.toFile().parentFile
-
- /**
- * @return the .kobalt directory where this build file will be compiled.
- */
- val dotKobaltDir: File get() = File(directory.parentFile.parentFile, KFiles.KOBALT_DOT_DIR).apply {
- mkdirs()
- }
-
- /**
- * @return the absolute directory of this project's location, assuming the build file is in
- * $project/kobalt/src/Build.kt.
- */
- val absoluteDir : File? get() {
- return path.parent?.parent?.parent?.toFile()
- }
}
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildSources.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildSources.kt
new file mode 100644
index 00000000..78425f18
--- /dev/null
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/build/BuildSources.kt
@@ -0,0 +1,61 @@
+package com.beust.kobalt.internal.build
+
+import com.beust.kobalt.homeDir
+import java.io.File
+import java.nio.file.*
+import java.nio.file.attribute.BasicFileAttributes
+
+/**
+ * The abstraction to represent a directory that contains source files. @param{root} is typically
+ * the root of the project and build files are searched under root/kobalt/src/ *kt.
+ */
+interface IBuildSources {
+ fun findSourceFiles() : List
+ val root: File
+ fun exists(): Boolean
+}
+
+class SingleFileBuildSources(val file: File) : IBuildSources {
+ override fun exists() = file.exists()
+ override fun findSourceFiles() = listOf(file)
+ override val root: File = file.parentFile.parentFile.parentFile
+ override fun toString() : String = file.path
+}
+
+class BuildSources(val file: File = File("")) : IBuildSources {
+
+ override val root = file
+
+ override fun findSourceFiles() : List {
+ return findBuildFiles(listOf(file))
+ }
+
+ override fun exists() = findSourceFiles().isNotEmpty()
+
+ override fun toString() = "{BuildSources " + findSourceFiles().joinToString(", ") + "}"
+
+ fun findBuildFiles(roots: List) : List {
+ val result = arrayListOf()
+ roots.forEach { file ->
+ Files.walkFileTree(Paths.get(file.path), object : SimpleFileVisitor() {
+ override fun preVisitDirectory(dir: Path?, attrs: BasicFileAttributes?): FileVisitResult {
+ if (dir != null) {
+ val path = dir.toFile()
+ if (path.name == "src" && path.parentFile.name == "kobalt") {
+ val sources = path.listFiles().filter { it.name.endsWith(".kt") }
+ result.addAll(sources)
+ }
+ }
+
+ return FileVisitResult.CONTINUE
+ }
+ })
+ }
+ return result
+ }
+}
+
+fun main(args: Array) {
+ val sources = BuildSources(File(homeDir("kotlin/kobalt"))).findSourceFiles()
+ println("sources: " + sources)
+}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/remote/GetDependenciesCommand.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/remote/GetDependenciesCommand.kt
deleted file mode 100644
index 7422c493..00000000
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/internal/remote/GetDependenciesCommand.kt
+++ /dev/null
@@ -1,14 +0,0 @@
-package com.beust.kobalt.internal.remote
-
-import com.beust.kobalt.Constants
-import java.io.PrintWriter
-import java.net.Socket
-
-fun main(argv: Array) {
- Socket("localhost", 1234).use { socket ->
- (PrintWriter(socket.outputStream, true)).use { out ->
- out.println("""{ "name" : "getDependencies", "buildFile":
- "/Users/beust/kotlin/kobalt/kobalt/src/${Constants.BUILD_FILE_NAME}"}""")
- }
- }
-}
\ No newline at end of file
diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt
index 7792455d..ff2d9fd9 100644
--- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt
+++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt
@@ -2,20 +2,23 @@ package com.beust.kobalt.maven
import com.beust.kobalt.KobaltException
import com.beust.kobalt.api.*
-import com.beust.kobalt.maven.aether.KobaltAether
+import com.beust.kobalt.maven.aether.Filters
+import com.beust.kobalt.maven.aether.KobaltMavenResolver
import com.beust.kobalt.maven.aether.Scope
import com.beust.kobalt.maven.dependency.FileDependency
import com.beust.kobalt.misc.KFiles
import com.beust.kobalt.misc.KobaltExecutors
import com.google.common.collect.ArrayListMultimap
+import org.eclipse.aether.graph.DependencyFilter
+import org.eclipse.aether.util.filter.OrDependencyFilter
import java.io.File
import java.util.*
import javax.inject.Inject
import javax.inject.Singleton
@Singleton
-class DependencyManager @Inject constructor(val executors: KobaltExecutors, val aether: KobaltAether)
- : IDependencyManager {
+class DependencyManager @Inject constructor(val executors: KobaltExecutors,
+ val resolver: KobaltMavenResolver) : IDependencyManager {
companion object {
fun create(id: String, optional: Boolean = false, projectDirectory: String? = null) =
@@ -57,12 +60,7 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, val
/**
* Create an IClasspathDependency from a Maven id.
*/
- override fun createMaven(id: String, optional: Boolean) : IClasspathDependency=
- if (KobaltAether.isRangeVersion(id)) {
- Kobalt.INJECTOR.getInstance(KobaltAether::class.java).resolve(id).dependency
- } else {
- aether.create(id, optional)
- }
+ override fun createMaven(id: String, optional: Boolean) : IClasspathDependency = resolver.create(id, optional)
/**
* Create an IClasspathDependency from a path.
@@ -72,12 +70,14 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, val
/**
* @return the source dependencies for this project, including the contributors.
*/
- override fun dependencies(project: Project, context: KobaltContext) = dependencies(project, context, false)
+ override fun dependencies(project: Project, context: KobaltContext, scopes: List)
+ = privateDependencies(project, context, listOf(Scope.COMPILE))
/**
* @return the test dependencies for this project, including the contributors.
*/
- override fun testDependencies(project: Project, context: KobaltContext) = dependencies(project, context, true)
+ override fun testDependencies(project: Project, context: KobaltContext)
+ = privateDependencies(project, context, listOf(Scope.COMPILE, Scope.TEST))
/**
* Transitive dependencies for the compilation of this project.
@@ -87,12 +87,15 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, val
// project.compileDependencies + project.compileRuntimeDependencies)
/**
- * @return the classpath for this project, including the IClasspathContributors.
- * allDependencies is typically either compileDependencies or testDependencies. If no dependencies
+ * @return the classpath for this project, including the IClasspathContributors. Excluded dependencies
+ * are removed from the result.
+ *
+ * @param{allDependencies} is typically either compileDependencies or testDependencies. If no dependencies
* are passed, they are calculated from the scope filters.
*/
override fun calculateDependencies(project: Project?, context: KobaltContext,
- scopeFilters: Collection,
+ dependencyFilter: DependencyFilter,
+ scopes: List,
vararg passedDependencies: List): List {
val result = arrayListOf()
@@ -100,9 +103,13 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, val
* Extract the correct dependencies from the project based on the scope filters.
*/
fun filtersToDependencies(project: Project, scopes: Collection): List {
- return arrayListOf().apply {
+ val result = arrayListOf().apply {
if (scopes.contains(Scope.COMPILE)) {
addAll(project.compileDependencies)
+ addAll(project.compileProvidedDependencies)
+ }
+ if (scopes.contains(Scope.COMPILEONLY)) {
+ addAll(project.compileOnlyDependencies)
}
if (scopes.contains(Scope.RUNTIME)) {
addAll(project.compileRuntimeDependencies)
@@ -111,22 +118,50 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, val
addAll(project.testDependencies)
}
}
+ return result.filter { ! it.optional }
}
val allDependencies : Array