Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/geo-win-build into lp:zorba/geo-module

2011-10-17 Thread Daniel Turcanu
FindGEOS.cmake from Windows should be made to search for libgeos.lib . I looked 
into it but I don't know how to do that.
-- 
https://code.launchpad.net/~zorba-coders/zorba/geo-win-build/+merge/78602
Your team Zorba Coders is subscribed to branch lp:zorba/geo-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 867107] Re: xqdoc dependency to zorba is wrong

2011-10-17 Thread David Graf
Yes, it is still a problem. It is very easy to reproduce:

1) make
2) rm bin/zorba
3 make xqdoc

=
make xqdoc
[100%] Built target zorba_simplestore
Scanning dependencies of target xqdoc-xml
[100%] Building XQDoc XML documentation ...
/bin/sh: bin/zorba: not found
make[3]: *** [CMakeFiles/xqdoc-xml] Error 127
make[2]: *** [CMakeFiles/xqdoc-xml.dir/all] Error 2
make[1]: *** [CMakeFiles/xqdoc.dir/rule] Error 2
make: *** [xqdoc] Error 2

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

Title:
  xqdoc dependency to zorba is wrong

Status in Zorba - The XQuery Processor:
  New

Bug description:
  Hello Sorin

  I guess you are the right guy for this:
  Today, I tried to execute 'make xqdoc' on a clean build. Unfortunately, it 
doesn't work because 'make xqdoc' is not dependent on zorba-cmd. Thus, I got 
the following error msg:

  [100%] Building XQDoc XML documentation for the modules...
  /bin/sh: ../../../bin/zorba: not found
  make[3]: *** [doc/zorba/xqdoc/CMakeFiles/xqdoc-xml] Error 127
  make[2]: *** [doc/zorba/xqdoc/CMakeFiles/xqdoc-xml.dir/all] Error 2
  make[1]: *** [doc/zorba/xqdoc/CMakeFiles/xqdoc.dir/rule] Error 2

  I though I can fix it be doing the following:

  Index: doc/zorba/xqdoc/CMakeLists.txt
  ===
  --- doc/zorba/xqdoc/CMakeLists.txt  (revision 2)
  +++ doc/zorba/xqdoc/CMakeLists.txt  (working copy)
  @@ -51,7 +51,7 @@
COMMENT Building XQDoc XML documentation for the modules...
)
#SET_TARGET_PROPERTIES(xqdoc-xml PROPERTIES EXCLUDE_FROM_DEFAULT_BUILD 1)
  -  ADD_DEPENDENCIES(xqdoc-xml zorba_simplestore)
  +  ADD_DEPENDENCIES(xqdoc-xml zorbacmd_simplestore)

ADD_CUSTOM_TARGET(xqdoc
  ${ZORBA_EXE_SCRIPT}
  Index: cmake_modules/ZorbaModule.cmake
  ===
  --- cmake_modules/ZorbaModule.cmake (revision 2)
  +++ cmake_modules/ZorbaModule.cmake (working copy)
  @@ -543,7 +543,7 @@
)
#SET_TARGET_PROPERTIES(xqdoc-xml-${PROJECT_NAME} PROPERTIES 
EXCLUDE_FROM_DEFAULT_BUILD 1)
IF(TARGET xqdoc-xml)
  -ADD_DEPENDENCIES(xqdoc-xml-${PROJECT_NAME} zorba_simplestore)
  +ADD_DEPENDENCIES(xqdoc-xml-${PROJECT_NAME} zorbacmd_simplestore)
  ADD_DEPENDENCIES(xqdoc-xml-${PROJECT_NAME} xqdoc-xml)
  ADD_DEPENDENCIES(xqdoc xqdoc-xml-${PROJECT_NAME})
ENDIF(TARGET xqdoc-xml)

  But it doesn't help. Can you take a look into it?

  Thanks
  David

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/867107/+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:~danielturcanu/zorba/plan-serializer into lp:zorba

2011-10-17 Thread Daniel Turcanu
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer 
into lp:zorba.

Requested reviews:
  Zorba Coders (zorba-coders)

For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528

Fixed bug #872850 about plan serializer crash
Updated unit test
-- 
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is requested to review the proposed merge of 
lp:~danielturcanu/zorba/plan-serializer into lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog	2011-10-12 21:30:46 +
+++ ChangeLog	2011-10-17 09:50:23 +
@@ -52,6 +52,7 @@
   * Added undo for node revalidation
   * Fixed bug #872796  (validate-in-place can interfere with other update primitives)
   * Fixed bug #872799 (validate-in-place can set incorrect types)
+  * Fixed bug #872850  (Serialization of Tumbling Window For fails)
 
 version 2.0.1
 

=== modified file 'src/compiler/expression/expr.cpp'
--- src/compiler/expression/expr.cpp	2011-09-16 19:36:18 +
+++ src/compiler/expression/expr.cpp	2011-10-17 09:50:23 +
@@ -1402,7 +1402,7 @@
 
 void eval_expr::serialize(::zorba::serialization::Archiver ar)
 {
-  serialize_baseclass(ar, (expr*)this);
+  serialize_baseclass(ar, (namespace_context_base_expr*)this);
   ar  theExpr;
   ar  theVars;
   ar  theArgs;

=== modified file 'src/compiler/expression/flwor_expr.cpp'
--- src/compiler/expression/flwor_expr.cpp	2011-08-15 14:17:09 +
+++ src/compiler/expression/flwor_expr.cpp	2011-10-17 09:50:23 +
@@ -70,6 +70,12 @@
 SERIALIZABLE_CLASS_VERSIONS(flwor_expr)
 END_SERIALIZABLE_CLASS_VERSIONS(flwor_expr)
 
+SERIALIZABLE_CLASS_VERSIONS(flwor_wincond)
+END_SERIALIZABLE_CLASS_VERSIONS(flwor_wincond)
+
+SERIALIZABLE_CLASS_VERSIONS(flwor_wincond::vars)
+END_SERIALIZABLE_CLASS_VERSIONS(flwor_wincond::vars)
+
 
 DEF_EXPR_ACCEPT (flwor_expr)
 
@@ -530,6 +536,25 @@
   set_flwor_clause(NULL);
 }
 
+void flwor_wincond::serialize(::zorba::serialization::Archiver ar)
+{
+  ar  theIsOnly;
+  ar  theInputVars;
+  ar  theOutputVars;
+  ar  theCondExpr;
+}
+
+void flwor_wincond::vars::serialize(::zorba::serialization::Archiver ar)
+{
+  ar  posvar;
+  ar  curr;
+  ar  prev;
+  ar  next;
+}
+
+flwor_wincond::vars::vars()
+{
+}
 
 flwor_wincond::vars::~vars()
 {

=== modified file 'src/compiler/expression/flwor_expr.h'
--- src/compiler/expression/flwor_expr.h	2011-08-15 14:17:09 +
+++ src/compiler/expression/flwor_expr.h	2011-10-17 09:50:23 +
@@ -360,19 +360,24 @@
   friend class ExprIterator;
 
 public:
-  struct vars
+  struct vars : public SerializeBaseClass
   {
 var_expr_t posvar;
 var_expr_t curr;
 var_expr_t prev;
 var_expr_t next;
 
+vars();
 ~vars();
 void set_flwor_clause(flwor_clause* c);
 
 void clone(vars cloneVars, expr::substitution_t subst) const;
 
 std::ostream put(std::ostream) const;
+  public:
+SERIALIZABLE_CLASS(vars)
+SERIALIZABLE_CLASS_CONSTRUCTOR(vars)
+void serialize(::zorba::serialization::Archiver ar);
   };
 
 protected:
@@ -382,6 +387,11 @@
   expr_t  theCondExpr;
 
 public:
+  SERIALIZABLE_CLASS(flwor_wincond)
+  SERIALIZABLE_CLASS_CONSTRUCTOR2(flwor_wincond, SimpleRCObject)
+  void serialize(::zorba::serialization::Archiver ar);
+
+public:
   flwor_wincond(
 static_context* sctx,
 bool isOnly,

=== added file 'test/rbkt/ExpQueryResults/zorba/windowing/windowing12.xml.res'
--- test/rbkt/ExpQueryResults/zorba/windowing/windowing12.xml.res	1970-01-01 00:00:00 +
+++ test/rbkt/ExpQueryResults/zorba/windowing/windowing12.xml.res	2011-10-17 09:50:23 +
@@ -0,0 +1,1 @@
+window1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52/windowwindow53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104/windowwindow105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156/windowwindow157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208/windowwindow209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260/windowwindow261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312/windowwindow313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 

[Zorba-coders] [Merge] lp:~diogo-simoes89/zorba/data-cleaning into lp:zorba/data-cleaning-module

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

2011-10-17 Thread Zorba Build Bot
The attempt to merge lp:~diogo-simoes89/zorba/data-cleaning 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:272 
(message):
  Validation queue job data-cleaning-2011-10-17T10-22-35.152Z is finished.
  The final status was:

  

  3 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/+merge/79530
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning 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 855709] Re: Construction of invalid regular expression

2011-10-17 Thread Gabriel Petrovay
Fix on:
https://code.launchpad.net/~zorba-coders/zorba/file_regex/+merge/79543

** Branch linked: lp:~zorba-coders/zorba/file_regex

** 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/855709

Title:
  Construction of invalid regular expression

Status in Zorba - The XQuery Processor:
  In Progress

Bug description:
  In modules/org/expath/ns/file.xq line 739, the line that reads:

  if (matches($path, concat(^\, $delim, +$))) then

  constructs an invalid regular expression when $delim is '/' as it
  would be on a Unix system.  Only explicitly permitted characters may
  be escaped and '/' isn't one of them.

  This bug is blocking the fix of bug #855715.

To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/855709/+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 874679] Re: Clean up second arg to ADD_TEST_DIRECTORY()

2011-10-17 Thread Gabriel Petrovay
You say:

ADD_TEST_DIRECTORY() needs to [...] accept a list of pairs [...] so it
can pass the bug number to EXPECTED_FAILURE().

And how is this disabling tests on Windows? (because EXPECTED_FAILURE is
not good enough on Windows)

-- 
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:
  New

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/geo-win-build into lp:zorba/geo-module

2011-10-17 Thread Gabriel Petrovay
At rev. 42 still not working.
-- 
https://code.launchpad.net/~zorba-coders/zorba/geo-win-build/+merge/78602
Your team Zorba Coders is subscribed to branch lp:zorba/geo-module.

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


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/geo-win-build into lp:zorba/geo-module

2011-10-17 Thread Gabriel Petrovay
Less segmentation faults but the same number of failures.

The following tests FAILED:
2891 - zorba_geo_module/geo/area10.xq (Failed)
2892 - zorba_geo_module/geo/area2.xq (Failed)
2893 - zorba_geo_module/geo/area3.xq (Failed)
2894 - zorba_geo_module/geo/area4.xq (Failed)
2895 - zorba_geo_module/geo/area5.xq (Failed)
2896 - zorba_geo_module/geo/area6.xq (Failed)
2899 - zorba_geo_module/geo/area9.xq (Failed)
2900 - zorba_geo_module/geo/as_binary1.xq (Failed)
2901 - zorba_geo_module/geo/as_text1.xq (Failed)
2902 - zorba_geo_module/geo/as_text2.xq (Failed)
2903 - zorba_geo_module/geo/as_text3.xq (SEGFAULT)
2904 - zorba_geo_module/geo/as_text4.xq (SEGFAULT)
2905 - zorba_geo_module/geo/as_text5.xq (SEGFAULT)
2906 - zorba_geo_module/geo/as_text6.xq (Failed)
2907 - zorba_geo_module/geo/as_text7.xq (Failed)
2908 - zorba_geo_module/geo/as_text8.xq (Failed)
2909 - zorba_geo_module/geo/as_text9.xq (Failed)
2910 - zorba_geo_module/geo/boundary1.xq (Failed)
2911 - zorba_geo_module/geo/boundary10.xq (Failed)
2912 - zorba_geo_module/geo/boundary11.xq (Failed)
2913 - zorba_geo_module/geo/boundary12.xq (Failed)
2914 - zorba_geo_module/geo/boundary13.xq (Failed)
2915 - zorba_geo_module/geo/boundary14.xq (Failed)
2916 - zorba_geo_module/geo/boundary2.xq (Failed)
2917 - zorba_geo_module/geo/boundary3.xq (Failed)
2921 - zorba_geo_module/geo/boundary7.xq (Failed)
2922 - zorba_geo_module/geo/boundary8.xq (Failed)
2923 - zorba_geo_module/geo/boundary9.xq (Failed)
2924 - zorba_geo_module/geo/bounding-polygons1.xq (Failed)
2925 - zorba_geo_module/geo/bounding-polygons2.xq (Failed)
2926 - zorba_geo_module/geo/bounding-polygons3.xq (Failed)
2927 - zorba_geo_module/geo/buffer1.xq (Failed)
2928 - zorba_geo_module/geo/buffer10.xq (Failed)
2929 - zorba_geo_module/geo/buffer2.xq (Failed)
2930 - zorba_geo_module/geo/buffer3.xq (Failed)
2933 - zorba_geo_module/geo/buffer6.xq (Failed)
2935 - zorba_geo_module/geo/buffer8.xq (Failed)
2936 - zorba_geo_module/geo/buffer9.xq (Failed)
2937 - zorba_geo_module/geo/centroid1.xq (Failed)
2938 - zorba_geo_module/geo/centroid10.xq (Failed)
2939 - zorba_geo_module/geo/centroid2.xq (Failed)
2940 - zorba_geo_module/geo/centroid3.xq (Failed)
2941 - zorba_geo_module/geo/centroid4.xq (Failed)
2942 - zorba_geo_module/geo/centroid5.xq (Failed)
2945 - zorba_geo_module/geo/centroid8.xq (Failed)
2947 - zorba_geo_module/geo/contains1.xq (Failed)
2948 - zorba_geo_module/geo/contains10.xq (Failed)
2949 - zorba_geo_module/geo/contains2.xq (Failed)
2950 - zorba_geo_module/geo/contains3.xq (Failed)
2951 - zorba_geo_module/geo/contains4.xq (Failed)
2955 - zorba_geo_module/geo/contains8.xq (Failed)
2956 - zorba_geo_module/geo/contains9.xq (Failed)
2957 - zorba_geo_module/geo/convex-hull1.xq (Failed)
2958 - zorba_geo_module/geo/convex-hull10.xq (Failed)
2959 - zorba_geo_module/geo/convex-hull2.xq (Failed)
2960 - zorba_geo_module/geo/convex-hull3.xq (Failed)
2961 - zorba_geo_module/geo/convex-hull3_3d.xq (Failed)
2962 - zorba_geo_module/geo/convex-hull4.xq (Failed)
2965 - zorba_geo_module/geo/convex-hull7.xq (Failed)
2967 - zorba_geo_module/geo/convex-hull9.xq (Failed)
2970 - zorba_geo_module/geo/coordinate_dimension3.xq (Failed)
2971 - zorba_geo_module/geo/coordinate_dimension4.xq (Failed)
2972 - zorba_geo_module/geo/coordinate_dimension5.xq (Failed)
2975 - zorba_geo_module/geo/crosses1.xq (Failed)
2976 - zorba_geo_module/geo/crosses10.xq (Failed)
2977 - zorba_geo_module/geo/crosses2.xq (Failed)
2978 - zorba_geo_module/geo/crosses3.xq (Failed)
2979 - zorba_geo_module/geo/crosses4.xq (Failed)
2980 - zorba_geo_module/geo/crosses5.xq (Failed)
2981 - zorba_geo_module/geo/crosses6.xq (Failed)
2985 - zorba_geo_module/geo/difference1.xq (SEGFAULT)
2986 - zorba_geo_module/geo/difference10.xq (Failed)
2987 - zorba_geo_module/geo/difference2.xq (Failed)
2988 - zorba_geo_module/geo/difference3.xq (Failed)
2989 - zorba_geo_module/geo/difference4.xq (Failed)
2990 - zorba_geo_module/geo/difference5.xq (Failed)
2991 - zorba_geo_module/geo/difference6.xq (Failed)
2992 - zorba_geo_module/geo/difference7.xq (Failed)
2994 - zorba_geo_module/geo/difference9.xq (SEGFAULT)
2996 - zorba_geo_module/geo/dimension10.xq (Failed)
2999 - zorba_geo_module/geo/dimension4.xq (Failed)
3000 - zorba_geo_module/geo/dimension5.xq (Failed)
3001 - zorba_geo_module/geo/dimension6.xq (Failed)
3002 - 

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

2011-10-17 Thread Matthias Brantner
Is it possible to create a test for this scenario?
-- 
https://code.launchpad.net/~zorba-coders/zorba/file_regex/+merge/79543
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/structuralrelationships2 into lp:zorba

2011-10-17 Thread Markos Zaharioudakis
 Hi Markos,
 i noticed that the parse uri function now uses strtoul without checking errno
 with the following effects:
 
 A missing component is considered 0:
 np:in-same-tree-of(xs:anyURI(zorba:..1.50),xs:anyURI(zorba:0.0.1.50))
 -true
 
 
 Out of ulong value-space == ULONG_MAX
 np:in-same-tree-of(xs:anyURI(zorba:.3
 .1.50),xs
 :anyURI(zorba:0.2
 .1.50)) -true
 
 Is this ok?
 
 Thanks
 Federico

No, it not ok of course. It's a bug (and a similar bug existed before the 
switch to strtoul as well). I think I have fixed it now (and added the above 2 
tests in position_err.xq). Take a look. 
-- 
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/78395
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/structuralrelationships2 into lp:zorba

2011-10-17 Thread Federico Cavalieri
  Hi Markos,
  i noticed that the parse uri function now uses strtoul without checking
 errno
  with the following effects:
 
  A missing component is considered 0:
  np:in-same-tree-of(xs:anyURI(zorba:..1.50),xs:anyURI(zorba:0.0.1.50))
  -true
 

 
  Out of ulong value-space == ULONG_MAX
  np:in-same-tree-
 of(xs:anyURI(zorba:.3
 
 .1.50),xs
 
 :anyURI(zorba:0.2
  .1.50)) -true
 
  Is this ok?
 
  Thanks
  Federico
 
 No, it not ok of course. It's a bug (and a similar bug existed before the
 switch to strtoul as well). I think I have fixed it now (and added the above 2
 tests in position_err.xq). Take a look.



Ok, thanks.

I think there is a bug in level because the ordpath of a document node as root 
contains a different
number of components than those of another kind of node as root. see ctor of 
ordpath.
Why
-- 
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/78395
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:~diogo-simoes89/zorba/data-cleaning into lp:zorba/data-cleaning-module

2011-10-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/~diogo-simoes89/zorba/data-cleaning/+merge/79530
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning 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 into lp:zorba/data-cleaning-module

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

Status: Rejected = Needs review

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

2011-10-17 Thread Chris Hillery
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning 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/+merge/79530
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning/+merge/79530
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning 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 into lp:zorba/data-cleaning-module

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

2011-10-17 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1. 
Got: 1 Pending.
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning/+merge/79530
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning 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 into lp:zorba/data-cleaning-module

2011-10-17 Thread Zorba Build Bot
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning 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/+merge/79530
-- 
https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning/+merge/79530
Your team Zorba Coders is requested to review the proposed merge of 
lp:~diogo-simoes89/zorba/data-cleaning 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:~zorba-coders/zorba/f-testPerfScript into lp:zorba

2011-10-17 Thread Matthias Brantner
Why does your commit remove so much useful stuff that was made before (e.g. 
from the ChangeLog and DownloadModules script)?
-- 
https://code.launchpad.net/~zorba-coders/zorba/f-testPerfScript/+merge/79622
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/file_regex into lp:zorba

2011-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/file_regex-2011-10-17T21-24-01.686Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/file_regex/+merge/79543
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/file_regex into lp:zorba

2011-10-17 Thread Zorba Build Bot
Validation queue job file_regex-2011-10-17T21-24-01.686Z is finished. The final 
status was:

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

2011-10-17 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve  1, Disapprove  1. 
Got: 1 Pending.
-- 
https://code.launchpad.net/~zorba-coders/zorba/file_regex/+merge/79543
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/file_regex into lp:zorba

2011-10-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/file_regex into lp:zorba has been 
updated.

Status: Approved = Needs review

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

2011-10-17 Thread Paul J. Lucas
Review: Approve


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

2011-10-17 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/file_regex-2011-10-17T21-48-00.92Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/file_regex/+merge/79543
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:~juan457/zorba/xqxq-api-changes into lp:zorba

2011-10-17 Thread Matthias Brantner
Review: Needs Fixing

Juan, could you please also provide some tests? The best places would either be 
doc/cxx/examples/context.cpp or test/unit/. Examples in the former file are 
mostly used for documentation purposes because those examples are linked from 
the web site. The latter directory contains unit tests.

Also, please don't forget to mention your changes in the ChangeLog.
-- 
https://code.launchpad.net/~juan457/zorba/xqxq-api-changes/+merge/79589
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/file_regex into lp:zorba

2011-10-17 Thread Zorba Build Bot
Validation queue job file_regex-2011-10-17T21-48-00.92Z is finished. The final 
status was:

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

2011-10-17 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/file_regex into lp:zorba has been 
updated.

Status: Approved = Needs review

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

2011-10-17 Thread Cezar Andrei
The proposal to merge lp:~zorba-coders/zorba/f-testPerfScript into lp:zorba has 
been updated.

Status: Needs review = Approved

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

2011-10-17 Thread Rodolfo Ochoa
According to this document, EOLs shouldn't be causing the problem...
http://doc.bazaar.canonical.com/bzr.2.4/en/user-reference/eol-help.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/itemfactory/+merge/79181
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/itemfactory into lp:zorba

2011-10-17 Thread Rodolfo Ochoa
I used previous version of the files and didn't touch anything else but the 
methods I added.
-- 
https://code.launchpad.net/~zorba-coders/zorba/itemfactory/+merge/79181
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