Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/email-module-dayofweek-fix into lp:zorba/email-module

2012-06-04 Thread Chris Hillery
Review: Approve

Not a terribly pretty solution, but then text processing in C++ never is. As 
far as I can tell it gets the job done.
-- 
https://code.launchpad.net/~zorba-coders/zorba/email-module-dayofweek-fix/+merge/108335
Your team Zorba Coders is subscribed to branch lp:zorba/email-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-991088 into lp:zorba

2012-06-04 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/bug-991088 
into lp:zorba.

Requested reviews:
  Markos Zaharioudakis (markos-za)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-991088/+merge/108535

Fixed bug #991088$ (raise XUST0001 in trycatch with mixed updating and simple 
clauses)
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-991088/+merge/108535
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog	2012-05-30 00:37:00 +
+++ ChangeLog	2012-06-04 10:13:32 +
@@ -1,5 +1,9 @@
 Zorba - The XQuery Processor
 
+version 2.7
+
+  * Fixed bug #991088$ (raise XUST0001 in trycatch with mixed updating and simple clauses)
+
 version 2.5
 
 New Features:

=== modified file 'src/compiler/expression/expr.cpp'
--- src/compiler/expression/expr.cpp	2012-05-16 17:25:48 +
+++ src/compiler/expression/expr.cpp	2012-06-04 10:13:32 +
@@ -23,6 +23,7 @@
 #include map
 
 #include diagnostics/assert.h
+#include diagnostics/util_macros.h
 #include diagnostics/xquery_diagnostics.h
 
 #include system/globalenv.h
@@ -190,15 +191,11 @@
   {
 if (theThenExpr-is_updating()  !theElseExpr-is_updating_or_vacuous())
 {
-  throw XQUERY_EXCEPTION(err::XUST0001, 
- ERROR_PARAMS(ZED(XUST0001_IF)),
- ERROR_LOC(get_loc()));
+  RAISE_ERROR(err::XUST0001, get_loc(), ERROR_PARAMS(ZED(XUST0001_IF)));
 }
 else if (theElseExpr-is_updating()  !theThenExpr-is_updating_or_vacuous())
 {
-  throw XQUERY_EXCEPTION(err::XUST0001, 
- ERROR_PARAMS(ZED(XUST0001_IF)),
- ERROR_LOC(get_loc()));
+  RAISE_ERROR(err::XUST0001, get_loc(), ERROR_PARAMS(ZED(XUST0001_IF)));
 }
 else
 {
@@ -1294,22 +1291,47 @@
 
 void trycatch_expr::compute_scripting_kind()
 {
-  theScriptingKind = SIMPLE_EXPR;
+  bool vacuous = true;
+
+  theScriptingKind = VACUOUS_EXPR;
 
   theScriptingKind |= theTryExpr-get_scripting_detail();
 
-  ulong numCatchClauses = (ulong)theCatchClauses.size();
-
-  for (ulong i = 0; i  numCatchClauses; ++i) 
+  if (theScriptingKind != VACUOUS_EXPR)
+vacuous = false;
+
+  csize numCatchClauses = theCatchClauses.size();
+
+  for (csize i = 0; i  numCatchClauses; ++i) 
   {
 const expr* catchExpr = theCatchExprs[i].getp();
-
 short catchKind = catchExpr-get_scripting_detail();
 
+if (catchKind == VACUOUS_EXPR)
+  continue;
+
+vacuous = false;
+
+if (!theSctx-is_feature_set(feature::scripting))
+{
+  if (is_updating()  !(catchKind  UPDATING_EXPR)  catchKind != VACUOUS_EXPR)
+  {
+RAISE_ERROR(err::XUST0001, catchExpr-get_loc(),
+ERROR_PARAMS(ZED(XUST0001_TRYCATCH)));
+  }
+
+  if (!is_updating()  !is_vacuous()  (catchKind  UPDATING_EXPR))
+  {
+RAISE_ERROR(err::XUST0001, catchExpr-get_loc(),
+ERROR_PARAMS(ZED(XUST0001_TRYCATCH)));
+  }
+}
+
 theScriptingKind |= catchKind;
   }
 
-  theScriptingKind = ~VACUOUS_EXPR;
+  if (!vacuous)
+theScriptingKind = ~VACUOUS_EXPR;
 
   if (theScriptingKind  UPDATING_EXPR)
 theScriptingKind = ~SIMPLE_EXPR;

=== modified file 'src/diagnostics/diagnostic_en.xml'
--- src/diagnostics/diagnostic_en.xml	2012-05-18 22:01:56 +
+++ src/diagnostics/diagnostic_en.xml	2012-06-04 10:13:32 +
@@ -1205,6 +1205,10 @@
 valuecomma expression with updating and non-updating branches/value
   /entry
 
+  entry key=TRYCATCH
+valuetry-catch expression with updating and non-updating clauses/value
+  /entry
+
   entry key=Generic
 valueupdating expression illegal here/value
   /entry

=== modified file 'src/diagnostics/pregenerated/dict_en.cpp'
--- src/diagnostics/pregenerated/dict_en.cpp	2012-05-18 22:01:56 +
+++ src/diagnostics/pregenerated/dict_en.cpp	2012-06-04 10:13:32 +
@@ -784,6 +784,7 @@
   { ~XUST0001_CONCAT, comma expression with updating and non-updating branches },
   { ~XUST0001_Generic, updating expression illegal here },
   { ~XUST0001_IF, conditional expression with updating and non-updating branch },
+  { ~XUST0001_TRYCATCH, try-catch expression with updating and non-updating clauses },
   { ~XUST0001_UDF_2, \$2\: function declared simple but body is updating },
   { ~XUST0002_Transform, transform expression witn non-updating or vacuous modify clause },
   { ~XUST0002_UDF_2, \$2\: function declared updating but body is not updating or vacuous },

=== added file 'test/rbkt/Queries/zorba/trycatch/trycatch14.spec'
--- test/rbkt/Queries/zorba/trycatch/trycatch14.spec	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/trycatch/trycatch14.spec	2012-06-04 10:13:32 +
@@ -0,0 +1,1 @@
+Error: http://www.w3.org/2005/xqt-errors:XUST0001

=== added file 'test/rbkt/Queries/zorba/trycatch/trycatch14.xq'
--- test/rbkt/Queries/zorba/trycatch/trycatch14.xq	1970-01-01 00:00:00 +
+++ 

[Zorba-coders] [Bug 991088] Re: Updating expressions in try/catch

2012-06-04 Thread Markos Zaharioudakis
I have a pending fix for this. However, the XUST0001 error will be
raised only if scripting is disabled. This is the same behaviour as for
the comma operator.


** Changed in: zorba
   Status: New = In Progress

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/991088

Title:
  Updating expressions in try/catch

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  I believe that the following two expressions should raise XUST0001:

  try { 1 } catch * { delete node a/ },
  try { delete node a/ } catch * { 1 }

  Source:
http://www.w3.org/TR/xquery-update-30/#id-try-catch

  Hope this helps,
  Christian

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/991088/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Bug 854506] Re: type errors cause ugly error msgs

2012-06-04 Thread Markos Zaharioudakis
** Changed in: zorba
   Status: New = In Progress

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/854506

Title:
  type errors cause ugly error msgs

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  A query like this

  declare function local:foo() as empty-sequence()
   {
 foo /
   };
   
   local:foo();

  causes the following error msg because of a type error:

  ?xml version=1.0 encoding=UTF-8?
  /Users/wcandillon/tmp/test.xq:3,3: type error [err:XPTY0004]: Cannot
  treat [NodeXQType elementNode nametest=[uri: , local: foo]
  content=[XQType ANY_TYPE_KIND*]] as [XQType NONE_KIND] when returning
  the result of the function local:foo().; raised at
  /Users/wcandillon/28msec/zorba/src/runtime/core/sequencetypes.cpp:557
  =
  local:foo#0 http://www.w3.org/2005/xquery-local-functions
  /Users/wcandillon/tmp/test.xq at line 6 column 1

  This is very ugly und not understandable for someone who is not
  involved in the zorba internals. We need an error msg like this:

  Stopped at line 1, column 17: [XPTY0004] Can't promote type xs:string
  to empty-sequence(): abc.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/854506/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/util-jvm-installable 
into lp:zorba/util-jvm-module.

Requested reviews:
  Zorba Coders (zorba-coders)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is requested to review the proposed merge of 
lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2012-05-18 10:36:07 +
+++ CMakeLists.txt	2012-06-04 11:46:30 +
@@ -40,11 +40,16 @@
   # Export this include directory by setting ZORBA_PROJECT_INCLUDE_DIRS
   SET (ZORBA_PROJECT_INCLUDE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}/include)
   INCLUDE_DIRECTORIES (${ZORBA_PROJECT_INCLUDE_DIRS})
+  INSTALL (FILES include/JavaVMSingleton.h DESTINATION include)
 
   # Set up a use file for other modules to import
   SET (ZORBA_PROJECT_USE_FILE
 	${CMAKE_CURRENT_SOURCE_DIR}/config/UtilJavaUse.cmake)
   ADD_SUBDIRECTORY (src)
+
+  # Set up configuration for install
+  ADD_SUBDIRECTORY (config)
+
   DONE_DECLARING_ZORBA_URIS ()
 
   ELSE (JNI_FOUND)

=== added file 'config/CMakeLists.txt'
--- config/CMakeLists.txt	1970-01-01 00:00:00 +
+++ config/CMakeLists.txt	2012-06-04 11:46:30 +
@@ -0,0 +1,37 @@
+# Copyright 2006-2012 The FLWOR Foundation.
+# 
+# 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.
+
+# We need to be able to install util-jvm and then build module projects
+# against that. The auto-config approach Zorba provides does not
+# support this, so we roll our own here. This should probably be
+# re-thought in the future.
+
+SET (UTILJVM_CMAKE_DIR share/cmake/zorba_util-jvm_module)
+
+# First, install the Use file. Doesn't need to be configured since it
+# just contains a macro.
+INSTALL (FILES UtilJavaUse.cmake DESTINATION ${UTILJVM_CMAKE_DIR})
+
+# Anyway, we set all the same variables to what they should be in the 
+# install image, and re-config Zorba's ExternalModuleConfig.cmake.in.
+SET (ZORBA_PROJECT_INCLUDE_DIRS ${CMAKE_INSTALL_PREFIX}/include)
+SET (ZORBA_PROJECT_USE_FILE ${CMAKE_INSTALL_PREFIX}/${UTILJVM_CMAKE_DIR}/UtilJavaUse.cmake)
+GET_TARGET_PROPERTY(_libpath util-jvm LOCATION)
+GET_FILENAME_COMPONENT(_libname ${_libpath}  NAME)
+SET (ZORBA_PROJECT_LIBRARIES ${CMAKE_INSTALL_PREFIX}/lib/${_libname})
+
+CONFIGURE_FILE(${Zorba_EXTERNALMODULECONFIG_FILE}
+  ${PROJECT_BINARY_DIR}/install/${PROJECT_NAME}Config.cmake @ONLY)
+INSTALL(FILES ${PROJECT_BINARY_DIR}/install/${PROJECT_NAME}Config.cmake
+  DESTINATION ${UTILJVM_CMAKE_DIR})

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Chris Hillery
Chris Hillery has proposed merging 
lp:~zorba-coders/zorba/data-formatting-find-java into 
lp:zorba/data-formatting-module.

Requested reviews:
  Zorba Coders (zorba-coders)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is requested to review the proposed merge of 
lp:~zorba-coders/zorba/data-formatting-find-java into 
lp:zorba/data-formatting-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2012-05-09 10:11:44 +
+++ CMakeLists.txt	2012-06-04 11:46:33 +
@@ -18,15 +18,16 @@
 
 PROJECT (zorba_data-formatting_module)
 
-
-# The util-jvm tool finds Java and JNI, so all standard cached JAVA_
-# variables will be set. util-jvm also caches Java_FOUND and JNI_FOUND
-# for us.
 FIND_PACKAGE (zorba_util-jvm_module QUIET)
 
 IF (zorba_util-jvm_module_FOUND)
   INCLUDE (${zorba_util-jvm_module_USE_FILE})
 
+  FIND_PACKAGE(JNI)
+  FIND_PACKAGE(Java)
+  FIND_PACKAGE (Zorba REQUIRED HINTS ${ZORBA_BUILD_DIR})
+  INCLUDE (${Zorba_USE_FILE})
+
   IF (JNI_FOUND)
 INCLUDE_DIRECTORIES (${zorba_util-jvm_module_INCLUDE_DIRS})
 

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Bug 1008504] [NEW] parse-xml-fragment triggers schema error

2012-06-04 Thread Alexander Kreutz
Public bug reported:

When running the attached example a schema validation error inside the
xml module is thrown:

/home/tryzorba/repo/zorba-2.2.0/build/URI_PATH/com/zorba-
xquery/www/modules/xml.xq:346,5: dynamic error [err:XQDY0027]: Schema
validation failed: no declaration found for element 'parse-xml-options
:strip-boundary-space': unexpected validity property

-Example:
xquery version 3.0;

import module namespace parse-xml = http://www.zorba-
xquery.com/modules/xml;

parse-xml:parse-xml-fragment(xmltest/xml,EDSlwf)

** Affects: zorba
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1008504

Title:
  parse-xml-fragment triggers schema error

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When running the attached example a schema validation error inside the
  xml module is thrown:

  /home/tryzorba/repo/zorba-2.2.0/build/URI_PATH/com/zorba-
  xquery/www/modules/xml.xq:346,5: dynamic error [err:XQDY0027]:
  Schema validation failed: no declaration found for element 'parse-
  xml-options:strip-boundary-space': unexpected validity property

  
-Example:
  xquery version 3.0;

  import module namespace parse-xml = http://www.zorba-
  xquery.com/modules/xml;

  parse-xml:parse-xml-fragment(xmltest/xml,EDSlwf)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1008504/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Bug 1008504] Re: parse-xml-fragment triggers schema error

2012-06-04 Thread David Graf
** Changed in: zorba
 Assignee: (unassigned) = David Graf (davidagraf)

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1008504

Title:
  parse-xml-fragment triggers schema error

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When running the attached example a schema validation error inside the
  xml module is thrown:

  /home/tryzorba/repo/zorba-2.2.0/build/URI_PATH/com/zorba-
  xquery/www/modules/xml.xq:346,5: dynamic error [err:XQDY0027]:
  Schema validation failed: no declaration found for element 'parse-
  xml-options:strip-boundary-space': unexpected validity property

  
-Example:
  xquery version 3.0;

  import module namespace parse-xml = http://www.zorba-
  xquery.com/modules/xml;

  parse-xml:parse-xml-fragment(xmltest/xml,EDSlwf)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1008504/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Bug 1008504] Re: parse-xml-fragment triggers schema error

2012-06-04 Thread David Graf
** Changed in: zorba
 Assignee: David Graf (davidagraf) = Nicolae Brinza (nbrinza)

** Changed in: zorba
   Importance: Undecided = Medium

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1008504

Title:
  parse-xml-fragment triggers schema error

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When running the attached example a schema validation error inside the
  xml module is thrown:

  /home/tryzorba/repo/zorba-2.2.0/build/URI_PATH/com/zorba-
  xquery/www/modules/xml.xq:346,5: dynamic error [err:XQDY0027]:
  Schema validation failed: no declaration found for element 'parse-
  xml-options:strip-boundary-space': unexpected validity property

  
-Example:
  xquery version 3.0;

  import module namespace parse-xml = http://www.zorba-
  xquery.com/modules/xml;

  parse-xml:parse-xml-fragment(xmltest/xml,EDSlwf)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1008504/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Cezar Andrei
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread Cezar Andrei
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/data-formatting-find-java into 
lp:zorba/data-formatting-module has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/data-formatting-find-java-2012-06-04T21-18-40.481Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/schema-tools-find-java into 
lp:zorba/schema-tools-module has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread Zorba Build Bot
Validation queue job data-formatting-find-java-2012-06-04T21-18-40.481Z is 
finished. The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/data-formatting-find-java into lp:zorba/data-formatting-module

2012-06-04 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/data-formatting-find-java into 
lp:zorba/data-formatting-module has been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
-- 
https://code.launchpad.net/~zorba-coders/zorba/data-formatting-find-java/+merge/108550
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/schema-tools-find-java-2012-06-04T21-50-51.66Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/util-jvm-installable into 
lp:zorba/util-jvm-module has been updated.

Commit Message changed to:

Add a cobbled-together but functional setup for installing util-jvm and 
allowing Java modules to be built based on an installed image.

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is requested to review the proposed merge of 
lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/schema-tools-find-java into 
lp:zorba/schema-tools-module has been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/schema-tools-find-java into lp:zorba/schema-tools-module

2012-06-04 Thread Zorba Build Bot
Validation queue job schema-tools-find-java-2012-06-04T21-50-51.66Z is 
finished. The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/schema-tools-find-java/+merge/108549
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/util-jvm-installable into 
lp:zorba/util-jvm-module has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/util-jvm-installable-2012-06-04T23-16-50.006Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread Zorba Build Bot
Validation queue job util-jvm-installable-2012-06-04T23-16-50.006Z is finished. 
The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/xqj_installers into lp:zorba

2012-06-04 Thread Chris Hillery
Review: Needs Fixing

Mostly looks excellent. I still think the SITEARCH issue is a problem, but I 
see that it also exists in the Java SWIG code as Rodolfo says, so I will file a 
separate bug report about that.

The only change I would request: Rodolfo, in swig/xqj/CMakeLists.txt, you have 
separate code using the new CMake UseJava functionality if the CMake version 
is greater than 2.8.6. I like that, because the ADD_CUSTOM_COMMAND(POST_BUILD) 
technique used otherwise is a kludge that doesn't get the dependencies right. 
Could you also use that same UseJava technique in swig/java/CMakeLists.txt ?
-- 
https://code.launchpad.net/~zorba-coders/zorba/xqj_installers/+merge/108257
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~zorba-coders/zorba/util-jvm-installable into lp:zorba/util-jvm-module

2012-06-04 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/util-jvm-installable into 
lp:zorba/util-jvm-module has been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
-- 
https://code.launchpad.net/~zorba-coders/zorba/util-jvm-installable/+merge/108548
Your team Zorba Coders is subscribed to branch lp:zorba/util-jvm-module.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Bug 1008504] Re: parse-xml-fragment triggers schema error

2012-06-04 Thread Chris Hillery
Pretty straightforward bug here: the deprecated backwards-compatible
version of parse-xml-fragment() converts the arguments string (EDSlwf
in the above example) into an parse-xml-options:options element. The
w flag introduces a parse-xml-options:strip-boundary-space/ element
into that options. But there's no strip-boundary-space element defined
by the schema. There doesn't seem to be anything at all related to
boundary or whitespace in the options schema, in fact, which leads
me to believe that perhaps that functionality simply never got
implemented.

-- 
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1008504

Title:
  parse-xml-fragment triggers schema error

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When running the attached example a schema validation error inside the
  xml module is thrown:

  /home/tryzorba/repo/zorba-2.2.0/build/URI_PATH/com/zorba-
  xquery/www/modules/xml.xq:346,5: dynamic error [err:XQDY0027]:
  Schema validation failed: no declaration found for element 'parse-
  xml-options:strip-boundary-space': unexpected validity property

  
-Example:
  xquery version 3.0;

  import module namespace parse-xml = http://www.zorba-
  xquery.com/modules/xml;

  parse-xml:parse-xml-fragment(xmltest/xml,EDSlwf)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1008504/+subscriptions

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp