Merge lp:~zorba-coders/zorba/checkout-modules into lp:zorba

[Chris Hillery]

Use "bzr co --lightweight" rather than "bzr branch" when downloading non-core modules.

[Chris Hillery (ceejatec)]

review: Approve

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/checkout-modules-2012-05-24T10-53-02.008Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job checkout-modules-2012-05-24T10-53-02.008Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 1 Approve.

[Chris Hillery (ceejatec)]

It seems that this is no longer desirable.

review: Disapprove

Unmerged revisions

10867. By Chris Hillery on 2012-05-24

Use "bzr co --lightweight" rather than "bzr branch" when downloading non-core
modules.

=== modified file 'modules/DownloadModules.cmake'
--- modules/DownloadModules.cmake	2012-05-03 12:31:51 +0000
+++ modules/DownloadModules.cmake	2012-05-24 10:36:20 +0000
@@ -103,7 +103,8 @@
           if (_modtag AND NOT notags)
             set (_modtagargs "-r" "${_modtag}")
           endif (_modtag AND NOT notags)
-          execute_process (COMMAND "${bzr}" branch "${_modurl}" "${_modname}"
+          execute_process (COMMAND "${bzr}" co --lightweight
+            "${_modurl}" "${_modname}"
             ${_modtagargs} WORKING_DIRECTORY "${outdir}" TIMEOUT 60
 	    RESULT_VARIABLE _status)