JonBCode / openjpeg

Automatically exported from code.google.com/p/openjpeg
Other
0 stars 0 forks source link

MD5 Checksum hangs under valgrind on MacOS X #410

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This is not a bug in the Test Suite but rather a workaround for CMake/Valgrind 
under MacOS X (I should report this to either or both of those projects as 
well).

Using CMake 2.8.12.2 & valgrind 3.10.0, md5 checksum hangs.
It would even be better if the md5 check wasn't run under valgrind but I don't 
know if it can be done (search for a while but didn't find any test property 
useful for this).

Attached is a patch that works around this problem. It prevents another cmake 
process being run & use file(MD5 instead.
Tested in CDash. 

Original issue reported on code.google.com by m.darb...@gmail.com on 6 Oct 2014 at 9:48

Attachments:

GoogleCodeExporter commented 9 years ago
+ cc Mathieu Malaterre

@Mathieu: could you have a look to this issue ?

Original comment by antonin on 22 Oct 2014 at 1:54

GoogleCodeExporter commented 9 years ago
Please do not apply the suggested patch, as explained by OP this is a hack. It 
solves the symptoms not the bug, plus it removes essential features of the test 
suite.

I do not see why running `cmake -E md5sum` would make valgrind hang...but I do 
not have access to a MacOSX box ATM

Original comment by mathieu.malaterre on 22 Oct 2014 at 2:00

GoogleCodeExporter commented 9 years ago
@Mathieu,

The way MD5 is computed is changed, that's all. It doesn't remove any essential 
features from the test suite as you can see on CDash (mbpro-mlr has this patch).

IMHO, while achieving non regression, it makes the file checkmd5refs.cmake 
easier to read. One could argue that the original is a hack, I asked myself why 
a process was spawned there... maybe if "file(MD5" was buggy & not `cmake -E 
md5sum` in some version of CMake.

Original comment by m.darb...@gmail.com on 22 Oct 2014 at 2:25

GoogleCodeExporter commented 9 years ago
@matthieu ok my mistake I missed the `file(MD5 ${pgxfullpath} output)` line.

It makes sense to remove this from the test suite, indeed. Patch can applied 
as-is. thx for the clarification.

Original comment by mathieu.malaterre on 22 Oct 2014 at 2:30

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r2914.

Original comment by antonin on 22 Oct 2014 at 10:52