[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 Michel Alexandre Salim michel+...@sylvestre.me changed: What|Removed |Added Flag|fedora-review? | -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 --- Comment #14 from Brandon bran...@pwnage.ca 2010-11-14 09:19:58 EST --- Take over, I explain why in his new bug. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 --- Comment #11 from W. Michael Petullo m...@flyn.org 2010-11-12 13:05:07 EST --- 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. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 --- Comment #12 from Michel Alexandre Salim fed...@michelsylvain.info 2010-11-12 16:56:14 EST --- 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? -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 --- Comment #10 from Michel Alexandre Salim fed...@michelsylvain.info 2010-11-11 07:25:36 EST --- Oops, yes, I mean Brandon. Will close as FE-DEADREVIEW if there is no response by next week. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 Michel Alexandre Salim fed...@michelsylvain.info changed: What|Removed |Added Flag||needinfo?(bran...@pwnage.ca ||) --- Comment #8 from Michel Alexandre Salim fed...@michelsylvain.info 2010-11-02 12:35:13 EDT --- 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! -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934
--- Comment #6 from Michel Alexandre Salim michael.silva...@gmail.com
2010-07-11 09:08:34 EDT ---
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.
--
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934
--- Comment #7 from Conrad Meyer kon...@tylerc.org 2010-07-11 12:44:08 EDT ---
(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.
--
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 Jochen Schmitt joc...@herr-schmitt.de changed: What|Removed |Added CC||joc...@herr-schmitt.de --- Comment #4 from Jochen Schmitt joc...@herr-schmitt.de 2010-07-08 15:59:58 EDT --- Regarding of the packaging guidlines the emacs and vim subpackages should be named as emacs-go and vim-go. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 --- Comment #5 from Brandon bran...@pwnage.ca 2010-07-08 16:32:34 EDT --- I'll fix those. There was a new release of Go on the 2nd of July, I'll have a new spec soon regardless. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 Michel Alexandre Salim michael.silva...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|michael.silva...@gmail.com Flag||fedora-review? --- Comment #3 from Michel Alexandre Salim michael.silva...@gmail.com 2010-07-04 10:57:51 EDT --- (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. -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 610934] Review Request: go - The Google Go Programming Language
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=610934 Randall Berry randyn3...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
