Installation of gv.cpp and gv.i

The autotools build installs tclpkg/gv/gv.cpp and tclpkg/gv/gv.i to include/. Is this intentional? I can’t imagine any user expects to find these files there.

I thought include/ should pretty much only include header files (.h) ?

Yep, that’s pretty much my understanding too. Which was why I was surprised to find a .cpp in there. I don’t know what .i is, but it looks like unprocessed SWIG input, which doesn’t seem to belong in include/ either.

It was intentional. Those are the only source files needed to build additional language bindings with SWIG.

I can agree that they are probably not used much.

Ah I see. So is include/ the right place for them? I have no objection to leaving them installed there if that is indeed the correct location.

{prefix}/include/graphviz/
as I recall, so I felt we could assume some leniency. I didn’t want to separate them.

Do you have suggestion?

John

{prefix}/share/graphviz/<something>/?

<something> = swig ?

Plenty of stuff under {prefix}/share/graphviz/ already:

share/graphviz/demo
share/graphviz/doc
share/graphviz/examples
share/graphviz/graphs
share/graphviz/gvedit
share/graphviz/gvpr
share/graphviz/lefty
share/graphviz/smyrna