Swig wrapper for cg-proc

[singh-lokendra]

guarded by ENABLE_PYTHON_BINDINGS

• updated cmake minimum version to 3.0.2 for finding python interpreter
• modified Grammar.hpp for including in swig wrapper
• updated travis distro to xenial

Usage:

cg = constraint_grammar.CGProc(dictionary_path)
command = ['cg-proc', '-w']
cg.cg_proc(len(command), command, input_file.name, output_file.name)

[sushain97]

did you modify the travis.yml to verify the python bindings build? (if needed)

[singh-lokendra]

do you mean 26c166f#diff-354f30a63fb0907d4ad57269548329e3R14?

[sushain97]

@TinoDidriksen could you please take a look at this PR?

[TinoDidriksen]

The Grammar.hpp changes cannot possibly be necessary. It's in the CG3 namespace, and all headers it includes put their things in the CG3 namespace. It should not need qualification.

Bumping CMake to 3.0 is fine. Only CentOS 7 needs special handling for that.

Also note that CG-3 is not a git project - this is a read-only copy of the svn repo, so pushing changes are a bit more fiddly.

[singh-lokendra]

The Grammar.hpp changes cannot possibly be necessary. It's in the CG3 namespace, and all headers it includes put their things in the CG3 namespace. It should not need qualification.

Grammar.hpp needs to be included https://github.com/TinoDidriksen/cg3/pull/37/files#diff-72aaa8a9e7ef602947a3afb265935f15R6 for making the inheritance visible to swig.
Without qualification, these errors are reported https://termbin.com/d1tu. Even swig v4.0 needs explicit qualification

[TinoDidriksen]

That is rather bizarre. Surely using namespace CG3 will do instead then.

[singh-lokendra]

Surely using namespace CG3 will do instead then.

Adding this before %include reports Error: Nothing known about namespace 'CG3'. And adding it after the include is still producing the above errors

[sushain97]

@TinoDidriksen any further thoughts on Grammar.hpp?

[TinoDidriksen]

Committed, without changing src/