Bug 1033797 - Review Request: golang-labix-mgo - The mgo rich MongoDB driver for Go
Summary: Review Request: golang-labix-mgo - The mgo rich MongoDB driver for Go
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lokesh Mandvekar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-22 23:27 UTC by Vincent Batts
Modified: 2015-08-17 12:48 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-17 12:48:38 UTC
Type: ---
Embargoed:
lsm5: fedora-review+


Attachments (Terms of Use)
script to pack the tagged snapshot of the source (517 bytes, application/x-shellscript)
2014-01-06 16:52 UTC, Vincent Batts
no flags Details

Description Vincent Batts 2013-11-22 23:27:08 UTC
Spec URL: http://vbatts.fedorapeople.org/rpmbuild/SPECS/golang-labix-mgo.spec
SRPM URL: http://vbatts.fedorapeople.org/rpmbuild/SRPMS/golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm
Description: The mgo rich MongoDB driver for Go
Fedora Account System Username: vbatts

$ rpmlint golang-labix-mgo.spec
golang-labix-mgo.spec:41: W: setup-not-quiet
golang-labix-mgo.spec: W: no-cleaning-of-buildroot %install
golang-labix-mgo.spec: W: no-cleaning-of-buildroot %clean
golang-labix-mgo.spec: W: no-buildroot-tag
golang-labix-mgo.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 5 warnings.

$ fedpkg scratch-build --srpm /home/vbatts/rpmbuild/SRPMS/golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm
Exception AttributeError: '_read_only' in <bound method write.__del__ of <git.config.write object at 0x1d78e50>> ignored
[====================================] 100% 00:00:00  57.36 KiB  95.22 KiB/sec
Building golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm for rawhide
Created task: 6215573
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=6215573
Watching tasks (this may be safely interrupted)...
6215573 build (rawhide, golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm): open (buildvm-21.phx2.fedoraproject.org)
  6215574 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, armv7hl): free
  6215575 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, x86_64): free
  6215576 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, i686): free
  6215574 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, armv7hl): free -> open (arm02-builder02.arm.fedoraproject.org)
  6215575 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, x86_64): free -> open (buildvm-04.phx2.fedoraproject.org)
  6215576 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, i686): free -> open (buildvm-01.phx2.fedoraproject.org)
  6215576 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, i686): open (buildvm-01.phx2.fedoraproject.org) -> closed
  0 free  3 open  1 done  0 failed
  6215575 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, x86_64): open (buildvm-04.phx2.fedoraproject.org) -> closed
  0 free  2 open  2 done  0 failed
  6215574 buildArch (golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm, armv7hl): open (arm02-builder02.arm.fedoraproject.org) -> closed
  0 free  1 open  3 done  0 failed
6215573 build (rawhide, golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm): open (buildvm-21.phx2.fedoraproject.org) -> closed
  0 free  0 open  4 done  0 failed

6215573 build (rawhide, golang-launchpad-goyaml-0-0.1.bzr50.el6.src.rpm) completed successfully

Comment 1 Vincent Batts 2013-11-22 23:30:27 UTC
sorry, pasted the wrong scratch build info


$ fedpkg scratch-build --srpm /home/vbatts/rpmbuild/SRPMS/golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm
Exception AttributeError: '_read_only' in <bound method write.__del__ of <git.config.write object at 0x2fede50>> ignored
[====================================] 100% 00:00:00 117.46 KiB 152.84 KiB/sec
Building golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm for rawhide
Created task: 6215585
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=6215585
Watching tasks (this may be safely interrupted)...
6215585 build (rawhide, golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm): open (buildvm-15.phx2.fedoraproject.org)
  6215586 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, armv7hl): free
  6215587 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, x86_64): free
  6215588 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, i686): free
  6215586 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, armv7hl): free -> open (arm02-builder05.arm.fedoraproject.org)
  6215587 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, x86_64): free -> open (buildvm-18.phx2.fedoraproject.org)
  6215588 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, i686): free -> open (buildvm-23.phx2.fedoraproject.org)
  6215587 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, x86_64): open (buildvm-18.phx2.fedoraproject.org) -> closed
  0 free  3 open  1 done  0 failed
  6215588 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, i686): open (buildvm-23.phx2.fedoraproject.org) -> closed
  0 free  2 open  2 done  0 failed
  6215586 buildArch (golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm, armv7hl): open (arm02-builder05.arm.fedoraproject.org) -> closed
  0 free  1 open  3 done  0 failed
6215585 build (rawhide, golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm): open (buildvm-15.phx2.fedoraproject.org) -> closed
  0 free  0 open  4 done  0 failed

6215585 build (rawhide, golang-labix-mgo-2013.11.18-0.1.bzr252.el6.src.rpm) completed successfully

Comment 2 Lokesh Mandvekar 2014-01-03 18:24:12 UTC
In this case since we have an upstream version number, the %{release} tag should be 1%{dist}.

Also, since upstream doesn't maintain tarballs, it's perhaps preferable to include in comments the steps to generate the tarball.

Comment 3 Lokesh Mandvekar 2014-01-03 19:32:59 UTC
The version number here is probably more like a snapshot release timestamp.

In that case the timestamp is probably better off in %{release} as per this: 
https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages

Comments?

Perhaps others wanna chime in on this. Here's how upstream names it: http://labix.org/mgo

Comment 4 Vincent Batts 2014-01-06 16:51:29 UTC
Since their "releases" is just a snapshot, and they do not even provide a tarball, I would like to keep the revno in the release.

I've attached a script to pack the source. I'll include this script in the package's git repo. Here is the spec, with cleaned source/revno (252 was the wrong revno for that tag).

SPEC: http://vbatts.fedorapeople.org/rpmbuild/SPECS/golang-labix-mgo.spec
SRPM: http://vbatts.fedorapeople.org/rpmbuild/SRPMS/golang-labix-mgo-2013.11.18-0.2.bzr251.el6.src.rpm

Comment 5 Vincent Batts 2014-01-06 16:52:12 UTC
Created attachment 846190 [details]
script to pack the tagged snapshot of the source

Comment 6 Lokesh Mandvekar 2014-01-06 18:52:51 UTC
the rev number needs an edit in the tarball filename too :)

Comment 7 Lokesh Mandvekar 2014-01-06 18:53:34 UTC
that being a minor issue, i'll do a final run and approve this in a bit

Comment 8 Lokesh Mandvekar 2014-01-06 19:00:48 UTC
Also, running 'sh pack_sources.sh' generates mgo-go1-253.tar.gz (using latest rev number). Perhaps we want to enter rev number in the script itself (if I'm reading it right, the script currently gets the latest rev number)

Comment 9 Vincent Batts 2014-01-06 19:12:52 UTC
sorry, i set the default tag to "go1", which is always the latest in their project.
For any given tag, like r2013.11.18, run it like
  TAG=r2013.11.18 sh pack_sources.sh

Comment 11 Lokesh Mandvekar 2014-01-06 19:47:15 UTC
Approved.

Only golang sources and docs packaged currently, this may change as golang packaging guidelines evolve.

Comment 13 Lokesh Mandvekar 2015-08-06 15:04:32 UTC
Jan, Fridolin, could you please take care of this package too? :)

Comment 14 Jan Chaloupka 2015-08-10 12:36:20 UTC
Packaged in golang-github-go-mgo-mgo. It can be closed.


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