The crc test does not deal properly with 64 bits machines where the int is 32bits and the long is 64bits.
Created attachment 136080 [details, diff] ebuild patch adding test crc patch for amd64
Created attachment 136081 [details, diff] test_crc patch
Looks to be a workaround instead of a fix. Has upstream been informed?
Yes, have a look at the mailing list archive (if any...). They "said" it's no matter of importance
I took a look under the hood, and all that's missing seems to be a cast to INT32 in cs/csparse.c to prevent the result to not "wrap around" as expected. The test passes with htis patch applied. I tested that clearsilver still works with pike/Public.Parser.ClearSilver. I attach a patch (clearsilver-0.10.5-csparse_crc.patch) and basically the same patch to the ebuild as Sylvain provided with just the patch-name changed. regards, tom
Created attachment 151456 [details, diff] Fix the crc "wrap-around" problem on amd64 machines
Created attachment 151458 [details, diff] patch clearsilver-0.10.5 to apply csparse_crc patch on amd64 machines
I sent the same patch to upstream (the clearsilver mailinglist at yahoo) - maybe there is a problem with this simple cast I don't see.
Are patches ok to go in?
@web-apps team: Please advise.
Re-add amd64 if we need us.
the test fails but installation is successfull
