Add JPIP options to Travis configuration #692
Conversation
|
I don't expect that this will work immediately, because there might be missing build dependencies (Java, more?). Maybe the JPIP server should be added later in a 2nd step. |
|
Most checks are now successful. Travis fails with clang (MacOSX) and x86_64-w64-mingw32-gcc (Windows cross) compilers. I assume that those build environments are incomplete. How can I see what is missing for JPIP? |
|
You can see some things in CDASH: http://my.cdash.org/index.php?project=OPENJPEG&date=2016-01-09 |
|
@mayeut, thanks. http://my.cdash.org/buildSummary.php?buildid=890011 contains the log messages for MacOSX, http://my.cdash.org/buildSummary.php?buildid=890008 those for Windows. The pull request here is ok: it detects faulty code in the JPIP library. Several format strings are not portable and fail for MacOSX and 64 bit Windows. I suggest to pull #659 - that fixes a precondition for JPIP. Then the format string problem can be addressed. Hopefully all checks will work then and #692 can be pulled. |
|
@stweil, I merged #659 , I did not see that @malaterre said it was ok to merge, sorry about the delay. Can you rebase this one please ? |
|
@mayeut, the patch is rebased now. 2 of the 4 original commits were no longer needed. |
This fixes issue uclouvain#691. Signed-off-by: Stefan Weil <[email protected]>
It has build dependencies which are missing on the Travis server. Signed-off-by: Stefan Weil <[email protected]>
|
@mayeut, the PR was now rebased once more. Could you please pull it? |
No description provided.