[Zorba-coders] [Merge] lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/DC-documentation-2012-05-01T06-35-37.444Z/log.html -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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
[Zorba-coders] [Merge] lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
The proposal to merge lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-988417-block-internal-module into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-988417-block-internal-module into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/104207 -- https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/104207 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-988417-block-internal-module into lp:zorba
Till Westmann has proposed merging
lp:~zorba-coders/zorba/bug-988417-block-internal-module into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
Chris Hillery (ceejatec)
Matthias Brantner (matthias-brantner)
Related bugs:
Bug #988417 in Zorba: "block internal modules"
https://bugs.launchpad.net/zorba/+bug/988417
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/104207
Enables a user to disable an built-in module. This is done running through URI
mapping (but not through URL resolution) during translation.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/104207
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/compiler/translator/translator.cpp'
--- src/compiler/translator/translator.cpp 2012-04-26 13:25:13 +
+++ src/compiler/translator/translator.cpp 2012-05-01 06:06:20 +
@@ -2840,6 +2840,11 @@
// importing module that X has been imported.
if (atlist == NULL && static_context::is_builtin_module(targetNS))
{
+// hust a test, this will throw, if the access is denied
+std::vector candidateURIs;
+theRootSctx->get_candidate_uris(targetNS,
+internal::EntityData::MODULE,
+candidateURIs);
theRootSctx->add_imported_builtin_module(targetNS);
#ifdef NDEBUG
// We cannot skip the math or the sctx introspection modules because they
=== modified file 'src/context/static_context.cpp'
--- src/context/static_context.cpp 2012-04-27 10:20:33 +
+++ src/context/static_context.cpp 2012-05-01 06:06:20 +
@@ -1526,6 +1526,21 @@
}
}
+void static_context::get_candidate_uris(
+zstring const& aUri,
+internal::EntityData::Kind aEntityKind,
+std::vector& oComponents) const
+{
+ // Create a simple EntityData that just reports the specified Kind
+ internal::EntityData const lData(aEntityKind);
+
+ apply_uri_mappers(aUri, &lData, internal::URIMapper::CANDIDATE, oComponents);
+ if (oComponents.size() == 0)
+ {
+oComponents.push_back(aUri);
+ }
+}
+
/***//**
=== modified file 'src/context/static_context.h'
--- src/context/static_context.h 2012-04-24 18:22:23 +
+++ src/context/static_context.h 2012-05-01 06:06:20 +
@@ -705,6 +705,15 @@
(zstring const& aUri, internal::EntityData::Kind aEntityKind,
std::vector& oComponents) const;
+ /**
+ * Given a URI, populate a vector with a list of candidate URIs. If
+ * no candidate URIs are available, the vector will be populated
+ * with (only) the input URI.
+ */
+ void get_candidate_uris
+ (zstring const& aUri, internal::EntityData::Kind aEntityKind,
+std::vector& oComponents) const;
+
void set_uri_path(const std::vector& aURIPath);
void get_uri_path(std::vector& oURIPath) const;
=== modified file 'test/api/userdefined_uri_resolution.cpp'
--- test/api/userdefined_uri_resolution.cpp 2012-04-24 12:39:38 +
+++ test/api/userdefined_uri_resolution.cpp 2012-05-01 06:06:20 +
@@ -79,11 +79,13 @@
virtual ~DenyAccessURIMapper() {}
virtual void mapURI(const zorba::String aUri,
-EntityData const* aEntityData,
+EntityData const*,
std::vector& oUris) throw ()
{
// Deny access to an URI that would otherwise work
-if(aUri == "http://www.zorba-xquery.com/tutorials/helloworld.xsd";) {
+if(aUri == "http://www.zorba-xquery.com/tutorials/helloworld.xsd"; ||
+ aUri == "http://www.zorba-xquery.com/modules/fetch"; ||
+ aUri == "http://expath.org/ns/file";) {
oUris.push_back(URIMapper::DENY_ACCESS);
}
}
@@ -259,7 +261,53 @@
return false;
}
-bool test_deny_access(Zorba* aZorba)
+bool test_deny_internal_module_access(Zorba* aZorba)
+{
+ StaticContext_t lContext = aZorba->createStaticContext();
+
+ DenyAccessURIMapper lMapper;
+ lContext->registerURIMapper(&lMapper);
+
+ try {
+XQuery_t lQuery = aZorba->compileQuery
+ ("import module namespace fetch = "
+"'http://www.zorba-xquery.com/modules/fetch'; "
+"1 + 1", lContext);
+std::cout << lQuery << std::endl;
+ } catch (ZorbaException& e) {
+std::cout << "Caught exception: " << e.what() << std::endl;
+if (e.diagnostic() == zerr::ZXQP0029_URI_ACCESS_DENIED) {
+ std::cout << "...the correct exception!" << std::endl;
+ return true;
+}
+ }
+ return false;
+}
+
+bool test_deny_external_module_access(Zorba* aZorba)
+{
+ StaticContext_t lContext = aZorba->createStaticContext();
+
+ DenyAccessURIMapper lMapper;
+ lContext->registerURIMapper(&lMapper);
+
+ try {
+XQuery_t lQuery = aZorba->compileQuery
+ ("import module namespace file = "
+"'http://expath.org/ns/file'; "
+"1 + 1", lContext);
+std::cout << lQuery << std::endl;
+ } catch (ZorbaException& e) {
+std::cout << "Caught exception: " << e.what
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-988417-block-internal-module into lp:zorba
Review: Approve Ah, yes, I guess you want the exception to be thrown anyway. Cool, that works. -- https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/103542 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-988417-block-internal-module into lp:zorba
I've changed this as proposed (I think) and added some tests. Having the exception throw directly seems to have the advantage that I get consistent behavior between internal and external modules - so that seems to be good. As resolution doesn't seem to happen in the current scenario, the only thing that's still not that nice is that we're updating the vector for something that's in principle a read-only operation. But, as you said, that's probably a minor overhead. -- https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/103542 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 974474] Re: invalid regex Block Escapes not reported for ICU
The XML Schema Part 2 specification includes the note:
Note: [Unicode Database] is subject to future revision. For example,
the mapping from code points to character properties might be updated.
All ·minimally conforming· processors ·must· support the character
properties defined in the version of [Unicode Database] that is current
at the time this specification became a W3C Recommendation. However,
implementors are encouraged to support the character properties defined
in any future version.
Since the Unicode Database can grow over time, there's no easy way to
know what is and isn't legal.
** Changed in: zorba
Status: Confirmed => 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/974474
Title:
invalid regex Block Escapes not reported for ICU
Status in Zorba - The XQuery Processor:
Won't Fix
Bug description:
When using ICU, a query such as:
fn:matches( "a", "\p{IsBasic-Latin}" )
returns "true" even though it should raise error err:FORX0002 since
IsBasic-Latin is not a valid Block Escape according to:
http://www.w3.org/TR/xmlschema-2/#charcter-classes
in [36]. (It should be IsBasicLatin -- no '-'.)
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/974474/+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 992037] Re: Miscellaneous warnings on Windows
** Changed in: zorba Status: Triaged => 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/992037 Title: Miscellaneous warnings on Windows Status in Zorba - The XQuery Processor: In Progress Bug description: First warning: include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/992037/+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-992304-compile-only into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-992304-compile-only into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-992304-compile-only into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 1 Approve. -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-992304-compile-only into lp:zorba
Validation queue job bug-992304-compile-only-2012-05-01T01-25-03.331Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-992304-compile-only into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-992304-compile-only-2012-05-01T01-25-03.331Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-990587-schema-tools into lp:zorba/schema-tools-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
Validation queue job bug-990587-schema-tools-2012-05-01T00-52-53.21Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-992304-compile-only into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-992304-compile-only into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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 992304] Re: Executing lib_only query invokes URI resolution process
I have figured out that this only happens when compiling the query as a
library module, as with the -l command-line option. This happens because
Zorba actually creates a separate query that imports the module
namespace and executes that, registering a custom URLResolver for that
namespace URI that returns the original query.
The reason this ends up hitting the web is because the full URI mapping
is also done. So, in this case, the URL "http://www.zorba-
xquery.com/foo" gets turned into a number of candidate URIs, one of
which is "http://www.zorba-xquery.com/foo.xq";. The custom URLResolver
recognizes only the original unchanged URI, so it returns nothing for
this URI, letting Zorba fall through to the built-in URLResolvers
including the HTTPURLResolver.
The fix is easy enough - simplify the custom URLResolver so it doesn't
even check the URL it is passed. We know it will always be the URL we
are expecting, even if it's been modified by URI Mapping, so we can just
return the query file resource directly. Tested locally and this works.
** Changed in: zorba
Assignee: (unassigned) => Chris Hillery (ceejatec)
** Branch linked: lp:~zorba-coders/zorba/bug-992304-compile-only
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/992304
Title:
Executing lib_only query invokes URI resolution process
Status in Zorba - The XQuery Processor:
New
Bug description:
The following query:
module namespace foo = "http://www.zorba-xquery.com/foo";;
declare function foo:test() {
if (fn:true())
then 1
else $a
};
(which is in the Zorba source as bin/test/mymod.xq) actually contacts
www.zorba-xquery.com via HTTP and attempts to download /foo.xq. This
means that the module namespace URI is undergoing URI resolution. I
cannot imagine why this would be the case, but it leads not only to
slowness but can cause the query to change behaviour depending on what
www.zorba-xquery.com chooses to return. (In the original test case,
the URL was at 28msec.com, and when 28msec.com changed from raising a
404 error to returning a 200 with an error HTML page, the query
started failing with a compilation error. Apparently Zorba was
attempting to actually compile the HTML as an XQuery.)
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/992304/+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-992304-compile-only into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-992304-compile-only into lp:zorba has been updated. Commit Message changed to: Don't check the URI passed to FakeLibraryModuleURLResolver - just resolve it. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-992304-compile-only into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194 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-992304-compile-only into lp:zorba
Chris Hillery has proposed merging
lp:~zorba-coders/zorba/bug-992304-compile-only into lp:zorba.
Requested reviews:
Chris Hillery (ceejatec)
Related bugs:
Bug #992304 in Zorba: "Executing lib_only query invokes URI resolution
process"
https://bugs.launchpad.net/zorba/+bug/992304
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194
--
https://code.launchpad.net/~zorba-coders/zorba/bug-992304-compile-only/+merge/104194
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog 2012-04-30 15:09:53 +
+++ ChangeLog 2012-05-01 01:02:21 +
@@ -44,7 +44,8 @@
* Fixed bug #949910 (has-children may be invoked on all nodes). Internally, zorba::store::Item::getChildren() now returns NULL on node classes without offspring (instead of raising an error).
* Fixed Bug #933490: Error ItemFactoryImpl::createBase64Binary with istream
* Fixed bug #867112 (Diagnostic Handler was not working on external APIs)
-
+ * Fixed bug #992304 (Compiling library modules could load data from
+ wrong source)
version 2.2
=== modified file 'bin/test/mymod.xq'
--- bin/test/mymod.xq 2012-04-30 23:31:06 +
+++ bin/test/mymod.xq 2012-05-01 01:02:21 +
@@ -14,7 +14,7 @@
: limitations under the License.
:)
-module namespace foo = "http://www.zorba-xquery.com/foo";;
+module namespace foo = "http://www.28msec.com/foo";;
declare function foo:test() {
if (fn:true())
=== modified file 'src/compiler/api/compiler_api.cpp'
--- src/compiler/api/compiler_api.cpp 2012-04-24 12:39:38 +
+++ src/compiler/api/compiler_api.cpp 2012-05-01 01:02:21 +
@@ -338,10 +338,8 @@
{
public:
FakeLibraryModuleURLResolver
- (zstring const& aLibraryModuleNamespace,
-zstring const& aLibraryModuleFilename, std::istream& aStream)
-: theLibraryModuleNamespace(aLibraryModuleNamespace),
- theLibraryModuleFilename(aLibraryModuleFilename),
+ (zstring const& aLibraryModuleFilename, std::istream& aStream)
+: theLibraryModuleFilename(aLibraryModuleFilename),
theStream(aStream)
{}
virtual ~FakeLibraryModuleURLResolver()
@@ -350,9 +348,14 @@
virtual internal::Resource* resolveURL
(zstring const& aUrl, internal::EntityData const* aEntityData)
{
-if (aUrl != theLibraryModuleNamespace) {
- return NULL;
-}
+// Since we know this URL resolver will only be used when compiling the
+// stub query, and the only URI that query will contain is the one we're
+// interested in, then we don't actually need to check the URL at all -
+// just return the stream. This is a good thing, because due to URI
+// mapping, aUrl will probably be different than the one we're expecting.
+// If we returned NULL here, Zorba's built-in URL Resolvers would attempt
+// to load the resource from somewhere else, and that may result in the
+// wrong information being compiled. See bug 992304.
assert (theStream.good());
// Pass a nullptr StreamReleaser; memory ownership of the istream remains
// with the caller of this method.
@@ -362,7 +365,6 @@
}
private:
- zstring theLibraryModuleNamespace;
zstring theLibraryModuleFilename;
std::istream& theStream;
};
@@ -400,7 +402,7 @@
aXQuery.clear();
aXQuery.seekg(0);
FakeLibraryModuleURLResolver* aFakeResolver =
-new FakeLibraryModuleURLResolver(lib_namespace.str(), aFileName, aXQuery);
+new FakeLibraryModuleURLResolver(aFileName, aXQuery);
theCompilerCB->theRootSctx->add_url_resolver(aFakeResolver);
// create a dummy main module and parse it
--
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-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-990587-schema-tools-2012-05-01T00-52-53.21Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
Validation queue job bug-990587-data-formatting-2012-05-01T00-21-37.286Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 992304] Re: Executing lib_only query invokes URI resolution process
** Summary changed:
- module declaration invokes URI resolution process
+ Executing lib_only query invokes URI resolution process
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/992304
Title:
Executing lib_only query invokes URI resolution process
Status in Zorba - The XQuery Processor:
New
Bug description:
The following query:
module namespace foo = "http://www.zorba-xquery.com/foo";;
declare function foo:test() {
if (fn:true())
then 1
else $a
};
(which is in the Zorba source as bin/test/mymod.xq) actually contacts
www.zorba-xquery.com via HTTP and attempts to download /foo.xq. This
means that the module namespace URI is undergoing URI resolution. I
cannot imagine why this would be the case, but it leads not only to
slowness but can cause the query to change behaviour depending on what
www.zorba-xquery.com chooses to return. (In the original test case,
the URL was at 28msec.com, and when 28msec.com changed from raising a
404 error to returning a 200 with an error HTML page, the query
started failing with a compilation error. Apparently Zorba was
attempting to actually compile the HTML as an XQuery.)
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/992304/+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 992037] Re: Miscellaneous warnings on Windows
For the first warning, the code is actually correct as-is. The warning is ONLY a warning. Hence, in this case, the warning should be disabled. -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/992037 Title: Miscellaneous warnings on Windows Status in Zorba - The XQuery Processor: Triaged Bug description: First warning: include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/992037/+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-990587-data-formatting into lp:zorba/data-formatting-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-990587-data-formatting-2012-05-01T00-21-37.286Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-rq-28msec into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-rq-28msec into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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/fix-rq-28msec into lp:zorba
Validation queue job fix-rq-28msec-2012-04-30T23-46-03.84Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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 992304] [NEW] module declaration invokes URI resolution process
Public bug reported:
The following query:
module namespace foo = "http://www.zorba-xquery.com/foo";;
declare function foo:test() {
if (fn:true())
then 1
else $a
};
(which is in the Zorba source as bin/test/mymod.xq) actually contacts
www.zorba-xquery.com via HTTP and attempts to download /foo.xq. This
means that the module namespace URI is undergoing URI resolution. I
cannot imagine why this would be the case, but it leads not only to
slowness but can cause the query to change behaviour depending on what
www.zorba-xquery.com chooses to return. (In the original test case, the
URL was at 28msec.com, and when 28msec.com changed from raising a 404
error to returning a 200 with an error HTML page, the query started
failing with a compilation error. Apparently Zorba was attempting to
actually compile the HTML as an XQuery.)
** Affects: zorba
Importance: Medium
Status: New
** 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/992304
Title:
module declaration invokes URI resolution process
Status in Zorba - The XQuery Processor:
New
Bug description:
The following query:
module namespace foo = "http://www.zorba-xquery.com/foo";;
declare function foo:test() {
if (fn:true())
then 1
else $a
};
(which is in the Zorba source as bin/test/mymod.xq) actually contacts
www.zorba-xquery.com via HTTP and attempts to download /foo.xq. This
means that the module namespace URI is undergoing URI resolution. I
cannot imagine why this would be the case, but it leads not only to
slowness but can cause the query to change behaviour depending on what
www.zorba-xquery.com chooses to return. (In the original test case,
the URL was at 28msec.com, and when 28msec.com changed from raising a
404 error to returning a 200 with an error HTML page, the query
started failing with a compilation error. Apparently Zorba was
attempting to actually compile the HTML as an XQuery.)
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/992304/+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/fix-rq-28msec into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix-rq-28msec-2012-04-30T23-46-03.84Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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/fix-rq-28msec into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-rq-28msec into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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/fix-rq-28msec into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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-988417-block-internal-module into lp:zorba
IMHO, the point of "built-in modules" was to prevent the overhead of *compiling* those modules, not bypassing URI resolution per se. It is true, though, that invoking URI resolution implies a bit of performance overhead. I believe that if only the URI mapping stage is invoked that the overhead will be relatively small. It probably is important to bypass the URL Resolution stage, because that's where things like making network connections to download the module might occur. I agree that using exceptions to communicate results isn't ideal, but I think the code reuse and API consolidation outweighs that concern in this case. However, it would actually be pretty easy to either modify apply_uri_mappers() or provide a slight variant of that method (with appropriate refactoring) which simply checked for DENY_ACCESS and returned a bool instead of throwing an exception. I could do that if you like. -- https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/103542 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/fix-rq-28msec into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-rq-28msec into lp:zorba has been updated. Description changed to: This module is not imported by anything; it is only used for checking zorbacmd's -l (compile library module) flag. So the namespace URI can be anything. I just changed it to a different site due to the changes in the way 28msec.com handles unknown URLs. (Why this test case contacted 28msec.com *at all* is a different bug.) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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/fix-rq-28msec into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187 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/fix-rq-28msec into lp:zorba
Chris Hillery has proposed merging lp:~zorba-coders/zorba/fix-rq-28msec into
lp:zorba.
Requested reviews:
Chris Hillery (ceejatec)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187
--
https://code.launchpad.net/~zorba-coders/zorba/fix-rq-28msec/+merge/104187
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'bin/test/mymod.xq'
--- bin/test/mymod.xq 2012-04-24 12:39:38 +
+++ bin/test/mymod.xq 2012-04-30 23:34:25 +
@@ -14,7 +14,7 @@
: limitations under the License.
:)
-module namespace foo = "http://www.28msec.com/foo";;
+module namespace foo = "http://www.zorba-xquery.com/foo";;
declare function foo:test() {
if (fn:true())
--
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-988417-block-internal-module into lp:zorba
It seemed to me that the whole point of distinguishing between built-in modules and other modules here was that one does not want to invoke the the existing URI resolution mechanisms. Wanting to keep this spirit of the existing code, I proposed this solution. If that's not needed, then that's fine with me. (The one thing that I'm not extremely fond of is that we would invoke exceptions to find the answer to a simple question, but I can live with that.) Obviously going through the URIMapper mechanism would have the great advantage of not forcing the user of the API to find out which modules are "internal" and which ones aren't. This would also make programs written agains this API more robust wrt. to changes in the internal/external classification of modules. -- https://code.launchpad.net/~zorba-coders/zorba/bug-988417-block-internal-module/+merge/103542 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-990587-schema-tools into lp:zorba/schema-tools-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-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-990587-schema-tools into lp:zorba/schema-tools-module
The attempt to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module 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 bug-990587-schema-tools-2012-04-30T21-51-49.171Z is finished. The final status was: 2 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/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-990587-schema-tools-2012-04-30T21-51-49.171Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 925143] Re: CollectionManager in SWIG apis
** Changed in: zorba Status: In Progress => Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/925143 Title: CollectionManager in SWIG apis Status in Zorba - The XQuery Processor: Fix Committed Bug description: Stijn Tonk has made the feature request to provide the CollectionManager API in the swig bindings. Currently, we only expose the DocumentManager. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/925143/+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:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
The proposal to merge lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
The attempt to merge lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message): Validation queue job DC-documentation-2012-04-30T17-10-39.284Z is finished. The final status was: 2 tests did not succeed - changes not commited. Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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
[Zorba-coders] [Merge] lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/DC-documentation-2012-04-30T17-10-39.284Z/log.html -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-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-990587-data-formatting into lp:zorba/data-formatting-module
The attempt to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module 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 bug-990587-data-formatting-2012-04-30T16-03-35.505Z is finished. The final status was: 2 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/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 992037] Re: Miscellaneous warnings on Windows
** Description changed: First warning: - include\zorba/transcode_stream.h(275) : warning C4355: 'this' : used in + include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) ** Branch linked: lp:~paul-lucas/zorba/bug-992037 ** Changed in: zorba Importance: Undecided => Low -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/992037 Title: Miscellaneous warnings on Windows Status in Zorba - The XQuery Processor: Triaged Bug description: First warning: include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/992037/+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 992037] [NEW] Miscellaneous warnings on Windows
Public bug reported: First warning: include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) ** Affects: zorba Importance: Undecided Assignee: Paul J. Lucas (paul-lucas) Status: Triaged -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/992037 Title: Miscellaneous warnings on Windows Status in Zorba - The XQuery Processor: Triaged Bug description: First warning: include\zorba\transcode_stream.h(275) : warning C4355: 'this' : used in base member initializer list Second warning: src\runtime\json\jsonml_array.cpp(201) : warning C4800: 'zorba::whitespace::type' : forcing value to bool 'true' or 'false' (performance warning) To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/992037/+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:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
The proposal to merge lp:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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:~diogo-simoes89/zorba/DC-documentation into lp:zorba/data-cleaning-module
Review: Approve -- https://code.launchpad.net/~diogo-simoes89/zorba/DC-documentation/+merge/103902 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/fix_bug_987830 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_987830/+merge/103370 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/fix_bug_962216 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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 962220] Re: full-text module
** Changed in: zorba Status: In Progress => Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/962220 Title: full-text module Status in Zorba - The XQuery Processor: Fix Committed Bug description: Implement a new module that provides basic full-text related functions. - accessing thesaurus - stemming - tokenization - accessing stop-words To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/962220/+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 986580] Re: fs::lsdir() improvements
** Changed in: zorba Status: In Progress => Fix Committed ** Changed in: zorba Milestone: None => 2.5 -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/986580 Title: fs::lsdir() improvements Status in Zorba - The XQuery Processor: Fix Committed Bug description: I noticed that this function was added. There are a few of problems with its implementation: 1. It forces the user to use a vector. What if I want to use a list? Or something else? 2. It uses std::string. The majority of the Zorba code base uses zstring. 3. It always reads the entire directory whether you want it or not. (WHat if there are 10,000 files in a directory?) Instead, it ought to provide an iterator that you can stop any time you please. 4. After the initial check for an error from opendir(), the code doesn't check for any subsequent errors, e.g., it does not check closedir() for error. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/986580/+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/fix_bug_962216 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_bug_962216 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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-990587-data-formatting into lp:zorba/data-formatting-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-990587-data-formatting-2012-04-30T16-03-35.505Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-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/fix_bug_962216 into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 1 Approve, 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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/fix_bug_962216 into lp:zorba
Validation queue job fix_bug_962216-2012-04-30T15-31-56.1Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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/fix_bug_987830 into lp:zorba
> The module files should contain syntactically valid XQuery code. The formal > semantics spec you cite is not concerned with syntax, and not everything > described there has a coresponding syntactic construct. The none type is one > example from the formal semantics that cannot be expressed syntactically. To > see what types can be expressed syntactically, you have to look here: > > http://www.w3.org/TR/xquery-30/#id-sequencetype-syntax I see this makes sense. One more thing though: where is the exception added for the fn:error function: http://www.w3.org/TR/xpath-functions-30/#func-error ? Thanks, Sorin -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_987830/+merge/103370 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-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_bug_962216 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix_bug_962216-2012-04-30T15-31-56.1Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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/fix_bug_962216 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_bug_962216 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 -- https://code.launchpad.net/~zorba-coders/zorba/fix_bug_962216/+merge/103281 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/bug925143 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug925143 into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 -- https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 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/bug925143 into lp:zorba
Validation queue job bug925143-2012-04-30T14-39-05.395Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 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-990587-data-formatting into lp:zorba/data-formatting-module
Review: Approve Tested it and it works. -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-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-990587-schema-tools into lp:zorba/schema-tools-module
Review: Approve Tested it and it works. -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug925143 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug925143-2012-04-30T14-39-05.395Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 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/bug925143 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug925143 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 -- https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 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/bug925143 into lp:zorba
Review: Approve Looks good. -- https://code.launchpad.net/~zorba-coders/zorba/bug925143/+merge/102791 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-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-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-990587-data-formatting into lp:zorba/data-formatting-module
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 1 Approve, 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
Validation queue job bug-990587-data-formatting-2012-04-30T10-02-42.29Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 990587] Re: Zorba with modules does not build if only JRE is installed (no javac or jar)
** Branch linked: lp:~zorba-coders/zorba/bug-990587-data-formatting ** Branch linked: lp:~zorba-coders/zorba/bug-990587-schema-tools -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/990587 Title: Zorba with modules does not build if only JRE is installed (no javac or jar) Status in Zorba - The XQuery Processor: New Bug description: If a target machine has the JRE installed but it doesn't have the "javac" and/or "jar" executables, Zorba fails to build. 1. Install JRE on the machine and make sure the full JSE is not installed (there should be no javac or jar executable) 2. Checkout the trunk 3. Checkout all the modules 4. run "cmake .." in the build folder The util-jvm and data-formatting modules fail to properly configure and this results in a Zorba cmake error instead of disabling the data- formatting module. The output I get: -- --- Module: util-jvm--- -- Looking for JNI -- Looking for Java -- Java version 1.6.0.24 configured successfully! -- Could NOT find Java (missing: Java_JAR_EXECUTABLE Java_JAVAC_EXECUTABLE) (found version "1.6.0.24") -- Building Zorba without util-jvm module. -- --- -- CMake Error at /home/colea/xquery_bzr/zorba_modules/schema-tools/CMakeLists.txt:22 (FIND_PACKAGE): Could not find module Findzorba_util-jvm_module.cmake or a configuration file for package zorba_util-jvm_module. Adjust CMAKE_MODULE_PATH to find Findzorba_util-jvm_module.cmake or set zorba_util-jvm_module_DIR to the directory containing a CMake configuration file for zorba_util-jvm_module. The file will have one of the following names: zorba_util-jvm_moduleConfig.cmake zorba_util-jvm_module-config.cmake -- Java and/or JNI not found; skipping schema-tools module. ... -- BEGIN Configuring module Data-Formatting CMake Error at /home/colea/xquery_bzr/zorba_modules/data-formatting/CMakeLists.txt:25 (FIND_PACKAGE): Could not find module Findzorba_util-jvm_module.cmake or a configuration file for package zorba_util-jvm_module. Adjust CMAKE_MODULE_PATH to find Findzorba_util-jvm_module.cmake or set zorba_util-jvm_module_DIR to the directory containing a CMake configuration file for zorba_util-jvm_module. The file will have one of the following names: zorba_util-jvm_moduleConfig.cmake zorba_util-jvm_module-config.cmake -- Java and/or JNI not found; skipping data-formating module. -- END Configuring module Data-Formatting ... -- Configuring incomplete, errors occurred! To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/990587/+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-990587-data-formatting into lp:zorba/data-formatting-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-990587-data-formatting-2012-04-30T10-02-42.29Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module has been updated. Commit Message changed to: Report gracefully if util-jvm module is not found. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-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-990587-data-formatting into lp:zorba/data-formatting-module
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-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-990587-schema-tools into lp:zorba/schema-tools-module
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079 Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-schema-tools into lp:zorba/schema-tools-module
Chris Hillery has proposed merging
lp:~zorba-coders/zorba/bug-990587-schema-tools into
lp:zorba/schema-tools-module.
Requested reviews:
Chris Hillery (ceejatec)
Nicolae Brinza (nbrinza)
Related bugs:
Bug #990587 in Zorba: "Zorba with modules does not build if only JRE is
installed (no javac or jar)"
https://bugs.launchpad.net/zorba/+bug/990587
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079
--
https://code.launchpad.net/~zorba-coders/zorba/bug-990587-schema-tools/+merge/104079
Your team Zorba Coders is subscribed to branch lp:zorba/schema-tools-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2012-04-05 11:51:23 +
+++ CMakeLists.txt 2012-04-30 10:01:27 +
@@ -19,43 +19,48 @@
# The util-jvm tool finds Java and JNI, so all standard cached JAVA_
# variables will be set. util-jvm also caches Java_FOUND and JNI_FOUND
# for us.
-FIND_PACKAGE (zorba_util-jvm_module REQUIRED)
-
-IF (JNI_FOUND)
- INCLUDE_DIRECTORIES (${zorba_util-jvm_module_INCLUDE_DIRS})
-
- IF (WIN32)
-SET (XMLBEANS_PATH $ENV{XMLBEANS_HOME}\\lib)
-FIND_FILE( XMLBEANS_JAR NAMES xmlbeans.jar xbean.jar
- PATHS ${XMLBEANS_PATH} )
-MESSAGE (STATUS "xmlbeans.jar looking path: " ${XMLBEANS_PATH} )
- ELSE (WIN32)
-FIND_FILE( XMLBEANS_JAR NAMES xmlbeans.jar xbean.jar
- PATHS /usr/share/java/ /usr/share/java/xmlbeans/ )
- ENDIF (WIN32)
- IF( EXISTS ${XMLBEANS_JAR} )
+FIND_PACKAGE (zorba_util-jvm_module QUIET)
+
+IF (zorba_util-jvm_module_FOUND)
+ IF (JNI_FOUND)
+INCLUDE_DIRECTORIES (${zorba_util-jvm_module_INCLUDE_DIRS})
+
+IF (WIN32)
+ SET (XMLBEANS_PATH $ENV{XMLBEANS_HOME}\\lib)
+ FIND_FILE( XMLBEANS_JAR NAMES xmlbeans.jar xbean.jar
+ PATHS ${XMLBEANS_PATH} )
+ MESSAGE (STATUS "xmlbeans.jar looking path: " ${XMLBEANS_PATH} )
+ELSE (WIN32)
+ FIND_FILE( XMLBEANS_JAR NAMES xmlbeans.jar xbean.jar
+ PATHS /usr/share/java/ /usr/share/java/xmlbeans/ )
+ENDIF (WIN32)
+IF( EXISTS ${XMLBEANS_JAR} )
-MESSAGE (STATUS "xmlbeans.jar found at: " ${XMLBEANS_JAR} )
-# if everything is found build this module
-SET (SCHEMA_TOOLS_VERSION 1.0)
-
-ENABLE_TESTING ()
-INCLUDE (CTest)
-
-SET_CMAKE_MODULE_PATH ()
-
-FIND_PACKAGE (Zorba REQUIRED HINTS "${ZORBA_BUILD_DIR}")
-INCLUDE ("${Zorba_USE_FILE}")
-
-ADD_SUBDIRECTORY ("src")
-ADD_SUBDIRECTORY ("srcJava")
-ADD_TEST_DIRECTORY("${PROJECT_SOURCE_DIR}/test")
-DONE_DECLARING_ZORBA_URIS ()
-
- ELSE( EXISTS ${XMLBEANS_JAR} )
-MESSAGE (STATUS "xmlbeans.jar not found; skipping schema-tools module.")
- ENDIF( EXISTS ${XMLBEANS_JAR} )
-
-ELSE (JNI_FOUND)
- MESSAGE (STATUS "Java and/or JNI not found; skipping schema-tools module.")
-ENDIF (JNI_FOUND)
+ MESSAGE (STATUS "xmlbeans.jar found at: " ${XMLBEANS_JAR} )
+ # if everything is found build this module
+ SET (SCHEMA_TOOLS_VERSION 1.0)
+
+ ENABLE_TESTING ()
+ INCLUDE (CTest)
+
+ SET_CMAKE_MODULE_PATH ()
+
+ FIND_PACKAGE (Zorba REQUIRED HINTS "${ZORBA_BUILD_DIR}")
+ INCLUDE ("${Zorba_USE_FILE}")
+
+ ADD_SUBDIRECTORY ("src")
+ ADD_SUBDIRECTORY ("srcJava")
+ ADD_TEST_DIRECTORY("${PROJECT_SOURCE_DIR}/test")
+ DONE_DECLARING_ZORBA_URIS ()
+
+ELSE( EXISTS ${XMLBEANS_JAR} )
+ MESSAGE (STATUS "xmlbeans.jar not found; skipping schema-tools module.")
+ENDIF( EXISTS ${XMLBEANS_JAR} )
+
+ ELSE (JNI_FOUND)
+MESSAGE (STATUS "Java and/or JNI not found; skipping schema-tools module.")
+ ENDIF (JNI_FOUND)
+
+ELSE (zorba_util-jvm_module_FOUND)
+ MESSAGE (STATUS "Zorba's util-jvm module not found; skipping schema-tools module.")
+ENDIF (zorba_util-jvm_module_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
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
The proposal to merge lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module has been updated. Commit Message changed to: Report gracefully if util-jvm module is not found. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 -- https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077 Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-990587-data-formatting into lp:zorba/data-formatting-module
Chris Hillery has proposed merging
lp:~zorba-coders/zorba/bug-990587-data-formatting into
lp:zorba/data-formatting-module.
Requested reviews:
Chris Hillery (ceejatec)
Nicolae Brinza (nbrinza)
Related bugs:
Bug #990587 in Zorba: "Zorba with modules does not build if only JRE is
installed (no javac or jar)"
https://bugs.launchpad.net/zorba/+bug/990587
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077
--
https://code.launchpad.net/~zorba-coders/zorba/bug-990587-data-formatting/+merge/104077
Your team Zorba Coders is subscribed to branch lp:zorba/data-formatting-module.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2012-04-11 13:45:56 +
+++ CMakeLists.txt 2012-04-30 10:00:25 +
@@ -22,28 +22,32 @@
# The util-jvm tool finds Java and JNI, so all standard cached JAVA_
# variables will be set. util-jvm also caches Java_FOUND and JNI_FOUND
# for us.
-FIND_PACKAGE (zorba_util-jvm_module REQUIRED)
-
-IF (JNI_FOUND)
- INCLUDE_DIRECTORIES (${zorba_util-jvm_module_INCLUDE_DIRS})
-
- ENABLE_TESTING ()
- INCLUDE (CTest)
-
- SET_CMAKE_MODULE_PATH ()
-
- FIND_PACKAGE (Zorba REQUIRED HINTS "${ZORBA_BUILD_DIR}")
- INCLUDE ("${Zorba_USE_FILE}")
-
- ADD_SUBDIRECTORY ("src")
- #ADD_TEST_DIRECTORY ("${CMAKE_SOURCE_DIR}/test")
-
- DONE_DECLARING_ZORBA_URIS ()
-
-ELSE (JNI_FOUND)
- MESSAGE ( STATUS "Java and/or JNI not found; skipping data-formating module.")
-ENDIF(JNI_FOUND)
-
+FIND_PACKAGE (zorba_util-jvm_module QUIET)
+
+IF (zorba_util-jvm_module_FOUND)
+ IF (JNI_FOUND)
+INCLUDE_DIRECTORIES (${zorba_util-jvm_module_INCLUDE_DIRS})
+
+ENABLE_TESTING ()
+INCLUDE (CTest)
+
+SET_CMAKE_MODULE_PATH ()
+
+FIND_PACKAGE (Zorba REQUIRED HINTS "${ZORBA_BUILD_DIR}")
+INCLUDE ("${Zorba_USE_FILE}")
+
+ADD_SUBDIRECTORY ("src")
+#ADD_TEST_DIRECTORY ("${CMAKE_SOURCE_DIR}/test")
+
+DONE_DECLARING_ZORBA_URIS ()
+
+ ELSE (JNI_FOUND)
+MESSAGE ( STATUS "Java and/or JNI not found; skipping data-formating module.")
+ ENDIF(JNI_FOUND)
+
+ELSE (zorba_util-jvm_module_FOUND)
+ MESSAGE (STATUS "Zorba's util-jvm module not found; skipping data-formatting module.")
+ENDIF (zorba_util-jvm_module_FOUND)
MESSAGE (STATUS " END Configuring module Data-Formatting ")
--
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 988090] Re: data-formatting should build without util-jvm
*** This bug is a duplicate of bug 990587 *** https://bugs.launchpad.net/bugs/990587 ** This bug has been marked a duplicate of bug 990587 Zorba with modules does not build if only JRE is installed (no javac or jar) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/988090 Title: data-formatting should build without util-jvm Status in Zorba - The XQuery Processor: New Bug description: The data-formatting module currently marks util-jvm as REQUIRED, and fails with a somewhat mysterious message if it is not there. We should (a) make it build without util-jvm (albeit without including the xsl- fo module), and (b) improve the error message to point to the util-jvm module. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/988090/+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 990587] Re: Zorba with modules does not build if only JRE is installed (no javac or jar)
This is a duplicate of bug 988090, but since this one has more detail I'll mark the other one as a dupe. Assigning to me as this is basically my stuff. ** Changed in: zorba Assignee: Cezar Andrei (cezar-andrei) => 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/990587 Title: Zorba with modules does not build if only JRE is installed (no javac or jar) Status in Zorba - The XQuery Processor: New Bug description: If a target machine has the JRE installed but it doesn't have the "javac" and/or "jar" executables, Zorba fails to build. 1. Install JRE on the machine and make sure the full JSE is not installed (there should be no javac or jar executable) 2. Checkout the trunk 3. Checkout all the modules 4. run "cmake .." in the build folder The util-jvm and data-formatting modules fail to properly configure and this results in a Zorba cmake error instead of disabling the data- formatting module. The output I get: -- --- Module: util-jvm--- -- Looking for JNI -- Looking for Java -- Java version 1.6.0.24 configured successfully! -- Could NOT find Java (missing: Java_JAR_EXECUTABLE Java_JAVAC_EXECUTABLE) (found version "1.6.0.24") -- Building Zorba without util-jvm module. -- --- -- CMake Error at /home/colea/xquery_bzr/zorba_modules/schema-tools/CMakeLists.txt:22 (FIND_PACKAGE): Could not find module Findzorba_util-jvm_module.cmake or a configuration file for package zorba_util-jvm_module. Adjust CMAKE_MODULE_PATH to find Findzorba_util-jvm_module.cmake or set zorba_util-jvm_module_DIR to the directory containing a CMake configuration file for zorba_util-jvm_module. The file will have one of the following names: zorba_util-jvm_moduleConfig.cmake zorba_util-jvm_module-config.cmake -- Java and/or JNI not found; skipping schema-tools module. ... -- BEGIN Configuring module Data-Formatting CMake Error at /home/colea/xquery_bzr/zorba_modules/data-formatting/CMakeLists.txt:25 (FIND_PACKAGE): Could not find module Findzorba_util-jvm_module.cmake or a configuration file for package zorba_util-jvm_module. Adjust CMAKE_MODULE_PATH to find Findzorba_util-jvm_module.cmake or set zorba_util-jvm_module_DIR to the directory containing a CMake configuration file for zorba_util-jvm_module. The file will have one of the following names: zorba_util-jvm_moduleConfig.cmake zorba_util-jvm_module-config.cmake -- Java and/or JNI not found; skipping data-formating module. -- END Configuring module Data-Formatting ... -- Configuring incomplete, errors occurred! To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/990587/+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
