Bug 1996798 - Review Request: ghc-cached-json-file - Locally cache a json file obtained by http
Summary: Review Request: ghc-cached-json-file - Locally cache a json file obtained by ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-08-23 17:26 UTC by Jens Petersen
Modified: 2023-11-02 06:06 UTC (History)
2 users (show)

Fixed In Version: ghc-cached-json-file-0.1.1-14.fc39
Clone Of:
Environment:
Last Closed: 2023-11-02 06:06:18 UTC
Type: ---
Embargoed:
spotrh: fedora-review+


Attachments (Terms of Use)

Description Jens Petersen 2021-08-23 17:26:50 UTC
Spec URL: https://petersen.fedorapeople.org/reviews/ghc-cached-json-file/ghc-cached-json-file.spec
SRPM URL: https://petersen.fedorapeople.org/reviews/ghc-cached-json-file/ghc-cached-json-file-0.1.0-1.fc35.src.rpm

Description:
A small library for caching a slow-changing remote json file in a specified
directory (under "~/.cache/"). When the json is requested and the cache file is
still fresh enough it is read directly from the local cache, otherwise the
cached file is refreshed first.


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

Comment 2 Package Review 2022-11-20 00:45:23 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 4 Jens Petersen 2023-02-02 02:37:38 UTC
This library is used by fbrnch via ghc-fedora-dists

Comment 6 Tom "spot" Callaway 2023-08-04 12:48:12 UTC
APPROVED (generated packages are so simple)

Good:

- rpmlint checks return:
ghc-cached-json-file-devel.x86_64: E: static-library-without-debuginfo /usr/lib64/ghc-9.2.6/lib/cached-json-file-0.1.1/libHScached-json-file-0.1.1-9nkhzHX7d8f8HC0nyuYTuZ.a
ghc-cached-json-file-prof.x86_64: E: static-library-without-debuginfo /usr/lib64/ghc-9.2.6/lib/cached-json-file-0.1.1/libHScached-json-file-0.1.1-9nkhzHX7d8f8HC0nyuYTuZ_p.a
ghc-cached-json-file.x86_64: W: no-documentation
ghc-cached-json-file-prof.x86_64: W: no-documentation
ghc-cached-json-file-prof.x86_64: W: devel-file-in-non-devel-package /usr/lib64/ghc-9.2.6/lib/cached-json-file-0.1.1/libHScached-json-file-0.1.1-9nkhzHX7d8f8HC0nyuYTuZ_p.a

All safe to ignore for Haskell packages.

- package meets naming guidelines
- package meets packaging guidelines
- license (BSD-3-Clause) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- doc subpackage okay
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files
- devel requires base package n-v-r

Comment 7 Jens Petersen 2023-08-04 12:53:47 UTC
Thank you very much 🙏

Comment 8 Fedora Admin user for bugzilla script actions 2023-08-04 12:57:39 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/ghc-cached-json-file


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