Bug 814924 - Review Request: jalv - a simple LV2 host
Summary: Review Request: jalv - a simple LV2 host
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Mattias Ellert
QA Contact: Fedora Extras Quality Assurance
Depends On: 814916
Blocks: FedoraAudio
TreeView+ depends on / blocked
Reported: 2012-04-21 10:27 UTC by Brendan Jones
Modified: 2012-05-28 01:28 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-05-28 01:28:28 UTC
Type: Bug
mattias.ellert: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Brendan Jones 2012-04-21 10:27:51 UTC
jalv is a simple but fully featured LV2 host for Jack. It runs LV2 plugins 
and exposes their ports as Jack ports, essentially making any LV2 plugin 
function as a Jack application. 

SPEC: http://bsjones.fedorapeople.org/lv2/jalv.spec
SRPM: http://bsjones.fedorapeople.org/lv2/jalv-1.0.0-1.fc16.src.rpm

rpmlint /home/bsjones/rpmbuild/RPMS/x86_64/jalv-1.0.0-1.fc16.x86_64.rpm /home/bsjones/rpmbuild/RPMS/x86_64/jalv-debuginfo-1.0.0-1.fc16.x86_64.rpm /home/bsjones/rpmbuild/SRPMS/jalv-1.0.0-1.fc16.src.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.

Comment 1 Mattias Ellert 2012-04-22 19:36:19 UTC
Fedora Review jalv 2012-04-22

$ rpmlint jalv*.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ Package named according to guidelines
+ Specfile named after package
+ Package license (MIT) is Fedora approved
+ Package license mateches license statements in sources
+ License file (COPYING) included as %doc
+ Specfile is written in readable American English
+ Package source matches upstream:

$ md5sum jalv-1.0.0.tar.bz2 jalv-srpm/jalv-1.0.0.tar.bz2 
f20f81dbb437f4e4ea10e00694f6cc4d  jalv-1.0.0.tar.bz2
f20f81dbb437f4e4ea10e00694f6cc4d  srpm/jalv-1.0.0.tar.bz2

- Package fails to compile due to several missing build requires:
    BuildRequires:  python
    BuildRequires:  suil-devel >= 0.6.0
    BuildRequires:  jack-audio-connection-kit-devel

[ Also my rebuild of the lilv package on which this package depends failed due to missing build requires on python and python-devel ]

+ No locales
+ No shared libraries
+ No bundled libraries
+ No duplicates in %files
+ Package owns directories it creates
+ File permissions are sane
+ Specfile uses macros consistently
+ Contains code
+ Documentation is small - need not be in subpackage
+ %doc not runtime essential
+ No static libraries
+ No development files
+ No subpackages
+ No libtool archives
+ Package does not own others' directories
+ Filenames valid UTF8

Summary: Only the missing build requires, otherwise OK.

Comment 2 Brendan Jones 2012-04-23 11:48:30 UTC
SPEC: http://bsjones.fedorapeople.org/lv2/jalv.spec
SRPM: http://bsjones.fedorapeople.org/lv2/jalv-1.0.0-2.fc16.src.rpm

Thanks. The BR's have been addressed.

Comment 3 Mattias Ellert 2012-04-24 15:22:36 UTC
Package approved.

Comment 4 Brendan Jones 2012-05-02 10:54:58 UTC
Thanks for taking this review

New Package SCM Request
Package Name: jalv
Short Description: A simple LV2 host
Owners: bsjones
Branches: f16 f17

Comment 5 Gwyn Ciesla 2012-05-02 12:16:56 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2012-05-15 09:54:22 UTC
jalv-1.0.0-2.fc17 has been submitted as an update for Fedora 17.

Comment 7 Fedora Update System 2012-05-15 16:41:20 UTC
jalv-1.0.0-2.fc17 has been pushed to the Fedora 17 testing repository.

Comment 8 Fedora Update System 2012-05-28 01:28:28 UTC
jalv-1.0.0-2.fc17 has been pushed to the Fedora 17 stable repository.

Note You need to log in before you can comment on or make changes to this bug.