Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small changes made while building and running WikiTrust. #1

Merged
merged 3 commits into from Jul 6, 2014

Conversation

makoshark
Copy link
Contributor

I am in the process of trying to build and use WikiTrust because I am evaluating it for use in some of my own research. I have made several changes to the documentation and build scripts — and some small changes in the code — in order to get the software to build and run on my system. I have also updated the documentation to make it more complete and accurate.

I have only very minimal OCaml skills but I think these changes are either not to the .ml files or are simple enough that I think they should be fine. And the software builds and work now on my system.

Thanks for building WikiTrust and thank so much for giving it to the world!

- Updated list of Debian/Ubuntu packages so that it is complete and
  includes packages that were created since 2009.
- Remove/replaced dead link to Jane Street ocaml packages.
- Fixed tab/space whitespace inconsistency that led to strange output.
On my system, the software failed to build because the camlzip package
could not be found, despite the fact that I had installed it. When I
updated the list of packages to "zip" instead of "camlzip" in the two
Makefiles, these errors were removed.
These changes fix a bug in which WikiTrust failed to build on my system.
One errors included that dbsocket was missing from every place that a
MySQL connection was opened. I was able to build the software by adding
variable, set to None, to every place this was called.

My understanding is that sockets are optional in new version of MySQL
but can be used instead of the standard port based method.

I have not yet tested this patch with a database and I am currently
using WikiTrust to parse XML dumps without a MySQL database present.
@makoshark
Copy link
Contributor Author

Are you waiting for something from me?

thumper17 added a commit that referenced this pull request Jul 6, 2014
Small changes made while building and running WikiTrust.
@thumper17 thumper17 merged commit 9dd056e into collaborativetrust:master Jul 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants