Page MenuHomeFreeBSD

[NEW-PORT] comms/py-ModbusTCP: Simple Modbus/TCP client for python
ClosedPublic

Authored by kiwi on Jan 10 2025, 3:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 14, 2:47 PM
Unknown Object (File)
Sun, Oct 12, 9:53 AM
Unknown Object (File)
Fri, Sep 26, 3:33 AM
Unknown Object (File)
Sat, Sep 20, 1:59 AM
Unknown Object (File)
Tue, Sep 16, 9:19 PM
Unknown Object (File)
Tue, Sep 16, 10:40 AM
Unknown Object (File)
Sep 15 2025, 4:48 AM
Unknown Object (File)
Sep 14 2025, 1:34 AM
Subscribers
None

Details

Summary
comms/py-ModbusTCP: Simple Modbus/TCP client for python

This port is a simple modbus TCP client for python.
A pure Python code without any extension or external module dependency

Approved by:    0mp (mentor)
Test Plan

Portlinted and portclippied.
Tested with poudriere on 14.2-RELEASE (and used on some of my projects)

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kiwi requested review of this revision.Jan 10 2025, 3:25 PM
kiwi created this revision.

Use distutils instead of pep517

0mp requested changes to this revision.Jan 13 2025, 2:26 PM
0mp added inline comments.
comms/py-ModbusTCP/Makefile
2

Empty line

comms/py-ModbusTCP/pkg-descr
2

Usually, it should be a paragraph of at least 2 sentences. Is there anything you could add here?

This revision now requires changes to proceed.Jan 13 2025, 2:26 PM
kiwi marked 2 inline comments as done.
kiwi edited the summary of this revision. (Show Details)

Updated pkg-descr for a better description, removed unused line in Makefile. Fixed the summary.

0mp requested changes to this revision.Jan 13 2025, 3:42 PM
0mp added inline comments.
comms/py-ModbusTCP/Makefile
3

Please use DISTVERSION by default. Only use PORTVERSION if necessary.

This revision now requires changes to proceed.Jan 13 2025, 3:42 PM

PORTVERSION -> DISTVERSION

kiwi marked an inline comment as done.Jan 13 2025, 3:51 PM

Done suggested changes !

Seems good to go! Thank you

This revision is now accepted and ready to land.Jan 14 2025, 1:28 PM