Patchwork [bitbake-devel] cvs fetcher

login
mail settings
Submitter Jate Sujjavanich
Date Sept. 19, 2012, 2:04 p.m.
Message ID <6C2434209962DC46B88345CA85C334A20183E5A9859A@Courier.syntech.org>
Download mbox | patch
Permalink /patch/36921/
State New
Headers show

Comments

Jate Sujjavanich - Sept. 19, 2012, 2:04 p.m.
While using a newer bitbake, I ran into some issues with a recipe that has a cvs SRC_URI. This patch fixes it, and I wanted to submit it for further review.


The localdata variable was removed but is required later in the download
function.
Add an extra space so that cvscmd's parameter is separated from the
command.
Signed-off-by: Jate Sujjavanich <jate.sujjavanich@myfuelmaster.com>
----------------------------- lib/bb/fetch2/cvs.py ----------------------------

Patch

diff --git a/lib/bb/fetch2/cvs.py b/lib/bb/fetch2/cvs.py
index 6a948c4..5249d85 100644
--- a/lib/bb/fetch2/cvs.py
+++ b/lib/bb/fetch2/cvs.py
@@ -111,8 +111,10 @@ 
         if ud.tag:
             options.append("-r %s" % ud.tag)
 
+        localdata = data.createCopy(d)
+        
         cvsbasecmd = d.getVar("FETCHCMD_cvs", True)
-        cvscmd = cvsbasecmd + "'-d" + cvsroot + "' co " + " ".join(options) + " " + ud.module
+        cvscmd = cvsbasecmd + " " + "'-d" + cvsroot + "' co " + " ".join(options) + " " + ud.module
         cvsupdatecmd = cvsbasecmd + "'-d" + cvsroot + "' update -d -P " + " ".join(options)
 
         if cvs_rsh: