On Thursday 18 December 2014 13:16:31 Paul Eggleton wrote:
> 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?

Oops, I missed the context was the autobuilder python code - ignore me ;)

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to