[PATCH] go: use a different goconfig package

[Justus Winter]

Quoting David Bremner (2013-04-24 00:51:11)
> Justus Winter <4winter at informatik.uni-hamburg.de> writes:
> 
> > The notmuch-addrlookup utility uses a third party library to read the
> > notmuch configuration file. The previously used implementation at
> > "github.com/kless/goconfig" vanished, so this patch switches to the
> > implementation at "github.com/msbranco/goconfig". As the
> > implementations differ at the API level, the code is updated
> > accordingly.
> 
> It's great that you fixed this, but the whole idea of something failing
> to build depending on the status of a website makes me pretty
> uncomfortable.
> 
> d

Ah, I pondered whether I should write more about the lib change. Maybe
I should have.

So there are two libs, kless and msbranco for short. kless is a fork
of msbranco, and it seems like the author abandoned his fork. So we
are now using the original lib, msbranco. msbranco seems to be in
"maintenance" mode, but has recently received updates to fix some
minor issues wrt. go 1.1. There are a lot of forks of msbranco on
github, so even if msbranco vanishes as well, we can pick the next
most promising fork.

So yeah, I feel the same way, it's unfortunately that I had to patch
this, but we're better of now (it builds again and we moved to a more
"vanilla" goconfig API (assuming the other forks do not change the api
as much as kless did)).

And I think you are being a bit harsh with the "status of the website"
comment. In case you are not familiar with go,
"github.com/kless/goconfig" is a package path, not a website. It just
happens to also be the host+resource part of a URL that points to the
github project page. So what happened here is that the VCS repository
specified by the package path vanished, not some web site (well, that
vanished too, but that didn't cause the build failure). If you happen
to have golang installed, type "go help remote" for more information.

Cheers,
Justus