Description of problem: Sereal 3.003 chomps off "\n" from the input data if math is used against the data prior to serialization. Version-Release number of selected component (if applicable): Sereal 3.003 How reproducible: my @a1 = grep { $_ % 3 == 0 } ( "1\n", "2\n", "3\n", "4\n", "5\n" ); my $frozen = encode_sereal(\@a1); my $a2 = decode_sereal($frozen); if ( $a2->[0] eq "3\n" ) { print "pass\n"; } else { print "fail\n"; } Actual results: fail Expected results: pass Additional info: Sereal 3.008 and 3.014 are fine and do not chomp "\n" leaving the data intact, similar to how Storable freeze and thaw behavior. Beginning with MCE 1.800, Sereal is enabled automatically if available. But, only if 3.008 or later. Best, Mario
I cannot bump Sereal version in epel7 because of package dependencies. As for newline chomp, could you provide a patch so I can merge and update the current version of package?
I have updated my test script 05_mce_grep.t in MCE 1.800 to chomp off "\n". Furthermore, MCE 1.800 enables Sereal only if 3.008 or higher. Otherwise, Storable is used. All is well in that sense and really not a problem for MCE or for existing packages that depend on Sereal 3.003. Please feel free to close this ticket if updating perl-Sereal to 3.008 or 3.014 is not feasible at this time. There are many bug fixes in Sereal between 3.003 and 3.008. So, am not sure if perl-Sereal in RedHat will ever reach 3.008 or 3.014. Kind regards, Mario
It's very unlikely that Sereal version will be increased in epel7, since some versioned dependencies were introduced in later versions, which break package builds. So there is no option for easy upgrade.
I had a look at this this morning. * perl-Sereal 3.008 needs perl-Sereal-Decoder 3.008 and perl-Sereal-Encoder 3.008 * perl-Sereal-Decoder 3.008 and perl-Sereal-Encoder 3.008 each want ExtUtils::MakeMaker 7.0 or later, whilst EL7 has 6.68 The EU::MM 7.0 dependency is expressed in the Makefile.PL of these dists, but I found that if I dropped the version requirement from the BuildRequires in the spec file, the packages built fine (and perl-Sereal built fine against them), without even any complaints about the version when running "perl Makefile.PL". The upstream commit where this requirement was introduced was: https://github.com/Sereal/Sereal/commit/d70a09082 The justification given was that the modules were failing to build properly with some older versions of EU::MM, but that doesn't appear to affect EL-7 builds, so perhaps there is a way forward here after all?
The module author explicitly introduced versioned EU::MM deps, if I remember correctly, to solve some bugs with old EU::MM, so it's not so straightforward. "ExtUtils::MakeMaker and CPAN::Meta need to be up to date or the META files miss stuff", et al. Although I can prepare "bumped" versions for test if anybody agrees to test updated versions thoroughly (please let me know in this case).
Discussed again, tested and reopened. Testing COPR: https://copr.fedorainfracloud.org/coprs/dfateyev/perl-Sereal/
perl-Sereal-3.014-1.el7 perl-Sereal-Encoder-3.014-1.el7 perl-Sereal-Decoder-3.014-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-6c9c1e1cdb
perl-Sereal-3.014-1.el7, perl-Sereal-Decoder-3.014-1.el7, perl-Sereal-Encoder-3.014-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-6c9c1e1cdb
perl-Sereal-3.014-1.el7, perl-Sereal-Decoder-3.014-1.el7, perl-Sereal-Encoder-3.014-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.