search

ndmitchell / hoogle

Haskell API search engine
http://hoogle.haskell.org/
Other
753 stars 137 forks source link

Some very minor fixes I found necessary on Mac OS X #7

Closed jwiegley closed 10 years ago

ndmitchell commented 11 years ago

The documentation patch looks good, the other one looks suspicious - once we clear that up, I'll pull. Thanks for the patches!

jwiegley commented 11 years ago

Hi Neil, I ran into that pattern match problem when merging together all of the 3688 databases on my system. Without the extra match, I'm simply unable to do this; but with it, I can at least get it to fudge past the database which causes the error.

ndmitchell commented 11 years ago

That suggests something more serious is wrong - something has corrupted the database. Are you able to determine which database needs that error to load, or which set of databases merged together raise that problem? Patching over it may well lead to the resulting database dying mysteriously when doing certain types of search or printing the answers to searches.

jwiegley commented 11 years ago

Still need to research this, although it is at lower priority just now. Also, a command-line flag for specifying additional directories to search for documentation in would be better than the forced hack I have in 09b3673.

ndmitchell commented 11 years ago

I'm still very concerned about putting in the missing pattern match, I can't see how it will do anything other than cause a blowup sometime later, with a far more opaque error message.

For the documentation directory, I really want a lot of defaults (and your patch there is certainly a good start), and then also a command line flag.