diff options
author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2016-09-19 08:08:13 +1200 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-20 15:11:07 +0100 |
commit | 383a4af0e9cf410961ee681f8a2f07b5818f3762 (patch) | |
tree | 7f21dd5d43ddae52cc281a92323b8b3c0f32d78a | |
parent | ce1904a1602de203293f901fe947d18105c5a1b0 (diff) | |
download | poky-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.py | 4 |
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 |