Bug 542760
Summary: | Review Request: mopac7 - Semi-empirical quantum mechanics suite | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Carl Byington <carl> |
Component: | Package Review | Assignee: | Susi Lehtola <susi.lehtola> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting, susi.lehtola |
Target Milestone: | --- | Flags: | susi.lehtola:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 1.15-8.fc12 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-01-12 23:26:49 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: | 542740 | ||
Bug Blocks: | 542765, 542767 |
Description
Carl Byington
2009-11-30 18:09:21 UTC
Why don't you package OpenMopac 7.1? It's still under maintenance, MOPAC7 is not... http://openmopac.net/ http://openmopac.net/Downloads/Mopac_7.1source.zip only contains microsoft .dsp style make files, nothing to help with building on linux. http://www.uku.fi/~thassine/projects/download/current/mopac7-1.15.tar.gz is the version used by the author of ghemical. http://www.five-ten-sg.com/mopac7.spec http://www.five-ten-sg.com/mopac7-1.15-5.fc12.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=1888970 rpmlint output: mopac7.x86_64: E: no-binary mopac7-devel.x86_64: W: no-documentation mopac7-libs.x86_64: W: no-documentation What you are actually doing with sed "s/\.\/src/\/usr\/bin/" run_mopac7 > %{buildroot}%{_bindir}/run_mopac7 is replacing the binary with a temporary libtool wrapper. What's the actual problem you want to solve? The sed line looks like you could do with another separator than /, using e.g. | would make the expressions a lot neater. Besides, you are shipping makefiles in the documentation. Try if %check make test works, if it does then IMHO you don't need to ship the tests directory at all. fixed. The .dat files in ./test seem to be samples that could also be used for testing. It seems reasonable to install those in %doc (without the Makefiles). The sed expressions were from mandriva - cleaned up now, and installed the real binary rather than the libtool wrapper. run_mopac7 is a convenience fortran wrapper that needs to reference the installed mopac7 binary. Not caught earlier since ghemical just needs the libraries, not the main mopac7 binary. http://www.five-ten-sg.com/mopac7.spec http://www.five-ten-sg.com/mopac7-1.15-6.fc12.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=1889228 rpmlint output: mopac7.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/mopac7 ['/usr/lib64'] mopac7-devel.x86_64: W: no-documentation mopac7-libs.x86_64: W: no-documentation 5 packages and 0 specfiles checked; 1 errors, 2 warnings. Get rid of the rpath. http://fedoraproject.org/wiki/Packaging/Guidelines#Removing_Rpath ** You have %{_bindir}/%{name} %{_bindir}/run_mopac7 in files, better change %{name} to mopac7. ** Btw, the build process doesn't seem to use f2c at all - gfortran is used to compile the Fortran stuff. ** There is no CCOPTIONS variable in the makefiles, drop CCOPTIONS="%{optflags}" from make. The used flags are already picked up by %configure. ** MUST: The package does not yet exist in Fedora. The Review Request is not a duplicate. OK MUST: The spec file for the package is legible and macros are used consistently. ~OK - I think you could do with a few clarifying comments in the %install section. MUST: The package must be named according to the Package Naming Guidelines. OK MUST: The spec file name must match the base package %{name}. OK MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. OK MUST: The License field in the package spec file must match the actual license. OK MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. OK MUST: The package MUST successfully compile and build into binary rpms. OK MUST: The spec file MUST handle locales properly. N/A MUST: Optflags are used and time stamps preserved. OK MUST: Packages containing shared library files must call ldconfig. OK MUST: A package must own all directories that it creates or require the package that owns the directory. OK MUST: Files only listed once in %files listings. OK MUST: Debuginfo package is complete. OK MUST: Permissions on files must be set properly. OK MUST: Clean section exists. OK MUST: Large documentation files must go in a -doc subpackage. N/A MUST: All relevant items are included in %doc. Items in %doc do not affect runtime of application. - Place the %doc in -libs instead of the main package, as the main package requires the libraries but not vice versa. MUST: Header files must be in a -devel package. OK MUST: Static libraries must be in a -static package. N/A MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. OK MUST: If a package contains library files with a suffix then library files ending in .so must go in a -devel package. OK MUST: In the vast majority of cases, devel packages must require the base package using a fully versioned dependency. OK MUST: Packages does not contain any .la libtool archives. OK MUST: Desktop files are installed properly. N/A MUST: No file conflicts with other packages andno general names. OK MUST: Buildroot cleaned before install. OK SHOULD: %{?dist} tag is used in release. OK SHOULD: If the package does not include license text(s) as separate files from upstream, the packager should query upstream to include it. OK SHOULD: The package builds in mock. OK Fixed; only the 64 bit version has rpath? Needs chrpath to get rid of it. http://www.five-ten-sg.com/mopac7.spec http://www.five-ten-sg.com/mopac7-1.15-7.fc12.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=1889314 Doesn't %configure sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=""|g' libtool sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool do the trick? possibly, but that seems much more fragile than using chrpath. Anyway, it's the recommended way. Chrpath is a last resort and should be treated as such. done. http://www.five-ten-sg.com/mopac7.spec http://www.five-ten-sg.com/mopac7-1.15-8.fc12.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=1898980 All issues have been fixed, the package has been APPROVED New Package CVS Request ======================= Package Name: mopac7 Short Description: Semi-empirical quantum mechanics suite Owners: carllibpst Branches: F-11 F-12 InitialCC: cvs done. mopac7-1.15-8.fc11 has been pushed to the Fedora 11 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 mopac7'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2010-0173 mopac7-1.15-8.fc12 has been pushed to the Fedora 12 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 mopac7'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-0200 mopac7-1.15-8.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report. mopac7-1.15-8.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. |