Bug 1795068 - Review Request: python-diskcache - Disk and file backed persistent cache
Summary: Review Request: python-diskcache - Disk and file backed persistent cache
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Fabio Valentini
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-01-26 21:16 UTC by Fabian Affolter
Modified: 2020-03-27 13:09 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-03-27 08:00:41 UTC
Type: ---
Embargoed:
decathorpe: fedora-review+


Attachments (Terms of Use)

Description Fabian Affolter 2020-01-26 21:16:27 UTC
Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-diskcache.spec
SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/python-diskcache-4.1.0-1.fc31.src.rpm

Project URL: http://www.grantjenks.com/docs/diskcache/

Description:
DiskCache is an Apache 2 licensed disk and file backed cache library,
written in pure-Python, and compatible with Django. The cloud-based
computing of 2019 puts a premium on memory. Gigabytes of empty space 
is left on disks asprocesses vie for memory. Among these processes is
Memcached (and sometimes Redis) which is used as a cache. Wouldn't it 
be nice to leverage.

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=41052840

rpmlint output:
$ rpmlint python-diskcache-4.1.0-1.fc31.src.rpm 
python-diskcache.src: W: spelling-error %description -l en_US asprocesses -> as processes, as-processes, processes
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint python3-diskcache-4.1.0-1.fc31.noarch.rpm 
python3-diskcache.noarch: W: spelling-error %description -l en_US asprocesses -> as processes, as-processes, processes
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Fedora Account System Username: fab

Comment 1 Fabio Valentini 2020-01-27 19:50:16 UTC
I can review this package.

Initial comments / suggestions / recommendations:

1) It looks like you're using pypi source tarball. You should be able to just use "%{pypi_source}" for that.

2) You could deduplicate the %descriptions (like I commented on python-shodan).

3) Please use trailing slashes for directories in %files:

%{python3_sitelib}/%{pypi_name}/
%{python3_sitelib}/%{pypi_name}-%{version}-py%{python3_version}.egg-info/

Other than that, looks good.

Let me know when the package is ready for the formal review.

Comment 2 Fabian Affolter 2020-03-18 10:05:47 UTC
Thanks for your help.

(In reply to Fabio Valentini from comment #1)
> 1) It looks like you're using pypi source tarball. You should be able to
> just use "%{pypi_source}" for that.

Fixed

> 2) You could deduplicate the %descriptions (like I commented on
> python-shodan).

I would like to keep it simple.

> 3) Please use trailing slashes for directories in %files:
> 
> %{python3_sitelib}/%{pypi_name}/
> %{python3_sitelib}/%{pypi_name}-%{version}-py%{python3_version}.egg-info/

Fixed


Updated files:
Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-diskcache.spec
SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/python-diskcache-4.1.0-2.fc31.src.rpm

Comment 3 Fabio Valentini 2020-03-18 11:38:58 UTC
Looks good now. Thanks!
Package approved.

Comment 4 Fabian Affolter 2020-03-18 16:13:26 UTC
Thanks for your support.

Comment 5 Gwyn Ciesla 2020-03-18 16:15:47 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/python-diskcache

Comment 6 Fedora Update System 2020-03-19 10:27:15 UTC
FEDORA-2020-ac573bf385 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-ac573bf385

Comment 7 Fedora Update System 2020-03-20 01:58:45 UTC
python-diskcache-4.1.0-2.fc32 has been pushed to the Fedora 32 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-ac573bf385

Comment 8 Fedora Update System 2020-03-20 03:08:24 UTC
python-diskcache-4.1.0-2.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-d79460fdf7

Comment 9 Fedora Update System 2020-03-20 03:12:23 UTC
python-diskcache-4.1.0-2.fc31 has been pushed to the Fedora 31 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-a1370763de

Comment 10 Fedora Update System 2020-03-27 08:00:41 UTC
FEDORA-2020-ac573bf385 has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 11 Fedora Update System 2020-03-27 10:42:35 UTC
FEDORA-2020-d79460fdf7 has been pushed to the Fedora 30 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 12 Fedora Update System 2020-03-27 13:09:03 UTC
FEDORA-2020-a1370763de has been pushed to the Fedora 31 stable repository.
If problem still persists, please make note of it in this bug report.


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