summaryrefslogtreecommitdiffstats
path: root/scripts/lib/recipetool
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2017-10-27 10:12:47 +1300
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-11-11 12:14:27 +0000
commit89ac9a6b8b1a21567cf8ef30e3fc8931eb808b39 (patch)
treeb9cdadc67ef01e9f961fbbb076f523c2af042f18 /scripts/lib/recipetool
parent4a523a4efea53b89aadef24570df2e13830a763b (diff)
downloadpoky-89ac9a6b8b1a21567cf8ef30e3fc8931eb808b39.tar.gz
recipetool: create: show a warning for github archive URLs
github archive URLs are not guaranteed to be stable [1] and thus we should show a warning if a user specifies one to recipetool create (or devtool add). [1] http://lists.openembedded.org/pipermail/openembedded-core/2017-September/142519.html (From OE-Core rev: 7e84a777aa924a237b4e604120ebf8a4b3ba53b2) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/recipetool')
-rw-r--r--scripts/lib/recipetool/create.py3
1 files changed, 3 insertions, 0 deletions
diff --git a/scripts/lib/recipetool/create.py b/scripts/lib/recipetool/create.py
index 055bdef436..54e7e85396 100644
--- a/scripts/lib/recipetool/create.py
+++ b/scripts/lib/recipetool/create.py
@@ -433,6 +433,9 @@ def create_recipe(args):
433 source = 'file://%s' % os.path.abspath(source) 433 source = 'file://%s' % os.path.abspath(source)
434 434
435 if scriptutils.is_src_url(source): 435 if scriptutils.is_src_url(source):
436 # Warn about github archive URLs
437 if re.match('https?://github.com/[^/]+/[^/]+/archive/.+(\.tar\..*|\.zip)$', source):
438 logger.warn('github archive files are not guaranteed to be stable and may be re-generated over time. If the latter occurs, the checksums will likely change and the recipe will fail at do_fetch. It is recommended that you point to an actual commit or tag in the repository instead (using the repository URL in conjunction with the -S/--srcrev option).')
436 # Fetch a URL 439 # Fetch a URL
437 fetchuri = reformat_git_uri(urldefrag(source)[0]) 440 fetchuri = reformat_git_uri(urldefrag(source)[0])
438 if args.binary: 441 if args.binary: