aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2016-09-16 16:28:10 +1200
committerPaul Eggleton <paul.eggleton@linux.intel.com>2016-09-19 08:06:10 +1200
commit18a6cbdd7dfdc478ad5199491c3a29033e6d99d6 (patch)
tree5ff23a2d891c2d6ecc969ee88a302fb0badbf7b3
parent89d68923d93f02e598929c552d5c5f6d9b607c2c (diff)
downloadopenembedded-core-contrib-paule/recipetool-fixes6.tar.gz
openembedded-core-contrib-paule/recipetool-fixes6.tar.bz2
openembedded-core-contrib-paule/recipetool-fixes6.zip
devtool: add: drop superfluous validation for recipe namepaule/recipetool-fixes6
Now that recipeutils.validate_pn() properly validates characters used in the name, we can drop this bit checking for '/' since that's not permitted by validate_pn(). (The FIXME comment here - that I myself apparently wrote - is questionable since that function was clearly never intended to allow '/', perhaps I was misled because it was broken and did so). Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
-rw-r--r--scripts/lib/devtool/standard.py4
1 files changed, 0 insertions, 4 deletions
diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py
index abbc0cb8f5..83191450be 100644
--- a/scripts/lib/devtool/standard.py
+++ b/scripts/lib/devtool/standard.py
@@ -85,10 +85,6 @@ def add(args, config, basepath, workspace):
if reason:
raise DevtoolError(reason)
- # FIXME this ought to be in validate_pn but we're using that in other contexts
- if '/' in args.recipename:
- raise DevtoolError('"/" is not a valid character in recipe names')
-
if args.srctree:
srctree = os.path.abspath(args.srctree)
srctreeparent = None