[Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Validation queue job feature-fix-internal-audit-constants-2013-04-09T00-50-45.574Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/feature-fix-internal-audit-constants-2013-04-09T00-50-45.574Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1, Needs Information < 1, Resubmit < 1. Got: 1 Approve, 1 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Validation queue job feature-fix-internal-audit-constants-2013-04-08T15-44-45.985Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/feature-fix-internal-audit-constants-2013-04-08T15-44-45.985Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 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/feature-fix-internal-audit-constants into lp:zorba
Till Westmann has proposed merging lp:~zorba-coders/zorba/feature-fix-internal-audit-constants into lp:zorba. Commit message: change name of generated constants to avoid conflicts Requested reviews: Till Westmann (tillw) Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 If we create an audit property with the last segment being "path" we get a conflict for the generated c++ constants. This conflict is avoided with this change. -- https://code.launchpad.net/~zorba-coders/zorba/feature-fix-internal-audit-constants/+merge/157676 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'src/api/auditprops_cpp.xq' --- src/api/auditprops_cpp.xq 2013-04-03 22:52:25 + +++ src/api/auditprops_cpp.xq 2013-04-08 15:44:24 + @@ -28,7 +28,7 @@ declare function local:group-path($group-spec as xs:string*) as xs:string { let $name := string-join($group-spec ! upper-case(.), "_") -let $path := $name || "_PATH" +let $path := $name || "_" return 'const char* ' || $path || "[]" === modified file 'src/api/pregenerated/auditprops.cpp' --- src/api/pregenerated/auditprops.cpp 2013-04-03 22:52:25 + +++ src/api/pregenerated/auditprops.cpp 2013-04-08 15:44:24 + @@ -1,5 +1,5 @@ -const char* XQUERY_COMPILATION_PATH[] = { "xquery", "compilation" }; -const PropertyGroupImpl XQUERY_COMPILATION(2, XQUERY_COMPILATION_PATH); +const char* XQUERY_COMPILATION_[] = { "xquery", "compilation" }; +const PropertyGroupImpl XQUERY_COMPILATION(2, XQUERY_COMPILATION_); const PropertyImpl XQUERY_COMPILATION_FILENAME(XQUERY_COMPILATION, "filename", 0, Property::STRING); const PropertyImpl XQUERY_COMPILATION_PARSE_DURATION(XQUERY_COMPILATION, "parse-duration", 1, Property::INT); const PropertyImpl XQUERY_COMPILATION_TRANSLATION_DURATION(XQUERY_COMPILATION, "translation-duration", 2, Property::INT); -- 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