I forgot to mention this is my first package and I need a sponsor.

*** Bug 557366 has been marked as a duplicate of this bug. ***

(In reply to comment #1)
> I forgot to mention this is my first package and I need a sponsor.    

no problem on that, I can sponsor. I'll take a look and start providing review feedback.

Regarding of the packaging guidlines the emacs and vim subpackages should be named as emacs-go and vim-go.

I'll fix those. There was a new release of Go on the 2nd of July, I'll have a new spec soon regardless.

Brandon,

When updating for the new release, please do the following:

- for the clone instruction, instead of -r release, please use -r release.2010-07-01 . This makes sure that the instruction would fetch the correct source even after subsequent releases

(aside: kind of curious how the release.2010-07-01 tag does *not* appear in the output of 'hg tags' until after I pull in some changes newer than the tag)

- use hg archive instead of invoking tar yourself. That would strip out the .hg metadata, and for those who track the Go upstream repo (as you, me as the reviewer, and any future co-maintainers likely do), it would avoid having to clone a new copy every time a new release come out because the working tree has been used for compilation. 

- if you're targeting only recent Fedora releases, consider using xz instead of bzip2 for the archive compression.

hg archive -t tar ../go-20100701.tar
cd ..
xz -9e -k go-*.tar # -k only used so the .tar is kept for re-compressing
bzip2 --best -k go-*.tar
ls -l go*.tar*

$ ls -l go*.tar*
-rw-rw-r--. 1 michel michel 20541440 Jul 11 14:54 go-20100701.tar
-rw-rw-r--. 1 michel michel  4700747 Jul 11 14:54 go-20100701.tar.bz2
-rw-rw-r--. 1 michel michel  3762540 Jul 11 14:54 go-20100701.tar.xz

- Here's the Emacs packaging guideline. You'd want to byte-compile the *.el files if possible, and only package the .elc files in the main Emacs subpackage. The source *.el should be in a separate emacs-go-el subpackage.

- there seems to be a bug in the files section. Since 8* is generated on ix86, and 6* on x86_64, you probably want

%ifarch %ix86
%{_bindir}/8*
%else
%{_bindir}/6*
%endif
... common filenames here ...

- you need to own %{_sysconfdir}/bash_completion.d 

- for Fedora 12 and above, BuildRoot no longer needs to be defined. For Fedora 13 and above, %clean is also no longer needed.
http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

So depending on which distributions you want to target, you can remove those. Keeping them is fine too.

(In reply to comment #6)
> - there seems to be a bug in the files section. Since 8* is generated on ix86,
> and 6* on x86_64, you probably want
> 
> %ifarch %ix86
> %{_bindir}/8*
> %else
> %{_bindir}/6*
> %endif
> ... common filenames here ...

Nah, some of the binaries are generated as 6* regardless of what architecture you're on.

Hi Conrad,

There's been several releases -- the last on October 27th. Is there an updated spec I can review? The pointed-out problems with checkout instruction and subpackages are still present in the spec file listed in the request.

Thanks!

It's Brandon's review, not mine.

Oops, yes, I mean Brandon.

Will close as FE-DEADREVIEW if there is no response by next week.

I would be happy to take over the maintenance of this package. I am presently working with several other people on a research OS that uses Go extensively. A Go package would be beneficial to our work. I may have some time this weekend to look at revising the RPM specification if we don't hear from Brandon.

Hi Michael,

That'd be great. If we don't hear by say Sunday morning, please create a new bug entry and close this one as a duplicate. I'll start the review then.

Thanks! Oh, and can I find out more about this research OS? seems interesting. You write the kernel in Go, or just the userspace?

*** This bug has been marked as a duplicate of bug 652987 ***

Take over, I explain why in his new bug.