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
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
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
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
>>>
>> 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
> > > 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
> > 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
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.
The xqdoc printe
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
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
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
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
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
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
> 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
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/
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
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-
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
20 matches
Mail list logo