[Zorba-coders] [Bug 874679] Re: Clean up second arg to ADD_TEST_DIRECTORY()

2012-04-17 Thread Chris Hillery
** Changed in: zorba
   Status: Confirmed = 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/874679

Title:
  Clean up second arg to ADD_TEST_DIRECTORY()

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  The second (optional) argument to ADD_TEST_DIRECTORY() contains a list
  of test cases that should be skipped (ie, not created at all). This is
  extremely dangerous. By bypassing them entirely, there is no way of
  detecting when the bug is fixed. We have already had serious problems
  caused by test removal hiding the introduction of new bugs in the
  past.

  I understand that removing a test that segfaults is necessary on
  Windows in order for the tests to run unattended. However, it should
  never be done on other platforms. So ADD_TEST_DIRECTORY() should
  instead mark the bugs as EXPECTED_FAILURE()s on other platforms. (I
  realize that *currently* all places that pass a list of segfaulting
  tests to ADD_TEST_DIRECTORY() are inside an IF(WIN32), but this is not
  sufficient - ADD_TEST_DIRECTORY() itself must enforce that the tests
  are skipped only on Windows, or else sooner or later someone will
  accidentally do it wrong. There's currently no documentation either,
  making this much more likely.)

  In order to do this, ADD_TEST_DIRECTORY() needs to either accept a
  list of pairs of the form (testname, bug ID), or else accept two
  parallel lists (testnames and bug IDs), so it can pass the bug number
  to EXPECTED_FAILURE().

  A slightly more difficult approach would be to introduce a new
  KNOWN_SEGFAULT() macro that handled this in conjunction with
  ADD_TEST_DIRECTORY(). I'm not totally sure this is a good idea,
  though, because it would (silently) not do anything for tests added by
  any means other than ADD_TEST_DIRECTORY().

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/874679/+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 857263] Re: Build/install doc is terrible

2012-04-17 Thread Chris Hillery
** 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/857263

Title:
  Build/install doc is terrible

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  The build.dox, installation.dox, build_ubuntu.dox, and several related
  files have numerous errors. Here are some:

  - build.dox mentions there are a number of build options, but does not
  document them. (It previously pointed to a page of *XQuery* options,
  but I deleted that link.)

  - installation.dox includes stuff about building (not just
  installing), and that stuff is all Makefile-specific.

  - installation.dox refers to setting LD_LIBRARY_PATH and so on which
  is not, as far as I know, required. The final section about ldconfig
  on Ubuntu is also obsolete and wrong as far as I know.

  - build_ubuntu.dox is not entirely accurate, but a lot of it is
  redundant with the main build instructions.

  - If we have build_ubuntu.dox, we should probably have separate pages
  for (at least) Fedora, Windows, and MacOS too. There is a lot of
  information specific to Linux, Windows, and MacOS on the main build
  page that could probably be spread around.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/857263/+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/bug928987 into lp:zorba

2012-04-17 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bug928987 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
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/bug928987 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug928987-2012-04-17T06-19-09.575Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
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


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

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug928987 into lp:zorba failed. 
Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job bug928987-2012-04-17T06-19-09.575Z is finished.  The
  final status was:

  

  1 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
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/bug928987 into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug928987 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug928987/+merge/99445
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/markos-scratch into lp:zorba

2012-04-17 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has 
been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
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/markos-scratch into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-04-17T08-43-58.329Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
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/markos-scratch into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue job markos-scratch-2012-04-17T08-43-58.329Z is finished. The 
final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
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/markos-scratch into lp:zorba

2012-04-17 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has 
been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102192
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/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Chris Hillery
Chris Hillery has proposed merging 
lp:~zorba-coders/zorba/bug-874679-add-test-directory into lp:zorba.

Requested reviews:
  Chris Hillery (ceejatec)
  Juan Zacarias (juan457)
Related bugs:
  Bug #874679 in Zorba: Clean up second arg to ADD_TEST_DIRECTORY()
  https://bugs.launchpad.net/zorba/+bug/874679

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'cmake_modules/ZorbaModule.cmake'
--- cmake_modules/ZorbaModule.cmake	2012-04-16 20:56:43 +
+++ cmake_modules/ZorbaModule.cmake	2012-04-17 09:46:23 +
@@ -629,41 +629,101 @@
 
 ENDMACRO(expected_failure)
 
-# Convenience macro for adding tests in a standard format.
+# Convenience macro for adding tests in a standard format. All test
+# cases (.xq files) in Queries subdirectory of the named directory
+# will be automatically added for CTest. Each test will be named
+# ${PROJECT_NAME}/filename, where filename is the path relative to
+# the Queries/ subdirectory.
+# 
+# The optional second argument exists for Windows. Normally,
+# EXPECTED_FAILURE() can be used even for tests that crash
+# (segfault). However, on Windows, when a test crashes, it pops up a
+# dialog box. This prevents tests running unattended.  Since it is not
+# possible with CMake to remove a test once added,
+# ADD_TEST_DIRECTORY() needs to be told not to add the test at all on
+# Windows platforms. Therefore, after the directory name, you may
+# additional arguments in the form (testname bugnumber testname
+# bugnumber ...) to ADD_TEST_DIRECTORY().  On Windows, the named tests
+# will be skipped entirely. On other platforms, they will be passed to
+# EXPECTED_FAILURE() with the corresponding bug number.
+#
+# This second argument should only be used for test cases that are 
+# currently *crashing* on Windows. If the test is simply failing, then
+# you should use EXPECTED_FAILURE() after calling ADD_TEST_DIRECTORY().
+#
 # Parameters:
 #   TEST_DIR - all the .xq files in this directory will be added as tests
-#   ARGV1- if this is present, it will be interpreted as a list of
-#  exceptions. The list items will be removed from the list of
-#  files found in TEST_DIR.
+#   (additional args)  - expected crashing tests, as discussed above.
 MACRO (ADD_TEST_DIRECTORY TEST_DIR)
   # QQQ error-check: Queries directory exists, some tests found...
   FILE(GLOB_RECURSE TESTFILES FOLLOW_SYMLINKS
 RELATIVE ${TEST_DIR}/Queries ${TEST_DIR}/Queries/*.xq)
 
-  FOREACH (EXCEPTION ${ARGV1})
-LIST (REMOVE_ITEM TESTFILES ${EXCEPTION})
-  ENDFOREACH (EXCEPTION)
+  # Convert extra arguments to two lists: test names and bug IDs
+  SET (crash_tests)
+  SET (crash_bugids)
+  SET (known_crashes ${ARGN})
+  LIST (LENGTH known_crashes num_crashes)
+  WHILE (num_crashes GREATER 0)
+LIST (GET known_crashes 0 _testcase)
+LIST (APPEND crash_tests ${_testcase})
+LIST (GET known_crashes 1 _bugid)
+LIST (APPEND crash_bugids ${_bugid})
+LIST (REMOVE_AT known_crashes 0 1)
+MATH (EXPR num_crashes ${num_crashes} - 2)
+  ENDWHILE (num_crashes GREATER 0)
 
+  IF(WIN32)
+SET(PATH_SEP ,)
+  ELSE(WIN32)
+SET(PATH_SEP :)
+  ENDIF(WIN32)
   SET(TESTCOUNTER 0)
   FOREACH(TESTFILE ${TESTFILES})
 SET(TESTNAME ${PROJECT_NAME}/${TESTFILE})
 
-IF(WIN32)
-  SET(PATH_SEP ,)
-ELSE(WIN32)
-  SET(PATH_SEP :)
-ENDIF(WIN32)
-ADD_TEST(${TESTNAME} ${Zorba_TESTDRIVER}
-  --rbkt-src ${TEST_DIR}
-  --module-path ${CMAKE_BINARY_DIR}/URI_PATH/${PATH_SEP}${SECONDARY_MODULE_PATHS}
-  ${TESTFILE})
-
-MATH(EXPR TESTCOUNTER ${TESTCOUNTER}+1)
-MATH(EXPR TESTMOD ${TESTCOUNTER}%100)
-IF (${TESTMOD} EQUAL 0)
-  MESSAGE(STATUS Adding another 100 Tests)
-ENDIF (${TESTMOD} EQUAL 0)
+# See if this test is in the known-crashing list
+SET (_crash_bugid)
+LIST (FIND crash_tests ${TESTNAME} _crash_idx)
+IF (${_crash_idx} GREATER -1)
+  # Get corresponding bug ID
+  LIST (GET crash_bugids ${_crash_idx} _crash_bugid)
+  # Remove from known-crashing list
+  LIST (REMOVE_AT crash_tests ${_crash_idx})
+ENDIF (${_crash_idx} GREATER -1)
+
+# On Windows, skip calling ADD_TEST() for any known crashing tests
+IF (WIN32 AND (${_crash_idx} GREATER -1) )
+  MESSAGE (STATUS WARNING: Skipping test case ${TESTNAME} which is expected to crash - bug ${_crash_bugid})
+ELSE (WIN32 AND (${_crash_idx} GREATER -1) )
+  ADD_TEST(${TESTNAME} ${Zorba_TESTDRIVER}
+	--rbkt-src ${TEST_DIR}
+	--module-path
+	${CMAKE_BINARY_DIR}/URI_PATH/${PATH_SEP}${SECONDARY_MODULE_PATHS}
+	${TESTFILE})
+
+  # On non-Windows, call EXPECTED_FAILURE() for known crashes
+  IF (${_crash_idx} GREATER -1)
+	MESSAGE (STATUS Marking test case ${TESTNAME} as expected to crash - bug ${bugid})
+	

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Chris Hillery
Chris Hillery has proposed merging 
lp:~zorba-coders/zorba/bug-874679-image-module into lp:zorba/image-module.

Requested reviews:
  Chris Hillery (ceejatec)
  Juan Zacarias (juan457)
Related bugs:
  Bug #874679 in Zorba: Clean up second arg to ADD_TEST_DIRECTORY()
  https://bugs.launchpad.net/zorba/+bug/874679

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-module.
=== modified file 'src/com/zorba-xquery/www/modules/image/CMakeLists.txt'
--- src/com/zorba-xquery/www/modules/image/CMakeLists.txt	2011-10-09 00:51:44 +
+++ src/com/zorba-xquery/www/modules/image/CMakeLists.txt	2012-04-17 09:47:22 +
@@ -47,39 +47,36 @@
   URI http://www.zorba-xquery.com/modules/image/animation;
   EXTRA_SOURCES image_commons
   LINK_LIBRARIES ${ImageMagick_LIBRARIES})
-  
-SET (EXCEPTION_LIST)
+
+SET (KNOWN_CRASHES)
 IF (WIN32)
-  # bug#867693
-  LIST (APPEND EXCEPTION_LIST
-image/basic_svg.xq
+  LIST (APPEND KNOWN_CRASHES
+zorba_image_module/image/basic_svg.xq 867693
   )
 
   IF (CMAKE_BUILD_TYPE STREQUAL Debug)
-# bug#867693
-LIST (APPEND EXCEPTION_LIST
-  image/basic_svg.xq
-  image/basic.xq
-  image/basic_convert.xq
-  image/basic_create.xq
-  image/paint_1.xq
-  image/paint_2.xq
-  image/paint_3.xq
-  image/paint_4.xq
-  image/paint_5.xq
-  image/paint_6.xq
-  image/paint_circles.xq
-  image/paint_different_lines.xq
-  image/paint_polygon.xq
-  image/paint_polyline.xq
-  image/paint_rectangles.xq
-  image/paint_stroked_polyline.xq
-  image/paint_text.xq
+LIST (APPEND KNOWN_CRASHES
+  zorba_image_module/image/basic.xq 867693
+  zorba_image_module/image/basic_convert.xq 867693
+  zorba_image_module/image/basic_create.xq 867693
+  zorba_image_module/image/paint_1.xq 867693
+  zorba_image_module/image/paint_2.xq 867693
+  zorba_image_module/image/paint_3.xq 867693
+  zorba_image_module/image/paint_4.xq 867693
+  zorba_image_module/image/paint_5.xq 867693
+  zorba_image_module/image/paint_6.xq 867693
+  zorba_image_module/image/paint_circles.xq 867693
+  zorba_image_module/image/paint_different_lines.xq 867693
+  zorba_image_module/image/paint_polygon.xq 867693
+  zorba_image_module/image/paint_polyline.xq 867693
+  zorba_image_module/image/paint_rectangles.xq 867693
+  zorba_image_module/image/paint_stroked_polyline.xq 867693
+  zorba_image_module/image/paint_text.xq 867693
 )
   ENDIF (CMAKE_BUILD_TYPE STREQUAL Debug)
 ENDIF (WIN32)
 
-ADD_TEST_DIRECTORY (${PROJECT_SOURCE_DIR}/test ${EXCEPTION_LIST})
+ADD_TEST_DIRECTORY (${PROJECT_SOURCE_DIR}/test ${KNOWN_CRASHES})
 
   ELSE (ImageMagick_FOUND)
 MESSAGE(STATUS Magick++   component found: ${ImageMagick_Magick++_FOUND})

-- 
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/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-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-874679-process-module into lp:zorba/process-module

2012-04-17 Thread Chris Hillery
Chris Hillery has proposed merging 
lp:~zorba-coders/zorba/bug-874679-process-module into lp:zorba/process-module.

Requested reviews:
  Chris Hillery (ceejatec)
  Juan Zacarias (juan457)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-process-module/+merge/102270
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-process-module/+merge/102270
Your team Zorba Coders is subscribed to branch lp:zorba/process-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2011-10-09 14:30:51 +
+++ CMakeLists.txt	2012-04-17 09:49:19 +
@@ -25,7 +25,7 @@
 
 ADD_SUBDIRECTORY(src)
 
-ADD_TEST_DIRECTORY (${PROJECT_SOURCE_DIR}/test ${EXCEPTION_LIST})
+ADD_TEST_DIRECTORY (${PROJECT_SOURCE_DIR}/test)
 
 IF (WIN32)
   EXPECTED_FAILURE (zorba_process_module/process.xq 866979)

-- 
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-874679-data-cleaning-module into lp:zorba/data-cleaning-module

2012-04-17 Thread Chris Hillery
Chris Hillery has proposed merging 
lp:~zorba-coders/zorba/bug-874679-data-cleaning-module into 
lp:zorba/data-cleaning-module.

Requested reviews:
  Chris Hillery (ceejatec)
  Juan Zacarias (juan457)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-data-cleaning-module/+merge/102271
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-data-cleaning-module/+merge/102271
Your team Zorba Coders is subscribed to branch lp:zorba/data-cleaning-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2011-12-22 13:56:52 +
+++ CMakeLists.txt	2012-04-17 09:49:20 +
@@ -23,7 +23,7 @@
 FIND_PACKAGE (Zorba REQUIRED HINTS ${ZORBA_BUILD_DIR})
 INCLUDE (${Zorba_USE_FILE})
 
-ADD_TEST_DIRECTORY(${PROJECT_SOURCE_DIR}/test ${EXCEPTION_LIST})
+ADD_TEST_DIRECTORY(${PROJECT_SOURCE_DIR}/test)
 
 ADD_SUBDIRECTORY(src)
 

-- 
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 874679] Re: Clean up second arg to ADD_TEST_DIRECTORY()

2012-04-17 Thread Chris Hillery
I have revamped the known-crashing test list arguments to
ADD_TEST_DIRECTORY(). It now defers to EXPECTED_FAILURE() on non-Win32
platforms. It also checks that all tests passed to it actually exist. I
extended the macro documentation accordingly.

The only module currently making use of this functionality is the image
module; I have modified it accordingly to pass bug numbers and name the
tests correctly.


** Branch linked: lp:~zorba-coders/zorba/bug-874679-add-test-directory

** Branch linked: lp:~zorba-coders/zorba/bug-874679-image-module

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

Title:
  Clean up second arg to ADD_TEST_DIRECTORY()

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  The second (optional) argument to ADD_TEST_DIRECTORY() contains a list
  of test cases that should be skipped (ie, not created at all). This is
  extremely dangerous. By bypassing them entirely, there is no way of
  detecting when the bug is fixed. We have already had serious problems
  caused by test removal hiding the introduction of new bugs in the
  past.

  I understand that removing a test that segfaults is necessary on
  Windows in order for the tests to run unattended. However, it should
  never be done on other platforms. So ADD_TEST_DIRECTORY() should
  instead mark the bugs as EXPECTED_FAILURE()s on other platforms. (I
  realize that *currently* all places that pass a list of segfaulting
  tests to ADD_TEST_DIRECTORY() are inside an IF(WIN32), but this is not
  sufficient - ADD_TEST_DIRECTORY() itself must enforce that the tests
  are skipped only on Windows, or else sooner or later someone will
  accidentally do it wrong. There's currently no documentation either,
  making this much more likely.)

  In order to do this, ADD_TEST_DIRECTORY() needs to either accept a
  list of pairs of the form (testname, bug ID), or else accept two
  parallel lists (testnames and bug IDs), so it can pass the bug number
  to EXPECTED_FAILURE().

  A slightly more difficult approach would be to introduce a new
  KNOWN_SEGFAULT() macro that handled this in conjunction with
  ADD_TEST_DIRECTORY(). I'm not totally sure this is a good idea,
  though, because it would (silently) not do anything for tests added by
  any means other than ADD_TEST_DIRECTORY().

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/874679/+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/bug-874679-process-module into lp:zorba/process-module

2012-04-17 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-process-module/+merge/102270
Your team Zorba Coders is subscribed to branch lp:zorba/process-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-874679-add-test-directory into lp:zorba

2012-04-17 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug-874679-add-test-directory into 
lp:zorba has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug-874679-add-test-directory-2012-04-17T10-03-56.024Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Matthias Brantner
Matthias Brantner has proposed merging 
lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq.

Requested reviews:
  Matthias Brantner (matthias-brantner)
  Markos Zaharioudakis (markos-za)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275

- plan serializer for jsoniq items and iterators
- some fixes for the build without json (still not compiling because of 
problems with the collection functions)
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.
=== modified file 'include/zorba/pregenerated/diagnostic_list.h'
--- include/zorba/pregenerated/diagnostic_list.h	2012-04-11 20:22:52 +
+++ include/zorba/pregenerated/diagnostic_list.h	2012-04-17 10:09:20 +
@@ -800,13 +800,13 @@
 
 namespace jerr {
 
+#if defined(ZORBA_WITH_JSON)
 extern ZORBA_DLL_PUBLIC JSONiqErrorCode JNSE0012;
 
 extern ZORBA_DLL_PUBLIC JSONiqErrorCode JNSE0013;
 
 extern ZORBA_DLL_PUBLIC JSONiqErrorCode JNSE0014;
 
-#if defined(ZORBA_WITH_JSON)
 extern ZORBA_DLL_PUBLIC JSONiqErrorCode JNTY0002;
 
 extern ZORBA_DLL_PUBLIC JSONiqErrorCode JNTY0003;

=== modified file 'src/api/serialization/serializer.cpp'
--- src/api/serialization/serializer.cpp	2012-04-11 20:22:52 +
+++ src/api/serialization/serializer.cpp	2012-04-17 10:09:20 +
@@ -2741,8 +2741,11 @@
 // the JSONIQ method is consider XML-based, although you will certainly
 // get errors if you attempt to serialize JDM this way.
 if (method != PARAMETER_VALUE_XML 
-method != PARAMETER_VALUE_XHTML 
-method != PARAMETER_VALUE_JSONIQ) {
+method != PARAMETER_VALUE_XHTML
+#ifdef ZORBA_WITH_JSON
+ method != PARAMETER_VALUE_JSONIQ
+#endif
+  ) {
   throw ZORBA_EXCEPTION(
 zerr::ZAPI0070_INVALID_SERIALIZATION_METHOD_FOR_SAX,
 ERROR_PARAMS( method )

=== modified file 'src/diagnostics/diagnostic_en.xml'
--- src/diagnostics/diagnostic_en.xml	2012-04-11 20:22:52 +
+++ src/diagnostics/diagnostic_en.xml	2012-04-17 10:09:20 +
@@ -2483,7 +2483,7 @@
 
 !--// CloudScript ///--
 
-diagnostic code=JNSE0012
+diagnostic code=JNSE0012 if=defined(ZORBA_WITH_JSON)
   commentIt is a dynamic error to serialize a sequence of less
   or more than one item with the JSON output method if the
   cloudscript-serialization-multiple-items is set to no.
@@ -2492,7 +2492,7 @@
   valueCannot serialize multiple top-level items as JSON/value
 /diagnostic
 
-diagnostic code=JNSE0013
+diagnostic code=JNSE0013 if=defined(ZORBA_WITH_JSON)
   commentIt is a dynamic error to serialize an atomic value not
   supported by JSON or a node with the JSON output method and with
   the cloudscript-serialization-extensions serialization parameter
@@ -2501,7 +2501,7 @@
   valueCannot serialize value as JSON: $1/value
 /diagnostic
 
-diagnostic code=JNSE0014
+diagnostic code=JNSE0014 if=defined(ZORBA_WITH_JSON)
   commentIt is a dynamic error to serialize a function with the
   JSON output method./comment
   valueCannot serialize a function item as JSON/value

=== modified file 'src/diagnostics/pregenerated/diagnostic_list.cpp'
--- src/diagnostics/pregenerated/diagnostic_list.cpp	2012-04-11 20:22:52 +
+++ src/diagnostics/pregenerated/diagnostic_list.cpp	2012-04-17 10:09:20 +
@@ -1176,6 +1176,7 @@
 
 namespace jerr {
 
+#if defined(ZORBA_WITH_JSON)
 JSONiqErrorCode JNSE0012( JNSE0012 );
 
 
@@ -1185,7 +1186,6 @@
 JSONiqErrorCode JNSE0014( JNSE0014 );
 
 
-#if defined(ZORBA_WITH_JSON)
 JSONiqErrorCode JNTY0002( JNTY0002 );
 
 

=== modified file 'src/diagnostics/pregenerated/dict_en.cpp'
--- src/diagnostics/pregenerated/dict_en.cpp	2012-04-11 20:22:52 +
+++ src/diagnostics/pregenerated/dict_en.cpp	2012-04-17 10:09:20 +
@@ -98,9 +98,15 @@
 #if defined(ZORBA_WITH_JSON)
   { JNDY0003, \$1\: pair with the same name already exists in object },
 #endif
+#if defined(ZORBA_WITH_JSON)
   { JNSE0012, Cannot serialize multiple top-level items as JSON },
+#endif
+#if defined(ZORBA_WITH_JSON)
   { JNSE0013, Cannot serialize value as JSON: $1 },
+#endif
+#if defined(ZORBA_WITH_JSON)
   { JNSE0014, Cannot serialize a function item as JSON },
+#endif
 #if defined(ZORBA_WITH_JSON)
   { JNTY0002, $1 },
 #endif

=== modified file 'src/runtime/json/json_constructors.cpp'
--- src/runtime/json/json_constructors.cpp	2012-04-12 09:18:43 +
+++ src/runtime/json/json_constructors.cpp	2012-04-17 10:09:20 +
@@ -94,7 +94,7 @@
   serialize_baseclass(ar, 
   (NaryBaseIteratorJSONArrayIterator, JSONArrayIteratorState*)this);
 
-  //ar  theCopyInputs; // TODO
+  SERIALIZE_BOOL_VEC(theCopyInputs);
 }
 
 
@@ -191,7 +191,7 @@
   serialize_baseclass(ar, 
   (NaryBaseIteratorJSONObjectIterator, JSONObjectIteratorState*)this);
 
-  //ar  

[Zorba-coders] [Bug 891650] Re: Use of last()

2012-04-17 Thread Markos Zaharioudakis
** Changed in: zorba
   Status: New = Fix Committed

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

Title:
  Use of last()

Status in Zorba - The XQuery Processor:
  Fix Committed

Bug description:
  The following query..

x//last()

  ..raises XPDY0002 instead of returning the integer 1.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/891650/+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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq has been updated.

Commit Message changed to:

plan serializer for jsoniq items and iterators

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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:~diogo-simoes89/zorba/data-cleaning-thesaurus into lp:zorba/data-cleaning-module

2012-04-17 Thread Diogo Simões
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-module has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-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-874679-add-test-directory into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue job bug-874679-add-test-directory-2012-04-17T10-03-56.024Z is 
finished. The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1, 
Needs Fixing  1, Pending  1. Got: 1 Approve, 1 Pending.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-874679-add-test-directory into 
lp:zorba has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-plan_serialization-2012-04-17T10-36-14.691Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq failed. Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job jsoniq-plan_serialization-2012-04-17T10-36-14.691Z is
  finished.  The final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-plan_serialization-2012-04-17T10-49-05.531Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq failed. Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job jsoniq-plan_serialization-2012-04-17T10-49-05.531Z is
  finished.  The final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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/jsoniq-plan_serialization into lp:~zorba-coders/zorba/jsoniq

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/jsoniq-plan_serialization into 
lp:~zorba-coders/zorba/jsoniq has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-plan_serialization/+merge/102275
Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/jsoniq.

-- 
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:~diogo-simoes89/zorba/data-cleaning-thesaurus into lp:zorba/data-cleaning-module

2012-04-17 Thread Diogo Simões
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-module has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-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:~diogo-simoes89/zorba/data-cleaning-thesaurus into lp:zorba/data-cleaning-module

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/data-cleaning-thesaurus-2012-04-17T11-34-47.556Z/log.html
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-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:~diogo-simoes89/zorba/data-cleaning-thesaurus into lp:zorba/data-cleaning-module

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-module failed. Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job data-cleaning-thesaurus-2012-04-17T11-34-47.556Z is
  finished.  The final status was:

  

  2 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-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:~diogo-simoes89/zorba/data-cleaning-thesaurus into lp:zorba/data-cleaning-module

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-module has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning-thesaurus/+merge/100683
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning-thesaurus into 
lp:zorba/data-cleaning-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 984031] [NEW] negative result for fib(95)

2012-04-17 Thread Matthias Brantner
Public bug reported:

declare function local:fib($n as xs:integer) as xs:integer
{
 if ($n eq 0) then 0
 else if ($n eq 1) then 1
 else local:fib($n - 1) + local:fib($n - 2)
};

local:fib(95)

gives  -4953053512429003327 but should return 31940434634990099905. This
seems to be a overflow in numerics. fib(90) gives the correct result.

** Affects: zorba
 Importance: Critical
 Assignee: Paul J. Lucas (paul-lucas)
 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/984031

Title:
  negative result for fib(95)

Status in Zorba - The XQuery Processor:
  New

Bug description:
  declare function local:fib($n as xs:integer) as xs:integer
  {
   if ($n eq 0) then 0
   else if ($n eq 1) then 1
   else local:fib($n - 1) + local:fib($n - 2)
  };

  local:fib(95)

  gives  -4953053512429003327 but should return 31940434634990099905.
  This seems to be a overflow in numerics. fib(90) gives the correct
  result.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/984031/+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 984031] Re: negative result for fib(95)

2012-04-17 Thread Paul J. Lucas
I thought we agreed a long time ago neither not to check for either
overflows or underflows when ZORBA_WITH_BIG_INTEGER=OFF.

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

Title:
  negative result for fib(95)

Status in Zorba - The XQuery Processor:
  New

Bug description:
  declare function local:fib($n as xs:integer) as xs:integer
  {
   if ($n eq 0) then 0
   else if ($n eq 1) then 1
   else local:fib($n - 1) + local:fib($n - 2)
  };

  local:fib(95)

  gives  -4953053512429003327 but should return 31940434634990099905.
  This seems to be a overflow in numerics. fib(90) gives the correct
  result.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/984031/+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 984031] Re: negative result for fib(95)

2012-04-17 Thread Paul J. Lucas
Since there's apparently no way to edit a comment, the previous comment
should read:

I thought we agreed a long time ago not to check for either overflows or
underflows when ZORBA_WITH_BIG_INTEGER=OFF.

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

Title:
  negative result for fib(95)

Status in Zorba - The XQuery Processor:
  New

Bug description:
  declare function local:fib($n as xs:integer) as xs:integer
  {
   if ($n eq 0) then 0
   else if ($n eq 1) then 1
   else local:fib($n - 1) + local:fib($n - 2)
  };

  local:fib(95)

  gives  -4953053512429003327 but should return 31940434634990099905.
  This seems to be a overflow in numerics. fib(90) gives the correct
  result.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/984031/+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/module-schema-tools into lp:zorba

2012-04-17 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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


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

2012-04-17 Thread Cezar Andrei
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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] [Bug 867156] Re: Import schema segfaults

2012-04-17 Thread Cezar Andrei
I'll take a look at it.

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

Title:
  Import schema segfaults

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Starting program: /Users/wcandillon/28msec/zorba/build/dist/bin/zorba -q 
CheckTaxonomy.xq -f
  Reading symbols for shared libraries 
... done
  Reading symbols for shared libraries . done
  Reading symbols for shared libraries  done
  Reading symbols for shared libraries ... done

  Program received signal EXC_BAD_ACCESS, Could not access memory.
  Reason: 13 at address: 0x
  0x7fff87b4b062 in std::istream::sentry::sentry ()
  (gdb) where
  #0  0x7fff87b4b062 in std::istream::sentry::sentry ()
  #1  0x7fff87b4b553 in std::istream::read ()
  #2  0x00010162168d in zorba::IstreamBinInputStream::readBytes ()
  #3  0x000103d921e3 in xercesc_2_8::XMLReader::refreshRawBuffer ()
  #4  0x000103d93b47 in xercesc_2_8::XMLReader::XMLReader ()
  #5  0x000103d93c39 in xercesc_2_8::XMLReader::XMLReader ()
  #6  0x000103d17eda in xercesc_2_8::ReaderMgr::createReader ()
  #7  0x000103cef908 in xercesc_2_8::IGXMLScanner::scanReset ()
  #8  0x000103cedc18 in xercesc_2_8::IGXMLScanner::scanDocument ()
  #9  0x000103c66998 in xercesc_2_8::AbstractDOMParser::parse ()
  #10 0x000103d58c77 in xercesc_2_8::TraverseSchema::preprocessImport ()
  #11 0x000103d57e72 in xercesc_2_8::TraverseSchema::preprocessChildren ()
  #12 0x000103d58690 in xercesc_2_8::TraverseSchema::preprocessSchema ()
  #13 0x000103d6d899 in xercesc_2_8::TraverseSchema::TraverseSchema ()
  #14 0x000103d6d972 in xercesc_2_8::TraverseSchema::TraverseSchema ()
  #15 0x000103d2eb5d in xercesc_2_8::SGXMLScanner::loadXMLSchemaGrammar ()
  #16 0x000103d2ee01 in xercesc_2_8::SGXMLScanner::loadGrammar ()
  #17 0x000103d9a0e5 in xercesc_2_8::XMLScanner::loadGrammar ()
  #18 0x000103d9a1a7 in xercesc_2_8::XMLScanner::loadGrammar ()
  #19 0x000103d2879a in xercesc_2_8::SAX2XMLReaderImpl::loadGrammar ()
  #20 0x000101614c4d in zorba::Schema::registerXSD ()
  #21 0x000100347e7a in zorba::TranslatorImpl::import_schema ()
  #22 0x000100290924 in zorba::TranslatorImpl::begin_visit ()
  #23 0x00010019f6e0 in zorba::SchemaImport::accept ()
  #24 0x0001001ab8ac in zorba::SIND_DeclList::accept ()
  #25 0x00010019f52e in zorba::Prolog::accept ()
  #26 0x00010019f21d in zorba::MainModule::accept ()
  #27 0x000100214d7d in zorba::translate_aux ()
  #28 0x0001002148eb in zorba::translate ()
  #29 0x0001005d87c6 in zorba::XQueryCompiler::normalize ()
  #30 0x0001005d89c3 in zorba::XQueryCompiler::compile ()
  #31 0x0001005da700 in zorba::XQueryCompiler::compile ()
  #32 0x00010004ba8a in zorba::XQueryImpl::doCompile ()
  #33 0x00010004c796 in zorba::XQueryImpl::compile ()
  #34 0x000153e8 in compileAndExecute ()
  #35 0x00019a87 in main ()

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/867156/+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 867156] Re: Import schema segfaults

2012-04-17 Thread Cezar Andrei
Zorba knows without any import about the built-in schema types but those
are only a small subset of schema4schema. Depending on which set is used
it is or is not required to import schema4schema.

** Changed in: zorba
 Assignee: Chris Hillery (ceejatec) = Cezar Andrei (cezar-andrei)

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

Title:
  Import schema segfaults

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Starting program: /Users/wcandillon/28msec/zorba/build/dist/bin/zorba -q 
CheckTaxonomy.xq -f
  Reading symbols for shared libraries 
... done
  Reading symbols for shared libraries . done
  Reading symbols for shared libraries  done
  Reading symbols for shared libraries ... done

  Program received signal EXC_BAD_ACCESS, Could not access memory.
  Reason: 13 at address: 0x
  0x7fff87b4b062 in std::istream::sentry::sentry ()
  (gdb) where
  #0  0x7fff87b4b062 in std::istream::sentry::sentry ()
  #1  0x7fff87b4b553 in std::istream::read ()
  #2  0x00010162168d in zorba::IstreamBinInputStream::readBytes ()
  #3  0x000103d921e3 in xercesc_2_8::XMLReader::refreshRawBuffer ()
  #4  0x000103d93b47 in xercesc_2_8::XMLReader::XMLReader ()
  #5  0x000103d93c39 in xercesc_2_8::XMLReader::XMLReader ()
  #6  0x000103d17eda in xercesc_2_8::ReaderMgr::createReader ()
  #7  0x000103cef908 in xercesc_2_8::IGXMLScanner::scanReset ()
  #8  0x000103cedc18 in xercesc_2_8::IGXMLScanner::scanDocument ()
  #9  0x000103c66998 in xercesc_2_8::AbstractDOMParser::parse ()
  #10 0x000103d58c77 in xercesc_2_8::TraverseSchema::preprocessImport ()
  #11 0x000103d57e72 in xercesc_2_8::TraverseSchema::preprocessChildren ()
  #12 0x000103d58690 in xercesc_2_8::TraverseSchema::preprocessSchema ()
  #13 0x000103d6d899 in xercesc_2_8::TraverseSchema::TraverseSchema ()
  #14 0x000103d6d972 in xercesc_2_8::TraverseSchema::TraverseSchema ()
  #15 0x000103d2eb5d in xercesc_2_8::SGXMLScanner::loadXMLSchemaGrammar ()
  #16 0x000103d2ee01 in xercesc_2_8::SGXMLScanner::loadGrammar ()
  #17 0x000103d9a0e5 in xercesc_2_8::XMLScanner::loadGrammar ()
  #18 0x000103d9a1a7 in xercesc_2_8::XMLScanner::loadGrammar ()
  #19 0x000103d2879a in xercesc_2_8::SAX2XMLReaderImpl::loadGrammar ()
  #20 0x000101614c4d in zorba::Schema::registerXSD ()
  #21 0x000100347e7a in zorba::TranslatorImpl::import_schema ()
  #22 0x000100290924 in zorba::TranslatorImpl::begin_visit ()
  #23 0x00010019f6e0 in zorba::SchemaImport::accept ()
  #24 0x0001001ab8ac in zorba::SIND_DeclList::accept ()
  #25 0x00010019f52e in zorba::Prolog::accept ()
  #26 0x00010019f21d in zorba::MainModule::accept ()
  #27 0x000100214d7d in zorba::translate_aux ()
  #28 0x0001002148eb in zorba::translate ()
  #29 0x0001005d87c6 in zorba::XQueryCompiler::normalize ()
  #30 0x0001005d89c3 in zorba::XQueryCompiler::compile ()
  #31 0x0001005da700 in zorba::XQueryCompiler::compile ()
  #32 0x00010004ba8a in zorba::XQueryImpl::doCompile ()
  #33 0x00010004c796 in zorba::XQueryImpl::compile ()
  #34 0x000153e8 in compileAndExecute ()
  #35 0x00019a87 in main ()

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/867156/+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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-17T15-15-59.439Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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] [Bug 984031] Re: negative result for fib(95)

2012-04-17 Thread Matthias Brantner
Yes, you are absolutely right. It's because I had Zorba compiled without
big number support.

** Changed in: zorba
   Status: New = Invalid

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

Title:
  negative result for fib(95)

Status in Zorba - The XQuery Processor:
  Invalid

Bug description:
  declare function local:fib($n as xs:integer) as xs:integer
  {
   if ($n eq 0) then 0
   else if ($n eq 1) then 1
   else local:fib($n - 1) + local:fib($n - 2)
  };

  local:fib(95)

  gives  -4953053512429003327 but should return 31940434634990099905.
  This seems to be a overflow in numerics. fib(90) gives the correct
  result.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/984031/+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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
failed. Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job module-schema-tools-2012-04-17T15-15-59.439Z is
  finished.  The final status was:

  

  No tests were run - build or configure step must have failed.

  Not commiting changes.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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


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

2012-04-17 Thread Cezar Andrei
Chris, can you check the cmake build dependencies for util-jvm module. I see 
the same error on my box when I use make -j12.

[ 95%] Built target modlib69_xqxq.xq
[ 95%] Built target modlib56_hmac.xq
Scanning dependencies of target modlib70_util-jvm.xq
[ 95%] Scanning dependencies of target modlib71_schema-tools.xq
.Building CXX object 
zorba_modules/zorba_util-jvm_module/src/CMakeFiles/modlib70_util-jvm.xq.dir/util-jvm.xq.src/util-jvm.cpp.o
make[2]: *** No rule to make target 
`LIB_PATH/com/zorba-xquery/www/modules/libutil-jvm_1.0.so', needed by 
`LIB_PATH/com/zorba-xquery/www/modules/libschema-tools_1.0.so'.  Stop.
make[2]: *** Waiting for unfinished jobs
[ 95%] Building CXX object 
zorba_modules/zorba_schema-tools_module/src/CMakeFiles/modlib71_schema-tools.xq.dir/schema-tools.xq.src/schema-tools.cpp.o
[ 95%] Building CXX object 
zorba_modules/zorba_util-jvm_module/src/CMakeFiles/modlib70_util-jvm.xq.dir/util-jvm.xq.src/JavaVMSingleton.cpp.o
Linking CXX shared library 
../../../LIB_PATH/com/zorba-xquery/www/modules/libutil-jvm_1.0.so
make[1]: *** 
[zorba_modules/zorba_schema-tools_module/src/CMakeFiles/modlib71_schema-tools.xq.dir/all]
 Error 2
make[1]: *** Waiting for unfinished jobs
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/bug-983628 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug-983628-2012-04-17T16-27-55.312Z/log.html
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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/markos-scratch into lp:zorba

2012-04-17 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch 
into lp:zorba.

Requested reviews:
  Markos Zaharioudakis (markos-za)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356

fix for bug #980526
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog	2012-04-16 20:56:43 +
+++ ChangeLog	2012-04-17 17:00:30 +
@@ -23,6 +23,7 @@
   * Fixed bug #906494 (default compile with D_FILE_OFFSET_BITS=64)
   * Fixed bug #912586, #912593 and #912722 (assertion failures with lax validation)
   * Fixed bug #921458 (file:read-text-lines() blocking)
+  * Fixed bug #980526 (no-copy rule bug due to global var being set in distant udf)
   * Fixed bug #949910 (has-children may be invoked on all nodes). Internally, zorba::store::Item::getChildren() now returns NULL on node classes without offspring (instead of raising an error).
 
 

=== modified file 'src/compiler/rewriter/tools/dataflow_annotations.cpp'
--- src/compiler/rewriter/tools/dataflow_annotations.cpp	2012-04-16 20:56:43 +
+++ src/compiler/rewriter/tools/dataflow_annotations.cpp	2012-04-17 17:00:30 +
@@ -834,12 +834,15 @@
   sources.erase(sources.begin() + i);
   --i;
 
-  // If this method is called to find the sources of an expr within the
+  // Note: If this method is called to find the sources of an expr within the
   // body of a function_item, then udfCaller-theFo will be NULL. 
+  while(udfCaller-theFo  varExpr-get_udf() != udfCaller-theFo-get_func())
+  {
+udfCaller = udfCaller-thePrev;
+  }
+
   if (udfCaller-theFo)
-  {
-ZORBA_ASSERT(varExpr-get_udf() == udfCaller-theFo-get_func());
-
+  {  
 fo_expr* foExpr = udfCaller-theFo;
 expr* foArg = foExpr-get_arg(varExpr-get_param_pos());
 std::vectorexpr* argSources;

=== modified file 'src/runtime/eval/eval.cpp'
--- src/runtime/eval/eval.cpp	2012-04-16 20:56:43 +
+++ src/runtime/eval/eval.cpp	2012-04-17 17:00:30 +
@@ -62,20 +62,6 @@
 {
 }
 
-///**
-
-/
-void
-EvalIteratorState::reset(PlanState aPlanState) 
-{
-  PlanIteratorState::reset(aPlanState);
-  // When an exception is thrown during an EvalIterator::nextImpl invocation
-  // or when the EvalIterator doesn't return all resulting items of an other reason,
-  // the PlanWrapper is not properly destroyed. Therefore, we destroy it in the reset
-  // to prevent troubles the next time the EvalIterator is used.
-  thePlanWrapper = 0;
-}
-
 
 ///**
 
@@ -116,6 +102,7 @@
 void EvalIterator::serialize(::zorba::serialization::Archiver ar)
 {
   ar.set_serialize_everything();
+
   serialize_baseclass(ar,
   (NaryBaseIteratorEvalIterator, EvalIteratorState*)this);
 

=== modified file 'src/runtime/eval/eval.h'
--- src/runtime/eval/eval.h	2012-04-16 20:56:43 +
+++ src/runtime/eval/eval.h	2012-04-17 17:00:30 +
@@ -34,11 +34,10 @@
   std::auto_ptrCompilerCB  ccb;
   std::auto_ptrdynamic_context dctx;
 
+public:
   EvalIteratorState();
 
   ~EvalIteratorState();
-
-  void reset(PlanState);
 };
 
 

=== added file 'test/rbkt/ExpQueryResults/zorba/no-copy/q21.xml.res'
=== added file 'test/rbkt/Queries/zorba/no-copy/q21.xq'
--- test/rbkt/Queries/zorba/no-copy/q21.xq	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/no-copy/q21.xq	2012-04-17 17:00:30 +
@@ -0,0 +1,86 @@
+import module namespace refl = http://www.zorba-xquery.com/modules/reflection;;
+
+declare namespace ann = http://www.zorba-xquery.com/annotations;;
+
+declare variable $serialization-params as element():=
+serialization-parameters/;
+
+
+declare %ann:sequential function 
+local:set-serialization-parameters($params as element()) as empty-sequence()
+{
+  $serialization-params := $params;
+};
+
+
+declare %ann:sequential function local:set-redirect($url as xs:string) as xs:integer
+{
+  1
+};
+
+
+declare %ann:sequential function local:save()
+{
+  variable $tmp := $serialization-params/..;
+  local:set-redirect(/wiki);
+};
+
+
+local:save()
+
+
+
+
+(:
+
+module namespace resp = http://www.28msec.com/modules/response;;
+
+declare namespace ann = http://www.zorba-xquery.com/annotations;;
+
+declare variable 
+$resp:serialization-params as element(serialization-parameters) :=
+resp:serializer-defaults-xml();
+
+
+declare function 
+resp:serializer-defaults-xml() as element(serialization-parameters)
+{
+  serialization-parameters/
+};
+
+
+declare %ann:sequential function 
+resp:set-serialization-parameters($params as element(serialization-parameters)) as 
+empty-sequence()
+{
+  $resp:serialization-params := $params;
+};
+
+
+declare %ann:sequential function 

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Juan Zacarias
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue job bug-983628-2012-04-17T16-27-55.312Z is finished. The final 
status was:

All tests succeeded!
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1, 
Needs Fixing  1, Pending  1. Got: 1 Approve.
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~paul-lucas/zorba/bug-983628 into lp:zorba has been 
updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Juan Zacarias
Review: Resubmit


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-add-test-directory into lp:zorba

2012-04-17 Thread Juan Zacarias
srry I approved it thinking it was another branch, I am checking the changes 
and will approve as soon as I check all you mentioned in the email
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-add-test-directory/+merge/102268
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


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

2012-04-17 Thread Markos Zaharioudakis
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
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/markos-scratch into lp:zorba

2012-04-17 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has 
been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
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/markos-scratch into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-04-17T18-12-56.039Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
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


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Rodolfo Ochoa
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Rodolfo Ochoa
The proposal to merge lp:~paul-lucas/zorba/bug-983628 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102348
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


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
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


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Rodolfo Ochoa
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
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/markos-scratch into lp:zorba

2012-04-17 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has 
been updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
-- 
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/102356
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug-983628-2012-04-17T18-44-54.123Z/log.html
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
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] [Bug 967284] Re: XQDoc doesn't support invalid XML entities

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  XQDoc doesn't support invalid XML entities

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Given the following HTML as part of an XQDoc comment:

  table
trtdfoonbsp;bar/td/tr
  /table

  one gets the error:

  can not parse as XML for xqdoc: loader parsing error: Entity 'nbsp'
  not defined; raised at /Users/pjl/src/flwor/zorba/repo/feature-
  ft_module/zorba/src/compiler/parsetree/parsenode_print_xqdoc_visitor.cpp:433

  The parser needs to accept nbsp; so a non-breaking space can be
  included in the HTML.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/967284/+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 960425] Re: import statement in XQDoc should not have line number

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  import statement in XQDoc should not have line number

Status in Zorba - The XQuery Processor:
  New

Bug description:
  The example import statement at the top of XQDoc documentation for an
  XQuery module should not have the single line number of 1 -- it's
  silly to have a single line numbered. For multi-line code snippets,
  line numbers are OK (but really should be used only when the
  surrounding documentation actually references specific line numbers).
  For a single-line, it's not as obvious that the 1 is a line number
  since the user can't see any other lines with increasing numbers,
  e.g., 2, 3, 

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/960425/+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 959833] Re: pre blocks in XQDoc have unequal spacing

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  pre blocks in XQDoc have unequal spacing

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Given:

  (:~
   : For example:
   : pre
   :  if ( ft:stem( apples ) eq apple ) ** WRONG **
   : /pre
   : Instead do:
   : pre
   :  if ( ft:stem( apples ) eq ft:stem( apple ) )  ** CORRECT **
   : /pre
   :)

  one gets what's shown in the attached screen-shot. As can be seen,
  there's more space below the line of code than above -- it should be
  equal.

  Additionally, the whitespace between the surrounding text should be
  increased -- see second attached screen-shot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/959833/+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 957416] Re: Links in XQDoc for variables don't show up

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  Links in XQDoc for variables don't show up

Status in Zorba - The XQuery Processor:
  New

Bug description:
  If I document an XQuery variable like:

  (:~
   : Predeclared constant for the Danish
   : a 
href=http://www.w3.org/TR/xmlschema-2/#language;codexs:language/code/a.
   :)
  declare variable $ft:lang-da as xs:language := xs:language(da);

  the generated HTML doesn't include the link to the w3.org site -- see
  attached screenshot. The term xs:language in the documentation should
  be a link but isn't.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/957416/+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 933284] Re: code in XQDoc @error is ignored

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  code in XQDoc @error is ignored

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Within the description of an @error in XQDoc documentation, the
  rendered HTML ignores use of code.../code, for example: the XQDoc:

  @error err:FTST0009 if codeft:current-lang()/code is unsupported.

  generates HTML where the ft:current-lang() is rendered in the same
  font as surrounding text and not in a distinct mono-spaced font.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933284/+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 933709] Re: XQDoc table cells should be vertically aligned at the top

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  XQDoc table cells should be vertically aligned at the top

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When doing XQDoc documentation and including a table, the table
  cells should have the CSS equivalent of the valign=top attribute,
  e.g. td valign=top. Don't forget the same for th.

  The default rendering in most browsers is to vertically center text
  which always looks ugly.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933709/+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 933718] Re: Not enough whitespace before headings in XQDoc

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  Not enough whitespace before headings in XQDoc

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When using h2 in XQDoc, there's not enough whitespace before the
  heading -- see attached screen-shot. The same is probably also true
  for h1, h3, h4, etc.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933718/+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 933713] Re: H2 in XQDoc has extra whitespace

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  H2 in XQDoc has extra whitespace

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When using an h2 element in XQDoc, it renders having extra
  whitespace to the left of the heading -- why?  It should be removed.
  (This probably also happens for h3, h4, etc.) See attached screen-
  shot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933713/+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 933271] Re: XQDoc th should be left-justified

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  XQDoc th should be left-justified

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When including a th within a table in XQDoc documentation, the
  text should be left-justified. (It invariably looks better.)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933271/+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 933257] Re: XQDoc table should have cell borders

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  XQDoc table should have cell borders

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When including a table in XQDoc documentation, the resulting HTML
  should render the table such that there are very thin, light gray
  lines between the table cells.

  Additionally, the table header cells should have a background cell
  color of a light gray as well.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933257/+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 930500] Re: @error XQDoc should use monospaced font for error code

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  @error XQDoc should use monospaced font for error code

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When using @error, the resulting HTML should render the error code in
  a monospaced font, e.g.:

   @error err:XQDY0027 if $options can not be validated against the
  json-options schema.

  The err:XQDY0027 should be monospaced to look better.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/930500/+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 933255] Re: XQDoc table should be centered

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  XQDoc table should be centered

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When including a table in XQDoc documentation, the resulting HTML
  should have the table centered.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/933255/+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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue job bug-983628-2012-04-17T18-44-54.123Z is finished. The final 
status was:

All tests succeeded!
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
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/module-schema-tools into lp:zorba

2012-04-17 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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:~paul-lucas/zorba/bug-983628 into lp:zorba

2012-04-17 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/bug-983628 into lp:zorba has been 
updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-983628/+merge/102367
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


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

2012-04-17 Thread Zorba Build Bot
There are additional revisions which have not been approved in review. Please 
seek review and approval of these new revisions.
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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/module-schema-tools into lp:zorba

2012-04-17 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Commit Message changed to:

Add java classpath to zorbacmd and to Zorba API.
Fix dynamic libraries loading.
Tested it works on Windows with both xsl-fo and xmlbeans modules. All tests are 
passing.

Fixed ExternalModule integration, fixed comment and indentation of previously 
property generated files: src/system/zorba_properties.h and 
bin/zorbacmdproperties_base.h.

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102001
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


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

2012-04-17 Thread Cezar Andrei
Review: Approve

Fixed erroneous merging of swig files.
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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/module-schema-tools into lp:zorba

2012-04-17 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/module-schema-tools-2012-04-17T19-20-05.192Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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] [Bug 930496] Re: pre in XQDoc @param doesn't work properly

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
Milestone: None = 2.5

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

Title:
  pre in XQDoc @param doesn't work properly

Status in Zorba - The XQuery Processor:
  New

Bug description:
  When writing XQDoc, if one has:

   : @param $x What $x is for, for example:
   : pre
   : yadda
   : yadda
   : /pre

  i.e., a pre block as part of the description of a @param, then both
  yaddas appear on the same line, i.e., newlines aren't respected
  despite being in a pre block.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/930496/+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 983628] Re: Latest Integer/Decimal/Float changes break Windows build

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
   Status: New = Fix Committed

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

Title:
  Latest Integer/Decimal/Float changes break Windows build

Status in Zorba - The XQuery Processor:
  Fix Committed

Bug description:
  I happen to think the Windows compiler is just plain wrong. The entire
  *point* of enable_if is to be able to use it via the C++ SFINAE
  technique.  The compiler happily accepts enable_if in other contexts.
  Sigh

  decimal.cpp
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(260) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::Decimal 
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(261) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::Decimal 
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(295) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImpl__int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(295) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImplunsigned __int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(296) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImpl__int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(296) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImplunsigned __int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(297) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImpl__int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(297) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImplunsigned __int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(298) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImpl__int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(298) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImplunsigned __int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(299) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImpl__int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(299) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=zorba::IntegerImplunsigned __int64
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(320) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=bool
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(320) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=bool
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(321) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=bool
  ]
  C:\dev\bug867112\src\zorbatypes\decimal.cpp(321) : error C2794: 'type' : is 
not a member of any direct or indirect base class of 
'std::tr1::enable_if_Test,_Type'
  with
  [
  _Test=false,
  _Type=bool
  ]
  

[Zorba-coders] [Bug 862971] Re: No error upon (almost) duplicate function declarations

2012-04-17 Thread Paul J. Lucas
** Changed in: zorba
 Assignee: (unassigned) = Markos Zaharioudakis (markos-za)

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

Title:
  No error upon (almost) duplicate function declarations

Status in Zorba - The XQuery Processor:
  New

Bug description:
  If you declare two XQuery functions that have the same name and
  signature (arity) in the same file, but the first is declared
  external and the second one isn't, Zorba allows the second
  declaration. It should be reported as an error.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/862971/+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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
failed. Below is the output from the failed tests.


CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 
(message):
  Validation queue job module-schema-tools-2012-04-17T19-20-05.192Z is
  finished.  The final status was:

  

  1 tests did not succeed - changes not commited.


Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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/module-schema-tools into lp:zorba

2012-04-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/module-schema-tools into lp:zorba 
has been updated.

Status: Approved = Needs review

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
-- 
https://code.launchpad.net/~zorba-coders/zorba/module-schema-tools/+merge/102377
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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Juan Zacarias
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-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-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Juan Zacarias
The proposal to merge lp:~zorba-coders/zorba/bug-874679-image-module into 
lp:zorba/image-module has been updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-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-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug-874679-image-module-2012-04-17T20-08-49.025Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-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-874679-image-module into lp:zorba/image-module

2012-04-17 Thread Zorba Build Bot
Validation queue job bug-874679-image-module-2012-04-17T20-08-49.025Z is 
finished. The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-874679-image-module/+merge/102269
Your team Zorba Coders is subscribed to branch lp:zorba/image-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


  1   2   >