Spec URL: https://www.cora.nwra.com/~orion/fedora/catalyst.spec SRPM URL: https://www.cora.nwra.com/~orion/fedora/catalyst-2.0-0.1.20201218git2fc94c5.fc34.src.rpm Description: Catalyst is an API specification developed for simulations (and other scientific data producers) to analyze and visualize data in situ. Originally, Catalyst was part of ParaView. However, with ParaView version 5.9, the Catalyst API was revamped quite extensively and release as an independent project. This repository includes the definition together with a lightweight implementation of this revamped Catalyst API. The documentation describing the details, including build instructions and how to use Catalyst for in situ analysis and visualization in simulations is available here: https://catalyst-in-situ.readthedocs.io/en/latest/index.html Fedora Account System Username: orion Scratch build - https://koji.fedoraproject.org/koji/taskinfo?taskID=61063693
> Originally, Catalyst was part of ParaView. However, with ParaView version > 5.9, the Catalyst API was revamped quite extensively and release as an > independent project. This repository includes the definition together with a > lightweight implementation of this revamped Catalyst API. > > The documentation describing the details, including build instructions and > how to use Catalyst for in situ analysis and visualization in simulations is > available here: https://catalyst-in-situ.readthedocs.io/en/latest/index.html "d" is missing. But alltogether, this is not very intesting for users. Maybe: Catalyst has been split out of ParaView. This package includes the definition together with a lightweight implementation of this Catalyst API. For details how to use Catalyst for in situ analysis and visualization in simulations, see https://catalyst-in-situ.readthedocs.io/en/latest/index.html. + pakcage name is OK + license is acceptable for Fedora (BSD 3-clause) + license is specified correctly + builds and installs OK + BR look correct + P:bundled() is used where appropriate + P/R look correct rpmlint: catalyst.src: W: spelling-error Summary(en_US) situ -> sit, suit, site catalyst.src: W: spelling-error %description -l en_US situ -> sit, suit, site catalyst.src: W: strange-permission catalyst-2fc94c55ba4eb994f27728141ebcf15c3435f306.tar.gz 660 catalyst.src: W: strange-permission catalyst-unbundle.patch 660 catalyst.src: W: strange-permission catalyst.spec 660 catalyst.src:24: W: unversioned-explicit-provides bundled(conduit) catalyst.x86_64: W: spelling-error Summary(en_US) situ -> sit, suit, site catalyst.x86_64: W: spelling-error %description -l en_US situ -> sit, suit, site catalyst.x86_64: W: no-documentation catalyst-devel.x86_64: W: no-documentation 5 packages and 0 specfiles checked; 0 errors, 10 warnings. Nothing important here. Package is APPROVED.
Oh, please add %license License.txt 3rdPartyLicenses.txt
Thanks for the review, will make the changes.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/catalyst
Checked in and built.