Bug 2367134 - Review Request: dfx-mgr - A tool for managing programable logic accelerators
Summary: Review Request: dfx-mgr - A tool for managing programable logic accelerators
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Javier Martinez Canillas
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/Xilinx/dfx-mgr
Whiteboard:
Depends On: 2367133
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-05-18 21:40 UTC by Peter Robinson
Modified: 2025-12-01 21:11 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-12-01 21:11:12 UTC
Type: ---
Embargoed:
javierm: fedora-review+


Attachments (Terms of Use)

Description Peter Robinson 2025-05-18 21:40:43 UTC
SPEC: https://pbrobinson.fedorapeople.org/dfx-mgr.spec
SRPM: https://pbrobinson.fedorapeople.org/dfx-mgr-2024.2-1.fc42.src.rpm

Description:
DFX-MGR provides infrastructure to abstract configuration and
hardware resource management for dynamic deployment of Xilinx
based accelerators (AKA FPGA) across different platforms.

FAS: pbrobinson


Reproducible: Always

Comment 1 Fedora Review Service 2025-11-20 15:36:55 UTC
Hello,
I do realize that this is possibly an old ticket. I am sorry that it hasn't been
reviewed yet. Let me trigger the Fedora Review Service to see if the package
builds successfully. Hopefully, a green check mark will attract some reviewer.

If I am resurrecting an old ticket that you are not interested in anymore, my
apologies, feel free to close it.

[fedora-review-service-build]

Comment 2 Fedora Review Service 2025-11-20 15:39:57 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9819860
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2367134-dfx-mgr/fedora-rawhide-x86_64/09819860-dfx-mgr/builder-live.log.gz

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.

Comment 3 Javier Martinez Canillas 2025-12-01 08:35:26 UTC
fedora-review -b 2367133
INFO: Processing bugzilla bug: 2367133
INFO: Getting .spec and .srpm Urls from : 2367133
INFO:   --> SRPM url: https://pbrobinson.fedorapeople.org/libdfx-2024.2.0-2.fc42.src.rpm
INFO:   --> Spec url: https://pbrobinson.fedorapeople.org/libdfx.spec
INFO: Using review directory: /home/javier/devel/packages/2367133-libdfx
INFO: Downloading .spec and .srpm files
INFO: Downloading (Source0): https://github.com/Xilinx/libdfx/archive/v2024.2.0/libdfx-2024.2.0.tar.gz
WARNING: Cannot download url: https://github.com/Xilinx/libdfx/archive/v2024.2.0/libdfx-2024.2.0.tar.gz
INFO: No upstream for (Source0): libdfx-2024.2.0.tar.gz

The correct URL seems to be https://github.com/Xilinx/libdfx/archive/refs/tags/xilinx_v2024.2.tar.gz

Comment 4 Javier Martinez Canillas 2025-12-01 08:37:11 UTC
Other than that, the package looks good to me.

Comment 5 Javier Martinez Canillas 2025-12-01 08:40:01 UTC
The last two comments were meant for bug #2367133 (libdfx - the dependency of this package). Sorry about that.

Comment 6 Javier Martinez Canillas 2025-12-01 08:54:44 UTC
The tarball URL for this package has changed too. Now is https://github.com/Xilinx/dfx-mgr/archive/refs/tags/xilinx_v2024.2.tar.gz

Comment 7 Javier Martinez Canillas 2025-12-01 09:05:41 UTC
Setting fedora-review+, please fix the source URL before committing the changes.

Comment 8 Fedora Admin user for bugzilla script actions 2025-12-01 09:25:46 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/dfx-mgr

Comment 9 Peter Robinson 2025-12-01 21:11:12 UTC
URL fixed, and updated to latest release.

Thanks!


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