Gentoo Archives: gentoo-catalyst

From: Matt Turner <mattst88@g.o>
To: gentoo-catalyst@l.g.o
Cc: Matt Turner <mattst88@g.o>
Subject: [gentoo-catalyst] [PATCH 13/21] catalyst: Remove support for source_matching="loose"
Date: Wed, 20 May 2020 03:43:08
Message-Id: 20200520034226.2870937-13-mattst88@gentoo.org
In Reply to: [gentoo-catalyst] [PATCH 01/21] targets: Remove copy_{file,symlink,lib} functions by Matt Turner
1 This does not seem like a useful feature to me.
2
3 Signed-off-by: Matt Turner <mattst88@g.o>
4 ---
5 catalyst/base/stagebase.py | 3 ---
6 catalyst/defaults.py | 1 -
7 catalyst/support.py | 6 +++---
8 etc/catalyst.conf | 12 ------------
9 4 files changed, 3 insertions(+), 19 deletions(-)
10
11 diff --git a/catalyst/base/stagebase.py b/catalyst/base/stagebase.py
12 index f39895fe..febaf969 100644
13 --- a/catalyst/base/stagebase.py
14 +++ b/catalyst/base/stagebase.py
15 @@ -130,8 +130,6 @@ class StageBase(TargetBase, ClearBase, GenBase):
16 decomp_opt=self.settings["decomp_opt"])
17 self.accepted_extensions = self.decompressor.search_order_extensions(
18 self.settings["decompressor_search_order"])
19 - log.notice("Source file specification matching setting is: %s",
20 - self.settings["source_matching"])
21 log.notice("Accepted source file extensions search order: %s",
22 self.accepted_extensions)
23 # save resources, it is not always needed
24 @@ -409,7 +407,6 @@ class StageBase(TargetBase, ClearBase, GenBase):
25 normpath(self.settings["storedir"] + "/builds/" +
26 self.settings["source_subpath"]),
27 self.accepted_extensions,
28 - self.settings["source_matching"] in ["strict"]
29 )
30 log.debug('Source path returned from file_check is: %s',
31 self.settings["source_path"])
32 diff --git a/catalyst/defaults.py b/catalyst/defaults.py
33 index 412cb956..14f671fe 100644
34 --- a/catalyst/defaults.py
35 +++ b/catalyst/defaults.py
36 @@ -45,7 +45,6 @@ confdefaults = {
37 "repos": "%(storedir)s/repos",
38 "sharedir": "/usr/share/catalyst",
39 "shdir": "%(sharedir)s/targets",
40 - "source_matching": "strict",
41 "storedir": "/var/tmp/catalyst",
42 "target_distdir": "/var/cache/distfiles",
43 "target_pkgdir": "/var/cache/binpkgs",
44 diff --git a/catalyst/support.py b/catalyst/support.py
45 index c4a5c797..a6a6854a 100644
46 --- a/catalyst/support.py
47 +++ b/catalyst/support.py
48 @@ -59,7 +59,7 @@ def cmd(mycmd, env=None, debug=False, fail_func=None):
49 print_traceback=False)
50
51
52 -def file_check(filepath, extensions=None, strict=True):
53 +def file_check(filepath, extensions=None):
54 '''Check for the files existence and that only one exists
55 if others are found with various extensions
56 '''
57 @@ -73,8 +73,8 @@ def file_check(filepath, extensions=None, strict=True):
58 ".CONTENTS") and not x.endswith(".CONTENTS.gz") and not x.endswith(".DIGESTS")]
59 if len(files) == 1:
60 return files[0]
61 - if len(files) > 1 and strict:
62 - msg = "Ambiguos Filename: %s\nPlease specify the correct extension as well" % filepath
63 + if len(files) > 1:
64 + msg = "Ambiguous Filename: %s\nPlease specify the correct extension as well" % filepath
65 raise CatalystError(msg, print_traceback=False)
66 target_file = None
67 for ext in extensions:
68 diff --git a/etc/catalyst.conf b/etc/catalyst.conf
69 index f64fe971..d33be15f 100644
70 --- a/etc/catalyst.conf
71 +++ b/etc/catalyst.conf
72 @@ -50,18 +50,6 @@ envscript="/etc/catalyst/catalystrc"
73 # (These options can be used together)
74 options="autoresume bindist kerncache pkgcache seedcache"
75
76 -# source_matching specifies how catalyst will match non-specific file names
77 -# if the filename is not found as an exact match.
78 -# ie: a filename without the extension specified. "/path/to/foo"
79 -#
80 -# possible values are:
81 -# "strict" meaning if more than one file of that name is present with any
82 -# file extension, then it will raise an exception.
83 -# "loose" meaning it will search for an existing filename with an added
84 -# extension from an ordered list of extensions determined from the
85 -# decompressor_search_order specification in the spec file or (default)
86 -source_matching="strict"
87 -
88 # port_logdir is where all build logs will be kept. This dir will be automatically cleaned
89 # of all logs over 30 days old. If left undefined the logs will remain in the build directory
90 # as usual and get cleaned every time a stage build is restarted.
91 --
92 2.26.2

Replies