Bug 1018540 - Review Request: golang-github-goraft-raft - A Go implementation of the Raft distributed consensus protocol
Summary: Review Request: golang-github-goraft-raft - A Go implementation of the Raft d...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Matthew Miller
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1018057
Blocks: 1018479
TreeView+ depends on / blocked
 
Reported: 2013-10-13 03:27 UTC by Lokesh Mandvekar
Modified: 2014-07-23 10:05 UTC (History)
9 users (show)

Fixed In Version: golang-github-goraft-raft-0-0.3.gite138889.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-29 03:48:25 UTC
Type: ---
Embargoed:
mattdm: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Lokesh Mandvekar 2013-10-13 03:27:19 UTC
Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-goraft-raft.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-goraft-raft-0-0.1.gite138889.fc21.src.rpm

Description: A Go implementation of the Raft distributed consensus protocol
Fedora Account System Username: lsm5

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6054102

$ rpmlint SPECS/golang-github-goraft-raft.spec SRPMS/golang-github-goraft-raft-0-0.1.gite138889.fc21.src.rpm RPMS/noarch/golang-github-goraft-raft-devel-0-0.1.gite138889.fc21.noarch.rpm
SPECS/golang-github-goraft-raft.spec:35: W: setup-not-quiet
golang-github-goraft-raft.src:35: W: setup-not-quiet
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

Comment 1 Lokesh Mandvekar 2013-10-13 03:28:30 UTC
packaging the upstream instead of the coreos fork

Comment 2 Matthew Miller 2013-10-14 15:17:57 UTC
Should have

BuildRequires: golang-googlecode-goprotobuf-devel

Comment 3 Matthew Miller 2013-10-14 16:41:29 UTC
(Or the equivalent  golang("[import path]") virtual provides.)

Comment 4 Matthew Miller 2013-10-14 17:34:55 UTC
Actually should be Requires, not BuildRequires.

Comment 5 Matthew Miller 2013-10-15 20:26:20 UTC
So, the devel package should have

Requires: golang("code.google.com/p/goprotobuf/proto")



Eventually, we'll make RPM generate these automatically.

Comment 6 Lokesh Mandvekar 2013-10-16 01:29:48 UTC
%changelog
* Tue Oct 15 2013 Lokesh Mandvekar <lsm5>  0-0.2.gite138889
- requires goprotobuf

Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-goraft-raft.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-goraft-raft-0-0.2.gite138889.fc21.src.rpm

Comment 7 Matthew Miller 2013-10-16 01:51:02 UTC
Review passed, with the caveat that it may need to be updated as the Go packaging guidelines progress beyond draft form.

Also, it wouldn't hurt to have a more descriptive description. But not a big deal for the review.

%description
This is a Go implementation of the Raft distributed consensus protocol.
Raft is a protocol by which a cluster of nodes can maintain a replicated state machine. The state machine is kept in sync through the use of a replicated log.

Comment 8 Lokesh Mandvekar 2013-10-18 19:47:00 UTC
New Package SCM Request
=======================
Package Name: golang-github-goraft-raft
Short Description: A Go implementation of the Raft distributed consensus protocol
Owners: lsm5
Branches: f19 f20 el6
InitialCC:

Comment 9 Gwyn Ciesla 2013-10-19 21:15:00 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2013-10-20 05:20:55 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.fc20

Comment 11 Fedora Update System 2013-10-20 05:26:03 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.fc19

Comment 12 Fedora Update System 2013-10-20 05:31:10 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.el6

Comment 13 Fedora Update System 2013-10-20 16:19:37 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 14 Peter Lemenkov 2013-10-23 11:07:23 UTC
Package Change Request
======================
Package Name: golang-github-goraft-raft
InitialCC: golang-sig

Comment 15 Gwyn Ciesla 2013-10-23 14:12:47 UTC
Done.

Comment 16 Fedora Update System 2013-10-29 03:48:25 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc19 has been pushed to the Fedora 19 stable repository.

Comment 17 Fedora Update System 2013-11-04 17:35:31 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 18 Fedora Update System 2013-11-10 07:37:49 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc20 has been pushed to the Fedora 20 stable repository.

Comment 19 Troy Dawson 2014-07-22 20:17:31 UTC
Package Change Request
======================
Package Name: golang-github-goraft-raft
New Branches: epel7
Owners: tdawson maxamillion

Comment 20 Gwyn Ciesla 2014-07-23 10:05:30 UTC
Git done (by process-git-requests).


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