Bug 2254928

Summary: Review Request: perl-Net-Async-SOCKS - Some degree of SOCKS5 proxy support in IO::Async
Product: [Fedora] Fedora Reporter: Chris Adams <linux>
Component: Package ReviewAssignee: Jitka Plesnikova <jplesnik>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jplesnik, package-review
Target Milestone: ---Flags: jplesnik: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-02-06 18:33:05 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:
Bug Depends On: 2254922    
Bug Blocks: 2254931    

Description Chris Adams 2023-12-17 22:54:46 UTC
Spec URL: https://cmadams.fedorapeople.org/perl-Net-Async-SOCKS/perl-Net-Async-SOCKS.spec
SRPM URL: https://cmadams.fedorapeople.org/perl-Net-Async-SOCKS/perl-Net-Async-SOCKS-0.003-1.fc39.src.rpm
Description:
Currently provides a very basic implementation of SOCKS_connect:

Fedora Account System Username: cmadams

Comment 1 Chris Adams 2023-12-17 22:55:15 UTC
Note: depends on perl-Protocol-SOCKS (review BZ 2254922)

Comment 3 Jitka Plesnikova 2024-01-30 15:06:35 UTC
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

BuildRequires
FIX: Please add build-requires
     perl(IO::Async::Loop) - lib/Net/Async/SOCKS.pm:36
     perl(IO::Async::Stream) - lib/Net/Async/SOCKS.pm:37
     perl(Protocol::SOCKS::Client) - lib/Net/Async/SOCKS.pm:34
     perl(Protocol::SOCKS::Constants) - lib/Net/Async/SOCKS.pm:35
TODO: Remove perl(Test::Refcount), it is not used in code

$ rpm -qp --requires perl-Net-Async-SOCKS-0.003-2.fc40.noarch.rpm | sort | uniq -c | grep -v rpmlib
      1 perl(Carp)
      1 perl(IO::Async::Loop)
      1 perl(IO::Async::Stream)
      1 perl-libs
      1 perl(Protocol::SOCKS::Client)
      1 perl(Protocol::SOCKS::Constants)
      1 perl(strict)
      1 perl(warnings)
Binary requires are Ok.

$ rpm -qp --provides perl-Net-Async-SOCKS-0.003-2.fc40.noarch.rpm | sort | uniq -c
      1 perl(Net::Async::SOCKS) = 0.003
      1 perl-Net-Async-SOCKS = 0.003-2.fc40
Binary provides are Ok.

$ rpmlint ./perl-Net-Async-SOCKS*
================================= rpmlint session starts =========================================
rpmlint: 2.4.0
configuration:
    /usr/lib/python3.11/site-packages/rpmlint/configdefaults.toml
    /etc/xdg/rpmlint/fedora-legacy-licenses.toml
    /etc/xdg/rpmlint/fedora-spdx-licenses.toml
    /etc/xdg/rpmlint/fedora.toml
    /etc/xdg/rpmlint/scoring.toml
    /etc/xdg/rpmlint/users-groups.toml
    /etc/xdg/rpmlint/warn-on-functions.toml
checks: 31, packages: 3
perl-Net-Async-SOCKS.noarch: W: incoherent-version-in-changelog 0.04-2 ['0.003-2.fc40', '0.003-2']
=== 2 packages and 1 specfiles checked; 0 errors, 1 warnings, 0 badness; has taken 0.2 s =========

FIX: Update different version in changelog

TODO: Explicitly name packaged files in %{_mandir}/man* and %{perl_vendorlib} directories
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists.

Otherwise the package is in line with Fedora and Perl packaging guide lines.
Please correct all 'FIX' items, consider fixing 'TODO' item.

Resolution:
Approved

Comment 5 Jitka Plesnikova 2024-02-06 08:15:58 UTC
The update looks good. 
The review was already approved.

Comment 6 Fedora Admin user for bugzilla script actions 2024-02-06 14:09:02 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-Net-Async-SOCKS

Comment 7 Fedora Update System 2024-02-06 14:46:10 UTC
FEDORA-2024-0dfd10281a (perl-Net-Async-SOCKS-0.003-3.fc39) has been submitted as an update to Fedora 39.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-0dfd10281a

Comment 8 Chris Adams 2024-02-06 18:33:05 UTC
Thanks for the review! I focused on the FIX bits and managed to not notice the Approved line. :)

Comment 9 Fedora Update System 2024-02-07 01:29:35 UTC
FEDORA-2024-0dfd10281a has been pushed to the Fedora 39 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-0dfd10281a \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-0dfd10281a

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 10 Fedora Update System 2024-02-15 00:59:34 UTC
FEDORA-2024-0dfd10281a (perl-Net-Async-SOCKS-0.003-3.fc39) has been pushed to the Fedora 39 stable repository.
If problem still persists, please make note of it in this bug report.