Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: libcapifax - Support for Sending/receiving faxes over CAPI capable devices
https://bugzilla.redhat.com/show_bug.cgi?id=591900
Summary: Review Request: libcapifax - Support for Sending/receiving faxes over CAPI capable devices Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: louis@lagendijk.xs4all.nl QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://www.fazant.net/libcapifax/libcapifax.spec SRPM URL: http://www.fazant.net/libcapifax/libcapifax-0.7.3-1.fc12.src.rpm Description: libcapifax is a library that supports sending/receiving faxes over capi via soft DSP (spandsp). It was originally released as part of the capifax package.
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=591900
Felix Kaechele felix@fetzig.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |felix@fetzig.org
--- Comment #1 from Felix Kaechele felix@fetzig.org 2010-05-14 05:54:23 EDT --- This package fails to build due to a missing libcapi. I believe this package requires the libcapi from http://www.tabos.org/ffgtk to work. The libcapi in isdn4k-utils will not work. And that basically poses a problem because the libcapi mentioned above conflicts with the libcapi in isdn4k-utils. This is not acceptable as per https://fedoraproject.org/wiki/Packaging:Conflicts because the libcapi from ffgtk is meant to replace the libcapi from isdn4k-utils.
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=591900
--- Comment #2 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-05-14 07:24:45 EDT --- Sorry, forgot to clarify this: the modular Capi replaces the old one in isdn4k-utils. The current isdn4k-utils CVS contains both. The isdn4k-utils in rawhide does already contain the new, modular libcapi20. The modular Capi is a drop-in replacement for the current version. The ABI is fully compatible.
I discussed this with the isdn4k-utils maintainer and he indicated that this version will probably be back-ported to F12 and F13 too. See BZ 566909, https://bugzilla.redhat.com/show_bug.cgi?id=566909.
I am planning to release libcapifax also for EPEL, with a separate libcapi package (which I plan to start the review for shortly, there are some small, mainly cosmetic issues that I want to solve first). That package will add the new libcapi20, but install it in a different location, so it does not conflict. It will then add the path through a file in /etc/ld.so.conf.d so that we can still override the isdn4k-utils libcapi. For a (not yet completely ready package see http://www.fazant.net/libcapi20/). This could also be used on F12 and F13 to bridge the gap untill isdn4k-utils is updated.
Libcapifax already builds in Rawhide. If you want to build on F12 or F13 you can either use the rebuild the rawhide version of isdn4k-utils (with the additiona patches in above BZ) or my libcapi package. Both will work
ffgtk (from tabos.org) will also be available soon as that was the reason to start the whole exercise in the first place.
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=591900
Louis Lagendijk louis@lagendijk.xs4all.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |566909
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=591900
--- Comment #3 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-05-14 07:36:14 EDT --- Here is the rpmlint output. I belive that all warnings can be skipped. I plan to release the package also for EPEL, hence the rpm-buildroot-usage warning.
[louis@travel capifax-0.7.3-jmb3]$ rpmlint libcapifax.spec ~/rpm/SRPMS/libcapifax-0.7.3-1.fc12.src.rpm ~/rpm/RPMS/x86_64/libcapifax-*fc12* libcapifax.spec:41: W: rpm-buildroot-usage %build rm -rf %{buildroot} libcapifax.src: W: spelling-error %description -l en_US capi -> capo, cap, carpi libcapifax.src: W: spelling-error %description -l en_US spandsp -> spandex, spanned, Spanish libcapifax.src: W: spelling-error %description -l en_US capifax -> capital, capitol, Capitol libcapifax.src:41: W: rpm-buildroot-usage %build rm -rf %{buildroot} libcapifax.x86_64: W: spelling-error %description -l en_US faxies -> faxes, faeries, falsies libcapifax.x86_64: W: spelling-error %description -l en_US capi -> capo, cap, carpi libcapifax.x86_64: W: spelling-error %description -l en_US spandsp -> spandex, spanned, Spanish libcapifax.x86_64: W: spelling-error %description -l en_US capifax -> capital, capitol, Capitol libcapifax.x86_64: W: incoherent-version-in-changelog 3.0.5a-1-jmb3 ['0.7.3-1.fc12.jmb3', '0.7.3-1.jmb3'] libcapifax.x86_64: W: spelling-error %description -l en_US capi -> capo, cap, carpi libcapifax.x86_64: W: spelling-error %description -l en_US spandsp -> spandex, spanned, Spanish libcapifax.x86_64: W: spelling-error %description -l en_US capifax -> capital, capitol, Capitol libcapifax-devel.x86_64: W: spelling-error Summary(en_US) capifax -> capital, capitol, Capitol libcapifax-devel.x86_64: W: spelling-error Summary(en_US) capifax -> capital, capitol, Capitol 7 packages and 1 specfiles checked; 0 errors, 15 warnings.
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=591900
--- Comment #4 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-05-14 07:56:50 EDT --- oops, please ignore the rpmlint output above, it contains some old rpm's I had still around. Here is the correct output: [louis@travel capifax-0.7.3-jmb3]$ rpmlint libcapifax.spec ~/rpm/SRPMS/libcapifax-0.7.3-1.fc12.src.rpm ~/rpm/RPMS/x86_64/libcapifax-* libcapifax.spec:41: W: rpm-buildroot-usage %build rm -rf %{buildroot} libcapifax.src: W: spelling-error %description -l en_US capi -> capo, cap, carpi libcapifax.src: W: spelling-error %description -l en_US spandsp -> spandex, spanned, Spanish libcapifax.src: W: spelling-error %description -l en_US capifax -> capital, capitol, Capitol libcapifax.src:41: W: rpm-buildroot-usage %build rm -rf %{buildroot} libcapifax.x86_64: W: spelling-error %description -l en_US capi -> capo, cap, carpi libcapifax.x86_64: W: spelling-error %description -l en_US spandsp -> spandex, spanned, Spanish libcapifax.x86_64: W: spelling-error %description -l en_US capifax -> capital, capitol, Capitol libcapifax-devel.x86_64: W: spelling-error Summary(en_US) capifax -> capital, capitol, Capitol 4 packages and 1 specfiles checked; 0 errors, 9 warnings.
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=591900
Felix Kaechele felix@fetzig.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |felix@fetzig.org Flag| |fedora-review?
--- Comment #5 from Felix Kaechele felix@fetzig.org 2010-05-14 08:27:33 EDT --- I do have a spec file for ffgtk available if you haven't started one yet. So if you're interested just drop me a line.
Other than that I'll take this review.
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=591900
Louis Lagendijk louis@lagendijk.xs4all.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |592487
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=591900
--- Comment #6 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-06-01 17:54:09 EDT --- Ping, Felix, can you please do the review? The isdn4k-utils is now in updates-testing
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=591900
Bug 591900 depends on bug 566909, which changed state.
Bug 566909 Summary: RFE: please change isdn4k-utils to use capi20.new https://bugzilla.redhat.com/show_bug.cgi?id=566909
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution| |ERRATA
--- Comment #7 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-07-17 17:57:28 EDT --- Ping, Felix are you still interested in doing the review for libcapifax?
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=591900
Louis Lagendijk louis@lagendijk.xs4all.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |
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=591900
Louis Lagendijk louis@lagendijk.xs4all.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW
--- Comment #8 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-07-29 17:46:20 EDT --- Reset fedora-review flag as Felix has not reacted since he said that he wanted to take the review
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=591900
Felix Kaechele felix@fetzig.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag| |fedora-review?
--- Comment #9 from Felix Kaechele felix@fetzig.org 2010-07-30 03:31:48 EDT --- Hi there, sorry for having this sit idle for so long. Here's your review:
Package Review ==============
Key: - = N/A x = Check ! = Problem ? = Not evaluated
=== REQUIRED ITEMS === [x] Package is named according to the Package Naming Guidelines. [x] Spec file name must match the base package %{name}, in the format %{name}.spec. [x] Package meets the Packaging Guidelines. [x] Package successfully compiles and builds into binary rpms on at least one supported architecture. Tested on: [!] Rpmlint output: libcapifax.spec:41: W: rpm-buildroot-usage %build rm -rf %{buildroot} [x] Package is not relocatable. [x] Buildroot is correct (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)) [x] Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [!] License field in the package spec file matches the actual license. License type: GPLv2+ [x] If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x] Spec file is legible and written in American English. [x] Sources used to build the package matches the upstream source, as provided in the spec URL. MD5SUM this package : 98d1aefd7d25496d3fd2f25c253bfe53 MD5SUM upstream package: 98d1aefd7d25496d3fd2f25c253bfe53 [x] Package is not known to require ExcludeArch [x] All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [-] The spec file handles locales properly. [x] ldconfig called in %post and %postun if required. [x] Package must own all directories that it creates. [-] Package requires other packages for directories it uses. [x] Package does not contain duplicates in %files. [x] Permissions on files are set properly. [x] Package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). [x] Package consistently uses macros. [x] Package contains code, or permissable content. [-] Large documentation files are in a -doc subpackage, if required. [x] Package uses nothing in %doc for runtime. [x] Header files in -devel subpackage, if present. [-] Static libraries in -devel subpackage, if present. [x] Package requires pkgconfig, if .pc files are present. [x] Development .so files in -devel subpackage, if present. [x] Fully versioned dependency in subpackages, if present. [x] Package does not contain any libtool archives (.la). [-] Package contains a properly installed %{name}.desktop file if it is a GUI application. [x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS === [!] Latest version is packaged. [x] Package does not include license text files separate from upstream. [-] Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x] Reviewer should test that the package builds in mock. [x] Package should compile and build into binary rpms on all supported architectures. Tested on: F12: http://koji.fedoraproject.org/koji/taskinfo?taskID=2357892 F13: http://koji.fedoraproject.org/koji/taskinfo?taskID=2357886 [x] Package functions as described. [x] Scriptlets must be sane, if used. [x] The placement of pkgconfig(.pc) files are correct. [-] File based requires are sane.
=== Issues === 1. rm -rf %{buildroot} is not needed in %build as the buildroot is populated in %install, not in %build 2. Specfile lists license as being GPLv2. Sources however specify GPLv2+ 3. Upstream of the patches lists a jmb4 version at http://www.tabos.org/ffgtk/download.php
=== Final Notes === 1. If the aforementioned issues are solved I shall approve this package.
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=591900
--- Comment #10 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-07-30 17:37:01 EDT --- Hello Felix Thanks for the review! I corrected the issues you mentioned: - removed the cleanup of the buildroot from %install AND %clean - Corrected license. good catch! I don't know how I missed that - Replaced my own fix for the pc file and replaced that with the jmb4 patch
New versions of the SRPM and specfiel are available from:
http://fazant.net/libcapifax/libcapifax-0.7.3-3.fc12.src.rpm http://fazant.net/libcapifax/libcapifax.spec
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=591900
--- Comment #11 from Felix Kaechele felix@fetzig.org 2010-08-07 16:35:08 EDT --- Sorry, I was busy last week again. I see that you removed the %clean section completely. It's required however and does require a rm -rf %{buildroot} also your changelog includes the info that the rm -rf %{buildroot} has moved from %build to %install. %install however does not contain rm -rf %{buildroot} (as it should). Seems that there was a little misunderstanding regarding that.
Other than that everything else seems fine.
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=591900
--- Comment #12 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-08-14 12:16:56 EDT --- My bad. I indeed misunderstood you.
I have been abroad this week, hence the late response. I put the rm -rf %{buildroot} back in install and %clean
http://fazant.net/libcapifax/libcapifax-0.7.3-4.fc12.src.rpm http://fazant.net/libcapifax/libcapifax.spec
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=591900
Felix Kaechele felix@fetzig.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #13 from Felix Kaechele felix@fetzig.org 2010-08-14 18:21:41 EDT --- Okay. Package looks good now.
APPROVED
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=591900
Louis Lagendijk louis@lagendijk.xs4all.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #14 from Louis Lagendijk louis@lagendijk.xs4all.nl 2010-08-21 10:46:01 EDT --- New Package SCM Request ======================= Package Name: libcapifax Short Description: Support for Sending/receiving faxes over CAPI capable devices Owners: llagendijk Branches: f12 f13 f14 el6 InitialCC:
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=591900
--- Comment #15 from Kevin Fenzi kevin@tummy.com 2010-08-23 16:57:12 EDT --- Git done (by process-git-requests).
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=591900
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
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=591900
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2010-08-25 18:13:55 EDT --- libcapifax-0.7.3-4.fc14 has been submitted as an update for Fedora 14. http://admin.fedoraproject.org/updates/libcapifax-0.7.3-4.fc14
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=591900
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2010-08-25 18:14:49 EDT --- libcapifax-0.7.3-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/libcapifax-0.7.3-4.fc13
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=591900
--- Comment #18 from Fedora Update System updates@fedoraproject.org 2010-08-25 18:15:57 EDT --- libcapifax-0.7.3-4.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/libcapifax-0.7.3-4.fc12
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=591900
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #19 from Fedora Update System updates@fedoraproject.org 2010-08-25 20:56:01 EDT --- libcapifax-0.7.3-4.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libcapifax'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/libcapifax-0.7.3-4.fc13
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=591900
--- Comment #20 from Fedora Update System updates@fedoraproject.org 2010-09-02 16:48:23 EDT --- libcapifax-0.7.3-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
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=591900
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |libcapifax-0.7.3-4.fc13 Resolution| |ERRATA Last Closed| |2010-09-02 16:48:27
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=591900
--- Comment #21 from Fedora Update System updates@fedoraproject.org 2010-09-02 16:48:50 EDT --- libcapifax-0.7.3-4.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
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=591900
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|libcapifax-0.7.3-4.fc13 |libcapifax-0.7.3-4.fc12
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=591900
--- Comment #22 from Fedora Update System updates@fedoraproject.org 2010-09-04 01:18:30 EDT --- libcapifax-0.7.3-4.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
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=591900
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|libcapifax-0.7.3-4.fc12 |libcapifax-0.7.3-4.fc14
package-review@lists.fedoraproject.org