This request is superseded by
request 849045
(Show diff)
Overview
Request 848923 superseded
initial version - requried for kanku
- Created by M0ses
- In state superseded
- Supersedes 848333
- Superseded by 849045
- Open review for devel:languages:perl / perl-IO-Compress-Lzma
- Open review for opensuse-review-team
- Open review for openSUSE:Factory:Staging:adi:66
Loading...
reviewer
@computersalat, @coolo, @dstoecker, @lrupp, @rjschwei, @smithfarm, @tinita: review reminder
source maintainer
I think we should replace the description. And the many empty lines in the second changelog entry should be stripped.
Login required, please
login
in order to comment
Request History
M0ses created request
initial version - requried for kanku
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:66"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:66"
That leaves the description. Is this really still "...DO NOT use in production code...". Otherwise the description text should be replaced by something more meaningful. I think this text is outdated. No mention of it in the online documentation.
Version 2.093 indeed says "WARNING -- This is a Beta release.": https://metacpan.org/pod/release/PMQS/IO-Compress-Lzma-2.093/lib/IO/Compress/Lzma.pm
Yeah I know, but the perl website doesn't issue a similar warning and the fact that this warning is there for some time I'd say this is a "well, we either forgot or'll never remove this notice" state.
Ah in 2.096 the warning was removed with "No Changes, No Changes, add test" as only changes, so It is safe to remove it in 2.093 as well.