Merge lp:~zorba-coders/zorba/not-packaging-external-jars into lp:zorba

Proposed by Matthias Brantner
Status: Merged
Approved by: Matthias Brantner
Approved revision: 10868
Merged at revision: 10867
Proposed branch: lp:~zorba-coders/zorba/not-packaging-external-jars
Merge into: lp:zorba
Diff against target: 58 lines (+6/-7)
3 files modified
ChangeLog (+3/-0)
cmake_modules/ZorbaModule.cmake (+3/-3)
test/rbkt/modules/java/CMakeLists.txt (+0/-4)
To merge this branch: bzr merge lp:~zorba-coders/zorba/not-packaging-external-jars
Reviewer Review Type Date Requested Status
Matthias Brantner Approve
Chris Hillery Approve
Review via email: mp+107895@code.launchpad.net

Commit message

added a cmake variable that allows to configure whether external jars are packaged or not (ZORBA_PACKAGE_EXTERNAL_JARS)

Description of the change

- added a cmake variable that allows to configure whether external jars are packaged or not (ZORBA_PACKAGE_EXTERNAL_JARS)
- removed a test that started to fail if the ZORBA_PACKAGE_EXTERNAL_JARS variable is turned on. the test was actually cheating and didn't have an actual jar
- extended changelog for schema-tools and xsl-fo

To post a comment you must log in.
Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
Revision history for this message
Matthias Brantner (matthias-brantner) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job not-packaging-external-jars-2012-05-30T01-18-06.466Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ChangeLog'
2--- ChangeLog 2012-05-23 16:11:13 +0000
3+++ ChangeLog 2012-05-30 00:41:21 +0000
4@@ -24,6 +24,8 @@
5 stream is seekable or not.
6 * Added new functions in the fetch module and the StaticContext to fetch
7 resources as binary
8+ * Added non-core schema-tools module for getting sample XMLSchema from
9+ XML instances and sample XML instances from XMLSchema.
10
11 Optimizations:
12 * optimized insertion into a collection (don't copy it if the node was
13@@ -64,6 +66,7 @@
14 * Fixed Bug #933490 (Error ItemFactoryImpl::createBase64Binary with istream)
15 * Fixed bug #867112 (Diagnostic Handler was not working on external APIs)
16 * Fixed bug #857842 (Assertion failed with simple content element with comments)
17+ * Fixed bug #928626 (no setting classpath in xsl-fo module)
18
19 version 2.2
20
21
22=== modified file 'cmake_modules/ZorbaModule.cmake'
23--- cmake_modules/ZorbaModule.cmake 2012-05-18 19:34:39 +0000
24+++ cmake_modules/ZorbaModule.cmake 2012-05-30 00:41:21 +0000
25@@ -481,16 +481,16 @@
26 # Iterate over all supplied jar files
27 FOREACH (_jar_file ${JAR_FILE})
28
29- IF (JAR_EXTERNAL)
30+ IF (JAR_EXTERNAL AND NOT ZORBA_PACKAGE_EXTERNAL_JARS)
31 # Put absolute path into classpath file
32 FILE (APPEND "${_CP_FILE}" "${_jar_file}\n")
33- ELSE (JAR_EXTERNAL)
34+ ELSE (JAR_EXTERNAL AND NOT ZORBA_PACKAGE_EXTERNAL_JARS)
35 # Copy jar to jars/ directory and add relative path to classpath file
36 GET_FILENAME_COMPONENT (_output_filename "${_jar_file}" NAME)
37 ADD_COPY_RULE ("LIB" "${_jar_file}" "jars/${_output_filename}" ""
38 "${JAR_TARGET}" 1 "${JAR_TEST_ONLY}")
39 FILE (APPEND "${_CP_FILE}" "${_output_filename}\n")
40- ENDIF (JAR_EXTERNAL)
41+ ENDIF (JAR_EXTERNAL AND NOT ZORBA_PACKAGE_EXTERNAL_JARS)
42
43 ENDFOREACH (_jar_file)
44 ENDMACRO (DECLARE_ZORBA_JAR)
45
46=== modified file 'test/rbkt/modules/java/CMakeLists.txt'
47--- test/rbkt/modules/java/CMakeLists.txt 2012-04-05 11:48:50 +0000
48+++ test/rbkt/modules/java/CMakeLists.txt 2012-05-30 00:41:21 +0000
49@@ -32,9 +32,5 @@
50 DECLARE_ZORBA_JAR(FILE ${JavaTest_JAR_FILE} TARGET JavaTest TEST_ONLY)
51 DECLARE_ZORBA_JAR(FILE ${JavaTest2_JAR_FILE} TARGET JavaTest2 TEST_ONLY)
52
53- # Pretend to tell Zorba about some system jars
54- DECLARE_ZORBA_JAR(EXTERNAL FILE
55- "${PROJECT_SOURCE_DIR}/MyJar.jar" "${PROJECT_SOURCE_DIR}/MyJar2.jar")
56-
57 ENDIF (Java_Development_FOUND)
58 ENDIF (${RESULT} GREATER -1)

Subscribers

People subscribed via source and target branches