From: Paul Eggleton Date: Sun, 18 Sep 2016 20:08:13 +0000 (+1200) Subject: devtool: add: drop superfluous validation for recipe name X-Git-Tag: uninative-1.4~67 X-Git-Url: https://code.ossystems.io/gitweb?a=commitdiff_plain;h=e010d9be3709cf3c607ffc03c3188abe4e1e9eb4;p=openembedded-core.git 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 Signed-off-by: Richard Purdie --- 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