Spec URL: http://www.techie.net/~talbert/miniz.spec SRPM URL: http://www.techie.net/~talbert/miniz-1.15r4-1.fc20.src.rpm Description: miniz.c is a lossless, high performance data compression library in a single source file that implements the zlib (RFC 1950) and Deflate (RFC 1951) compressed data format specification standards. It supports the most commonly used functions exported by the zlib library, but is a completely independent implementation so zlib's licensing requirements do not apply. miniz.c also contains simple to use functions for writing .PNG format image files and reading/writing/appending .ZIP format archives. miniz's compression speed has been tuned to be comparable to zlib's, and it also has a specialized real-time compressor function designed to compare well against fastlz/minilzo. Fedora Account System Username: swt2c Koji Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=7869152
Please ask upstream to remove the precompiled exe files from the source.
Done: https://code.google.com/p/miniz/issues/detail?id=40
And sorry to hit you with things one by one here, but: license should be "Unlicense" -- see https://fedoraproject.org/wiki/Licensing/Unlicense and https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Software_License_List
I'm also unsure about including the source .c file in -devel -- that seems out of the ordinary, but seems to be in line with the _intent_ of upstream as a copylib (http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Copylibs)
(In reply to Matthew Miller from comment #4) > I'm also unsure about including the source .c file in -devel -- that seems > out of the ordinary, but seems to be in line with the _intent_ of upstream > as a copylib > (http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Copylibs) Yes, this is certainly out of the ordinary. :-) Upstream doesn't provide a separate header file - their intention is for you to #define MINIZ_HEADER_FILE_ONLY and then #include the C file if you want just the header. Thus I think we have to include the C file in -devel so that we can have other packages link against miniz.
(In reply to Scott Talbert from comment #5) > (In reply to Matthew Miller from comment #4) > > I'm also unsure about including the source .c file in -devel -- that seems > > out of the ordinary, but seems to be in line with the _intent_ of upstream > > as a copylib > > (http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Copylibs) > > Yes, this is certainly out of the ordinary. :-) Well, I guess upstream is not aware about the fact, bundling zlib is THE historic precedence for banning static linkage and generally consider bundling to be a crap design. That said, I regret not to be able to have negative feelings on this package. In short, I feel this package should not be included into Fedora.
(In reply to Ralf Corsepius from comment #6) > That said, I regret not to be able to have negative feelings on this > package. In short, I feel this package should not be included into Fedora. Sorry, typo: ... I regret not to be able to deny negative feelings ...
this package already submitted for review by Petr just a day before. Please communicate with him. His submission is https://bugzilla.redhat.com/show_bug.cgi?id=1152653
Thanks all for the feedback. Since Petr beat me by about 150 bugs, I'll close mine and let him have this one. :-) In his packaging, he has extracted a header file from the C file, so I think that might address Ralf's concerns about allowing static linking. *** This bug has been marked as a duplicate of bug 1152653 ***
Heh, I was going to make the suggestion of pulling out a .h file — that encourages people to use it as a shared library (and, prevents it from being compiled in by accident).