undefined: readAvailableDictionary

[so-jelly]

Using the readme sample code on go playground

../gopath155601560/pkg/mod/github.com/tjarratt/[email protected]/babble.go:22:12: undefined: readAvailableDictionary

[CamiloGarciaLaRotta]

🙂 I run into the same issue. Anytime babble is run from within a non Windows environment in which it doesn't have access to /usr/share/dict/words it will panic. For example while running it as a non admin user in a container.

I see for Windows @tjarratt copied over the complete list of words, so we could default to it for unix environments if the words file is not found

[CamiloGarciaLaRotta]

Ahh it appears the Windows dictionary is not yet used

babble/main.go

Line 19 in aa3777d

// TODO: break the random word functionality into windows && unix helpers

[tjarratt]

Hello @CamiloGarciaLaRotta thanks for looking into this a bit. I haven't actually touched this code in years but I think you're right. My goal at the time was to reduce the build size for unix, since including the dictionary for windows was 3.5 megabytes. I've tried too do a quick test right now to see how much of a difference this makes but haven't managed to get it to build :(

I'm not sure which unix systems don't normally have /usr/share/dict/words (I thought that was more widely available) but it would be nice if the dictionary was only compiled in when it can't otherwise be found at runtime.

It's also worth noting @CamiloGarciaLaRotta that the main.go file was included mostly as a proof-of-concept. I think I'm likely going to delete it or move it somewhere else since it doesn't even build for me at the moment.

[jonashdown]

docker images of unix/linux might not have /usr/share/dict/words present. It might be worth documenting that this is a non-go dependency. on Ubuntu it can be solved by running sudo apt-get install -y wbritish