Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cdrkit - cdrtools replacement https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224365 ------- Additional Comments From tibbs@xxxxxxxxxxx 2007-01-26 13:32 EST ------- This fails to build for me: CMake Error: Error: found a Linux system but no libcap header. Install libcap-dev. Adding BuildRequires: libcap-devel gets me past that, but then the build fails later: [ 70%] Building C object genisoimage/CMakeFiles/genisoimage.dir/jte.o /builddir/build/BUILD/cdrkit-1.1.2/genisoimage/jte.c:18:18: error: zlib.h: No such file or directory /builddir/build/BUILD/cdrkit-1.1.2/genisoimage/jte.c: In function 'flush_gzip_chunk': /builddir/build/BUILD/cdrkit-1.1.2/genisoimage/jte.c:581: error: 'z_stream' undeclared (first use in this function) [etc.] Adding BR: zlib-devel gets through that and allows the build to complete. The resulting packages don't install properly: Error: Missing Dependency: /usr/local/bin/perl is needed by package icedax Error: Missing Dependency: perl(v5.8.1) is needed by package genisoimage Some rpmlint complaints: W: cdrkit summary-ended-with-dot A collection of CD/DVD utilities. W: genisoimage summary-ended-with-dot Creates an image of an ISO9660 filesystem. W: wodim summary-ended-with-dot A command line CD/DVD recording program. W: icedax summary-ended-with-dot A utility for sampling/copying .wav files from digital audio CDs. Best for summaries to not end with dots. E: icedax invalid-dependency /usr/local/bin/perl E: icedax wrong-script-interpreter /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl "/usr/local/bin/perl" W: icedax doc-file-dependency /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl /usr/local/bin/perl W: icedax doc-file-dependency /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl perl(strict) W: icedax doc-file-dependency /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl perl(IO::Handle) W: icedax doc-file-dependency /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl perl(Env) W: icedax doc-file-dependency /usr/share/doc/icedax-1.1.2/icedax/tracknames.pl perl(Socket) These are all related. Basically, documentation should not be executable, and examples in documentation shouldn't pull in dependencies (which they won't if they're not executable). If you want to be nice, you can fix up the /usr/local/bin/perl bits. W: cdrkit unversioned-explicit-provides dvdrecord W: cdrkit unversioned-explicit-obsoletes cdrecord W: cdrkit unversioned-explicit-provides cdrecord W: cdrkit unversioned-explicit-obsoletes cdrecord-mkisofs W: cdrkit unversioned-explicit-provides cdrecord-mkisofs W: cdrkit unversioned-explicit-obsoletes mkisofs W: cdrkit unversioned-explicit-provides mkisofs W: cdrkit unversioned-explicit-obsoletes cdrecord-cdda2wav W: cdrkit unversioned-explicit-provides cdrecord-cdda2wav W: cdrkit unversioned-explicit-obsoletes cdda2wav W: cdrkit unversioned-explicit-provides cdda2wav These really shouldn't be unversioned; we can't predict what will happen in the future with these package. So it's best to obsolete cdrecord <= current version and provide cdrecord = current version + epsilon. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review