On Friday, December 13, 2013 07:07:13 PM Brian Dolbec wrote:
> Use the new configured snapshot_name and portdir settings
> Use the portdir setting rather than hard-coded path
> ---
> catalyst | 14 +++++++++-----
> modules/generic_stage_target.py | 6 ++++--
> modules/snapshot_target.py | 14 +++++++++-----
> modules/tinderbox_target.py | 4 ++--
> 4 files changed, 24 insertions(+), 14 deletions(-)
>
> diff --git a/catalyst b/catalyst
> index ba26f3c..19ec77e 100755
> --- a/catalyst
> +++ b/catalyst
> @@ -61,11 +61,15 @@ def parse_config(myconfig):
> myconf={}
> config_file=""
>
> - confdefaults={ "storedir":"/var/tmp/catalyst",\
> -
> "sharedir":"/usr/share/catalyst","distdir":"/usr/portage/distfiles",\
> - "portdir":"/usr/portage","options":"",\
> - "snapshot_cache":"/var/tmp/catalyst/snapshot_cache",\
> - "hash_function":"crc32"}
> + confdefaults={
> + "hash_function": "crc32",
> + "portdir": "/usr/portage",
> + "repo_name": "portage",
> + "sharedir": "/usr/lib/catalyst",
> + "snapshot_name": "portage-",
> + "snapshot_cache": "/var/tmp/catalyst/snapshot_cache",
> + "storedir": "/var/tmp/catalyst",
> + }
pleas keep the formatting style; no comma on the last element and closing
brace on the same line
>
> # first, try the one passed (presumably from the cmdline)
> if myconfig:
> diff --git a/modules/generic_stage_target.py
> b/modules/generic_stage_target.py index 848aca2..c2888b6 100644
> --- a/modules/generic_stage_target.py
> +++ b/modules/generic_stage_target.py
> @@ -410,7 +410,8 @@ class generic_stage_target(generic_target):
>
> def set_snapshot_path(self):
>
> self.settings["snapshot_path"]=normpath(self.settings["storedir"]+\
> -
> "/snapshots/portage-"+self.settings["snapshot"]+".tar.xz")
> + "/snapshots/" + self.settings["snapshot_name"] +
> + self.settings["snapshot"]+".tar.xz")
please be consistent with spaces between operators; follow whatever PEP8 says
to do this case :)
I'd also suggest using the str.format() method instead of concatenating a
large number of strings and variables, it's much more readable IMHO:
self.settings["snapshot_path"] = os.path.normpath(
"{0}/snapshots/{1}{2}.tar.xz".format(
self.settings["storedir"], self.settings["snapshot_name"],
self.settings["snapshot"]))
>
> if os.path.exists(self.settings["snapshot_path"]):
> self.settings["snapshot_path_hash"]=\
> @@ -418,7 +419,8 @@ class generic_stage_target(generic_target):
>
> hash_function=self.settings["hash_function"],verbose=False)
> else:
>
self.settings["snapshot_path"]=normpath(self.settings["storedir"]+\
> -
> "/snapshots/portage-"+self.settings["snapshot"]+".tar.bz2")
> + "/snapshots/" + self.settings["snapshot_name"] +
> + self.settings["snapshot"]+".tar.bz2")
see my previous comment
>
> if os.path.exists(self.settings["snapshot_path"]):
> self.settings["snapshot_path_hash"]=\
> diff --git a/modules/snapshot_target.py b/modules/snapshot_target.py
> index e93a86a..29d6e87 100644
> --- a/modules/snapshot_target.py
> +++ b/modules/snapshot_target.py
> @@ -18,8 +18,9 @@ class snapshot_target(generic_stage_target):
> self.settings=myspec
> self.settings["target_subpath"]="portage"
> st=self.settings["storedir"]
> -
self.settings["snapshot_path"]=normpath(st+"/snapshots/portage-"+self.set
> tings["version_stamp"]\ - +".tar.bz2")
> + self.settings["snapshot_path"]=normpath(st + "/snapshots/"
> + + self.settings["snapshot_name"]
> + + self.settings["version_stamp"] + ".tar.bz2")
again
>
self.settings["tmp_path"]=normpath(st+"/tmp/"+self.settings["target_subpa
> th"])
>
> def setup(self):
> @@ -46,11 +47,14 @@ class snapshot_target(generic_stage_target):
> if not os.path.exists(mytmp):
> os.makedirs(mytmp)
>
> - cmd("rsync -a --delete --exclude /packages/ --exclude
> /distfiles/
> --exclude /local/ --exclude CVS/ --exclude .svn
> --filter=H_**/files/digest-* "+\ -
> self.settings["portdir"]+"/
> "+mytmp+"/portage/","Snapshot failure",env=self.env) +
> cmd("rsync -a
> --delete --exclude /packages/ --exclude /distfiles/ " + +
> "--exclude
> /local/ --exclude CVS/ --exclude .svn --filter=H_**/files/digest-* " +
> + self.settings["portdir"] + "/ " + mytmp + "/%s/" %
> self.settings["repo_name"], + "Snapshot failure",env=self.env)
>
> print "Compressing Portage snapshot tarball..."
> - cmd("tar -I lbzip2 -cf "+self.settings["snapshot_path"]+" -C
"+mytmp+"
> portage",\ + cmd("tar -I lbzip2 -cf " +
self.settings["snapshot_path"] + "
> -C " + + mytmp + " %s" % self.settings["repo_name"],
> "Snapshot creation failure",env=self.env)
>
> self.gen_contents_file(self.settings["snapshot_path"])
> diff --git a/modules/tinderbox_target.py b/modules/tinderbox_target.py
> index 46fe082..d6d3ea3 100644
> --- a/modules/tinderbox_target.py
> +++ b/modules/tinderbox_target.py
> @@ -29,8 +29,8 @@ class tinderbox_target(generic_stage_target):
> raise CatalystError,"Tinderbox aborting due to error."
>
> def set_cleanables(self):
> -
> self.settings["cleanables"]=["/etc/resolv.conf","/var/tmp/*","/root/*",\
> - "/usr/portage"]
> +
> self.settings["cleanables"]=["/etc/resolv.conf","/var/tmp/*","/root/*",
> + self.settings['portdir']]
spaces between the elements of a list pls.
>
> def set_action_sequence(self):
> #Default action sequence for run method
signature.asc
Description: This is a digitally signed message part.
