{"action":"created","issue":{"url":"https://api.github.com/repos/querycert/qcert/issues/26","repository_url":"https://api.github.com/repos/querycert/qcert","labels_url":"https://api.github.com/repos/querycert/qcert/issues/26/labels{/name}","comments_url":"https://api.github.com/repos/querycert/qcert/issues/26/comments","events_url":"https://api.github.com/repos/querycert/qcert/issues/26/events","html_url":"https://github.com/querycert/qcert/issues/26","id":196222068,"number":26,"title":"Document new prerequisites and integrate non-Make steps into Makefiles","user":{"login":"joshuaauerbachwatson","id":19649275,"avatar_url":"https://avatars.githubusercontent.com/u/19649275?v=3","gravatar_id":"","url":"https://api.github.com/users/joshuaauerbachwatson","html_url":"https://github.com/joshuaauerbachwatson","followers_url":"https://api.github.com/users/joshuaauerbachwatson/followers","following_url":"https://api.github.com/users/joshuaauerbachwatson/following{/other_user}","gists_url":"https://api.github.com/users/joshuaauerbachwatson/gists{/gist_id}","starred_url":"https://api.github.com/users/joshuaauerbachwatson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshuaauerbachwatson/subscriptions","organizations_url":"https://api.github.com/users/joshuaauerbachwatson/orgs","repos_url":"https://api.github.com/users/joshuaauerbachwatson/repos","events_url":"https://api.github.com/users/joshuaauerbachwatson/events{/privacy}","received_events_url":"https://api.github.com/users/joshuaauerbachwatson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"joshuaauerbachwatson","id":19649275,"avatar_url":"https://avatars.githubusercontent.com/u/19649275?v=3","gravatar_id":"","url":"https://api.github.com/users/joshuaauerbachwatson","html_url":"https://github.com/joshuaauerbachwatson","followers_url":"https://api.github.com/users/joshuaauerbachwatson/followers","following_url":"https://api.github.com/users/joshuaauerbachwatson/following{/other_user}","gists_url":"https://api.github.com/users/joshuaauerbachwatson/gists{/gist_id}","starred_url":"https://api.github.com/users/joshuaauerbachwatson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshuaauerbachwatson/subscriptions","organizations_url":"https://api.github.com/users/joshuaauerbachwatson/orgs","repos_url":"https://api.github.com/users/joshuaauerbachwatson/repos","events_url":"https://api.github.com/users/joshuaauerbachwatson/events{/privacy}","received_events_url":"https://api.github.com/users/joshuaauerbachwatson/received_events","type":"User","site_admin":false},"assignees":[{"login":"joshuaauerbachwatson","id":19649275,"avatar_url":"https://avatars.githubusercontent.com/u/19649275?v=3","gravatar_id":"","url":"https://api.github.com/users/joshuaauerbachwatson","html_url":"https://github.com/joshuaauerbachwatson","followers_url":"https://api.github.com/users/joshuaauerbachwatson/followers","following_url":"https://api.github.com/users/joshuaauerbachwatson/following{/other_user}","gists_url":"https://api.github.com/users/joshuaauerbachwatson/gists{/gist_id}","starred_url":"https://api.github.com/users/joshuaauerbachwatson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshuaauerbachwatson/subscriptions","organizations_url":"https://api.github.com/users/joshuaauerbachwatson/orgs","repos_url":"https://api.github.com/users/joshuaauerbachwatson/repos","events_url":"https://api.github.com/users/joshuaauerbachwatson/events{/privacy}","received_events_url":"https://api.github.com/users/joshuaauerbachwatson/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-12-17T13:27:45Z","updated_at":"2016-12-20T11:05:20Z","closed_at":null,"body":"Now that (syntactic, non-sexp) SQL is being offered as a first-class source language, the prerequisites and build are more complicated and not all of that is seamless yet.\r\n\r\nBecause of a needed change to the ocaml build, ocamlfind is now a pre-requisite.  This might or might not matter in practice, because my impression is that the normal process of installing the existing pre-reqs via opam will get you a copy of ocamlfind.\r\n\r\nA Java pre-req is currently documented as optional unless you are using the Java runtime or the query runner.   In fact, it is required if you are using SQL as a source language.  Furthermore:\r\n\r\nThe 'make' in sqlParser will fail unless an ant script is run first.  This is not explicit in the Makefile.  And it means that ant is now a prerequisite.\r\n\r\nThe qcert binary will not work for -source sql unless a second ant script is run to package up Java code and move it to 'bin'.\r\n\r\nContinuing to document the Java pre-req as optional is becoming strained because the various Makefiles really need to invoke the Java compiler and invoke ant if the result is not to be inconvenient and puzzling.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/querycert/qcert/issues/comments/268216239","html_url":"https://github.com/querycert/qcert/issues/26#issuecomment-268216239","issue_url":"https://api.github.com/repos/querycert/qcert/issues/26","id":268216239,"user":{"login":"joshuaauerbachwatson","id":19649275,"avatar_url":"https://avatars.githubusercontent.com/u/19649275?v=3","gravatar_id":"","url":"https://api.github.com/users/joshuaauerbachwatson","html_url":"https://github.com/joshuaauerbachwatson","followers_url":"https://api.github.com/users/joshuaauerbachwatson/followers","following_url":"https://api.github.com/users/joshuaauerbachwatson/following{/other_user}","gists_url":"https://api.github.com/users/joshuaauerbachwatson/gists{/gist_id}","starred_url":"https://api.github.com/users/joshuaauerbachwatson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshuaauerbachwatson/subscriptions","organizations_url":"https://api.github.com/users/joshuaauerbachwatson/orgs","repos_url":"https://api.github.com/users/joshuaauerbachwatson/repos","events_url":"https://api.github.com/users/joshuaauerbachwatson/events{/privacy}","received_events_url":"https://api.github.com/users/joshuaauerbachwatson/received_events","type":"User","site_admin":false},"created_at":"2016-12-20T11:05:20Z","updated_at":"2016-12-20T11:05:20Z","body":"In a discussion with @jeromesimeon we identified two courses of action with pros and cons.\r\n\r\n1.   Make Java (and ant) a hard pre-requisite and unify the build so that the sql parser is built along with qcert.\r\n\r\n- Pro: once the user satisfies the pre-reqs (once), the build is simpler and everything works.\r\n- Con: it seems a shame to require Java and ant when much of the system can be built and used without them\r\n\r\n2.  Document that Java and ant are needed for sql parsing (Java is already an optional pre-req for the query runner and the Java runtime).   Let the build of the sql parser be separate or only optionally integrated into the main makefile via a flag.\r\n\r\n- Pro: users who aren't interested in SQL have the same pre-reqs as before\r\n- Con: it complicates the build and could be confusing to users who just want to install the system and don't yet know exactly how they will use it.\r\n\r\n@jeromesimeon said he would think about this some more and ask others.  I am waiting for guidance."}}
{"actor":{"display_login":"joshuaauerbachwatson"}}