← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/xqj_installers into lp:zorba

 

Review: Needs Fixing

Mostly looks excellent. I still think the SITEARCH issue is a problem, but I see that it also exists in the Java SWIG code as Rodolfo says, so I will file a separate bug report about that.

The only change I would request: Rodolfo, in swig/xqj/CMakeLists.txt, you have separate code using the new CMake "UseJava" functionality if the CMake version is greater than 2.8.6. I like that, because the ADD_CUSTOM_COMMAND(POST_BUILD) technique used otherwise is a kludge that doesn't get the dependencies right. Could you also use that same UseJava technique in swig/java/CMakeLists.txt ?
-- 
https://code.launchpad.net/~zorba-coders/zorba/xqj_installers/+merge/108257
Your team Zorba Coders is subscribed to branch lp:zorba.


References