Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing source for the JPIP library and executables (issue #658) #659

Merged
merged 1 commit into from
May 10, 2016

Conversation

stweil
Copy link
Contributor

@stweil stweil commented Nov 17, 2015

They all need opj_malloc and other functions from opc_malloc.c.

Signed-off-by: Stefan Weil sw@weilnetz.de

…ain#658)

They all need opj_malloc and other functions from opc_malloc.c.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
@stweil
Copy link
Contributor Author

stweil commented Dec 23, 2015

Ping? This patch waits to be pulled.
Joyeux Noël, Stefan

@malaterre
Copy link
Collaborator

@mayeut the actual real issue is that this got unoticed. I'd like to keep this open for now, until #691 is fixed.

@stweil
Copy link
Contributor Author

stweil commented Mar 7, 2016

PR #692 fixes issue #691. I suggest to apply at first PR #659, then #692 (reversed order would break checks first and fix them later).

@stweil
Copy link
Contributor Author

stweil commented Apr 23, 2016

@malaterre, @mayeut, what about my suggestion? Can we proceed like that, or do you see more open related issues?

@malaterre
Copy link
Collaborator

Ok for me !

@stweil
Copy link
Contributor Author

stweil commented May 10, 2016

@mayeut, I'd appreciate if I could get this removed from my to-do list. Could you please pull both requests?

@mayeut mayeut merged commit aae066d into uclouvain:master May 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants