Bug 2416748
| Summary: | Review Request: Silo - Mesh and Field I/O Library and Scientific Database | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Christoph Junghans <junghans> | ||||
| Component: | Package Review | Assignee: | Phil Wyett <philip.wyett> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | rawhide | CC: | package-review, philip.wyett | ||||
| Target Milestone: | --- | Flags: | philip.wyett:
fedora-review+
|
||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| URL: | https://silo.llnl.gov/ | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | --- | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2026-04-12 23:59:56 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: | |||||||
| Attachments: |
|
||||||
|
Description
Christoph Junghans
2025-11-24 13:50:44 UTC
Copr build: https://copr.fedorainfracloud.org/coprs/build/9832877 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2416748-silo/fedora-rawhide-x86_64/09832877-Silo/fedora-review/review.txt Found issues: - A package with this name already exists. Please check https://src.fedoraproject.org/rpms/silo Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names Please know that there can be false-positives. --- 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. Hi, I will have a look at this one. Regards Phil Hi,
This episode will be builds.
Builds
======
* Local x86_64: Fail
Snippet:
-4.12.0/tools/python/pysilo.cpp
In file included from /builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/src/silo/silo_private.h:64,
from /builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pydbfile.h:55,
from /builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.cpp:53:
/builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/redhat-linux-build/include/config.h:79:9: warning: ‘HAVE_INTTYPES_H’ redefined
79 | #define HAVE_INTTYPES_H
| ^~~~~~~~~~~~~~~
In file included from /usr/include/python3.14/pyconfig.h:6,
from /usr/include/python3.14/Python.h:14,
from /builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.cpp:51:
/usr/include/python3.14/pyconfig-64.h:654:9: note: this is the location of the previous definition
654 | #define HAVE_INTTYPES_H 1
| ^~~~~~~~~~~~~~~
In file included from /builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.cpp:54:
/builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.cpp: In function ‘PyObject* PyInit_Silo()’:
/builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.h:85:13: error: either all initializer clauses should be designated or none of them should be
85 | .m_name = name, \
| ^
/builddir/build/BUILD/Silo-4.12.0-build/Silo-4.12.0/tools/python/pysilo.cpp:257:5: note: in expansion of macro ‘PY_SILO_MOD_DEF’
257 | PY_SILO_MOD_DEF(siloModule, "Silo", &SiloMethods[0]);
| ^~~~~~~~~~~~~~~
* COPR: Faill (all)
https://copr.fedorainfracloud.org/coprs/kathenas/generic-fedora-rawhide-all-with-review-not-production/build/10255189/
Regards
Phil
Hi, Please remove the version depends with cmake. 3.12 has been long since superseded. Regards Phil (In reply to Phil Wyett from comment #3) > * COPR: Faill (all) Fixed (In reply to Phil Wyett from comment #4) > Please remove the version depends with cmake. 3.12 has been long since > superseded. Fixed as well. Updates here: Spec URL: https://junghans.fedorapeople.org/Silo.spec SRPM URL: https://junghans.fedorapeople.org/Silo-4.12.0-2.fc45.src.rpm TaskID: https://koji.fedoraproject.org/koji/taskinfo?taskID=144189437 Created attachment 2136389 [details]
The .spec file difference from Copr build 9832877 to 10304077
Copr build: https://copr.fedorainfracloud.org/coprs/build/10304077 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2416748-silo/fedora-rawhide-x86_64/10304077-Silo/fedora-review/review.txt Found issues: - A package with this name already exists. Please check https://src.fedoraproject.org/rpms/silo Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names Please know that there can be false-positives. --- 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. Hi, Sorry for the late reply. An infection in my foot has with the help of high dose antibiotics has helped to the point of decent concentration. You are in the 'packager' group, so I would login to packages is source page below and 'Take' this orphaned package for at least fedora, but I would also hope EPEL. https://src.fedoraproject.org/rpms/silo Builds and checks of the spec file etc. look good. Marking as '+' and many thanks for the great work. This package may now have to pass through the retirement process, link below. https://docs.fedoraproject.org/en-US/package-maintainers/Package_Retirement_Process/ Regards Phil The Pagure repository was created at https://src.fedoraproject.org/rpms/Silo The Pagure repository was created at https://src.fedoraproject.org/rpms/Silo FEDORA-2026-daad06faac (Silo-4.12.0-2.fc45) has been submitted as an update to Fedora 45. https://bodhi.fedoraproject.org/updates/FEDORA-2026-daad06faac FEDORA-2026-daad06faac (Silo-4.12.0-2.fc45) has been pushed to the Fedora 45 stable repository. If problem still persists, please make note of it in this bug report. |