Spec URL: http://vicodan.fedoraproject.org/BitchX.spec SRPM URL: http://vicodan.fedoraproject.org/BitchX-1.2-1.fc17.src.rpm Description: BitchX: The ultimate IRC client
Sorry correct URLs are here: Spec URL: http://vicodan.fedorapeople.org/BitchX.spec SRPM URL: http://vicodan.fedorapeople.org/BitchX-1.2-1.fc17.src.rpm Description: BitchX: The ultimate IRC client
I can do the review/sponsorship.
Why this review request is restricted?
marking unrestricted
scratch build: http://vicodan.fedorapeople.org/BitchX-1.2-1.fc17.src.rpm rpmlint: BitchX.x86_64: E: devel-dependency ncurses-devel BitchX.x86_64: W: incoherent-version-in-changelog 1.2 ['1.2-1.fc17', '1.2-1'] BitchX.x86_64: W: invalid-url URL: http://www.BitchX.org/ <urlopen error [Errno -2] Name or service not known> BitchX.x86_64: W: spurious-executable-perm /usr/lib64/bx/help/6_Functions/open.bz ... (bazillions of these spurious-executable-perm warnings) ... BitchX.x86_64: E: script-without-shebang /usr/lib64/bx/script/auto_resume BitchX.x86_64: W: no-manual-page-for-binary BitchX-1.2c01-svn BitchX.x86_64: W: no-manual-page-for-binary scr-bx naming: ok 1. MUST remove .spec tags: Vendor: BitchX Packager: Dan Mashal <vicodan> See: https://fedoraproject.org/wiki/Packaging/Guidelines#Tags 2. MUST remove seemingly erroneous Requires: ncurses ncurses-devel openssl xmlsec1-openssl-devel runtime shlib dependencies should get calculated automatically by rpm, no need to list these by hand. -devel runtime deps are almost certainly wrong here too. license: NOT ok 3. licensecheck. reveals many sources to be GPLv2+ and many without license attribution at all. Top level directory contains a COPYRIGHT file containing BSD licence. Can probably infer that all files without contrary licensing details are indeed BSD. So, you have a choice, to either use MUST: change to License: GPLv2+ or MUST: change to License: BSD and GPLv2+ depending on how pedantic you want to be. SHOULD: poke upstream to add license attribution to all source files (or at least the .c, .h ones). For example, /* * wserv.c - little program to be a pipe between a screen or * xterm window to the calling ircII process. * * Written by Troy Rollo, Copyright 1992 Troy Rollo * Finished by Matthew Green, Copyright 1993 Matthew Green * Support for 4.4BSD by Jeremy Nelson, Copyright 1997 EPIC Software Labs * * Works by opening up the unix domain socket that ircII bind's * before calling wserv, and which ircII also deleted after the * connection has been made. */ 4. As this is a GUI app, MUST: add a .desktop file so it appears in DE menus See: https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files macros: ok scriptlets: n/a So, looks like we've 4 MUST fix items in need of some love and fixing before I can approve this.
Rex, Please re-review. Per our IRC conversation this is not a GUI app. Removed: 1) Vendora and Packager tags from spec file 2) Removed -devel requirements from spec file 3) Updated License Also please note at this time BitchX.org is down so I have updated the spec file to use fedorapeople.org. Thanks, Dan
OK, for documentation purposes, this latest .spec has some other issues or gressed in a couple of ways: 2. still not completely fixed, still has unnecessary Requires: 5. source tarball has precompiled .o files 6. BuildRequires: compat-gcc34 added, not needed (or used) here
Removed Requires: tag from spec. Removed .o files from tarball Removed compat-gcc34 from BuildRequires. Please let me know if any other issues.
On other thing I mentioned in irc, but not explicitly here. Whenever making changes, please bump Release.
7. MUST %{_libdir}/bx/ dir is unowned, replace %{_libdir}/bx/* with %{_libdir}/bx/ (or equivalent) 5. is seems not yet fixed, latest srpm I just tried, http://vicodan.fedorapeople.org/BitchX-1.2-1.fc17.src.rpm still includes a bunch of .o files
Updated spec files per your request, tarball source RPM, change log, release version.
Scratch builds fail, looks like we're missing BuildRequires: openssl-devel after adding that, still getting an odd failure on i686: http://koji.fedoraproject.org/koji/taskinfo?taskID=4020520 I'll have to look deeper.
Try one of these.. From the original BitchX.spec: BuildRequires: ncurses ncurses-devel openssl xmlsec1-openssl-devel glib2-devel compat-gcc-34 compat-gcc-34-c++ Requires: ncurses openssl glibc
I just built with the latest source and spec on Fedora 17 i686. http://vicodan.fedorapeople.org/BitchX-1.2-4.fc17.i686.rpm http://vicodan.fedorapeople.org/BitchX-1.2-4.fc17.srpm Also make sure you have the latest source (it has changed) http://vicodan.fedorapeople.org/BitchX1.2.tar.gz
http://vicodan.fedorapeople.org/BitchX-1.2-4.fc17.src.rpm
The build failure is trying to link x86-64 .o files into a x86 build. Note for example that wterm.o is *not* being built, so it must already have been present - was that the older .tar.gz which included some .o files? Dan seems to have fixed that.
Per comment #12, still missing BuildRequires: openssl-devel and fwiw, no need for BuildRequires: ncurses openssl after adding the missing BuildRequires to your latest src.rpm, here's another try at a koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4024686
ok, sans my .spec typo (BR: openssl-openssl, haha), another one: http://koji.fedoraproject.org/koji/taskinfo?taskID=4024692
ok, I think all my MUST fix items have been resolved, let's consider this good and APPROVED. I'll leave it to you to add BuildRequires: openssl-devel so that it builds ok. next step is to request scm access, https://fedoraproject.org/wiki/PackageMaintainers/Join#Add_Package_to_Source_Code_Management_.28SCM.29_system_and_Set_Owner
New Package SCM Request ======================= Package Name: BitchX Short Description: The Ultimate IRC client. Owners: vicodan Branches: f15 f16 f17 InitialCC: rdieter
Git done (by process-git-requests).
Thanks Jon!
BitchX-1.2-4.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/BitchX-1.2-4.fc17
BitchX-1.2-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/BitchX-1.2-4.fc16
BitchX-1.2-4.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/BitchX-1.2-4.fc15
BitchX-1.2-5.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/BitchX-1.2-5.fc16
BitchX-1.2-5.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/BitchX-1.2-5.fc17
BitchX-1.2-5.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/BitchX-1.2-5.fc15
Package BitchX-1.2-5.fc17: * should fix your issue, * was pushed to the Fedora 17 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing BitchX-1.2-5.fc17' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-7046/BitchX-1.2-5.fc17 then log in and leave karma (feedback).
BitchX-1.2-5.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.
BitchX-1.2-5.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report.
BitchX-1.2-5.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.
Package Change Request ====================== Package Name: BitchX New Branches: el5 el6 Owners: vicodan