Doesn't build on weekly about goirc

[fluffle]

Urg.

I've not had time to keep up with the weekly changes, sorry. I was hoping Go 1 would be released sometime soon, so I could just upgrade to that all in one go ;-)

Are you using the master or the release branch? The former is definitely a little more complicated -- you'll need the logging and event libraries that I split out into github.com/fluffle/goevent and github./com/fluffle/golog respectively. Last time I really looked at this stuff, goinstall could follow these dependencies. What's changed with the new 'go' tool?

I might be able to find some time this weekend to sort it out; I'm hoping it'll just be a case of running gofix over the code and creating a new 'weekly' branch.

Sorry for the breakage.
--alex

[peterbourgon]

I might be able to find some time this weekend to sort it out; I'm hoping it'll just be a case of running gofix over the code and creating a new 'weekly' branch.

I wish :( The (from my POV) complex interdependencies between
event/logging/state/client/etc. means 'gofix' needs to be applied to a
lot of different things. And I wasn't able to figure out the
dependency graph to do it.

No big worries; I'm using a workaround for the time being (ie. another
IRC library :) And for the record I really recommend the "gb" tool
over go install or Makefiles or anything like that.

[fluffle]

I used gb to build things locally, but of course goinstall works slightly differently and is^Wwas the "standard" way to install third-party libraries for Go. I appreciate the dependencies aren't particularly simple, but it didn't make sense to bundle more generic components like the event registry and logging along with the client.

Let me see what I can do...

[fluffle]

Run into some problems with function inlining in 6g :-(

http://code.google.com/p/go/issues/detail?id=2877

And some problems with gomock -- reflect.DeepEqual appears to be barfing when comparing functions now.

panic: runtime error: comparing uncomparable type func(...interface {})

We'll see how things go. The "weekly" branch for goevent, golog, and goirc build with the current weekly, but I can't test the latter because of the reflect problems. I should have gone to bed hours ago. Nothing's ever simple ;-)

[fluffle]

Turns out functions are no longer comparable. Also, the changes to the time module exposed some embarassing fail in my rateLimit test. I believe this should compile with the current weekly, modulo the problem in the link in my previous post. I explain a workaround in that bug -- nothing like a bit of hand-editing .a files :-)

--alex