https://bugzilla.redhat.com/show_bug.cgi?id=2226668
Bug ID: 2226668 Summary: Review Request: gismo - A C++ library for isogeometric analysis (IGA) Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: topazus@outlook.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://topazus.fedorapeople.org/rpms/gismo.spec SRPM URL: https://topazus.fedorapeople.org/rpms/gismo-21.12.0-1.20230630gitf3e05c3.fc3... Description: G+Smo (Geometry + Simulation Modules, pronounced "gismo") is an open-source C++ library that brings together mathematical tools for geometric design and numerical simulation. It implements the relatively new paradigm of isogeometric analysis, which suggests the use of a unified framework in the design and analysis pipeline. G+Smo is an object-oriented, cross-platform, template C++ library and follows the generic programming principle, with a focus on both efficiency and ease of use. The library aims at providing access to high quality, open-source software to the forming isogeometric numerical simulation community and beyond.
Fedora Account System Username: topazus
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
Felix Wang topazus@outlook.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Doc Type|--- |If docs needed, set a value
--- Comment #1 from Felix Wang topazus@outlook.com --- [fedora-review-service-build]
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #2 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6645833 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please make sure the package builds successfully at least for Fedora Rawhide.
- If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field
--- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service
If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #3 from Felix Wang topazus@outlook.com --- Spec URL: https://topazus.fedorapeople.org/rpms/gismo.spec SRPM URL: https://topazus.fedorapeople.org/rpms/gismo-21.12.0-1.20230630gitf3e05c3.fc3...
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #4 from Felix Wang topazus@outlook.com --- update to 23.09.
Spec URL: https://topazus.fedorapeople.org/rpms/gismo.spec SRPM URL: https://topazus.fedorapeople.org/rpms/gismo-23.09.0-1.fc40.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #5 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6646434 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please make sure the package builds successfully at least for Fedora Rawhide.
- If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field
--- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service
If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #6 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Created attachment 1999708 --> https://bugzilla.redhat.com/attachment.cgi?id=1999708&action=edit The .spec file difference from Copr build 6646434 to 6646435
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #7 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6646435 (succeeded)
Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please take a look if any issues were found.
--- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service
If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
Benson Muite benson_muite@emailplus.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@fedoraproject.org |benson_muite@emailplus.org Flags| |fedora-review? CC| |benson_muite@emailplus.org Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #8 from Benson Muite benson_muite@emailplus.org --- Initial comments: a) Can tests be run? b) Should additional functionality be enabled, for example, Eigen, MPI, UMFPACK, TAUCS, SUPERLU?
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #9 from Benson Muite benson_muite@emailplus.org --- c) Warning when running fedora-review:
[!]: Large data in /usr/share should live in a noarch subpackage if package is arched. Note: Arch-ed rpms have a total of 43612160 bytes in /usr/share gismo- examples-23.09.0-1.fc40.aarch64.rpm:43591680 See:
https://fedoraproject.org/wiki/Packaging:ReviewGuidelines#Package_Review_Gui... Probably want to make a noarch package examples-data which would then be required by the examples package.
Product: Fedora Version: rawhide Component: Package Review
Benson Muite benson_muite@emailplus.org has canceled Package Review package-review@lists.fedoraproject.org's request for Benson Muite benson_muite@emailplus.org's needinfo: Bug 2226668: Review Request: gismo - A C++ library for isogeometric analysis (IGA) https://bugzilla.redhat.com/show_bug.cgi?id=2226668
--- Comment #11 from Benson Muite benson_muite@emailplus.org --- Happy to finish review.
https://bugzilla.redhat.com/show_bug.cgi?id=2226668
Benson Muite benson_muite@emailplus.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(benson_muite@emai | |lplus.org) |
--- Comment #11 from Benson Muite benson_muite@emailplus.org --- Happy to finish review.
package-review@lists.fedoraproject.org