Bug 492969
Summary: | Review Request: lv2dynparam - LV2 dynamic parameters extension | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Orcan Ogetbil <oget.fedora> |
Component: | Package Review | Assignee: | Mattias Ellert <mattias.ellert> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, nando, notting |
Target Milestone: | --- | Flags: | mattias.ellert:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 2-1.fc10 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-04-27 21:36:19 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 492971, 492990 |
Description
Orcan Ogetbil
2009-03-30 23:18:17 UTC
koji rawhide build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1266783 I tried to make a mock build on Fedora 10 of this package. It fails for both i386 and x86_64 with the following error during configure: configure: error: The pkg-config script could not be found or is too old. Make sure it is in your PATH or set the PKG_CONFIG environment variable to the full path to pkg-config. Alternatively, you may set the environment variables LV2_CFLAGS and LV2_LIBS to avoid the need to call pkg-config. See the pkg-config man page for more details. To get pkg-config, see <http://pkg-config.freedesktop.org/>. See `config.log' for more details. error: Bad exit status from /var/tmp/rpm-tmp.gW9MDo (%build) I added the update-testing repo to the mock config for the build, and the build requirements were picked up: DEBUG backend.py:554: /usr/bin/yum --installroot /var/lib/mock/fedora-10-x86_64-19081239177430616696239/root/ install 'lv2core-devel' DEBUG util.py:280: Executing command: /usr/bin/yum --installroot /var/lib/mock/fedora-10-x86_64-19081239177430616696239/root/ install 'lv2core-devel' DEBUG util.py:256: ================================================================================ DEBUG util.py:256: Package Arch Version Repository Size DEBUG util.py:256: ================================================================================ DEBUG util.py:256: Installing: DEBUG util.py:256: lv2core-devel x86_64 3.0-2.fc10 updates-testing 9.2 k DEBUG util.py:256: Installing for dependencies: DEBUG util.py:256: lv2core x86_64 3.0-2.fc10 updates-testing 11 k DEBUG util.py:256: Transaction Summary DEBUG util.py:256: ================================================================================ DEBUG util.py:256: Install 2 Package(s) DEBUG util.py:256: Update 0 Package(s) DEBUG util.py:256: Remove 0 Package(s) DEBUG util.py:256: Total download size: 20 k DEBUG util.py:256: Installed: DEBUG util.py:256: lv2core-devel.x86_64 0:3.0-2.fc10 DEBUG util.py:256: Dependency Installed: DEBUG util.py:256: lv2core.x86_64 0:3.0-2.fc10 DEBUG util.py:319: Child returncode was: 0 However, the pkg-config tools was not installed in the mock environment. The reason seems to be a missing requires on pkgconfig in the lv2core-devel package currently in the testing repo. According to the packaging guidelines section 1.22.1: "Packages containing pkgconfig(.pc) files must Requires: pkgconfig (for directory ownership and usability)." [ellert@ellert ~]$ rpm -q --requires -p lv2core-devel-3.0-2.fc10.x86_64.rpm lv2core = 3.0-2.fc10 rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 Thanks for pointing this. I am one of the co-maintainers of lv2core and I fixed the issue and rebuilt the package in devel and in F-10. Meanwhile, you can add BR: pkgconfig to the SPEC file of lv2dynparam to *work around* the mock build issue. (I put the new lv2core SRPM from koji first in by mock chain build.) Fedora review lv2dynparam-2-1.fc10.src.rpm 2009-04-09 * OK ! Needs attention * rpmlint output - OK $ rpmlint *.rpm lv2dynparam.spec lv2dynparam-devel.x86_64: W: no-documentation 4 packages and 1 specfiles checked; 0 errors, 1 warnings. * Package is named according to guidelines * The spec file is named after the package * Package is licensed using a Fedora approved license (GPLv2) * The package license matches the license as stated in the sources * The license file in the sources (COPYING) is included in the package * The specfile is written in legible English * Sources matches upstream - and is the latest released version 708a7ce7e6604af2a58f521e343509d5 lv2dynparam1-2.tar.bz2 708a7ce7e6604af2a58f521e343509d5 SRPM/lv2dynparam1-2.tar.bz2 * Package builds in mock (Fedora 10) * BuildRequires are sane * Package calls ldconfig as appropriate * Package owns directories it creates * No double listed files * File permissions are sane and %files section has %defattr * %clean clear buildroot * The specfile uses macros consistently * Package contains code * %doc is not essential for running * headers are in -devel * No static libraries * -devel Requires: pkgconfig * .so files are in devel * -devel requires main with fully qualified version * No .la files * Package does not own other's directories * %install clears buildroot * Installed file names are valid UTF-8 Package approved. Thanks a lot! New Package CVS Request ======================= Package Name: lv2dynparam Short Description: LV2 dynamic parameters extension Owners: oget Branches: F-10 InitialCC: cvs done. lv2dynparam-2-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/lv2dynparam-2-1.fc10 lv2dynparam-2-1.fc10 has been pushed to the Fedora 10 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 lv2dynparam'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3734 lv2dynparam-2-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. |