You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@polygene.apache.org by pa...@apache.org on 2017/04/10 09:53:59 UTC

[1/2] polygene-java git commit: POLYGENE-29 Remove Groovy from published dependencies

Repository: polygene-java
Updated Branches:
  refs/heads/develop 27cad2880 -> 425af3228


POLYGENE-29 Remove Groovy from published dependencies

It\u2019s only needed for tests and bring a big load of useless stuff on the
classpath if you don\u2019t plan to use Groovy.


Project: http://git-wip-us.apache.org/repos/asf/polygene-java/repo
Commit: http://git-wip-us.apache.org/repos/asf/polygene-java/commit/255b5fc6
Tree: http://git-wip-us.apache.org/repos/asf/polygene-java/tree/255b5fc6
Diff: http://git-wip-us.apache.org/repos/asf/polygene-java/diff/255b5fc6

Branch: refs/heads/develop
Commit: 255b5fc63dd89d1763d04887805b08cf4835bb4c
Parents: 27cad28
Author: Paul Merlin <pa...@apache.org>
Authored: Mon Apr 10 11:41:16 2017 +0200
Committer: Paul Merlin <pa...@apache.org>
Committed: Mon Apr 10 11:48:25 2017 +0200

----------------------------------------------------------------------
 libraries/scripting/build.gradle           | 3 +--
 libraries/scripting/src/docs/scripting.txt | 7 +++----
 2 files changed, 4 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/polygene-java/blob/255b5fc6/libraries/scripting/build.gradle
----------------------------------------------------------------------
diff --git a/libraries/scripting/build.gradle b/libraries/scripting/build.gradle
index bacd28e..b885232 100644
--- a/libraries/scripting/build.gradle
+++ b/libraries/scripting/build.gradle
@@ -30,11 +30,10 @@ dependencies {
   api polygene.core.api
   api polygene.core.bootstrap
 
-  implementation libraries.groovy
-
   runtimeOnly polygene.core.runtime
 
   testImplementation polygene.core.testsupport
+  testImplementation libraries.groovy
 
   testRuntimeOnly libraries.logback
 }

http://git-wip-us.apache.org/repos/asf/polygene-java/blob/255b5fc6/libraries/scripting/src/docs/scripting.txt
----------------------------------------------------------------------
diff --git a/libraries/scripting/src/docs/scripting.txt b/libraries/scripting/src/docs/scripting.txt
index c94ed76..7198fc4 100644
--- a/libraries/scripting/src/docs/scripting.txt
+++ b/libraries/scripting/src/docs/scripting.txt
@@ -43,8 +43,7 @@ should be supported. On Java.Net there is a https://java.net/projects/scripting/
 
 
 It is important to understand that the ScriptEngines must be present on the classpath, or they will not be available,
-with the exception of JavaScript which uses the built-in Nashorn engine and Groovy that is an explicit dependency of
-this library.
+with the exception of JavaScript which uses the built-in Nashorn engine.
 
 We welcome additional information and samples of languages that has been tried and tested.
 
@@ -53,7 +52,7 @@ We welcome additional information and samples of languages that has been tried a
 |=======
 |Language | Supported | Tested | License | Implementation
 |JavaScript | Yes | Yes | ? |Nashorn, built-in Java 8 and later
-|Groovy | Yes | Yes |Apache |Apache Groovy, dependency exist from this library
+|Groovy | Yes | Yes |Apache |Apache Groovy
 |Ruby | Yes | No |(EPL\|GPL\|LGPL) + Ruby |JRuby
 |Python | Yes | No |https://opensource.org/licenses/Python-2.0[Python] |Jython, must use jython-standalone
 |Kotlin | Yes | No |Apache |Jetbrains
@@ -91,7 +90,7 @@ The following languages may or may not be supported, and was found at https://ja
 [options="header"]
 |=======
 | Language | Comment
-| Java | Implementation exists at https://github.com/nickman/javax-scripting/ which is forked from https://java.net/projects/scripting. It doesn't support Invocable, and additional work on ScriptMixin is required to support this. Althernatively, fork the ScriptEngine implementation and support Invocable in it.
+| Java | Implementation exists at https://github.com/nickman/javax-scripting/ which is forked from https://java.net/projects/scripting. It doesn't support Invocable, and additional work on ScriptMixin is required to support this. Alternatively, fork the ScriptEngine implementation and support Invocable in it.
 |=======
 
 include::../../../../libraries/scripting/src/docs/javascript.txt[]


[2/2] polygene-java git commit: POLYGENE-29 Better test assertions to pass on JDK 9

Posted by pa...@apache.org.
POLYGENE-29 Better test assertions to pass on JDK 9


Project: http://git-wip-us.apache.org/repos/asf/polygene-java/repo
Commit: http://git-wip-us.apache.org/repos/asf/polygene-java/commit/425af322
Tree: http://git-wip-us.apache.org/repos/asf/polygene-java/tree/425af322
Diff: http://git-wip-us.apache.org/repos/asf/polygene-java/diff/425af322

Branch: refs/heads/develop
Commit: 425af322810e81286f42576444d88a601847068f
Parents: 255b5fc
Author: Paul Merlin <pa...@apache.org>
Authored: Mon Apr 10 11:50:52 2017 +0200
Committer: Paul Merlin <pa...@apache.org>
Committed: Mon Apr 10 11:50:52 2017 +0200

----------------------------------------------------------------------
 .../org/apache/polygene/library/scripting/ScriptMixinTest.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/polygene-java/blob/425af322/libraries/scripting/src/test/java/org/apache/polygene/library/scripting/ScriptMixinTest.java
----------------------------------------------------------------------
diff --git a/libraries/scripting/src/test/java/org/apache/polygene/library/scripting/ScriptMixinTest.java b/libraries/scripting/src/test/java/org/apache/polygene/library/scripting/ScriptMixinTest.java
index df51580..ba12e68 100644
--- a/libraries/scripting/src/test/java/org/apache/polygene/library/scripting/ScriptMixinTest.java
+++ b/libraries/scripting/src/test/java/org/apache/polygene/library/scripting/ScriptMixinTest.java
@@ -78,9 +78,9 @@ public class ScriptMixinTest
         domain2.inc();
         domain2.inc();
         domain3.inc();
-        assertThat(domain1.count(), equalTo(4.0) );
-        assertThat(domain2.count(), equalTo(3.0) );
-        assertThat(domain3.count(), equalTo(1.0) );
+        assertThat(domain1.count(), equalTo(4.0D) );
+        assertThat(domain2.count(), equalTo(3.0D) );
+        assertThat(domain3.count(), equalTo(1.0D) );
     }
 
     @Test