Bug 750099
Summary: | Review Request: airinv - C++ Simulated Airline Inventory Management System Library | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Denis Arnaud <denis.arnaud_fedora> |
Component: | Package Review | Assignee: | Thomas Spura <tomspur> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | dwmw2, notting, package-review, tomspur |
Target Milestone: | --- | Flags: | tomspur:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2012-02-12 19:44:23 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: | 702987, 728649 | ||
Bug Blocks: | 760594 |
Description
Denis Arnaud
2011-10-30 23:39:48 UTC
Hmm, doesn't build here. It seems the rmol config file for cmake is missing. -- Requires RMOL-0.25 CMake Error at config/project_config_embeddable.cmake:763 (find_package): Could not find a configuration file for package RMOL. Set RMOL_DIR to the directory containing a CMake configuration file for RMOL. The file will have one of the following names: RMOLConfig.cmake rmol-config.cmake Call Stack (most recent call first): config/project_config_embeddable.cmake:319 (get_rmol) CMakeLists.txt:42 (get_external_libs) (In reply to comment #1) > Hmm, doesn't build here. It seems the rmol config file for cmake is missing. > > > -- Requires RMOL-0.25 > CMake Error at config/project_config_embeddable.cmake:763 (find_package): > Could not find a configuration file for package RMOL. Yes, as mentioned, it requires rmol-0.25, which is currently in the testing repository: https://admin.fedoraproject.org/updates/rmol I have added it in the buildroot override for Fedora 15. The BuildRequires line may be (at least until rmol-0.25 reaches the stable repository for the distribution on which you test AirInv) changed to: BuildRequires: readline-devel, stdair-devel, airrac-devel, rmol-devel >= 0.25 Alternatively, rmol-0.25 may be installed with (on almost all the versions of Fedora and EPEL): yum --enablerepo=updates-testing install rmol-devel yum -y install stdair-devel airrac-devel ------- Nevertheless, I just noticed that a new version of StdAir and AirRAC must be, in turn, re-delivered. As is, AirInv compilation will not complete. I shall work on it shortly. Review: - name ok - descriptions ok - BR ok - doc has own COPYING and doesn't R the main package - ok - library correct installed - ldconfig called - no static lib - %files ok - license ok (headers are missing, but as you are part of upstream, you know you should have some ;)) - check is there - macros everywhere - doc is noarch - R pkgconfig ok as you are building for el* - BR there - defattr there - sources match: ccd19f9a48abe5d506c43a28544d96f5 airinv-0.1.1.tar.bz2 - koji build worked: http://koji.fedoraproject.org/koji/taskinfo?taskID=3561152 - $ rpmlint /home/tom/rpmbuild/RPMS/*/airinv* /home/tom/rpmbuild/SRPMS/airinv-0.1.1-1.fc16.src.rpm 5 packages and 0 specfiles checked; 0 errors, 0 warnings. NOTICE: - It looks strange, that _datadir/airinv is owned by the devel package and not by the main package... Usually the main package would own such a directory and put application data in it. But as only the devel package puts files there: ok NEEDSWORK: - This is in the build log: Warning: the PDF reference manual (/home/tom/rpmbuild/BUILD/airinv-0.1.1/doc/latex/refman.pdf) has failed to build. You can perform a simple re-build (make in the doc/latex sub-directory). and the refman.pdf is indeed missing. Please check why it fails and fix. _____________________________________________________________________________ As there is only a minor doc fix needed, this package is: _____________________________________________________________________________ APPROVED Thank you very much for that review, Thomas! I have eventually found the bug hindering the generation of the PDF reference manual. It has been fixed in the new upstream release (0.1.2). ------------------------------------------------------------------------------- New Package SCM Request ======================= Package Name: airinv Short Description: C++ Simulated Airline Inventory Management System Library Owners: denisarnaud Branches: f15 f16 el5 el6 InitialCC: denisarnaud ------------------------------------------------------------------------------- Git done (by process-git-requests). Removed badly formatted InitialCC section. AirInv has been packaged for F16, F15 and EL6. For instance: http://bit.ly/xkYe9O |