summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2016-09-19 08:08:13 +1200
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-09-20 15:11:07 +0100
commit383a4af0e9cf410961ee681f8a2f07b5818f3762 (patch)
tree7f21dd5d43ddae52cc281a92323b8b3c0f32d78a
parentce1904a1602de203293f901fe947d18105c5a1b0 (diff)
downloadpoky-383a4af0e9cf410961ee681f8a2f07b5818f3762.tar.gz
devtool: add: drop superfluous validation for recipe name
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). (From OE-Core rev: e010d9be3709cf3c607ffc03c3188abe4e1e9eb4) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-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):
85 if reason: 85 if reason:
86 raise DevtoolError(reason) 86 raise DevtoolError(reason)
87 87
88 # FIXME this ought to be in validate_pn but we're using that in other contexts
89 if '/' in args.recipename:
90 raise DevtoolError('"/" is not a valid character in recipe names')
91
92 if args.srctree: 88 if args.srctree:
93 srctree = os.path.abspath(args.srctree) 89 srctree = os.path.abspath(args.srctree)
94 srctreeparent = None 90 srctreeparent = None