Bug 550234
Summary: | Review Request: tnt - C++ templates for scientific computing | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Matt Chan <talcite> |
Component: | Package Review | Assignee: | Peter Lemenkov <lemenkov> |
Status: | CLOSED DEFERRED | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, i, lemenkov, notting, susi.lehtola |
Target Milestone: | --- | Flags: | lemenkov:
fedora-review+
dennis: 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: | 2010-10-28 07:55:37 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: | 201449, 518949 |
Description
Matt Chan
2009-12-24 04:56:26 UTC
Matt, I don't sse you in the list of those persons, who requested membership in 'Fedora Packager CVS Commit Group'. Please, go to FAS and apply for membership here. Thanks for the heads up Peter. I've applied for the packager group now. Matt Unblocking FE-NEEDSPONSOR since I just sponsored Matt. I'll review it - The name of this package should be 'tnt', not 'tnt-devel'. - Srap the BuildRequires and Requires lines, they are empty. - Instead of cp -a %{_builddir}/tnt/*.h %{buildroot}%{_includedir}/ you can just use cp -a *.h %{buildroot}%{_includedir}/ because you are in the builddir already. I've implemented the last 2 changes but I've held back releasing the spec because the 1st is giving me trouble. The project consists only of header files. Should it still be named 'tnt' and not 'tnt-devel'? When I name the package 'tnt', rpmlint complains about having headers in a non-devel package. Matt What you need to do is create a -devel subpackage. For example, see the GSL spec file at http://cvs.fedoraproject.org/viewvc/rpms/gsl/devel/gsl.spec on how to do this. The trick is, that you do not proclaim a %files section for the main package. That way the main rpm won't be created. However, if in the future the project implements a runtime library, then we will have the opportunity to expand the package in Fedora. Ah. I did not know you could create a devel subpackage by itself. Here are the new URLs with the changes implemented: SPEC: http://mattchan.homelinux.net:55555/tnt/tnt.spec SRPM: http://mattchan.homelinux.net:55555/tnt/tnt-3.0.11-0.2.beta.fc12.src.rpm Thanks for the feedback. Matt REVIEW: + rpmlint is not silent, but this the only warning may be safely ignored: [petro@Sulaco SPECS]$ rpmlint ../RPMS/noarch/tnt-devel-3.0.11-0.2.beta.fc12.noarch.rpm tnt-devel.noarch: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [petro@Sulaco SPECS]$ + The package is named according to the Package Naming Guidelines. + The spec file name matches the base package %{name}, in the format %{name}.spec. + The package meets the Packaging Guidelines. + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The License field in the package spec file matches the actual license. 0 Upstream doesn't provide the file file, containing the text of the license(s) for the package. + The spec file is written in American English. + The spec file for the package is legible. + The sources used to build the package, match the upstream source, as provided in the spec URL. [petro@Sulaco SOURCES]$ sha256sum tnt_3_0_11.zip* f0779ef0090df4e5143d4cfcdc2b8c9a9f1abb6befcc97c0969b6e7f134b5dd1 tnt_3_0_11.zip f0779ef0090df4e5143d4cfcdc2b8c9a9f1abb6befcc97c0969b6e7f134b5dd1 tnt_3_0_11.zip.1 [petro@Sulaco SOURCES]$ + The package successfully compiles and builds into binary rpms on at least one primary architecture. + All build dependencies are listed in BuildRequires. 0 No need to handle locales. 0 No shared library files. + The package does NOT bundle copies of system libraries. + The package is not designed to be relocatable. + The package owns all directories that it creates. + The package does not list a file more than once in the spec file's %files listings. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + The package consistently uses macros. + The package contains code, or permissible content. 0 No extremely large documentation files. 0 The package doesn't contain %doc files at all. + C header files packaged in devel sub-package. 0 No static libraries. 0 No pkgconfig(.pc) files. 0 The package doesn't contain library files with a suffix (e.g. libfoo.so.1.1). 0 No devel sub-package. + The package does NOT contain any .la libtool archives. 0 Not a GUI application. + The package does not own files or directories already owned by other packages. + At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + All filenames in rpm packages are valid UTF-8. APPROVED. sed s/"0 No devel sub-package."// Ping, Matt! Sorry for the lack of updates. I've been waiting on an upstream takeover of the STEP/UTAH components by the BRL-CAD project. I'll push them a bit and see if we can get things moving along. Are there any other actions to take on these two packages, or are they ready to be included into Fedora? Matt (In reply to comment #12) > Sorry for the lack of updates. I've been waiting on an upstream takeover of the > STEP/UTAH components by the BRL-CAD project. I'll push them a bit and see if we > can get things moving along. > > Are there any other actions to take on these two packages, or are they ready to > be included into Fedora? > > Matt Sorry for the delay - I completely overlooked your comment. What packages are you talking about? Anyway, this partucular package is (from the packaging view) is in a very good shape, so there is no need to wait for other packages before submitting this one. Ok. Are there any steps we need to take in order to submit them, or is it an automatic process as soon as the review flags are cleared? Matt Sorry, I neglected to mention that the packages I was referring to in comment 12 are: - NIST STEP Class Library - UTAH raster toolkit These two projects are bundled with BRLCAD but have had source changes and the upstream is dead. The maintainers of the BRLCAD project are in the process of taking over maintainership for them. I didn't mention it in comment 12, but TkHtml3 and OpenNURBS also need to be separated out from BRLCAD. There's been a bit of progress which I'll post back to the main BRLCAD bug. Matt (In reply to comment #14) > Ok. Are there any steps we need to take in order to submit them, or is it an > automatic process as soon as the review flags are cleared? You must fill separate review request for each package, if I understood you correctly (English is not my native language, as you may already noticed). Anyway, there is no need to wait for them - you should move further with this particular package. if you just don't know what to do next here, then you can refer to this link: https://fedoraproject.org/wiki/PackageMaintainers/Join#Add_Package_to_CVS_and_Set_Owner New Package CVS Request ======================= Package Name: tnt Short Description: C++ templates for scientific computing Owners: talcite Branches: F-12 F-13 EL-5 InitialCC: CVS Done, Does this need a EL-6 branch? Yes, please include an EL-6 branch Package Change Request ====================== Package Name: tnt New Branches: EL-6 Owners: talcite CVS Done Ping, Matt. Ping, again. Last ping. I'll close this ticket next week if there will be no answer. Hi Peter, Sorry I haven't been very responsive on this in awhile. School has been taking up a lot of time lately and I haven't been able to balance everything in life. I also don't know how the progress is going in the BRL-CAD project on re-factoring the code to be consistent with fedora guidelines. To my knowledge, the upstream for STEP and UTAH still have not been taken over. It may be best to close this ticket for awhile. I will be graduating in April and may have more time in the future to come back to this. There may also be new developments in the project by that time. Matt (In reply to Matt Chan from comment #24) > Hi Peter, > > Sorry I haven't been very responsive on this in awhile. > > School has been taking up a lot of time lately and I haven't been able to > balance everything in life. I also don't know how the progress is going in > the BRL-CAD project on re-factoring the code to be consistent with fedora > guidelines. To my knowledge, the upstream for STEP and UTAH still have not > been taken over. > > It may be best to close this ticket for awhile. I will be graduating in > April and may have more time in the future to come back to this. There may > also be new developments in the project by that time. > > Matt ping after 3 months about tntv3. |