[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 Daniel Novotny changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Flag|fedora-review? |fedora-review+ --- Comment #12 from Daniel Novotny 2010-03-31 09:05:54 EDT --- seems OK now, thanks, review+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 --- Comment #11 from Vitezslav Crhonek 2010-03-29 11:10:26 EDT --- (In reply to comment #8) [snip] > > BAD debuginfo package looks complete. - rpmlint warns > expect-debuginfo.i386: W: spurious-executable-perm > /usr/src/debug/expect-5.43/exp_main_tk.c > (maybe that is nothing to worry about, let's see your comments) Honestly, don't know why it's executable. Permissions are preserved from upstream. > > BAD rpmlint is silent. > W: patch-not-applied Patch7: expect-5.43.0-tcl8.5.6.patch > (this minor warning can be easily corrected) Fixed. > > OK final provides and requires look sane. > OK %check is present and all tests pass. > > BAD shared libraries should be added to the regular linker search paths. > every binary RPM package which contains shared library files (not just > symlinks) in any of the dynamic linker's default paths, must call ldconfig in > %post and %postun. > -there's /usr/lib/libexpect5.43.so and no call to /sbin/ldconfig > I think that this library is used by Tcl only, which knows the path to the library from /usr/lib64/tcl8.5/expect5.43/pkgIndex.tcl file. [snip] FYI, there's new version in devel branch... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 --- Comment #10 from Daniel Novotny 2010-03-29 09:38:14 EDT --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 Daniel Novotny changed: What|Removed |Added CC||vcrho...@redhat.com --- Comment #9 from Daniel Novotny 2010-01-14 10:13:44 EST --- cc-ing maintainer -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 Daniel Novotny changed: What|Removed |Added Status|NEW |ASSIGNED CC||dnovo...@redhat.com AssignedTo|nob...@fedoraproject.org|dnovo...@redhat.com --- Comment #8 from Daniel Novotny 2010-01-14 09:59:46 EST --- seems no-one is assigned here now, I will continue with the review: OK source files match upstream: e6aaab98967f6410099b40f2b3ddebb4 expect-5.43.0.tar.bz2 OK source contains full URL OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK build root is correct. OK license field matches the actual license (public domain). OK license is open source-compatible. License text included in package. OK latest version is being packaged. OK BuildRequires are proper. OK compiler flags are appropriate. OK %clean is present. OK package builds in mock. BAD debuginfo package looks complete. - rpmlint warns expect-debuginfo.i386: W: spurious-executable-perm /usr/src/debug/expect-5.43/exp_main_tk.c (maybe that is nothing to worry about, let's see your comments) BAD rpmlint is silent. W: patch-not-applied Patch7: expect-5.43.0-tcl8.5.6.patch (this minor warning can be easily corrected) OK final provides and requires look sane. OK %check is present and all tests pass. BAD shared libraries should be added to the regular linker search paths. every binary RPM package which contains shared library files (not just symlinks) in any of the dynamic linker's default paths, must call ldconfig in %post and %postun. -there's /usr/lib/libexpect5.43.so and no call to /sbin/ldconfig OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK no scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK headers in -devel. OK no pkgconfig files. OK no libtool .la droppings. OK not a GUI app. - expectk can interact or create gui apps, but isn't one itself, so that's ok the most important issue is imho the /sbin/ldconfig call when we add shared library to /usr/lib -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225743] Merge Review: expect
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225743 Vitezslav Crhonek changed: What|Removed |Added Status|MODIFIED|NEW AssignedTo|vcrho...@redhat.com |nob...@fedoraproject.org Flag|fedora-review- |fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
