Bug 256268 - [NEW PORT] sysutils/py-concurrent-log-handler: RotatingFileHandler replacement with concurrency, gzip
Summary: [NEW PORT] sysutils/py-concurrent-log-handler: RotatingFileHandler replacemen...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Lewis Cook
URL: https://pypi.org/project/concurrent-l...
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-30 17:30 UTC by risner
Modified: 2021-12-21 16:59 UTC (History)
3 users (show)

See Also:
koobs: maintainer-feedback+


Attachments
git staged diff (1.52 KB, patch)
2021-05-30 17:30 UTC, risner
no flags Details | Diff
Updated file (1.50 KB, patch)
2021-05-31 14:24 UTC, risner
no flags Details | Diff
Final staged git diff. (1.50 KB, patch)
2021-05-31 18:35 UTC, risner
no flags Details | Diff
FIxed issues (1.62 KB, patch)
2021-06-02 01:20 UTC, risner
no flags Details | Diff
Typo in required versions. Fixed. (1.63 KB, patch)
2021-06-02 01:22 UTC, risner
no flags Details | Diff
Noticed I never updated to include the category in the patch as requested. (2.12 KB, patch)
2021-12-04 13:04 UTC, risner
risner: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description risner 2021-05-30 17:30:49 UTC
Created attachment 225380 [details]
git staged diff
Comment 1 risner 2021-05-31 14:24:14 UTC
Created attachment 225404 [details]
Updated file

Ran through portlint and resolved issues.
New git staged diff.
Comment 2 risner 2021-05-31 18:35:39 UTC
Created attachment 225422 [details]
Final staged git diff.

More cleanup and fixed incorrect license.
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2021-06-02 00:01:14 UTC
Review items:

  - Add LICENSE_FILE when a license file file is provided in the distribution file(s).
  - setup.py declares dependencies not references in port:

  install_requires = [
    'portalocker<=1.7.1; python_version < "3"',
    'portalocker>=1.4.0; python_version >= "3"',
  ]
Comment 4 risner 2021-06-02 01:20:13 UTC
Created attachment 225486 [details]
FIxed issues

Obsoleted the old patch, updated with the license file and the missing dep.

Thanks
Comment 5 risner 2021-06-02 01:22:15 UTC
Created attachment 225487 [details]
Typo in required versions. Fixed.

Take 2
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2021-06-02 02:34:55 UTC
Only other thing (for future reference), is when producing patches, generate them against the ports root, so the category/path is available
Comment 7 risner 2021-06-02 12:37:43 UTC
Acknowledged. Thanks.
Comment 8 risner 2021-12-04 13:04:01 UTC
Created attachment 229891 [details]
Noticed I never updated to include the category in the patch as requested.
Comment 9 commit-hook freebsd_committer freebsd_triage 2021-12-21 16:56:22 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=40131f2c50e4af829e15c6a2aca0d07445bea2b0

commit 40131f2c50e4af829e15c6a2aca0d07445bea2b0
Author:     Lewis Cook <lcook@FreeBSD.org>
AuthorDate: 2021-12-21 16:49:39 +0000
Commit:     Lewis Cook <lcook@FreeBSD.org>
CommitDate: 2021-12-21 16:54:57 +0000

    sysutils/py-concurrent-log-handler: Add new port

    PR:     256268

 sysutils/Makefile                                  |  1 +
 sysutils/py-concurrent-log-handler/Makefile (new)  | 18 ++++++++++++++++++
 sysutils/py-concurrent-log-handler/distinfo (new)  |  3 +++
 sysutils/py-concurrent-log-handler/pkg-descr (new) | 11 +++++++++++
 4 files changed, 33 insertions(+)
Comment 10 Lewis Cook freebsd_committer freebsd_triage 2021-12-21 16:59:16 UTC
Committed with minor changes (replaced `PORTVERSION` with `DISTVERSION` [1]), thanks!

[1] https://docs.freebsd.org/en/books/porters-handbook/makefiles/#makefile-versions