Bug 2323713 - Review Request: modpoll - A command line tool for Modbus and MQTT
Summary: Review Request: modpoll - A command line tool for Modbus and MQTT
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/gavinying/modpoll
Whiteboard:
Depends On:
Blocks: IoT
TreeView+ depends on / blocked
 
Reported: 2024-11-04 19:35 UTC by Peter Robinson
Modified: 2025-12-07 11:22 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)
The .spec file difference from Copr build 9769597 to 9775466 (832 bytes, patch)
2025-11-07 04:19 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 9775466 to 9883282 (733 bytes, patch)
2025-12-07 11:06 UTC, Fedora Review Service
no flags Details | Diff

Description Peter Robinson 2024-11-04 19:35:03 UTC
SPEC: https://pbrobinson.fedorapeople.org/modpoll.spec
SRPM: https://pbrobinson.fedorapeople.org/modpoll-1.3.3-1.fc41.src.rpm

Description:
A command line tool for Modbus and MQTT

FAS: pbrobinson

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=125512021

Comment 1 Fedora Review Service 2024-11-04 19:37:05 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8209906
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2323713-modpoll/fedora-rawhide-x86_64/08209906-modpoll/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 2 Package Review 2025-11-05 00:45:33 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the
NEEDINFO flag.

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group.

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.

Comment 3 Peter Robinson 2025-11-05 09:46:22 UTC
yes

Comment 4 Benson Muite 2025-11-05 13:55:20 UTC
[fedora-review-service-build]

Comment 5 Fedora Review Service 2025-11-05 13:57:01 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9769597
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2323713-modpoll/fedora-rawhide-x86_64/09769597-modpoll/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 6 Benson Muite 2025-11-05 14:01:41 UTC
Can you update to the latest release 1.5.1?

Pymodbus in Fedora is 3.9.2:
https://src.fedoraproject.org/rpms/pymodbus

The Package seems to need 3.6.9:
https://github.com/gavinying/modpoll/blob/master/pyproject.toml#L13

Maybe it would work if the dependency restriction is relaxed?

Comment 7 Peter Robinson 2025-11-06 09:43:20 UTC
Yes, when I submitted this is was the right version, I updated all that in my local/copr build but given no one had picked this up in a year I'd not updated it.

SPEC: https://pbrobinson.fedorapeople.org/modpoll.spec
SRPM: https://pbrobinson.fedorapeople.org/modpoll-1.5.1-1.fc43.src.rpm

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=138771786

Comment 8 Fedora Review Service 2025-11-07 04:19:22 UTC
Created attachment 2113148 [details]
The .spec file difference from Copr build 9769597 to 9775466

Comment 9 Fedora Review Service 2025-11-07 04:19:24 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9775466
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2323713-modpoll/fedora-rawhide-x86_64/09775466-modpoll/fedora-review/review.txt

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.

Comment 11 Fedora Review Service 2025-12-07 11:06:22 UTC
Created attachment 2117790 [details]
The .spec file difference from Copr build 9775466 to 9883282

Comment 12 Fedora Review Service 2025-12-07 11:06:24 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9883282
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2323713-modpoll/fedora-rawhide-x86_64/09883282-modpoll/fedora-review/review.txt

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.

Comment 13 Peter Robinson 2025-12-07 11:22:54 UTC
Fixed the missing v in the source URL too.


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