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

2011-10-04 Thread Chris Hillery
Review: Approve I like it. I'd leave the link from the index page there - having a specific section marked "tutorials" will maybe encourage folks to write some more over time. If not, we can easily move that later. -- https://code.launchpad.net/~danielturcanu/zorba/web_crawler_tutorial/+merge/7

[Zorba-coders] [Merge] lp:~danielturcanu/zorba/web_crawler_tutorial into lp:zorba

2011-10-04 Thread Chris Hillery
The proposal to merge lp:~danielturcanu/zorba/web_crawler_tutorial into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~danielturcanu/zorba/web_crawler_tutorial/+merge/77179 -- https://code.launchpad.net/~danielturcanu/zorba/web

[Zorba-coders] [Merge] lp:~ceejatec/zorba/sf-bug-migrate into lp:zorba

2011-10-05 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/sf-bug-migrate into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~ceejatec/zorba/sf-bug-migrate/+merge/78209 Changes all references to Sourceforge URLs for Zorba to

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

2011-10-05 Thread Chris Hillery
Review: Disapprove This isn't a good fix. The "path" of a URI isn't a filesystem path; it's a specific part of a URI (see RFC 3986). The path portion of a URI cannot be "absolute" or "relative". So it doesn't make sense to have this functionality in the URI class. fn:resolve-uri() is not suppo

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

2011-10-05 Thread Chris Hillery
The path component of a file: URI on Windows can contain C:/ (or C%3A/), but that's not directly relevant. fn:resolve-uri() works with exactly URIs. It is not defined to accept filesystem paths. Thus, if the input is "C:/foo", it will interpret that strictly as a URI (and most likely fail since

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

2011-10-05 Thread Chris Hillery
The W3C doesn't care about filesystems. :) encode-for-uri() just escapes illegal characters in URIs. Fortunately, EXPath cares: the File module has functions to go back and forth. file:path-to-uri() converts a filesystem path to a file: URI, and file:path-to-uri() (which is a bit oddly named) d

[Zorba-coders] [Bug 857263] Re: Build/install doc is terrible

2011-10-05 Thread Chris Hillery
** Changed in: zorba Importance: Undecided => High -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/857263 Title: Build/install doc is terrible Status in Zorba - The XQuery Processor: New B

[Zorba-coders] [Bug 850652] Re: Numerous iterplans tests depend on missing queries

2011-10-05 Thread Chris Hillery
** Changed in: zorba Assignee: (unassigned) => Markos Zaharioudakis (markos-za) ** Changed in: zorba Importance: Undecided => Medium -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/850652

[Zorba-coders] [Bug 866962] Re: Remote-queue log message HTML'ized

2011-10-05 Thread Chris Hillery
No longer relevant as the old remote queue is dead. ** Changed in: zorba Status: New => Won't Fix -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/866962 Title: Remote-queue log message HT

[Zorba-coders] [Bug 849811] Re: test bug

2011-10-05 Thread Chris Hillery
Doesn't seem to be any way to delete bugs, so marked this as fixed. ** Changed in: zorba Status: New => Fix Released -- You received this bug notification because you are a member of Zorba Coders, which is a bug assignee. https://bugs.launchpad.net/bugs/849811 Title: test bug Status i

[Zorba-coders] [Bug 866847] Re: Remotequeue does not accept some svn changelist names

2011-10-05 Thread Chris Hillery
No longer relevant as the old remote queue is dead. (Well, ok, it's still there for non-core modules, but that should be taken care of in the next week or so too.) ** Changed in: zorba Status: Incomplete => Won't Fix -- You received this bug notification because you are a member of Zorba

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

2011-10-06 Thread Chris Hillery
HTTP and FTP URIs certainly don't work with filesystem paths. File: URIs only do by convention, and there are (somewhat hand-wavy) rules for mapping filesystem paths to file: URIs. But a filesystem path is-not-a file: URI, and so a filesystem path is never an appropriate argument for fn:resolve-

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/tester-branch into lp:zorba. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
Review: Approve Will break -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 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

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/tester-branch into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 Your

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/tester-branch into lp:zorba has been updated. Status: Approved => Rejected For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 Your team

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/tester-branch into lp:zorba has been updated. Status: Rejected => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 Your team

[Zorba-coders] [Merge] lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module

2011-10-06 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 -- https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module

2011-10-06 Thread Chris Hillery
Review: Approve Will break. -- https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 Your team Zorba Coders is subscribed to branch lp:zorba/oauth-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launc

[Zorba-coders] [Merge] lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 -- https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 Y

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/tester-branch into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 Your

[Zorba-coders] [Merge] lp:~ceejatec/zorba/tester-branch into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/tester-branch into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 -- https://code.launchpad.net/~ceejatec/zorba/tester-branch/+merge/78366 Your

[Zorba-coders] [Merge] lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 -- https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 Y

[Zorba-coders] [Merge] lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/test-oauth into lp:zorba/oauth-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 -- https://code.launchpad.net/~ceejatec/zorba/test-oauth/+merge/78367 Y

[Zorba-coders] [Merge] lp:~ceejatec/zorba/download-modules into lp:zorba

2011-10-06 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/download-modules into lp:zorba. Requested reviews: David Graf (davidagraf) For more details, see: https://code.launchpad.net/~ceejatec/zorba/download-modules/+merge/78376 -- https://code.launchpad.net/~ceejatec/zorba/download-modules

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

2011-10-06 Thread Chris Hillery
Review: Approve Seems perfectly reasonable. -- https://code.launchpad.net/~davidagraf/zorba/allow_c_files/+merge/77816 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

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/download-modules into lp:zorba

2011-10-06 Thread Chris Hillery
Review: Approve David looked it over and said things were OK. -- https://code.launchpad.net/~ceejatec/zorba/download-modules/+merge/78376 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

[Zorba-coders] [Merge] lp:~ceejatec/zorba/download-modules into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/download-modules into lp:zorba has been updated. Commit Message changed to: Updated DownloadModules to get modules from Launchpad. (Reviewed by David Graf) For more details, see: https://code.launchpad.net/~ceejatec/zorba/download-modules/+merge/78376 --

[Zorba-coders] [Merge] lp:~ceejatec/zorba/download-modules into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/download-modules into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/download-modules/+merge/78376 -- https://code.launchpad.net/~ceejatec/zorba/download-modules/+merge/78

[Zorba-coders] [Merge] lp:~davidagraf/zorba/allow_c_files into lp:zorba

2011-10-06 Thread Chris Hillery
The proposal to merge lp:~davidagraf/zorba/allow_c_files into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~davidagraf/zorba/allow_c_files/+merge/77816 -- https://code.launchpad.net/~davidagraf/zorba/allow_c_files/+merge/77816

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

2011-10-06 Thread Chris Hillery
Sorry, but we implement a spec. You've suggested at least two changes which would break our compliance with that spec, so I don't have much choice but to quote the spec back at you. It sounds like what you want would be best served by an extension to the Expath file module. As I said in my prev

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

2011-10-07 Thread Chris Hillery
It looks like all the absolute path changes are still in there... -- https://code.launchpad.net/~danielturcanu/zorba/mytrunk/+merge/78621 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net U

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-07 Thread Chris Hillery
Review: Approve Seems reasonable. -- https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78671 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://lau

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

2011-10-07 Thread Chris Hillery
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78671 -- https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/

[Zorba-coders] [Bug 871062] Re: test/rbkt/zorba/xqdoc/moduleDocumentation failing on Windows

2011-10-08 Thread Chris Hillery
Which test is it - moduleDocumentation or link1? Neither seem to be mentioned in the CMakeLists.txt file linked above... -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/871062 Title: test/rbkt/z

[Zorba-coders] [Bug 871062] Re: test/rbkt/zorba/xqdoc/moduleDocumentation failing on Windows

2011-10-08 Thread Chris Hillery
And what is the error output? -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/871062 Title: test/rbkt/zorba/xqdoc/moduleDocumentation failing on Windows Status in Zorba - The XQuery Processor:

[Zorba-coders] [Bug 871061] Re: test/rbkt/zorba/versioning/link1 failing on Windows

2011-10-08 Thread Chris Hillery
What is the error output? -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/871061 Title: test/rbkt/zorba/versioning/link1 failing on Windows Status in Zorba - The XQuery Processor: New Bug des

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/download-tagged-modules into lp:zorba

2011-10-11 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/download-tagged-modules into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/download-tagged-modules/+merge/79035 -- https://code.launchpad.net

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/download-tagged-modules into lp:zorba

2011-10-11 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/download-tagged-modules/+merge/79035 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.ne

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/download-tagged-modules into lp:zorba

2011-10-11 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/download-tagged-modules into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/download-tagged-modules/+merge/79035 -- https://code.launchpad.net/~zorba-coders/zorba/

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0

2011-10-11 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0. Requested reviews: Chris Hillery (ceejatec) Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/2.0-download-tagged

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0

2011-10-11 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/2.0-download-tagged-modules/+merge/79037 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/2.0. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0

2011-10-11 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0 has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/2.0-download-tagged-modules/+merge/79037 -- https://code.launchpa

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0

2011-10-11 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/2.0-download-tagged-modules into lp:~zorba-coders/zorba/2.0 has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/2.0-download-tagged-modules/+merge/79037 -- https://code.launchpa

[Zorba-coders] [Bug 872500] Re: Crash in xqdoc query

2011-10-11 Thread Chris Hillery
le not found; raised at /home/ceej/zo/src/src/runtime/xqdoc/xqdoc_impl.cpp:89 Jonathan - what version of the source are you building? Can you tell me your exact CMake line so I can try it with the same options? ** Changed in: zorba Assignee: Chris Hillery (ceejatec) => Jonathan Robie (j

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

2011-10-11 Thread Chris Hillery
Review: Needs Fixing IMHO, path_to_uri.xq / .spec / .xml.res shouldn't be checked in with this change, since it's not related to the bug being fixed. path_to_uri.spec isn't relevant and shouldn't be checked in at all, since the variable $input-context it declares isn't used by path_to_uri.xq.

[Zorba-coders] [Bug 872386] Re: inconsistent build and install directories

2011-10-11 Thread Chris Hillery
This happens because DECLARE_ZORBA_MODULE() uses a glob pattern to find all .cpp files in the module's source directory. When you make a change which adds new .cpp files, as you did here, it's unfortunately necessary to re-run CMake. There's no way CMake can detect this change automatically, so it

[Zorba-coders] [Merge] lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba

2011-10-12 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba. Requested reviews: Daniel Turcanu (danielturcanu) For more details, see: https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3c-bugfix/+merge/79068 -- https://code.launchpad.net/~ceejatec/zorba

[Zorba-coders] [Merge] lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba

2011-10-12 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3c-bugfix/+merge/79068 -- https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba

2011-10-12 Thread Chris Hillery
Daniel - I'm asking you to review this since you originally explained that the regexp was in error in this zorba-coders thread: https://lists.inf.ethz.ch/mailman/private/zorba-coders/2011-August/011953.html See the W3C bug mentioned in the commit message. This change modifies the regexp to the

[Zorba-coders] [Bug 872731] Re: Steps-leading-lone-slash-8a test fails in Zorba 2.0 branch

2011-10-12 Thread Chris Hillery
*** This bug is a duplicate of bug 867256 *** https://bugs.launchpad.net/bugs/867256 ** This bug has been marked a duplicate of bug 867256 "instance of document-node(element(x))" in predicate -- You received this bug notification because you are a member of Zorba Coders, which is the regi

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

2011-10-12 Thread Chris Hillery
Review: Needs Information Still wondering what the #include additions are about. They don't seem related or necessary... -- https://code.launchpad.net/~danielturcanu/zorba/mytrunk/+merge/79120 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zor

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba

2011-10-12 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3c-bugfix/+merge/79068 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/~zo

[Zorba-coders] [Merge] lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba

2011-10-12 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/xslt-schema-w3c-bugfix into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3c-bugfix/+merge/79068 -- https://code.launchpad.net/~ceejatec/zorba/xslt-schema-w3

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

2011-10-12 Thread Chris Hillery
But it doesn't look like like you're using any such settings here...? -- https://code.launchpad.net/~danielturcanu/zorba/mytrunk/+merge/79120 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.n

[Zorba-coders] [Bug 872958] [NEW] Need way to detect non-core modules in build

2011-10-12 Thread Chris Hillery
Public bug reported: We have a few test cases in core Zorba which depend on non-core modules. We would like to add some more, including "integration" tests and utility scripts. What we need is some CMake flag which is set by the existence of a given non-core module, so the core CMakeLists.txt fil

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

2011-10-12 Thread Chris Hillery
Rodolfo - it looks from the diffs like maybe the files XQuery.i and Zorba.i have Windows line endings, or something? The whole files are showing up as one big diff, even though it doesn't look like everything changed. Any idea what's going on there? -- https://code.launchpad.net/~zorba-coders/z

[Zorba-coders] [Bug 874676] [NEW] Remove SET_CMAKE_MODULE_PATH

2011-10-14 Thread Chris Hillery
Public bug reported: This macro wasn't in Zorba 2.0. Therefore, using it in modules means that those modules require Zorba 2.1 for building, even though they (as far as I know) don't need it for running. There will probably be cases in the future where such dependencies are un-avoidable, but in t

[Zorba-coders] [Bug 867125] Re: xqdoc does no dependency checking on inputs

2011-10-14 Thread Chris Hillery
It's actually easily possible in CMake to introduce dependencies on any files. So it would be easy to ensure that the "doc" target also depends on the stylesheets and so forth. -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https

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

2011-10-14 Thread Chris Hillery
Public bug reported: The second (optional) argument to ADD_TEST_DIRECTORY() contains a list of test cases that should be skipped (ie, not created at all). This is extremely dangerous. By bypassing them entirely, there is no way of detecting when the bug is fixed. We have already had serious proble

[Zorba-coders] [Bug 867125] Re: xqdoc does no dependency checking on inputs

2011-10-14 Thread Chris Hillery
In that case, the "doc" target should have dependencies on the xqdoc2xhtml module. No matter what, a build target always does one thing: take a set of inputs (which could be data, code, whatever) and produce some outputs. In all cases, the target should depend on all inputs, so that it only runs w

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

2011-10-17 Thread Chris Hillery
The proposal to merge lp:~diogo-simoes89/zorba/data-cleaning into lp:zorba/data-cleaning-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~diogo-simoes89/zorba/data-cleaning/+merge/79530 -- https://code.launchpad.net/~diogo-simoes89

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

2011-10-18 Thread Chris Hillery
Looks like mostly re-formatting changes other than the addition of the test case; is that correct? Out of curiosity, what was the motivation to add explicit fn:data() calls and proc: prefixes? The proc: prefixes seem like they're changing behaviour in some form, so I'd like to know why. -- htt

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

2011-10-18 Thread Chris Hillery
Sorry, I still don't understand why you added the zorba/config.h include. I was able to fix the included test case locally by doing only the change in strings_impl.cpp; it doesn't seem like the dict_XX_cpp.xq change is related at all. And I still think that adding that #include is a bad idea unl

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

2011-10-18 Thread Chris Hillery
See the second paragraph of my description: "I understand that removing a test that segfaults is necessary on Windows in order for the tests to run unattended. However, it should never be done on other platforms. So ADD_TEST_DIRECTORY() should instead mark the bugs as EXPECTED_FAILURE()s on other

[Zorba-coders] [Bug 877829] [NEW] Need to expose Zorba build flags to modules

2011-10-18 Thread Chris Hillery
Assignee: Chris Hillery (ceejatec) Status: New ** Changed in: zorba Importance: Undecided => Medium ** Changed in: zorba Assignee: (unassigned) => Chris Hillery (ceejatec) -- You received this bug notification because you are a member of Zorba Coders, which

[Zorba-coders] [Bug 850641] Re: 3 core Zorba tests depend on non-core modules

2011-10-18 Thread Chris Hillery
This may be a dupe of https://bugs.launchpad.net/zorba/+bug/872958 - certainly fixing that bug would render this one moot. Should it be marked as such? ** Changed in: zorba Importance: Undecided => Medium -- You received this bug notification because you are a member of Zorba Coders, which is

[Zorba-coders] [Bug 872958] Re: Need way to detect non-core modules in build

2011-10-18 Thread Chris Hillery
** Changed in: zorba Assignee: (unassigned) => Chris Hillery (ceejatec) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/872958 Title: Need way to detect non-core modules in build Status

[Zorba-coders] [Bug 862091] Re: Files missing from email module

2011-10-18 Thread Chris Hillery
For now, I'm marking this as targeting the next release, since this is one of the issues reported by the Fedora package maintainer and we are trying to fix all of his problems for 2.1. ** Changed in: zorba Importance: Medium => High ** Changed in: zorba Milestone: None => 2.1 -- You rece

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

2011-10-18 Thread Chris Hillery
> Why do all the generate files show up as changed? That's showing up as a problem on one of Rodolfo's branches also. I assume it's line-ending problems but I can't imagine how it's happening. -- https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765 Your team Zorba Code

[Zorba-coders] [Bug 862091] Re: Files missing from email module

2011-10-19 Thread Chris Hillery
** Changed in: zorba Assignee: Juan Zacarias (juan457) => Chris Hillery (ceejatec) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/862091 Title: Files missing from email module Status

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

2011-10-19 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/f-testPerfScript/+merge/79628 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorb

[Zorba-coders] [Bug 862091] Re: Files missing from email module

2011-10-20 Thread Chris Hillery
iding whether they are all still required. ** Changed in: zorba Assignee: Chris Hillery (ceejatec) => Juan Zacarias (juan457) ** Summary changed: - Files missing from email module + Files missing from email module (and others?) -- You received this bug notification because you are a m

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-21 Thread Chris Hillery
Review: Approve Juan - you'll need to vote "Approve" also to kick off the remote queue. -- https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post

[Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-21 Thread Chris Hillery
The proposal to merge lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 -- https://code.launchpad.net/~juan457/zorba/email-

[Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-21 Thread Chris Hillery
The proposal to merge lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 -- https://code.launchpad.net/~juan457/zorba/email-

[Zorba-coders] [Merge] lp:~ceejatec/zorba/feature-module-installation into lp:zorba

2011-10-21 Thread Chris Hillery
Chris Hillery has proposed merging lp:~ceejatec/zorba/feature-module-installation into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~ceejatec/zorba/feature-module-installation/+merge/80040 -- https://code.launchpad.net

[Zorba-coders] [Merge] lp:~ceejatec/zorba/feature-module-installation into lp:zorba

2011-10-21 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/feature-module-installation into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ceejatec/zorba/feature-module-installation/+merge/80040 -- https://code.launchpad.net/~ceejatec/zorba/feat

[Zorba-coders] [Merge] lp:~ceejatec/zorba/feature-module-installation into lp:zorba

2011-10-21 Thread Chris Hillery
The proposal to merge lp:~ceejatec/zorba/feature-module-installation into lp:zorba has been updated. Commit Message changed to: Replace ZORBA_MODULES_INSTALL_DIR with four different paths: core and non-core versions of URI and Library paths. URI path is used for resolving URIs; Library path is

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/feature-module-installation into lp:zorba

2011-10-21 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~ceejatec/zorba/feature-module-installation/+merge/80040 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.ne

Re: [Zorba-coders] [Merge] lp:~ceejatec/zorba/feature-module-installation into lp:zorba

2011-10-21 Thread Chris Hillery
I realize this needs Changelog and doc updates. -- https://code.launchpad.net/~ceejatec/zorba/feature-module-installation/+merge/80040 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsu

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

2011-10-24 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug_872732 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug_872732/+merge/79272 -- https://code.launchpad.net/~zorba-coders/zorba/bug_872732/+merge/79272 Yo

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

2011-10-24 Thread Chris Hillery
1. It looks like your editor is still randomly changing whitespace on certain lines, resulting in extraneous diffs... I hope you can figure out what's causing that and disable it. 2. In testdriver_mt, would it be possible to create the XML results on-disk as you go, rather than collecting the

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-25 Thread Chris Hillery
I ran the test manually in your build on the remote queue machine, and it succeeded. So I guess the remote queue machine is acting up again. I'm re-approving this proposal, hopefully it will work this time. -- https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 Your team

[Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-25 Thread Chris Hillery
The proposal to merge lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 -- https://code.launchpad.net/~juan457/zorba/email-

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental

2011-10-25 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental. Requested reviews: Zorba Coders (zorba-coders) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/ruby-rpath/+merge/80406 -- https://code.launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental

2011-10-25 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/ruby-rpath/+merge/80406 -- https://code.launchpad.net/~zorba-coders

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental

2011-10-25 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/ruby-rpath into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/ruby-rpath/+merge/80406 -- https://code.launchpad.net/~zorba-coders

Re: [Zorba-coders] [Merge] lp:~juan457/zorba/email-module-bugfixes into lp:zorba/email-module

2011-10-25 Thread Chris Hillery
You need two Approve votes to actually merge changes, so you should vote Approve yourself and then mark the proposal Approved. -- https://code.launchpad.net/~juan457/zorba/email-module-bugfixes/+merge/79984 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: h

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/ruby-rpath into lp:zorba

2011-10-25 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/ruby-rpath into lp:zorba. Requested reviews: Chris Hillery (ceejatec) Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/ruby-rpath/+merge/80409 -- https://code.launchpad.net

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

2011-10-25 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/ruby-rpath/+merge/80409 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-code

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

2011-10-25 Thread Chris Hillery
Chris Hillery has proposed merging lp:zorba into lp:~zorba-coders/zorba/zorba-experimental. Requested reviews: Zorba Coders (zorba-coders) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 -- https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge

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

2011-10-25 Thread Chris Hillery
The proposal to merge lp:zorba into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 -- https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 Your

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

2011-10-25 Thread Chris Hillery
The proposal to merge lp:zorba into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 -- https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 Your

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

2011-10-25 Thread Chris Hillery
The proposal to merge lp:zorba into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 -- https://code.launchpad.net/~zorba-coders/zorba/trunk/+merge/80410 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba

2011-11-01 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-thesaurus-uriresolver/+merge/80979 -- https://code.launchpad.net/~zorba-c

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba

2011-11-01 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-thesaurus-uriresolver/+merge/80979 -- https://code.launchpad.net/~zorba-c

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba

2011-11-02 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-thesaurus-uriresolver/+merge/80979 -- https://code.launchpad.net/~zorba-c

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba

2011-11-02 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-thesaurus-uriresolver/+merge/80979 -- https://code.launchpad.net/~zorba-c

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba

2011-11-02 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/feature-thesaurus-uriresolver into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-thesaurus-uriresolver/+merge/80979 -- https://code.launchpad.net/~zorba-c

<    5   6   7   8   9   10   11   12   13   14   >