Review: Needs Fixing
--
https://code.launchpad.net/~zorba-coders/zorba/feature-node_copy/+merge/97562
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
The function should be marked as a map and as propagating-input-nodes.
Also, the must-copy-input-nodes property is not alwayts tree. It is true if the
preserve mode is no-preserve. In general, I admit that must-copy-input-nodes is
not a good name for this property; it should be called
requires-s
William Candillon has proposed merging lp:~zorba-coders/zorba/phpapi into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/97612
Minor update on the PHP example page.
--
https://code.launchp
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/97612
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
M
Matthias Brantner has proposed merging
lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba.
Requested reviews:
Paul J. Lucas (paul-lucas)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
proper return
The proposal to merge lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyz
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-analyze_string-2012-03-15T15-01-13.07Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mai
Why isn't code-generation improved so it *can* be done via code-generation
instead?
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba
> Why isn't code-generation improved so it *can* be done via code-generation
> instead?
Automatic code-generation is going to be even more important (and necessary) in
light of the forthcoming LLVM stuff. There's going to be even more code that
ought to be generated automatically, e.g., C thunk
I have fixed the failing test, which apparently fails only in the Release build.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists
Validation queue job bug-analyze_string-2012-03-15T15-01-13.07Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1,
Needs Fixing < 1, Pending < 1. Got: 3 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://laun
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
Your
The proposal to merge lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671
--
https://code.launchpad.net/~zorba-coders/zorba/bug-analyz
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292
Your
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
Your
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
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-coder
> Why isn't code-generation improved so it *can* be done via code-generation
> instead?
There are plenty of cases to specify the user-defined types. Handling each of
them automatically (i.e. generate the correct create_* function call to create
the type) is not worth it for the 2 functions that w
> > Why isn't code-generation improved so it *can* be done via code-generation
> > instead?
>
> Automatic code-generation is going to be even more important (and necessary)
> in light of the forthcoming LLVM stuff. There's going to be even more code
> that ought to be generated automatically, e.g
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
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-coder
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
Your
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
Your
It seems the remote queue is stuck again, probably because I have approved it
before anyone else. I'll resubmit.
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
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/~z
The PROJECT_NAMES need to be quoted because they contain spaces. That is,
"Zorba C-API".
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Sorin Marian Nasoi has proposed merging
lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
Sorin Marian Nasoi (sorin.marian.nasoi)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_pr
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Your team Zorba Coders is requested to review the proposed merge of
lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-
You have been requested to review the proposed merge of
lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Fixed a namespace clash by adding a "-schema" to the impo
Works great for me.
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
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/
> Works great for me.
It still needs one more Approve to be merged :).
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
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/~z
The proposal to merge lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warn
> - the unparsed-text-lines function doesn't really stream because it
> materializes all the lines in a vector before returning them. Instead, the
> iterator should keep the istream in it's state and return the lines as they
> are parsed from the stream.
Just one question about this, the Item retur
Public bug reported:
We should make sure the modules (and schemas used in them) are named
with the SAME pattern.
See thread: https://groups.google.com/forum/?hl=en&fromgroups#!topic
/zorba-dev/S10aasc-tnA
** Affects: zorba
Importance: Undecided
Status: New
--
You received this bu
Public bug reported:
adding the following two functions to the XQXQ module should allow for an
arbitrary static context to be set up as required (e.g. declaring namespaces,
enabling the hof option, ...).
Hence, it should not be required anymore to make changes to the original query.
declare %a
> > - the unparsed-text-lines function doesn't really stream because it
> > materializes all the lines in a vector before returning them. Instead, the
> > iterator should keep the istream in it's state and return the lines as they
> > are parsed from the stream.
> Just one question about this, the
Review: Approve
The change looks good. One question: is using the same prefix that the original
module used likely to cause any conflicts? Are those prefixes generally unique
as well as the module prefixes?
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/
Public bug reported:
Matthias mentioned that a test which has an obviously wrong expected
result
(
> The change looks good. One question: is using the same prefix that the
> original module used likely to cause any conflicts? Are those prefixes
> generally unique as well as the module prefixes?
I think that that is what Dana meant by suggesting uniformity: as a result I
have added bug lp:956318
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix
into lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
--
https://code.launchpad.net/~zorba
Sounds reasonable.
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
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://laun
Review: Needs Information
Does this also work for schema imports without a prefix?
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-15T21-48-35.202Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Your team Zorba Coders is subscribed to
The attempt to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into
lp:zorba failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job
fix_xqdoc_schema_import_prefix-2012-03-15T
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix
into lp:zorba has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
--
https://code.launchpad.net/~zorba
> Does this also work for schema imports without a prefix?
Are you referring to:
import schema default element namespace "SOMETHING"; ?
The answer would be no, the current changes are not tested for this because we
have no modules at this point using this kind of schema imports.
The xqdoc printe
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
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://launc
> > Does this also work for schema imports without a prefix?
> Are you referring to:
> import schema default element namespace "SOMETHING"; ?
>
> The answer would be no, the current changes are not tested for this because we
> have no modules at this point using this kind of schema imports.
>
> T
> > > Does this also work for schema imports without a prefix?
> > Are you referring to:
> > import schema default element namespace "SOMETHING"; ?
> >
> > The answer would be no, the current changes are not tested for this because
> we
> > have no modules at this point using this kind of schema im
>>>
>> Yes, that's what I mean. It looks like the xqdoc visitor could crash with
>> this
>> change. That's why I think the branch shouldn't be merged before it's tested.
> Nope, I disagree: this merge does not introduce the problem you mentioned
> because it was already there :)
I agree with So
Review: Approve
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97701
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-coder
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been
updated.
Commit Message changed to:
Fully streaming parse-xml module.
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97701
--
https://code.launchpad.net/~nbrinza/zorba/parse-fr
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix
into lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
--
https://code.launchpad.net/~zorba
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Your team Zorba Coders is subscribed to
Validation queue job fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix
into lp:zorba has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708
--
https://code.launchpad.net/~zorba-coder
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
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/~z
The proposal to merge lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
has been updated.
Commit Message changed to:
Fixed warnings that appear in Fedora.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
--
https://code.launchpad.net
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_warnings_fedora-2012-03-16T06-26-11.832Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705
Your team Zorba Coders is subscribed to branch lp:zorba.
--
60 matches
Mail list logo
