Request 101629 (accepted)
drop pre-2005 compatibility patch which influences the output of bzip2. it is incredibly weird that one cannot reproduce the same bzip2 after bunzip2 on a non-opensuse system with the same bzip2 version. so, since the patch in question is only there to allow incredibly old bzip2 versions to decompre [+]
Created
by apachelogger almost 6 years ago
In
state accepted
1 package maintainer
(show who)
Submit package home:apachelogge...anches:Archiving / bzip2 to package Archiving / bzip2
| [-] [+] | Changed | bzip2.changes |
| [-] [+] | Changed | bzip2.spec ^ |
| [-] [+] | Deleted | bzip2-maxlen20.patch ^ |
| @@ -1,38 +0,0 @@ ---- compress.c -+++ compress.c -@@ -444,11 +444,9 @@ - /*-- - Recompute the tables based on the accumulated frequencies. - --*/ -- /* maxLen was changed from 20 to 17 in bzip2-1.0.3. See -- comment in huffman.c for details. */ - for (t = 0; t < nGroups; t++) - BZ2_hbMakeCodeLengths ( &(s->len[t][0]), &(s->rfreq[t][0]), -- alphaSize, 17 /*20*/ ); -+ alphaSize, 20 ); - } - - -@@ -485,7 +483,7 @@ - if (s->len[t][i] > maxLen) maxLen = s->len[t][i]; - if (s->len[t][i] < minLen) minLen = s->len[t][i]; - } -- AssertH ( !(maxLen > 17 /*20*/ ), 3004 ); -+ AssertH ( !(maxLen > 20), 3004 ); - AssertH ( !(minLen < 1), 3005 ); - BZ2_hbAssignCodes ( &(s->code[t][0]), &(s->len[t][0]), - minLen, maxLen, alphaSize ); ---- huffman.c -+++ huffman.c -@@ -137,7 +137,10 @@ - Huffman codes with a maximum length of 17 bits. However, in - order to preserve backwards compatibility with bitstreams - produced by versions pre-1.0.3, the decompressor must still -- handle lengths of up to 20. */ -+ handle lengths of up to 20. -+ -+ SuSE: we changed it back to 20 to be compatible to older -+ bzip2 versions */ - - for (i = 1; i <= alphaSize; i++) { - j = weight[i] >> 8; | ||
Comments for request 101629 (0)
There's nothing to be done right now
Request History
|
|
created request | 1327579119 almost 6 years ago |
drop pre-2005 compatibility patch which influenc [+] |
||
|
|
Request got accepted | 1328017077 almost 6 years ago |
This makes sense. Thanks! |
||