[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-big-integer into lp:zorba
Chris Hillery has proposed merging lp:~zorba-coders/zorba/fix-big-integer into lp:zorba. Requested reviews: Zorba Coders (zorba-coders) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into lp:zorba. === modified file 'CMakeLists.txt' --- CMakeLists.txt 2011-12-21 14:40:33 + +++ CMakeLists.txt 2012-01-04 09:13:30 + @@ -16,7 +16,6 @@ ENABLE_TESTING() INCLUDE(CTest) - # overwriting the source and binary directories with the current zorba ones # this is usefull for other projects reusing zorba as a subcomponent SET(CMAKE_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}) -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Commit Message changed to: ignore for now For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix-big-integer-2012-01-04T09-15-05.17Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/gen-flwor-opt into lp:zorba
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/gen-flwor-opt into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/gen-flwor-opt/+merge/87444 general flwor compile-time optimizations -- https://code.launchpad.net/~zorba-coders/zorba/gen-flwor-opt/+merge/87444 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'src/compiler/expression/fo_expr.cpp' --- src/compiler/expression/fo_expr.cpp 2011-11-23 15:16:51 + +++ src/compiler/expression/fo_expr.cpp 2012-01-04 09:24:32 + @@ -38,7 +38,7 @@ { if (v.begin_visit(*this)) { -accept_children(v); +accept_children(v); } v.end_visit(*this); @@ -147,6 +147,17 @@ return theFunction->getName(); } +void fo_expr::add_arg(expr_t e) +{ + theArgs.push_back(e); + compute_scripting_kind(); +} + +void fo_expr::add_args(const std::vector &args) +{ + theArgs.insert(theArgs.end(), args.begin(), args.end()); + compute_scripting_kind(); +} void fo_expr::compute_scripting_kind() { @@ -179,7 +190,7 @@ ERROR_PARAMS(ZED(XUST0001_CONCAT)), ERROR_LOC(theArgs[i]->get_loc())); } - + if (i > 0 && !is_updating() && !is_vacuous() && (argKind & UPDATING_EXPR)) { throw XQUERY_EXCEPTION(err::XUST0001, @@ -248,7 +259,7 @@ expr_t fo_expr::clone(substitution_t& subst) const { - if (get_func()->getKind() == + if (get_func()->getKind() == FunctionConsts::ZORBA_STORE_COLLECTIONS_STATIC_DML_COLLECTION_1) { expr::subst_iter_t i = subst.find(this); === modified file 'src/compiler/expression/fo_expr.h' --- src/compiler/expression/fo_expr.h 2012-01-03 12:10:06 + +++ src/compiler/expression/fo_expr.h 2012-01-04 09:24:32 + @@ -79,11 +79,25 @@ const store::Item* get_fname() const; +<<< TREE csize num_args() const { return theArgs.size(); } expr* get_arg(csize i) const { return theArgs[i].getp(); } void set_arg(csize i, expr* e) { theArgs[i] = e; } +=== + ulong num_args() const { return (ulong)theArgs.size(); } + + expr* get_arg(ulong i) const { return theArgs[i].getp(); } + + const std::vector& get_args() const {return theArgs;} + + void set_arg(ulong i, expr* e) { theArgs[i] = e; } +>>> MERGE-SOURCE + + void add_arg(expr_t e); + + void add_args(const std::vector &args); void compute_scripting_kind(); === modified file 'src/compiler/rewriter/rules/flwor_rules.cpp' --- src/compiler/rewriter/rules/flwor_rules.cpp 2011-09-02 20:43:46 + +++ src/compiler/rewriter/rules/flwor_rules.cpp 2012-01-04 09:24:32 + @@ -173,7 +173,7 @@ expr* whereExpr; if ((whereExpr = flwor.get_where()) != NULL && !flwor.has_sequential_clauses()) { -const expr_tools::var_ptr_set& whereVars = +const expr_tools::var_ptr_set& whereVars = expr_tools::get_varset_annotation(whereExpr); expr_tools::var_ptr_set diff; @@ -229,7 +229,7 @@ ulong domainCount = TypeOps::type_max_cnt(tm, *domainType); const var_expr* pvar = fc->get_pos_var(); - if (pvar != NULL && + if (pvar != NULL && expr_tools::count_variable_uses(&flwor, pvar, &rCtx, 1) == 0) { MODIFY(fc->set_pos_var(NULL)); @@ -340,7 +340,7 @@ let_clause_t save = lc; MODIFY(flwor.remove_clause(i)); const QueryLoc& loc = var->get_loc(); -var_expr_t fvar = new var_expr(sctx, loc, var_expr::for_var, var->get_name()); +var_expr_t fvar = new var_expr(sctx, loc, var_expr::for_var, var->get_name()); for_clause_t fc = new for_clause(sctx, loc, fvar, domainExpr); flwor.add_clause(i, fc); @@ -376,7 +376,7 @@ if ((whereExpr = flwor.get_where()) != NULL) { - rchandle ifExpr = + rchandle ifExpr = new if_expr(whereExpr->get_sctx(), LOC(whereExpr), whereExpr, @@ -384,7 +384,7 @@ fo_expr::create_seq(whereExpr->get_sctx(), LOC(whereExpr))); fix_if_annotations(ifExpr); - + return ifExpr.getp(); } @@ -406,7 +406,7 @@ FOR, LET, or WINDOW clauses of a flwor expr. / static void collect_flw_vars( -const flwor_expr& flwor, +const flwor_expr& flwor, expr_tools::VarSetAnnVal& vars) { for (uint32_t i = 0; i < flwor.num_clauses(); ++i) @@ -499,9 +499,9 @@ Check if it is OK to fold (inline) a FOR/LET var X that we know is referenced only once withing its flwor expr. - For a LET var, varQuant is always QUNAT_ONE. + For a LET var, varQuant is always QUNAT_ONE. - For a FOR var, varQuant is the quantifier of the type of the domain expr. + For a FOR var, varQuant is the quantifier of the type of the domain expr. It can be either QUANT_ONE or QUANT_QU
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-big-integer into lp:zorba
The attempt to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:273 (message): Validation queue job fix-big-integer-2012-01-04T09-15-05.17Z 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/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Commit Message changed to: Ensure clean test run when Zorba built with ZORBA_WITH_BIG_INTEGER. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87443 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
Chris Hillery has proposed merging lp:~zorba-coders/zorba/fix-big-integer into lp:zorba. Requested reviews: Zorba Coders (zorba-coders) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into lp:zorba. === added file 'test/rbkt/ExpQueryResults/zorba/udf/udf-fib-rec-BIG_INTEGER.xml.res' --- test/rbkt/ExpQueryResults/zorba/udf/udf-fib-rec-BIG_INTEGER.xml.res 1970-01-01 00:00:00 + +++ test/rbkt/ExpQueryResults/zorba/udf/udf-fib-rec-BIG_INTEGER.xml.res 2012-01-04 10:05:55 + @@ -0,0 +1,1 @@ +354224848179261915075 === added file 'test/rbkt/Queries/zorba/udf/udf-fib-rec.spec' --- test/rbkt/Queries/zorba/udf/udf-fib-rec.spec 1970-01-01 00:00:00 + +++ test/rbkt/Queries/zorba/udf/udf-fib-rec.spec 2012-01-04 10:05:55 + @@ -0,0 +1,1 @@ +Result: "$RBKT_SRC_DIR/ExpQueryResults/zorba/udf/udf-fib-rec.xml.res" "$RBKT_SRC_DIR/ExpQueryResults/zorba/udf/udf-fib-rec-BIG_INTEGER.xml.res" \ No newline at end of file -- 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Commit Message changed to: Ensure clean test run when Zorba build with ZORBA_WITH_BIG_INTEGER. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix-big-integer-2012-01-04T10-10-05.121Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
Validation queue job fix-big-integer-2012-01-04T10-10-05.121Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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 904656] Re: Debugger: history in debugger client
** Changed in: zorba Status: New => Fix Committed ** Changed in: zorba Milestone: None => 2.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/904656 Title: Debugger: history in debugger client Status in Zorba - The XQuery Processor: Fix Committed Bug description: Would be nice do be able to scroll through the preceding commands by clicking 'arrow up' and 'arrow down'. As in gdb. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/904656/+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/fix-big-integer into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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/fix-big-integer into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-big-integer into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 -- https://code.launchpad.net/~zorba-coders/zorba/fix-big-integer/+merge/87448 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix-big-integer into 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 868329] Re: URI resolver does not work well on Windows
Followups in the discussion on zorba-coders: "File API extension for resolving path against a base URI/path" ** Changed in: zorba Assignee: Gabriel Petrovay (gabipetrovay) => (unassigned) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/868329 Title: URI resolver does not work well on Windows Status in Zorba - The XQuery Processor: New Bug description: When resolving a Windows absolute path relative to another URI, URI resolver cannot see that it is an absolute path (does not check for C:\) and treats it as relative path. It only checks for Linux style absolute paths. See test zorba/resolving/path_to_uri.xq in branch lp:~danielturcanu/zorba/mytrunk To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/868329/+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/get-namespace-bindings into lp:zorba
I've changed to use zorba::store::StoreConsts::NsScoping. Regarding vector vs. map, after thinking about it a bit, I agree with you that map<> would be at least a bit better user experience. However, ItemFactory::createElementNode() takes a vector> already, and that can't be changed. I think probably it's best that getNamespaceBindings() be consistent with that API. In fact, probably createElementNode() should be changed to use the same typedef as getNamespaceBindings(). Should I move the typedef for NsBindings into store_consts.h? -- https://code.launchpad.net/~zorba-coders/zorba/get-namespace-bindings/+merge/87187 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 867205] Re: Add debugger tests
** Tags removed: v2.1 ** Tags added: test tests -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/867205 Title: Add debugger tests Status in Zorba - The XQuery Processor: New Bug description: Since the replacement of the debugger the old unit test have become obsolete and were therefore deleted. New tests are required. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/867205/+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/bug-885512 into lp:zorba
Daniel Turcanu has proposed merging lp:~zorba-coders/zorba/bug-885512 into lp:zorba. Requested reviews: Zorba Coders (zorba-coders) Related bugs: Bug #885512 in Zorba: "Windows 8 Developer Preview bugs" https://bugs.launchpad.net/zorba/+bug/885512 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 The bug was fixed in november. Paul did a fix, and the user had to do a clean rebuild. The bug was not related to Windows 8. -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into lp:zorba. === modified file 'src/util/string_util.h' --- src/util/string_util.h 2011-10-11 17:59:20 + +++ src/util/string_util.h 2012-01-04 11:24:25 + @@ -28,6 +28,12 @@ #include "cxx_util.h" #include "stl_util.h" +#ifdef WIN32 +// Windows annoyingly defines these as macros. +#undef max +#undef min +#endif /* WIN32 */ + namespace zorba { namespace ztd { -- 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-885512 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-885512 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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-885512 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-885512-2012-01-04T11-24-02.503Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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-885512 into lp:zorba
Validation queue job bug-885512-2012-01-04T11-24-02.503Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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/fix_bug_dc into lp:zorba/data-cleaning-module
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_dc/+merge/86706 Your team Zorba Coders is subscribed to branch 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/bug-885512 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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-885512 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-885512 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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/debugger_broken_mac_build into lp:zorba
Gabriel Petrovay has proposed merging lp:~zorba-coders/zorba/debugger_broken_mac_build into lp:zorba. Requested reviews: Gabriel Petrovay (gabipetrovay) Paul J. Lucas (paul-lucas) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 Fixes the two problems on mac that break the build: - usage of wait() without parameters - moved the "sys/wait" include in "process_listener.h" since that uses pid_t that is available once this header is included. -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'bin/debugger/main.cpp' --- bin/debugger/main.cpp 2012-01-03 12:10:06 + +++ bin/debugger/main.cpp 2012-01-04 13:05:28 + @@ -378,7 +378,8 @@ //tCommandLineHandler.execute(); #ifndef WIN32 -wait(); +int lStatus; +wait(&lStatus); #endif delete theClient; === modified file 'bin/debugger/process_listener.cpp' --- bin/debugger/process_listener.cpp 2012-01-03 12:10:06 + +++ bin/debugger/process_listener.cpp 2012-01-04 13:05:28 + @@ -19,7 +19,6 @@ #include #ifdef ZORBA_HAVE_PTHREAD_H -# include # include #endif === modified file 'bin/debugger/process_listener.h' --- bin/debugger/process_listener.h 2012-01-03 12:10:06 + +++ bin/debugger/process_listener.h 2012-01-04 13:05:28 + @@ -21,6 +21,7 @@ #ifdef ZORBA_HAVE_PTHREAD_H # include +# include typedef pthread_t ThreadId; typedef pid_t ProcessId; typedef int ExitCode; -- 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/debugger_broken_mac_build into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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
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/87478 fixed wrong assertion in var_expr::remove_set_expr -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'src/compiler/expression/var_expr.cpp' --- src/compiler/expression/var_expr.cpp 2012-01-03 12:10:06 + +++ src/compiler/expression/var_expr.cpp 2012-01-04 14:32:24 + @@ -244,6 +244,7 @@ { assert(theKind == local_var || theKind == prolog_var); + bool found = false; std::vector::iterator ite = theSetExprs.begin(); std::vector::iterator end = theSetExprs.end(); for (; ite != end; ++ite) @@ -251,11 +252,12 @@ if (*ite == e) { theSetExprs.erase(ite); + found = true; break; } } - assert(ite != end); + ZORBA_ASSERT(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/markos-scratch into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 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
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/87478 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 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
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-01-04T14-34-53.065Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 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
Validation queue job markos-scratch-2012-01-04T14-34-53.065Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 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-885512 into lp:zorba
Daniel: have you fixed the warnings on Windows Developer 8 Preview? -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/bug-885512 into 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
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/87478 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/87478 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/debugger_broken_mac_build into lp:zorba
Should you check the result of "lStatus"? -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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/debugger_broken_mac_build into lp:zorba
I want to quit, whatever the status is. What would be the solution in this case? -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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/debugger_broken_mac_build into lp:zorba
If you just want to quit, then it doesn't matter what the status is. But if you just want to quit, why do you need to wait() at all? -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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/debugger_broken_mac_build into lp:zorba
The main process would exit and you get a console prompt even though the zorba debugger did not finish. -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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/debugger_broken_mac_build into lp:zorba
You get a console prompt on a non-Windows platform?? If not, then just #ifdef out the wait completely on Mac/Linux. -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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/debugger_broken_mac_build into lp:zorba
I have removed the wait call. The behavior is the same. I thought that the usage of wait is a safe way to not delete theClient too early and crashing the child process when this still tries to access the client. -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 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/debugger_broken_mac_build into lp:zorba
Gabriel Petrovay has proposed merging lp:~zorba-coders/zorba/debugger_broken_mac_build into lp:zorba. Requested reviews: Gabriel Petrovay (gabipetrovay) Paul J. Lucas (paul-lucas) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 Fixes the two problems on mac that break the build: - usage of wait() without parameters - moved the "sys/wait" include in "process_listener.h" since that uses pid_t that is available once this header is included. -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'bin/debugger/main.cpp' --- bin/debugger/main.cpp 2012-01-03 12:10:06 + +++ bin/debugger/main.cpp 2012-01-04 16:49:37 + @@ -214,10 +214,7 @@ return pID; } -// pID > 0 - // Watch the process -//ProcessListener* lPl = new ProcessListener(pID, &onExitProcess); return 0; @@ -375,12 +372,6 @@ theClient = new XqdbClient(lPort); theClient->start(); -//tCommandLineHandler.execute(); - -#ifndef WIN32 -wait(); -#endif - delete theClient; } catch (...) { === modified file 'bin/debugger/process_listener.cpp' --- bin/debugger/process_listener.cpp 2012-01-03 12:10:06 + +++ bin/debugger/process_listener.cpp 2012-01-04 16:49:37 + @@ -19,7 +19,6 @@ #include #ifdef ZORBA_HAVE_PTHREAD_H -# include # include #endif === modified file 'bin/debugger/process_listener.h' --- bin/debugger/process_listener.h 2012-01-03 12:10:06 + +++ bin/debugger/process_listener.h 2012-01-04 16:49:37 + @@ -21,6 +21,7 @@ #ifdef ZORBA_HAVE_PTHREAD_H # include +# include typedef pthread_t ThreadId; typedef pid_t ProcessId; typedef int ExitCode; -- 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/debugger_broken_mac_build into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 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/debugger_broken_mac_build into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 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/debugger_broken_mac_build into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87469 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-885512 into lp:zorba
Review: Approve Those warnings about secure strcpy_s should not appear in a normal build because there is a flag that disables them. -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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/debugger_broken_mac_build into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/debugger_broken_mac_build into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 -- https://code.launchpad.net/~zorba-coders/zorba/debugger_broken_mac_build/+merge/87506 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/doxygen into lp:zorba
> - I couldn't find the examples.html file linked by "set of examples". Where is > this file supposed to come from. Which link are you referring to? > - Why is the Introduction section removed from ft_intro.dox? Shouldn't we be > consistent with others (e.g. schema_lifecycle.dox still has an introduction > section)? It should be also removed in schema_lifecycle.dox. There are a lot of such fixes to be done. > - How is the generated index page going to look like in the build directory if > the table is removed? Will this also be generated to reflect the same layout > as the web site's documentation? It looks how it should be in the local documentation. -- https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/87276 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 911956] [NEW] Zorba fails to build with gcc 4.7.0
Public bug reported: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). ** Affects: zorba Importance: Undecided Status: New -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/911956 Title: Zorba fails to build with gcc 4.7.0 Status in Zorba - The XQuery Processor: New Bug description: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/911956/+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 911956] Re: Zorba fails to build with gcc 4.7.0
** Patch added: "add missing include " https://bugs.launchpad.net/bugs/911956/+attachment/2657559/+files/zorba-2.1.0-unistd.patch -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/911956 Title: Zorba fails to build with gcc 4.7.0 Status in Zorba - The XQuery Processor: New Bug description: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/911956/+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 911956] Re: Zorba fails to build with gcc 4.7.0
** Changed in: zorba Milestone: None => 2.2 ** Changed in: zorba Assignee: (unassigned) => Matthias Brantner (matthias-brantner) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/911956 Title: Zorba fails to build with gcc 4.7.0 Status in Zorba - The XQuery Processor: New Bug description: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/911956/+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/bug911956 into lp:zorba
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug911956 into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) Martin (martin-gieseking) Related bugs: Bug #911956 in Zorba: "Zorba fails to build with gcc 4.7.0" https://bugs.launchpad.net/zorba/+bug/911956 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 fixed build for gcc 4.7.x (fixes bug #911956) -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'test/unit/datetime.cpp' --- test/unit/datetime.cpp 2011-06-14 17:26:33 + +++ test/unit/datetime.cpp 2012-01-04 20:21:23 + @@ -17,6 +17,7 @@ #include #include #include +#include #ifdef WIN32 # include -- 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 911956] Re: Zorba fails to build with gcc 4.7.0
** Branch linked: lp:~zorba-coders/zorba/bug911956 -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/911956 Title: Zorba fails to build with gcc 4.7.0 Status in Zorba - The XQuery Processor: New Bug description: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/911956/+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/bug911956 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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 911956] Re: Zorba fails to build with gcc 4.7.0
** 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/911956 Title: Zorba fails to build with gcc 4.7.0 Status in Zorba - The XQuery Processor: In Progress Bug description: In order to build the Zorba 2.1.0 packages for Fedora 17 (with gcc 4.7.x) I have to apply the attached patch which adds a missing include. Maybe you can apply it to the trunk as well. The compilation error is related to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49745 (iostream no longer includes unistd.h indirectly). To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/911956/+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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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/bug911956 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug911956-2012-01-04T20-23-03.043Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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/bug911956 into lp:zorba
Validation queue job bug911956-2012-01-04T20-23-03.043Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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/bug911956 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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-885512 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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-885512 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-885512 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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-885512 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-885512-2012-01-04T20-50-58.653Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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-885512 into lp:zorba
Validation queue job bug-885512-2012-01-04T20-50-58.653Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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-885512 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-885512 into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 -- https://code.launchpad.net/~zorba-coders/zorba/bug-885512/+merge/87459 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/bug911956 into lp:zorba
Review: Needs Information Since most Windows compilers don't provide unistd.h, I suggest to use my patch attached to the bug report which excludes the header file from Windows builds. This shouldn't be a problem as the prototype of sleep() included by unistd.h is defined separately in the patched source file. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87529 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/bug911956 into lp:zorba
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug911956 into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) Martin Gieseking (martin-gieseking) Related bugs: Bug #911956 in Zorba: "Zorba fails to build with gcc 4.7.0" https://bugs.launchpad.net/zorba/+bug/911956 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 fixed build for gcc 4.7.x (fixes bug #911956) -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'test/unit/datetime.cpp' --- test/unit/datetime.cpp 2011-06-14 17:26:33 + +++ test/unit/datetime.cpp 2012-01-04 21:38:19 + @@ -21,6 +21,8 @@ #ifdef WIN32 # include # define sleep(s) Sleep(s*1000) +#else +# include #endif #include -- 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Commit Message changed to: fixed build for gcc 4.7.x (fixes bug #911956) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/get-namespace-bindings into lp:zorba
> Regarding vector vs. map, after thinking about it a bit, I agree with you that > map<> would be at least a bit better user experience. However, > ItemFactory::createElementNode() takes a vector> already, and that > can't be changed. I think probably it's best that getNamespaceBindings() be > consistent with that API. Fair enough. > > In fact, probably createElementNode() should be changed to use the same > typedef as getNamespaceBindings(). Should I move the typedef for NsBindings > into store_consts.h? No, I don't think so. Because the internal NsBindings uses zstring (not zorba::String) -- https://code.launchpad.net/~zorba-coders/zorba/get-namespace-bindings/+merge/87187 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
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/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug911956-2012-01-04T21-48-05.284Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Validation queue job bug911956-2012-01-04T21-48-05.284Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/phpapi into lp:zorba
Review: Needs Fixing - One thing that I don't understand is the relationship with the existing PHP binding. - The first three changes in swig/php/generate_proxy.php.in should be reverted (renaming of libPrefix to prefix) - executeToURI is only capable of writing to files. Also, it currently doesn't stream. - importQueryFromURI only works for file URIs. - The parse functionality should probably be removed. It merges two functionalities which don't really belong together (retrieving of data and parsing xml). The latter can also be done in XQuery. If this philosophy is to push more functionality into XQuery, the parse functionality should be removed or made symmetric to the existing C++ and XQuery data manager APIs. - getItem doesn't have a comment and is incomplete. Shouldn't it support more/all XQuery types? -- https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/86856 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug911956-2012-01-04T22-15-03.81Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/get-namespace-bindings into lp:zorba
> No, I don't think so. Because the internal NsBindings uses zstring (not > zorba::String) I was talking about the public zorba::ItemFactory::createElementNode() method. It takes a std::vector > just like getNamespaceBindings() does. -- https://code.launchpad.net/~zorba-coders/zorba/get-namespace-bindings/+merge/87187 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/get-namespace-bindings into lp:zorba
> > No, I don't think so. Because the internal NsBindings uses zstring (not > > zorba::String) > > I was talking about the public zorba::ItemFactory::createElementNode() method. > It takes a std::vector > just like > getNamespaceBindings() does. Oh, sorry. Yes, that would make sense. -- https://code.launchpad.net/~zorba-coders/zorba/get-namespace-bindings/+merge/87187 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/bug911956 into lp:zorba
Validation queue job bug911956-2012-01-04T22-15-03.81Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug911956-2012-01-04T22-41-01.706Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Validation queue job bug911956-2012-01-04T22-41-01.706Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87539 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/bug911956 into lp:zorba
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug911956 into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) Martin Gieseking (martin-gieseking) Related bugs: Bug #911956 in Zorba: "Zorba fails to build with gcc 4.7.0" https://bugs.launchpad.net/zorba/+bug/911956 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87545 fixed build for gcc 4.7.x (fixes bug #911956). Something is wrong with launchpad and preceding proposals. Hence, I'm trying to propose again. -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87545 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'test/unit/datetime.cpp' --- test/unit/datetime.cpp 2011-06-14 17:26:33 + +++ test/unit/datetime.cpp 2012-01-04 23:06:42 + @@ -21,6 +21,8 @@ #ifdef WIN32 # include # define sleep(s) Sleep(s*1000) +#else +# include #endif #include -- 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/bug911956 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug911956 into lp:zorba has been updated. Commit Message changed to: fixed build for gcc 4.7.x (fixes bug #911956). For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87545 -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87545 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/bug911956 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug911956/+merge/87545 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