diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2015-09-25 10:34:15 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2015-09-25 14:46:57 +0200 |
commit | a2ccb6406bb1b016aaf31a47c659d06bce0cfc6a (patch) | |
tree | dd60c18e41983f87902b4a81e8f761770b541281 /bpkg/manifest.cxx | |
parent | e83df84b8d0b73896421b52bead0eaa625d4586d (diff) |
Add role member to the repository_manifest class
Diffstat (limited to 'bpkg/manifest.cxx')
-rw-r--r-- | bpkg/manifest.cxx | 49 |
1 files changed, 49 insertions, 0 deletions
diff --git a/bpkg/manifest.cxx b/bpkg/manifest.cxx index 1ab9c72..bc1531d 100644 --- a/bpkg/manifest.cxx +++ b/bpkg/manifest.cxx @@ -36,6 +36,9 @@ namespace bpkg // Utility functions // static const strings priority_names ({"low", "medium", "high", "security"}); + static const strings repository_role_names ( + {"base", "prerequisite", "complement"}); + static const string spaces (" \t"); inline static bool @@ -1126,6 +1129,20 @@ namespace bpkg bad_value (e.what ()); } } + else if (n == "role") + { + if (role) + bad_name ("role redefinition"); + + auto b (repository_role_names.cbegin ()); + auto e (repository_role_names.cend ()); + auto i (find (b, e, v)); + + if (i == e) + bad_value ("unrecognized role"); + + role = static_cast<repository_role> (i - b); + } else bad_name ("unknown name '" + n + "' in repository manifest"); } @@ -1133,6 +1150,10 @@ namespace bpkg // Verify all non-optional values were specified. // // - location can be omitted + // - role can be omitted + // + if (role && location.empty () != (*role == repository_role::base)) + bad_value ("invalid role"); } void repository_manifest:: @@ -1143,9 +1164,34 @@ namespace bpkg if (!location.empty ()) s.next ("location", location.string ()); + if (role) + { + if (location.empty () != (*role == repository_role::base)) + throw serialization (s.name (), "invalid role"); + + auto r (static_cast<size_t> (*role)); + assert (r < repository_role_names.size ()); + s.next ("role", repository_role_names[r]); + } + s.next ("", ""); // End of manifest. } + repository_role repository_manifest:: + effective_role () const + { + if (role) + { + if (location.empty () != (*role == repository_role::base)) + throw logic_error ("invalid role"); + + return *role; + } + else + return location.empty () ? + repository_role::base : repository_role::prerequisite; + } + // repository_manifests // repository_manifests:: @@ -1175,6 +1221,9 @@ namespace bpkg if (empty () || !back ().location.empty ()) throw serialization (s.name (), "base repository manifest expected"); + // @@ Should we check that there is location in all except the last + // entry? + // for (const repository_manifest& r: *this) r.serialize (s); |