public inbox for gentoo-catalyst@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-catalyst] [PATCH 2/5] Rename config option 'portage_overlay' -> 'repos'
@ 2021-02-04  0:35 Felix Bier
  0 siblings, 0 replies; only message in thread
From: Felix Bier @ 2021-02-04  0:35 UTC (permalink / raw
  To: gentoo-catalyst@lists.gentoo.org

This commit renames the config option 'portage_overlay' to 'repos'.
Also, it renames the method 'portage_overlay' to 'process_repos'
and 'set_portage_overlay' to 'set_repos'. Motivation:

 1) The Gentoo wiki states 'repository' as the preferred term,
    and 'overlay' only as a colloquial
    (https://wiki.gentoo.org/wiki/Ebuild_repository),
    reducing the distinction between the main repository
    and other 'overlay' repositories.

 3) This avoids confusion with filesystem overlays
    (via the config option 'overlay' and 'root_overlay').

 2) The plural form highlights the possibility of specifying
    multiple repositories, continuing 2861038a.

Signed-off-by: Felix Bier <felix.bier@rohde-schwarz.com>
---
 catalyst/base/stagebase.py           | 30 ++++++++++++++--------------
 catalyst/targets/livecd_stage2.py    |  2 +-
 catalyst/targets/stage1.py           |  6 +++---
 catalyst/targets/stage2.py           |  6 +++---
 catalyst/targets/stage3.py           |  6 +++---
 catalyst/targets/stage4.py           |  2 +-
 doc/catalyst-config.5.txt            |  2 +-
 doc/catalyst-spec.5.txt              |  2 +-
 examples/generic_stage_template.spec |  4 ++--
 examples/livecd-stage1_template.spec |  4 ++--
 examples/livecd-stage2_template.spec |  4 ++--
 examples/netboot_template.spec       |  4 ++--
 examples/stage4_template.spec        |  4 ++--
 13 files changed, 38 insertions(+), 38 deletions(-)

diff --git a/catalyst/base/stagebase.py b/catalyst/base/stagebase.py
index 46cb1fda..fedc8f87 100644
--- a/catalyst/base/stagebase.py
+++ b/catalyst/base/stagebase.py
@@ -81,14 +81,14 @@ class StageBase(TargetBase, ClearBase, GenBase):
             "ldflags",
             "pkgcache_path",
             "portage_confdir",
-            "portage_overlay",
+            "repos",
             "portage_prefix",
         ])
         self.prepare_sequence = [
             self.unpack,
             self.config_profile_link,
             self.setup_confdir,
-            self.portage_overlay,
+            self.process_repos,
         ]
         self.build_sequence = [
             self.bind,
@@ -206,7 +206,7 @@ class StageBase(TargetBase, ClearBase, GenBase):
         self.set_linuxrc()
         self.set_busybox_config()
         self.set_overlay()
-        self.set_portage_overlay()
+        self.set_repos()
         self.set_root_overlay()
 
         # This next line checks to make sure that the specified variables exist on disk.
@@ -586,13 +586,13 @@ class StageBase(TargetBase, ClearBase, GenBase):
                 del self.settings[self.settings["spec_prefix"] +
                                   "/busybox_config"]
 
-    def set_portage_overlay(self):
-        if "portage_overlay" in self.settings:
-            if isinstance(self.settings['portage_overlay'], str):
-                self.settings["portage_overlay"] = \
-                    self.settings["portage_overlay"].split()
-            log.info('portage_overlay directories are set to: %s',
-                     ' '.join(self.settings['portage_overlay']))
+    def set_repos(self):
+        if 'repos' in self.settings:
+            if isinstance(self.settings['repos'], str):
+                self.settings['repos'] = \
+                    self.settings['repos'].split()
+            log.info('repos directories are set to: %s',
+                     ' '.join(self.settings['repos']))
 
     def set_overlay(self):
         if self.settings["spec_prefix"] + "/overlay" in self.settings:
@@ -831,10 +831,10 @@ class StageBase(TargetBase, ClearBase, GenBase):
         except OSError as e:
             raise CatalystError(f'Could not write {repo_conf_chroot}: {e}') from e
 
-    def portage_overlay(self):
+    def process_repos(self):
         """ We copy the contents of our repos to get_repo_location(repo_name) """
-        if "portage_overlay" in self.settings:
-            for x in self.settings["portage_overlay"]:
+        if 'repos' in self.settings:
+            for x in self.settings['repos']:
                 if os.path.exists(x):
                     name = get_repo_name(x)
 
@@ -1145,8 +1145,8 @@ class StageBase(TargetBase, ClearBase, GenBase):
             clear_path(target)
 
         # Remove our overlays
-        if "portage_overlay" in self.settings:
-            for repo_path in self.settings["portage_overlay"]:
+        if 'repos' in self.settings:
+            for repo_path in self.settings['repos']:
                 repo_name = get_repo_name(repo_path)
 
                 repo_conf = self.get_repo_conf_path(repo_name)
diff --git a/catalyst/targets/livecd_stage2.py b/catalyst/targets/livecd_stage2.py
index ff4ea62a..774702f8 100644
--- a/catalyst/targets/livecd_stage2.py
+++ b/catalyst/targets/livecd_stage2.py
@@ -42,7 +42,7 @@ class livecd_stage2(StageBase):
         "livecd/xdm",
         "livecd/xinitrc",
         "livecd/xsession",
-        "portage_overlay",
+        "repos",
     ])
 
     def __init__(self, spec, addlargs):
diff --git a/catalyst/targets/stage1.py b/catalyst/targets/stage1.py
index 5a154e76..0ea5f92b 100644
--- a/catalyst/targets/stage1.py
+++ b/catalyst/targets/stage1.py
@@ -66,9 +66,9 @@ class stage1(StageBase):
         if "ldflags" in self.settings:
             self.settings["LDFLAGS"] = self.settings["ldflags"]
 
-    def set_portage_overlay(self):
-        StageBase.set_portage_overlay(self)
-        if "portage_overlay" in self.settings:
+    def set_repos(self):
+        StageBase.set_repos(self)
+        if "repos" in self.settings:
             log.warning(
                 'Using an overlay for earlier stages could cause build issues.\n'
                 "If you break it, you buy it.  Don't complain to us about it.\n"
diff --git a/catalyst/targets/stage2.py b/catalyst/targets/stage2.py
index 0b7244e3..786f1020 100644
--- a/catalyst/targets/stage2.py
+++ b/catalyst/targets/stage2.py
@@ -39,9 +39,9 @@ class stage2(StageBase):
         if "ldflags" in self.settings:
             self.settings["LDFLAGS"] = self.settings["ldflags"]
 
-    def set_portage_overlay(self):
-        StageBase.set_portage_overlay(self)
-        if "portage_overlay" in self.settings:
+    def set_repos(self):
+        StageBase.set_repos(self)
+        if "repos" in self.settings:
             log.warning(
                 'Using an overlay for earlier stages could cause build issues.\n'
                 "If you break it, you buy it.  Don't complain to us about it.\n"
diff --git a/catalyst/targets/stage3.py b/catalyst/targets/stage3.py
index cfc0dbf9..d20ed679 100644
--- a/catalyst/targets/stage3.py
+++ b/catalyst/targets/stage3.py
@@ -17,9 +17,9 @@ class stage3(StageBase):
     def __init__(self, spec, addlargs):
         StageBase.__init__(self, spec, addlargs)
 
-    def set_portage_overlay(self):
-        StageBase.set_portage_overlay(self)
-        if "portage_overlay" in self.settings:
+    def set_repos(self):
+        StageBase.set_repos(self)
+        if "repos" in self.settings:
             log.warning(
                 'Using an overlay for earlier stages could cause build issues.\n'
                 "If you break it, you buy it.  Don't complain to us about it.\n"
diff --git a/catalyst/targets/stage4.py b/catalyst/targets/stage4.py
index f8383f75..ff1d4dca 100644
--- a/catalyst/targets/stage4.py
+++ b/catalyst/targets/stage4.py
@@ -15,7 +15,7 @@ class stage4(StageBase):
     ])
     valid_values = required_values | frozenset([
         "boot/kernel",
-        "portage_overlay",
+        "repos",
         "stage4/empty",
         "stage4/fsscript",
         "stage4/gk_mainargs",
diff --git a/doc/catalyst-config.5.txt b/doc/catalyst-config.5.txt
index ebd99b55..003d40b6 100644
--- a/doc/catalyst-config.5.txt
+++ b/doc/catalyst-config.5.txt
@@ -115,7 +115,7 @@ Defaults to `${storedir}/repos`.
 
 *repo_basedir*::
 The target repository directory to contain the primary repo (e.g.,
-gentoo repo) and any overlays.  The default location is
+gentoo repo) and any other repos. The default location is
 `/var/db/repos`.
 
 *repo_name*::
diff --git a/doc/catalyst-spec.5.txt b/doc/catalyst-spec.5.txt
index 4c1df857..10eb0bb8 100644
--- a/doc/catalyst-spec.5.txt
+++ b/doc/catalyst-spec.5.txt
@@ -80,7 +80,7 @@ This is an optional directory containing portage configuration files
 `/etc/portage` and should be consistent across all targets to minimize
 problems.
 
-*portage_overlay*::
+*repos*::
 This option specifies the location of the ebuild repositories that you would
 like to have used when building this target. It takes a space-separated list
 of directory names. (example: `/usr/local/portage`).
diff --git a/examples/generic_stage_template.spec b/examples/generic_stage_template.spec
index 8c8889bf..9d91d07d 100644
--- a/examples/generic_stage_template.spec
+++ b/examples/generic_stage_template.spec
@@ -93,8 +93,8 @@ portage_confdir:
 # like to have used when building this target. It takes a space-separated list
 # of directory names.
 # example:
-# portage_overlay: /usr/local/portage
-portage_overlay:
+# repos: /usr/local/portage
+repos:
 
 # This allows the optional directory containing the output packages for
 # catalyst.  Mainly used as a way for different spec files to access the same
diff --git a/examples/livecd-stage1_template.spec b/examples/livecd-stage1_template.spec
index 269a1e3d..b9edb87d 100644
--- a/examples/livecd-stage1_template.spec
+++ b/examples/livecd-stage1_template.spec
@@ -56,8 +56,8 @@ portage_confdir:
 # like to have used when building this target. It takes a space-separated list
 # of directory names.
 # example:
-# portage_overlay: /usr/local/portage
-portage_overlay:
+# repos: /usr/local/portage
+repos:
 
 # This allows the optional directory containing the output packages for
 # catalyst.  Mainly used as a way for different spec files to access the same
diff --git a/examples/livecd-stage2_template.spec b/examples/livecd-stage2_template.spec
index 4f50919f..8db17ed7 100644
--- a/examples/livecd-stage2_template.spec
+++ b/examples/livecd-stage2_template.spec
@@ -56,8 +56,8 @@ portage_confdir:
 # like to have used when building this target. It takes a space-separated list
 # of directory names.
 # example:
-# portage_overlay: /usr/local/portage
-portage_overlay:
+# repos: /usr/local/portage
+repos:
 
 # This allows the optional directory containing the output packages for
 # catalyst.  Mainly used as a way for different spec files to access the same
diff --git a/examples/netboot_template.spec b/examples/netboot_template.spec
index ad9d993c..07b3b53f 100644
--- a/examples/netboot_template.spec
+++ b/examples/netboot_template.spec
@@ -10,8 +10,8 @@ source_subpath: default/stage3-mips-uclibc-mips3-2006.126
 # like to have used when building this target. It takes a space-separated list
 # of directory names.
 # example:
-# portage_overlay: /usr/local/portage
-portage_overlay:
+# repos: /usr/local/portage
+repos:
 
 boot/kernel: ip22r4k ip22r5k ip27r10k ip28r10k ip30r10k ip32r5k
 boot/kernel/ip22r4k/sources: =mips-sources-2.6.14.5
diff --git a/examples/stage4_template.spec b/examples/stage4_template.spec
index aea70e86..5fbf6a50 100644
--- a/examples/stage4_template.spec
+++ b/examples/stage4_template.spec
@@ -56,8 +56,8 @@ portage_confdir:
 # like to have used when building this target. It takes a space-separated list
 # of directory names.
 # example:
-# portage_overlay: /usr/local/portage
-portage_overlay:
+# repos: /usr/local/portage
+repos:
 
 # This allows the optional directory containing the output packages for
 # catalyst.  Mainly used as a way for different spec files to access the same
-- 
2.30.0



^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-02-04  0:35 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-04  0:35 [gentoo-catalyst] [PATCH 2/5] Rename config option 'portage_overlay' -> 'repos' Felix Bier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox