On Tuesday 16 December 2014 20:14:23 Flanagan, Elizabeth wrote: > 2014-11-19 15:12 GMT+00:00 Bryan Evenson <beven...@melinkcorp.com>: > > All, > > > > I'm getting close to implementing this feature, but I'm having some issues > > getting auto.conf formatted correctly. See below.> > >> -----Original Message----- > >> From: Bryan Evenson > >> Sent: Monday, November 17, 2014 9:08 AM > >> To: Bryan Evenson; yocto@yoctoproject.org; elizabeth.flana...@intel.com > >> Subject: RE: autobuilder: How to set PREMIRRORS? > >> > >> All, > >> > >> After looking through the autobuilder code, I don't see anywhere in which > >> PREMIRRORS can be set and used. I see in CreateAutoConf.py that > >> PREMIRRORS is always set to "". I don't think it'd be that difficult to > >> add PREMIRRORS as a CreateAutoConf parameter for the buildsets. I can > >> work on the change and submit the patch. But before I start I have a > >> few questions: > >> > >> 1. I plan on implementing the PREMIRRORS parameter as an array, similar > >> to > >> 'layerdirs' for CreateBBLayersConf. For example: > >> > >> {'CreateAutoConf' : {'PREMIRRORS' : ['git://.*/.* > >> http://our/local/mirror/path/mirror/sources/ ', > >> > >> 'ftp://.*/.* http://our/local/mirror/path /mirror/sources/ ', > >> 'http://.*/.* http://our/local/mirror/path/mirror/sources/ ', > >> 'https://.*/.* http://our/local/mirror/path/mirror/sources/ ']} > >> > >> would add the following to auto.conf: > >> PREMIRRORS = 'git://.*/.* http://our/local/mirror/path/mirror/sources/ > >> \ > >> > >> ftp://.*/.* http://our/local/mirror/path /mirror/sources/ \ > >> http://.*/.* http://our/local/mirror/path/mirror/sources/ \ > >> https://.*/.* http://our/local/mirror/path/mirror/sources/ ' > >> > >> Does this sound reasonable, or would a different parameter format be > >> preferred? > > I would actually do this similar to how we have DEVKERNEL_MUT_REPO. > > PREMIRRORS = ['git://.*/.* > http://our/local/mirror/path/mirror/sources/', 'ftp://.*/.* > http://our/local/mirror/path /mirror/sources/', 'http://.*/.* > http://our/local/mirror/path/mirror/sources/', 'https://.*/.* > http://our/local/mirror/path/mirror/sources/']
I don't think we support setting a variable to a list like this - does that actually work? Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre -- _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto