aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2016-09-01 09:57:48 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2016-09-01 09:57:48 +0200
commite7f4231e935edae79ba2cd4d73a6e49591188bcc (patch)
treefa2dc172f5398c815591e9a133875b4f6989df1c
parent89a2881d1128fd799479aca1883687553a359395 (diff)
Add support for specifying directory in config.dist.archives
-rw-r--r--build2/dist/init.cxx9
-rw-r--r--build2/dist/operation.cxx38
2 files changed, 37 insertions, 10 deletions
diff --git a/build2/dist/init.cxx b/build2/dist/init.cxx
index 2bf26e0..428ca5c 100644
--- a/build2/dist/init.cxx
+++ b/build2/dist/init.cxx
@@ -41,13 +41,18 @@ namespace build2
// Note: some overridable, some not.
//
+ // config.dist.archives is a list of archive extensions that can be
+ // optionally prefixed with a directory. If it is relative, then it is
+ // prefixed with config.dist.root. Otherwise, the archive is written
+ // to the absolute location.
+ //
v.insert<abs_dir_path> ("config.dist.root", true);
- v.insert<strings> ("config.dist.archives", true);
+ v.insert<paths> ("config.dist.archives", true);
v.insert<path> ("config.dist.cmd", true);
v.insert<dir_path> ("dist.root");
v.insert<path> ("dist.cmd");
- v.insert<strings> ("dist.archives");
+ v.insert<paths> ("dist.archives");
v.insert<bool> ("dist", variable_visibility::target); // Flag.
diff --git a/build2/dist/operation.cxx b/build2/dist/operation.cxx
index 12bcc61..4628350 100644
--- a/build2/dist/operation.cxx
+++ b/build2/dist/operation.cxx
@@ -45,10 +45,13 @@ namespace build2
static void
install (const path& cmd, file&, const dir_path&);
- // cd <root> && tar|zip ... <pkg>.<ext> <pkg>
+ // cd <root> && tar|zip ... <dir>/<pkg>.<ext> <pkg>
//
static void
- archive (const dir_path& root, const string& pkg, const string& ext);
+ archive (const dir_path& root,
+ const string& pkg,
+ const dir_path& dir,
+ const string& ext);
static void
dist_execute (action, const action_targets& ts, bool)
@@ -289,8 +292,22 @@ namespace build2
//
if (auto l = rs->vars["dist.archives"])
{
- for (const string& e: cast<strings> (l))
- archive (dist_root, dist_package, e);
+ for (const path& p: cast<paths> (l))
+ {
+ dir_path d (p.relative () ? dist_root : dir_path ());
+ d /= p.directory ();
+
+ const string& s (p.string ());
+ size_t i (path::traits::find_leaf (s));
+
+ if (i == string::npos)
+ fail << "invalid archive '" << s << "' in dist.archives";
+
+ if (s[i] == '.') // Skip dot if specified.
+ ++i;
+
+ archive (dist_root, dist_package, d, string (s, i));
+ }
}
}
@@ -386,13 +403,16 @@ namespace build2
}
static void
- archive (const dir_path& root, const string& pkg, const string& e)
+ archive (const dir_path& root,
+ const string& pkg,
+ const dir_path& dir,
+ const string& e)
{
string a (pkg + '.' + e);
// Delete old archive for good measure.
//
- path ap (root / path (a));
+ path ap (dir / path (a));
if (file_exists (ap, false))
rmfile (ap);
@@ -401,9 +421,11 @@ namespace build2
//
cstrings args;
if (e == "zip")
- args = {"zip", "-rq", a.c_str (), pkg.c_str (), nullptr};
+ args = {"zip", "-rq",
+ ap.string ().c_str (), pkg.c_str (), nullptr};
else
- args = {"tar", "-a", "-cf", a.c_str (), pkg.c_str (), nullptr};
+ args = {"tar", "-a", "-cf",
+ ap.string ().c_str (), pkg.c_str (), nullptr};
if (verb >= 2)
print_process (args);