[gnome-continuous-yocto/gnomeostree-3.28-rocko: 2513/8267] devtool: add: drop superfluous validation for recipe name



commit 383a4af0e9cf410961ee681f8a2f07b5818f3762
Author: Paul Eggleton <paul eggleton linux intel com>
Date:   Mon Sep 19 08:08:13 2016 +1200

    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>

 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 abbc0cb..8319145 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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]