[Bug 652987] Review Request: go - The Go programming language

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=652987

--- Comment #25 from d. johnson <drjohnson1@xxxxxxxxx> 2011-01-27 20:33:01 EST ---
Thank you Renich. This version does indeed build in koji.


rpmlint has a few complaints about it however.

go.x86_64: E: statically-linked-binary /usr/bin/goinstall
go.x86_64: E: statically-linked-binary /usr/bin/ebnflint
go.x86_64: E: statically-linked-binary /usr/bin/gofmt
go.x86_64: E: statically-linked-binary /usr/bin/godoc
go.x86_64: E: statically-linked-binary /usr/bin/govet
go.x86_64: E: statically-linked-binary /usr/bin/goyacc
go.x86_64: E: statically-linked-binary /usr/bin/cgo
go.x86_64: E: statically-linked-binary /usr/bin/hgpatch

4 packages and 0 specfiles checked; 8 errors, 325 warnings.

The above errors should really be fixed,
http://fedoraproject.org/wiki/Packaging/Guidelines#Staticly_Linking_Executables

A bunch of static libraries were built, and should be in a -static package
rather than the main one if included at all.
http://fedoraproject.org/wiki/Packaging/Guidelines#Packaging_Static_Libraries

Many of the binaries are without manual pages or documentation. (Upstream?)


Trivial .spec fix?

go.x86_64: W: spurious-executable-perm /usr/share/doc/go-20110120/progs/run
go.x86_64: W: spurious-executable-perm /usr/share/doc/go-20110120/prog.sh
go.x86_64: W: spurious-executable-perm /usr/share/doc/go-20110120/makehtml
go.x86_64: W: non-conffile-in-etc /etc/bash_completion.d/go
go.x86_64: W: spurious-executable-perm
/usr/share/doc/go-20110120/codelab/wiki/test.sh

These subpackages should likely include a LICENSE file if nothing else:

go-emacs.x86_64: W: no-documentation
go-vim.x86_64: W: no-documentation

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]