Use staging rather than master as the default base for `master` commits

The 'master' versions are meant to be applied to staging, and aren't
generally expected to be used directly by downstreams.  If there's a
difference between master and staging, having it apply to staging is
more useful.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

xsa: Add set-patches command to set patches for a particular xsa + version

...rather than editing the metadata file manually.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

xsatool: Update README.md with more complete usage information

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

xsa: Return success when backport succeeds

Forgot to change the "unimplemented" message and return value after
implementing the 'backport' command.

While we're here, make backport a bit more chatty about what it's
doing.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Bring patch name for master in line with standard practice

Standard practice is to have the patch that applies to master be
simply "xsaNNN.patch" (or xsaNNN/*.patch in case of a series).

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Fix situation where there's no recipe for a prerequisite

It's not uncommon to "open up" several XSAs in a row, and to begin
work on subsequent ones before the backports to previous ones are
done.

If when making the baseline for a given Xen version for an XSA, one of
the prerequisete XSAs doesn't have a recipe for that Xen version, just
skip making the baseline for that version.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement reset-base

Implement reset-base, to reset the 'base' version to the upstream
stable trees without rebasing.

While we're here, document 'repo update' and add some comments.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Include fakeups for all 206 backports

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

repo init: Enable git rerere on the repos by default

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

README: Include a brief description of how to build.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

README: Mention mirrordir and gitproxy

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

meta: Get rid of VersionMeta.Latest

NB that the action of resetting outstanding XSAs will be manual as of
this commit.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

repo init: Add --mirrordir and --gitproxy options

...to set the appropriate proxy location.  Also add options to systemtest
to allow testing with those options.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

ToolConfig: Add MirrorDir and ProxyURL options

If available, use in Repo.Build when building Xen to create .config.

To do this:

- Make a XenBuildConfig type that contains methods for adding configs
for URLs, as well as both the repo path and revision (for qemu-based patches)

- Write unit tests for these methods

Use these in Repo.Build.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Delete all existing branches when not making from a clean state

Except `master`, of course.

Make sure to open all repos before cleaning branches.  (xsa.git will typically not be open
after `repo init` if it wasn't cloned.)

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Repo: Refuse to execute without a command except for 'git clone'

Running with a non-open repo could destroy the wrong data.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Implement --build

Always run 'test-apply'; run 'test-build' afterwards if --build is specified.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Implement --clean option to start from scratch

Pass all arguments to systemtest.  Use the `flag` package to read `-clean`.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Always re-run 'repo init'

Always run `repo init`.  If `tmp` already exists, delete `tmp/.xsatool`.

Set a limit that ToolConfig.Load won't go up higher than, so that when
starting the test from 'scratch' we don't get "real" configurations
from any higher-level directoiries by accident.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

systemtest: Abort the failed rebase

So that subsequent git commands (from another test) work

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Implement Repo.RebaseAbort

Wish there were a better interface, but here we are.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Miscelaneous fixes

Add missing arguments, remove a duplicated stanza

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Run go fmt

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Include README.md

This is based on the design document posted before, but is updated
with more workflow description, as well as future development plans.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Delete stackgit data when deleting or resetting branches

If a developer has modified a branch to be a stackgit branch as part
of development, simply deleting or resetting that branch with git
commands will "work", but will leave leftover state lying around which
may cause problems later.

Before deleting or resetting a branch, check to see if the branch is a
stackgit branch.  If so, use `stg branch --delete --force` to get rid of all
the stackgit data.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement backport

Implement Repo.Rebase() (with full head / base / onto)

Use this to implement Recipe.Backport()

Finally, implement xsatool [nnn] backport:
 - Look for the latest xenversion that doens't have a "patch" branch
 - Rebase next version from that onto it with 'git rebase'.

Always only do one, so that the caller has a chance to test the build
and look it over before continuing the backport process.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

XSAMeta: Keep SupportedVersions sorted in reverse order

There are many situations where we want to test things on master first
and then go backwards through supported versions (e.g., building,
looking for backports, &c).  Keep the SupportedVersions in this order.

To do this:
 - Refactor IsGreaterEqualThen to be usable by both it and IsGreaterThan
 - Make a XenVersionSlice type which implements sort.Interface, using the above
 - Re-sort XSAMeta.SupportedVersions on insert and load.

Also:

 - Have XSAMeta.NewRecipe() add the new SupportedVersions, rather
than having the caller add it manuall.

 - Keep XenVersion.PrevVersion() around, even though we're not using
 it for now, in case it becomes useful in the future.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Handle prerequisites

When making baselines, iterate through prerequisite list and apply all
patches.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Build: Use mirror paths for qemu unless there are patches

Cloning from the working tree is running into build problems saying it
can't find the right changeset, even though the changeset is present
in the working tree.  For now, use the mirror directory.

This needs to be sorted out properly somehow.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Use new branching scheme

New branching scheme:

 - $v-stable based on the recorded stable branch at the time the xsa
   was created

 - $v-baseline; the above, but with 'prerequisites' (outstanding XSAs
   not published yet)

 - $v: Branch with actual patches, only present when patches exist (or
   are being created)

To do this:

 - Separate Recipe.Apply() into ApplyBaseline (which only does -stable
   and -baseline) and ApplyPatches (which makes the main branch)

 - Get rid of IsApplied(), which doesn't do what it says on the tin

 - Implement HasPatchBranch, which detect whether the "patch"
   branch(es) exist for a given recipe

Then:

 - During Init, call only ApplyBaseline for all branches; only call
   ApplyPatches for master.

 - When calling sync-patches with 'all', have VersionsFromArgs filter
   out patches based on HasPatchBranch.  (sync-branches and test will still
   test all supported branches for 'all'.)

 - Add a test for the failure path of sync-branches with a
   non-existent branch

While we're here:

 - Use fmt.Print instead of fmt.Printf when there are no format tokens

 - Add missing '\n' to many status output messages

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Use StableBranch to create VVV-baseline branch

Get rid of XenVersionFull from the recipe and replace it with
XenVersion for convenience.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add stable branch reference to TreeRecipe

Add XenRepo.MakeStableBranch() and XenRepo.GetRefVersion(), which will
make a "stable branch" and get the ref for the appropriate version.

Check out the 'stable' branch for each version when creating a new
repo, and take a reference.

Also update the testing infrastructure to set up the tree such that
the stable branches reflect the appropriate branches at the time.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement 'repo update'

Implement Repo.Branches() with a pattern.  Add code to handle empty
output and 'stray' values.

Implement MainRepoUpdate, which will call 'git fetch' and 'git merge'
on master and stable-* branches.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

testing: Test master build

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

test: Check for patches in recipe rather than relying on the existence of a branch

Implement Recipe.HasPatches(), which returns true if any of the trees
has patches.

Call it in test.go:test() before applying and/or building.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Change to in-binary test

`go test` apparently has a default 10-minute timeout.  This isn't
enough for all the tests in the "system test", and there's no way to
change the default except to manually add a --timeout parameter to "go
test".

Instead pull the test out into a special xsatool command.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

testing: Implement 206 'implement and sync' tests

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: Accept an empty set of versions as 'all'

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Apply: Always check out local master rather than origin/master

To make testing work

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Testing: Test xsa 206 init

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: Implement 'init' command

New method of creating a new xsa:  "xsatool NNN init [trees]"

This will:
- For a given set of trees
- And all the currently-supported versions
- Create an XSA that contains all the currently-outstanding XSAs for that tree and set of versions
- And create branches for all of those versions in the correct tree
- And add it to the list of currently-outstanding XSAs

At that point, the user can go into the appropriate tree and create patches.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Recipe: Allow MakePatches to detect and tolerate "empty" trees

In order to move to a world where all supported branches are created
automatically, with prerequisites applied, as "empty" when the XSA is
created, allow Recipe.MakePatches() to detect and tolerate "empty"
versions:

- Have MakePatches() return the number of trees which contained
  patches

- Instead of verifying the reference, get the hashes for base and
  branch and compare them; simply return 0 if they are the same

- Don't bother copying patches if none are to be created.

Modify xsa.go:sync() to handle this case as well.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Implement ParseRef to change a ref into a changeset

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: add loadGlobal option

To indicate whether a command should try to load the global metadata.

Signed-of-by: George Dunlap <george.dunlap@citrix.com>

global init: Always create a 'master' version as well

This will help is keep track of un-released XSAs.

TODO: Still need a good way to reset this when we've published the
XSAs.

While we're here, fix a typo.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

system_test: Check for specific versions, not just presence

Check for the specific versions, not just the presence of the
branches.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

VersionMeta: Make XSA list per-tree

Not all XSAs apply to each tree; therefore to which XSAs need to be
applied as prerequisites for a given XSA, we need to store them
per-tree.

Change this to a map[Tree][]int.  Create SecurityMeta.InitVersion() to
initialize an empty version properly.

Also:
- explicitly list in an XSA which trees are supported (unused)
- Add SecurityMeta.AddXSA(), which will update VersionMeta based on xsa.Trees

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

XSAMeta: Store recipes in a map[XenVersion] rather than an array

This makes accessing it easier, and also makes the json easier for
humans to parse.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Recipe: Add ForEachTree method to do something for each tree

Have MakePatches, IsApplied, and Apply methods use it rather than
ForEachXenRepo().

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Use full path for AM operations

...so that Repo.Am() works regardless of your cwd.

Factor "normalization" into normalizePath.

Implement globToFullPath, which will take a glob and return a slice of
normalized filenames.

Use globToFullPath() for Repo.Am() and Repo.AmClean().

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Testing: Test "xsatool global update" initialization

Re-constructing the history of XSA 206, go back to version 4.4.

Check to make sure that after the initialization we have exactly the
versions we expect.

Also sketch out a plan for our 'system testing'.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Testing: Add a testing "input indirection" layer

If called from main(), print the string and query the user.

Otherwise, look up the response in the index.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Testing: Add a wrapper for XsaMain so that calling it is less ugly

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

global: Refer to global SecurityMeta structure, not a local one

In part so that the testing harness can see what's going on afterwards.

While we're here add a newline to the "manually add XSAs" message.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

XenRepo: Don't automatically fetch on open to make testing more predictable

This may need to be revisited at some point.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

system_test: Add versions and lack of metadata to 'state'

Delete all versions >= 4.9, and any tags after 4.8.0 or 4.7.2.

Also delete xsa.git/*.meta

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Add Repo.DeleteTag

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Testing: Introduce 'system' test

To begin with, just clone repos and begin to set them up in a specific
state, so that we can test operations.

To make testing iteration faster, set FullClone == false.  After test
development is done, this should be set to "true" to make the cloning
continues to work.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: Call OpenRepos when loading config

Rather than having to open them each place they're used.

Also do some minor reorganization of variables, just code motion.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: Open xsa repo in OpenRepos

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Add Repo.ResetBranch

To check out a particular branch.  This will be used in testing to set
up a particular view of the tree.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Refactor main to be more test-friendly

Have a special function, XsaMain(), which does the work and is passed
parameters and a return value.  Have a wrapper which interacts with
the operating system under normal conditions.  This allows the testing
infrastructure to simulate being called as closely as possible.

Get rid of global references to 'xsa' entirely; instead pass
references to a local copy on the stack.

Collect all remaining global state into a struct called 'G', and reset
its state every time XsaMain() is called.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

XenRepo: Quieten tagsToVersions

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Replace Prerequisite glob with xsa list

Punt on actually implementing applying the XSA until later.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Run go fmt

No functional change.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add global config

Modify global metadata to be a map (for easier readability).

Modify per-version metadata to include most recent full version and
XSAs since that point.

Add 'global update' command which will create a metadata file if not
there, and update it if there.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Fix global.Load() to return IsNotExist() error

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement unit tests for various parsing / comparsions

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

xen: Implement XenRepo.GetVersions()

Have GetVersions() fetch the tags from the repo, parse the releases,
and return a list of versions present in the tree.

Also implement XenVersion.Check() and XenVersion.IsGreaterEqualThan().

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Add Tags() method to get tags

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Move XenVersion and XenVersionFull into xen.go

Code-motion only (apart from dependency changes)

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Extend XenVersion parsing: fv.Check() and fv.Point()

Add an internal function to parse XenVersionFull into XenVersion and
Point.  Have .Check(), .Point(), and .XenVersion() be wrappers of
this.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add NNN new-recipe command

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

repo init: Actually save config file

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

sync-patches: Actually update and write the recipe file

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Use a string for Tree rather than an int; make maps instead of arrays

This is primarily to get better config files, where the indexes are
strings rather than integers or lists.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

A number of miscellaneous fixes

* In main(), actually include the xsa number in the printf when the load fails

* Load and save the actual xsa file (xsa.git/xsaNNN.meta) rather than
  the global meta file.  "Normalize" (set the xsa number for recipes) on load.

* Use the correct path for QEMU_TRADITIONAL_URL

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Fix config load

Have ToolConfig.Load return an error which returns true for
os.IsNotExist().

When the config isn't present, initialize it.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Introduce Tree type to avoid hard-coding tree names

Replace repos.{xen,qemuu,qemut}, config.*Path, and Recipes.{Xen,Qemu*}
with arrays indexed by Tree.  Replace Get*Path with GetPath(t).
Replace XRType with Tree.  Introduce ForEachCodeTree, ForEachTree, and
ForEachXenRepo helper functions to run a specific function for each
tree / repo; rewrite lots of code to use those functions instead of
doing code duplication.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add XSA.Load() and Save(), and use it rather than hardcoding

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Do a bit more towards global security metadata

Still need to sort out the Time serialization issue; leave a comment
to this effect.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Load xsatool config from a file

Recursively look for ".xsatool" in all directories going up until
there are no more.

Add 'repo info' command to read config and print info

Also:

 - Use config.GetXXXPath() to get normalized path rather than
   accessing config.XXXPath directly

 - Change config.GetXXXPath() to return NULL if either config.rootDir
   or config.XXXPath are undefined

 - Separate out XSA fake loading into a separate flag

 - use xenproject.org rather than xen.org for default configuration

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

hack: Include hack recipes for both 206 and 212

NB that 212 requires you to make an e-mail-able version.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement test[-apply|-build]

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Finish sync-patch, add sync-branch as well

Have MakePatches include a flag deciding whether to sync or not.

If set, after creating all appropriate patches in /tmp, make the
appropriate directory/ies in xsa.git and copy the files there (or just
copy a single patch if the file is a single patch).

Also implement sync-branch which will delete and re-create xsa/NNN/VVV
from xsa.git according to the recipe.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

main: Add partial implementation of sync-patches command

Partial implementation of sync-patches, which will look for trees
which contain xsa/NNN/VVV and xsa/NNN/VVV-baseline and run
`git format-patch` to generate patches.

To do this:
 - Implement Repo.FormatPatch()
 - Implement XSAMeta.GetRecipe(version) to get a recipe for a specific version
 - Modify IsApplied() to look for both $prefix/NNN/VVV and $prefix/NNN/VVV-baseline
 - Implement Recipe.MakePatches()
 - Modify IsApplied() to return only an error; but make return a
   distinguishable type to say what branch wasn't applied

A full implementation would also copy the patches into xsa.git either
as a single patch or a directory full of patches, as appropriate.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

recipe: Have branchName return both branch and baseline branch names

...to make sure we have consistent naming.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Main: Fake up config and xsa info

"Fake up" config and XSA info in Main(), so that we can start writing
functionality as though the config loading already existed, rather
than have to store it all in test.go.

This should make it easier to extend functionality as needed in a way
which will actually be useful once the config loading stuff is ready.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Recipe: Break down per-tree recipe into prerequisites and patches

Create a new struct, TreeRecipe, to be the recipe for a particular
tree (Xen, QemuU or QemuT).  Have this contain separate prereq and
patches sections.

When building, create a branch $branch-baseline to apply the Prereq
patches to, and then base $branch based on the baseline.

The idea here is to make an easy way to programmatically sync changes
to the tree back to xsa.git.

Also unconditionally use local qemu trees, even if we don't have local
patches, because we know they'll be up to date with upstream changes
(since we pull at start-of-day).

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

test: Make separate backport and test frameworks

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Recipe: Make more flexible for both backporting and testing

Make the interface a bit more flexible, so it can be used both for
backporting (which will create branches named xsa/NNN/VVV), and for
testing (which will create branches named test/NNN/VVV).

Add branchName() helper to calculate branch name, rather than storing
a single tempBranch.

Add IsApplied() method, which will (at the moment) only check for the
existence of the target branches in the appropriate trees.

Also:

- Use AmClean() rather than Am() to make it easier to follow up a
  failure by cherry-picking

- Fix a bug in Build() which would build the wrong version

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Recipe: Don't clean up temporary branches automatically (for now)

When doing testing, we probably want to remove temporary branches; but
when doing backporting we most certainly do not.  This is something
which still needs to be figured out.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

git: Implement VerifyRef and AmClean

VerifyRef returns true if the specified ref exists.

AmClean() is like Am(), but will reset the tree to a useable state if
the Am fails.  This is useful for situations like backporting where
you typically want to cherry-pick on a failed `git am` to take
advantage of the three-way merge functionality.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Make XenVersion string-only

Hopefully this will make it easier to store in a human-readable
format.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Allow Recipes to have multiple globs, implement Recipe.Build()

Allow each recipe to have multiple "globs" associated with each tree.

Implement Recipe.Build(), which will construct the appropriate config
file to make the build use the appropriate branches, and then
configure and run make.

While we're here fix some mistaken status messages in Recipe.Cleanup(), and
fix XenRepo.CheckoutVersion when XenVersion == Master.

And update the "test" to XSA211.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Git: Misc functionality and fixes

Have Repo.Am() check to make sure that the glob matched at least one
file; if not, throw an error.

Do "git clean -ffdx" after checkout in MakeBranch and Checkout.

Add GetPath to return the full path of a repository.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Refactor arguments to make "repo" target

Rather than "repoinit" as a single argument, make the first argument
the target (either an xsa or the repo), and separate commands based on
that.

Also fix a bug that causes a range check if there aren't enough
arguments.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Implement basic repoinit

Clone xen.git, qemu-xen.git, qemu-xen-traditional.git, and xsa.git,
and set up a default .xsatool config file.  Disable pushes on the xen
and qemu branches.

Modify gitCmd() to not add the -C prefix if r.Path isn't set.

Implement GitClone and XenRepoClone, and Repo.DisablePush.

Move global repo s into a global structure called 'repo'; and also
include a plain Repo for xsa.

Implement ToolConfig type, with Load and Save.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add draft infrastructure for different commands

Signed-off-by: George Dunlap <george.dunlap@citrix.com>

Add support for all repos, and test different versions.

Also rename the structure to XSAMeta, in preparation for adding other
metadata.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>