Bug 1431763 - Review Request: golang-github-oschwald-geoip2-golang - GeoIP2 lookup library for Go
Summary: Review Request: golang-github-oschwald-geoip2-golang - GeoIP2 lookup library ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Athos Ribeiro
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1431759
Blocks: 1427634
TreeView+ depends on / blocked
 
Reported: 2017-03-13 17:18 UTC by Fabio Valentini
Modified: 2017-05-04 18:24 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-05-02 23:14:58 UTC
Type: ---
Embargoed:
athoscribeiro: fedora-review+


Attachments (Terms of Use)

Description Fabio Valentini 2017-03-13 17:18:01 UTC
Spec URL: https://decathorpe.fedorapeople.org/packages/golang-github-oschwald-geoip2-golang.spec

SRPM URL: https://decathorpe.fedorapeople.org/packages/golang-github-oschwald-geoip2-golang-1.0.0-1.0.git706a710.fc25.src.rpm

Description: GeoIP2 lookup library for Go

Fedora Account System Username: decathorpe


This package is one of the (indirect) dependencies of syncthing. I can't provide a koji scratch build yet, since it depends on golang-github-oschwald-maxminddb-golang.

Comment 1 Fabio Valentini 2017-03-30 09:45:20 UTC
koji scratch build, as all dependencies are available on rawhide now: https://koji.fedoraproject.org/koji/taskinfo?taskID=18679011

Comment 2 Athos Ribeiro 2017-04-17 14:45:45 UTC
Hi Fabio,

I am taking this one.

- Conditional clauses on line 144 are duplicated.

- The guidelines for post releases have changed [1], in special, the release tag for post release should not be in X.Y format (use just the X) and the date the snapshot was taken must be included there.

- The test data could be included as a separate source file, but that's not a blocker here.

[1] https://fedoraproject.org/wiki/Packaging:Versioning

Comment 3 Fabio Valentini 2017-04-17 15:08:08 UTC
Spec URL: https://decathorpe.fedorapeople.org/packages/golang-github-oschwald-geoip2-golang.spec

SRPM URL: https://decathorpe.fedorapeople.org/packages/golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc26.src.rpm

- Redundant conditional removed
- Release: tag fixed
- updated to newer snapshot

For now I would rather not include another git repository snapshot tarball for the test data. Maybe I'll add it in the future, if there is demand for enabling the tests.

Comment 4 Athos Ribeiro 2017-04-17 15:17:21 UTC
Package looks good now!

Approved

Comment 5 Fabio Valentini 2017-04-17 15:24:09 UTC
Thanks for the review!

Comment 6 Gwyn Ciesla 2017-04-17 16:41:01 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/golang-github-oschwald-geoip2-golang

Comment 7 Fedora Update System 2017-04-17 17:10:36 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-43dcb207fa

Comment 8 Fedora Update System 2017-04-17 17:25:06 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-55388bed37

Comment 9 Fedora Update System 2017-04-17 17:35:00 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6261d56f95

Comment 10 Fedora Update System 2017-04-18 13:22:38 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc25 has been pushed to the Fedora 25 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-2017-55388bed37

Comment 11 Fedora Update System 2017-04-18 17:52:13 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc24 has been pushed to the Fedora 24 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-2017-6261d56f95

Comment 12 Fedora Update System 2017-04-18 19:53:23 UTC
golang-github-oschwald-geoip2-golang-1.0.0-1.20170314.git0fd242d.fc26 has been pushed to the Fedora 26 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-2017-43dcb207fa

Comment 13 Fedora Update System 2017-04-24 09:03:41 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-dd3997794d

Comment 14 Fedora Update System 2017-04-24 09:07:08 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2017-c4a6754314

Comment 15 Fedora Update System 2017-04-25 03:53:38 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc25 has been pushed to the Fedora 25 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-2017-dd3997794d

Comment 16 Fedora Update System 2017-04-25 19:25:23 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc24 has been pushed to the Fedora 24 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-2017-c4a6754314

Comment 17 Fedora Update System 2017-05-02 23:14:58 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2017-05-04 18:24:01 UTC
golang-github-oschwald-geoip2-golang-1.1.0-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, 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.