Bug 2416748 - Review Request: Silo - Mesh and Field I/O Library and Scientific Database
Summary: Review Request: Silo - Mesh and Field I/O Library and Scientific Database
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Phil Wyett
QA Contact: Fedora Extras Quality Assurance
URL: https://silo.llnl.gov/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-11-24 13:50 UTC by Christoph Junghans
Modified: 2026-04-12 23:59 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2026-04-12 23:59:56 UTC
Type: ---
Embargoed:
philip.wyett: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 9832877 to 10304077 (1.02 KB, patch)
2026-04-09 07:37 UTC, Fedora Review Service
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Github llnl Silo issues 525 0 None open Build fails with python3.14 2026-03-24 20:31:26 UTC

Description Christoph Junghans 2025-11-24 13:50:44 UTC
Spec URL: https://junghans.fedorapeople.org/Silo.spec
SRPM URL: https://junghans.fedorapeople.org/Silo-4.12.0-1.fc44.src.rpm
TaskID: https://koji.fedoraproject.org/koji/taskinfo?taskID=139232429
Description: Silo is a C/Fortran API for reading and writing a wide variety of scientific data to binary files. Silo files can be easily shared and exchanged between wholly independently developed applications running on disparate computing platforms. Consequently, Silo facilitates the development of general purpose tools for processing scientific data. One of the more popular tools to process Silo data is the VisIt visualization tool. Silo supports a variety of mesh types including simple curves, gridless (point), structured, unstructured-zoo and unstructured-arbitrary, block structured AMR, and constructive solid geometry (CSG).
Fedora Account System Username: junghans

Comment 1 Fedora Review Service 2025-11-24 14:00:29 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.

Comment 2 Phil Wyett 2026-03-23 19:13:30 UTC
Hi,

I will have a look at this one.

Regards

Phil

Comment 3 Phil Wyett 2026-03-23 19:25:39 UTC
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

Comment 4 Phil Wyett 2026-03-23 19:26:28 UTC
Hi,

Please remove the version depends with cmake. 3.12 has been long since superseded.

Regards

Phil

Comment 5 Christoph Junghans 2026-04-08 13:27:50 UTC
(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

Comment 6 Fedora Review Service 2026-04-09 07:37:28 UTC
Created attachment 2136389 [details]
The .spec file difference from Copr build 9832877 to 10304077

Comment 7 Fedora Review Service 2026-04-09 07:37:30 UTC
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.

Comment 8 Phil Wyett 2026-04-11 23:08:29 UTC
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

Comment 9 Fedora Admin user for bugzilla script actions 2026-04-12 23:17:33 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/Silo

Comment 10 Fedora Admin user for bugzilla script actions 2026-04-12 23:18:31 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/Silo

Comment 11 Fedora Update System 2026-04-12 23:56:26 UTC
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

Comment 12 Fedora Update System 2026-04-12 23:59:56 UTC
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.


Note You need to log in before you can comment on or make changes to this bug.