Merge lp:~zorba-coders/zorba/xqdoc_fix into lp:zorba

Proposed by William Candillon
Status: Merged
Approved by: Sorin Marian Nasoi
Approved revision: 10970
Merged at revision: 10972
Proposed branch: lp:~zorba-coders/zorba/xqdoc_fix
Merge into: lp:zorba
Diff against target: 19 lines (+5/-2)
1 file modified
modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq (+5/-2)
To merge this branch: bzr merge lp:~zorba-coders/zorba/xqdoc_fix
Reviewer Review Type Date Requested Status
Sorin Marian Nasoi Approve
Matthias Brantner Approve
Review via email: mp+118175@code.launchpad.net

Commit message

Fix make doc target when multiple version of the same module exists.

Description of the change

Fix make doc target when multiple version of the same module exists.

To post a comment you must log in.
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Matthias Brantner (matthias-brantner) :
review: Approve
Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job xqdoc_fix-2012-08-06T10-55-57.96Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq'
2--- modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq 2012-07-24 08:48:48 +0000
3+++ modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq 2012-08-03 18:43:23 +0000
4@@ -324,10 +324,13 @@
5 variable $manifestXML := fn:parse-xml(file:read-text($zorbaManifestPath));
6
7 variable $moduleManifests := $manifestXML/z:manifest/z:module;
8-
9+
10 for $module in $moduleManifests
11+ let $uri := $module/z:uri/text()
12+ group by $uri
13+ let $module := if(count($module) gt 1) then $module[@version = max($module/@version)] else $module
14 return
15- insert node <module uri="{data($module/z:uri)}"
16+ insert node <module uri="{$uri}"
17 isCore="{data($module/@isCore)}"
18 version="{if (exists(data($module/@version))) then data($module/@version) else ''}"
19 projectRoot="{data($module/z:projectRoot)}"/> as last into $xqdoc2html:ZorbaManifest;

Subscribers

People subscribed via source and target branches