[PATCH] use /var/tmp instead of /tmp

Vratislav Podzimek vpodzime at redhat.com
Wed Oct 3 07:08:55 UTC 2012


Looks good to me.

On Tue, 2012-10-02 at 11:36 -0700, Brian C. Lane wrote:
> From: "Brian C. Lane" <bcl at redhat.com>
> 
> Also add a --tmp option so users can specify a top level tmp dir.
> ---
>  src/pylorax/treebuilder.py | 2 +-
>  src/sbin/lorax             | 6 +++++-
>  2 files changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/src/pylorax/treebuilder.py b/src/pylorax/treebuilder.py
> index b6e5527..3ff0e6f 100644
> --- a/src/pylorax/treebuilder.py
> +++ b/src/pylorax/treebuilder.py
> @@ -146,7 +146,7 @@ class RuntimeBuilder(object):
>              runcmd(["depmod", "-a", "-F", ksyms, "-b", root, kver])
>              generate_module_info(moddir+kver, outfile=moddir+"module-info")
>  
> -    def create_runtime(self, outfile="/tmp/squashfs.img", compression="xz", compressargs=[], size=1):
> +    def create_runtime(self, outfile="/var/tmp/squashfs.img", compression="xz", compressargs=[], size=1):
>          # make live rootfs image - must be named "LiveOS/rootfs.img" for dracut
>          workdir = joinpaths(os.path.dirname(outfile), "runtime-workdir")
>          if size:
> diff --git a/src/sbin/lorax b/src/sbin/lorax
> index 1754e95..f3aa9c3 100755
> --- a/src/sbin/lorax
> +++ b/src/sbin/lorax
> @@ -109,6 +109,8 @@ def main(args):
>              action="store_false", default=True, dest="domacboot")
>      optional.add_option("--logfile", default="./lorax.log",
>              help="Path to logfile")
> +    optional.add_option("--tmp", default="/var/tmp",
> +            help="Top level temporary directory" )
>  
>      # add the option groups to the parser
>      parser.add_option_group(required)
> @@ -142,6 +144,8 @@ def main(args):
>  
>      setup_logging(opts)
>  
> +    tempfile.tempdir = opts.tmp
> +
>      # create the temporary directory for lorax
>      tempdir = tempfile.mkdtemp(prefix="lorax.", dir=tempfile.gettempdir())
>  
> @@ -169,7 +173,7 @@ def main(args):
>  
> 
>  def get_yum_base_object(installroot, repositories, mirrorlists=[],
> -                        tempdir="/tmp", proxy=None, excludepkgs=[]):
> +                        tempdir="/var/tmp", proxy=None, excludepkgs=[]):
>  
>      def sanitize_repo(repo):
>          if repo.startswith("/"):

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list