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-19 09:06:52 +0100 |
| commit | e010d9be3709cf3c607ffc03c3188abe4e1e9eb4 (patch) | |
| tree | 5e59cfe37ee0a0ec5010315ead1e9055d81a38db /scripts/lib/devtool | |
| parent | c5d5a1baf98a11676537fb5e9f8ec4409e30c1fd (diff) | |
| download | openembedded-core-e010d9be3709cf3c607ffc03c3188abe4e1e9eb4.tar.gz openembedded-core-e010d9be3709cf3c607ffc03c3188abe4e1e9eb4.tar.bz2 openembedded-core-e010d9be3709cf3c607ffc03c3188abe4e1e9eb4.zip | |
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).
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/devtool')
| -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): 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 |