Bug 1018542 - Review Request: go-bindata - A small utility which generates Go code from any file
Summary: Review Request: go-bindata - A small utility which generates Go code from any...
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:
Blocks: 1018543
TreeView+ depends on / blocked
 
Reported: 2013-10-13 05:02 UTC by Lokesh Mandvekar
Modified: 2014-07-01 22:59 UTC (History)
8 users (show)

Fixed In Version: go-bindata-0-0.4.git79847ab.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-29 03:41:22 UTC
Type: ---
Embargoed:
mattdm: fedora-review+


Attachments (Terms of Use)

Description Lokesh Mandvekar 2013-10-13 05:02:31 UTC
Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-jteeuwen-go-bindata.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-jteeuwen-go-bindata-0-0.1.git79847ab.fc21.src.rpm

Description:
A small utility which generates Go code from any file

This tool converts any file into managable Go source code. Useful for
embedding binary data into a go program. The file data is optionally gzip
compressed before being converted to a raw byte slice.

Fedora Account System Username: lsm5

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

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

Comment 1 Matthew Miller 2013-10-14 14:22:56 UTC
It looks like this one should be built as a binary, non-devel package providing the go-bindata tool, not the code.

Comment 2 Matthew Miller 2013-10-14 17:52:03 UTC
The above also implies that the package should be "go-bindata", not the longer import-path-based name.

Comment 3 Matthew Miller 2013-10-14 21:39:59 UTC
So, no -devel subpackage. Also, BuildRequires golang but no Requires -- the result is stand-alone.

Comment 4 Matthew Miller 2013-10-14 21:44:26 UTC
I think this should do it for the %build section:


%build

mkdir -p src/github.com/jteeuwen/
ln -s $(pwd) src/github.com/jteeuwen/go-bindata
export GOPATH=$(pwd)
go build -v -a

Comment 5 Lokesh Mandvekar 2013-10-14 23:38:18 UTC
%changelog
* Mon Oct 14 2013 Lokesh Mandvekar <lsm5>  0-0.2.git79847ab
- defattr removed
- only go-bindata installed, no devel package

Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-jteeuwen-go-bindata.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-jteeuwen-go-bindata-0-0.1.git79847ab.fc21.src.rpm

Comment 6 Matthew Miller 2013-10-15 00:00:44 UTC
Except the package should actually be renamed to "go-bindata". :)

Comment 7 Lokesh Mandvekar 2013-10-15 01:16:08 UTC
(In reply to Matthew Miller from comment #6)
> Except the package should actually be renamed to "go-bindata". :)

yup :)

%changelog
* Mon Oct 14 2013 Lokesh Mandvekar <lsm5>  0-0.3.git79847ab
- package name change to go-bindata


Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/go-bindata.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/go-bindata-0-0.3.git79847ab.fc21.src.rpm

Comment 8 Matthew Miller 2013-10-15 01:50:31 UTC
Looks good. As with all of the Go packages this may need to be updated as the draft packaging guidelines are developed.

Comment 9 Lokesh Mandvekar 2013-10-15 02:29:21 UTC
New Package SCM Request
=======================
Package Name: go-bindata
Short Description: A small utility which generates Go code from any file
Owners: lsm5
Branches: f19 f20 el6
InitialCC:

Comment 10 Jens Petersen 2013-10-15 06:02:25 UTC
Git done (by process-git-requests).

Comment 11 Vincent Batts 2013-10-16 18:33:04 UTC
(In reply to Matthew Miller from comment #4)
> I think this should do it for the %build section:
> 
> 
> %build
> 
> mkdir -p src/github.com/jteeuwen/
> ln -s $(pwd) src/github.com/jteeuwen/go-bindata
> export GOPATH=$(pwd)
> go build -v -a

does this still build fine if the rpm is installed on the host that is trying to do a new build of the RPM?
Is setting of GOBIN needed? or passing '-o %{buildroot}/usr/bin/go-bindata' to go build?

Comment 12 Matthew Miller 2013-10-16 19:43:20 UTC
(In reply to Vincent Batts from comment #11)
> does this still build fine if the rpm is installed on the host that is
> trying to do a new build of the RPM?
> Is setting of GOBIN needed? or passing '-o %{buildroot}/usr/bin/go-bindata'
> to go build?

I'm pretty sure go-bindata just needs to be in the path. However, I see that etcd's mod/dashboard build script wants to build the binary into $PWD (not even $GOBIN!) and calls it as ./go-bindata.

We should either patch or ignore that build script (which we'll have to do anyway because it's got other oddities).

Let me know if I'm missing something here.

Comment 13 Lokesh Mandvekar 2013-10-18 16:54:01 UTC
(In reply to Vincent Batts from comment #11)
> 
> does this still build fine if the rpm is installed on the host that is
> trying to do a new build of the RPM?
> Is setting of GOBIN needed? or passing '-o %{buildroot}/usr/bin/go-bindata'
> to go build?

It does build fine when already installed.

I'm guessing everything else is good to go for this package (?)

Comment 14 Lokesh Mandvekar 2013-10-18 17:18:19 UTC
%changelog
* Fri Oct 18 2013 Lokesh Mandvekar <lsm5>  0-0.4.git79847ab
- buildrequires golang

Comment 15 Matthew Miller 2013-10-18 17:30:44 UTC
Yeah I think it's just waiting builds :)

Comment 16 Fedora Update System 2013-10-18 17:42:40 UTC
go-bindata-0-0.4.git79847ab.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/go-bindata-0-0.4.git79847ab.fc20

Comment 17 Fedora Update System 2013-10-18 17:56:15 UTC
go-bindata-0-0.4.git79847ab.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/go-bindata-0-0.4.git79847ab.fc19

Comment 18 Fedora Update System 2013-10-18 18:02:29 UTC
go-bindata-0-0.4.git79847ab.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/go-bindata-0-0.4.git79847ab.el6

Comment 19 Fedora Update System 2013-10-19 00:12:55 UTC
go-bindata-0-0.4.git79847ab.fc20 has been pushed to the Fedora 20 testing repository.

Comment 20 Peter Lemenkov 2013-10-23 11:11:57 UTC
Package Change Request
======================
Package Name: go-bindata
InitialCC: golang-sig

Comment 21 Gwyn Ciesla 2013-10-23 14:14:14 UTC
Done.

Comment 22 Fedora Update System 2013-10-29 03:41:22 UTC
go-bindata-0-0.4.git79847ab.fc19 has been pushed to the Fedora 19 stable repository.

Comment 23 Fedora Update System 2013-11-03 17:57:56 UTC
go-bindata-0-0.4.git79847ab.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 24 Fedora Update System 2013-11-10 06:18:40 UTC
go-bindata-0-0.4.git79847ab.fc20 has been pushed to the Fedora 20 stable repository.


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