Dub.fetch: Deprecate Dependency overload, only accept Version[range] ...

Because fetching a path-based or repository dependency doesn't
make much sense.

Merge pull request #2373 from Geod24/move-more-stuff ...

PackageManager: Move some `Location`-specific routines to `Location`

Signed-off-by: Jan Jurzitza <gh@webfreak.org>
Merged-on-behalf-of: Jan Jurzitza <gh@webfreak.org>

PackageManager: Move routine to write local pkg list to Location struct

PackageManager: Move routine to write override in Location struct

improve some output coloring ...

See #2347

change TAG_WIDTH const to a variable value ...

use it for making error-only output be more readable

Improve searchPackages naming+docs ...

See https://github.com/dlang/dub/commit/ca988572b58d34a4d9384caeabbdb2fd8d7d0cd5#r80490672

Trivial: Remove redundant cast

PackageManager: Move Location-specific code to Location type ...

Because having a nested function that looks up an index in a field
and goes through this indirection all the time doesn't seem like
the best design anymore.

Trivial: Document PackageManager.Location, add attributes

Deprecate Dependency.versionSpec ...

All places which assumed that Dependency contained a VersionRange have been culled,
so we can now start removing those properties.

Trivial: Move relevant documentation from Dependency.versionSpec to VersionRange

Overrides: Make it clear that only VersionRange can be overriden ...

The code was always assuming overrides were VersionRange,
as trying to do 'dub add-override' would call 'Dependency'
constructor with a string, which forwards to VersionRange.fromString.
However, when using dub as a library, this assumption was much less obvious,
notably because the fields are public.
With this change, we make sure library users do not misuse overrides.

Use std.sumtype for PackageOverride too ...

Like Dependency, we only were using one of those two fields,
but it was not readily obvious before, and we relied on one
of them being empty to read the other one.

Trivial: Pass a Repository to loadGitPackage

SDL: Use visit instead of multiple if when parsing Dependency

Remove `Dub.updatePackageSearchPath` ...

This function was essentially two functions: One when an override was set,
another one for the common case. However, the function is private,
and the number of call sites is limited, so it's better to inline
the call at the caller site. Once that was done, it became obvious
it only ever has an effect once, when the Dub instance is constructed,
as we don't modify this environment variable.
Thus, we can inline it in the constructor, and remove the dead branches.

Configy: Implement soundexer-based key-matching in strict mode ...

This scales much better, e.g. when using Configy with dub's 54 config keys.

Integrate Configy colored output with dub's

Remove a few usages of old version-spec approach from test-suite

improve man page generation

make command exit codes more consistent

dub search: Improve text, rename parameter ...

The 'query' name is inaccurate, as we only ever expect a package name.

Dub.fetch: Remove a redundant getPackage call ...

Since storeFetchedPackage already returns the Package.

Trivial: Deprecate a compatibility overload of Dub.remove

dub.fetch: Do not catch Exception when a local package can't be loaded ...

Given how simple the function being called is,
there is no reason for us to ignore this error.

Also add options when we combine buildsettings (#2363) ...

* Also add options when we combine buildsettings

* include all the buildsettings in add(bs)

Co-authored-by: WebFreak001 <gh@webfreak.org>

Revert "Default `dub init` to SDL" ...

This reverts commit 3d0d867.

Default `dub init` to SDL ...

Átila wants to throw out dub.json, so this is the first step to discouraging dub.json

Dependency: Expose `visit` to `match` on `SumType` ...

To do so, we need to expose `VersionRange`.
The function is a free function as it otherwise triggers
a deprecation for dual contexts due to the template parameters.