From c861c537649f30802f883cfd782395dc075a270f Mon Sep 17 00:00:00 2001 From: "jingui.ren" Date: Wed, 28 Feb 2018 17:20:03 +0800 Subject: [PATCH] fix bugs about linux path which includes abnormal character Change-Id: If45e69e345ecf8ca6847ce7e0033b801ed31533b --- gitbuildsys/cmd_build.py | 7 ++++--- gitbuildsys/cmd_export.py | 11 ++++++----- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/gitbuildsys/cmd_build.py b/gitbuildsys/cmd_build.py index fcd0ad8..9e2d2fa 100644 --- a/gitbuildsys/cmd_build.py +++ b/gitbuildsys/cmd_build.py @@ -198,9 +198,10 @@ def prepare_repos_and_build_conf(args, arch, profile): if not distconf.endswith('.conf') or '-' in os.path.basename(distconf): raise GbsError("build config file must end with .conf, and can't " "contain '-'") - dist = os.path.basename(distconf)[:-len('.conf')] + dist = "'"+os.path.basename(distconf)[:-len('.conf')]+"'" cmd_opts += ['--dist=%s' % dist] - cmd_opts += ['--configdir=%s' % os.path.dirname(distconf)] + path = "'"+os.path.dirname(distconf)+"'" + cmd_opts += ['--configdir=%s' % path] return cmd_opts @@ -716,7 +717,7 @@ def main(args): else: cmd += ['--noinit'] - cmd += ['--path=%s' % workdir] + cmd += ['--path=%s' % "'"+str(workdir)+"'"] if args.ccache: cmd += ['--ccache'] diff --git a/gitbuildsys/cmd_export.py b/gitbuildsys/cmd_export.py index 70071fb..0141f3e 100644 --- a/gitbuildsys/cmd_export.py +++ b/gitbuildsys/cmd_export.py @@ -254,13 +254,13 @@ def main(args): if args.commit and args.include_all: raise Usage("--commit can't be specified together with --include-all") - + log.debug("export begin") workdir = args.gitdir try: repo = RpmGitRepository(workdir) except GitRepositoryError, err: raise GbsError(str(err)) - + log.debug("read repo path") utils.read_localconf(repo.path) utils.git_status_checker(repo, args) workdir = repo.path @@ -276,10 +276,11 @@ def main(args): commit = 'HEAD' orphan_packaging = configmgr.get('packaging_branch', 'orphan-devel') spec_commit_id = orphan_packaging if orphan_packaging else commit + log.debug("check packaging_dir") packaging_dir = get_packaging_dir(args) main_spec, rest_specs = utils.guess_spec(workdir, packaging_dir, args.spec, spec_commit_id) - + log.debug("check outdir") if args.outdir: outdir = args.outdir else: @@ -290,7 +291,7 @@ def main(args): raise GbsError('no write permission to outdir: %s' % outdir) else: mkdir_p(outdir) - + log.debug("check tmpdir") tmpdir = configmgr.get('tmpdir', 'general') tempd = utils.Temp(prefix=os.path.join(tmpdir, '.gbs_export_'), directory=True) @@ -336,7 +337,7 @@ def main(args): else: outdir = "%s/%s-%s-%s" % (outdir, spec.name, spec.upstreamversion, spec.release) - + log.debug("check outdir") if os.path.exists(outdir): if not os.access(outdir, os.W_OK|os.X_OK): raise GbsError('no permission to update outdir: %s' % outdir) -- 2.7.4