[pkg-go] EARLY DRAFT: pkg-go plans

[Ximin Luo]

On 09/03/14 21:56, Michael Stapelberg wrote:
> Hi,
> 
> I have been collecting a few thoughts on how I think pkg-go should work
> as a team (and what kind of awesome infrastructure we should have) in
> this document:
> 
> https://docs.google.com/document/d/1s4V3XeUBou8c0m1JGygDNcMQ6QJIYUnOHgoy0CTrTRQ/edit?usp=sharing
> 
> Please feel free to comment on the document itself or in this email
> thread.
> 
> The content of the document is still very rough and unpolished, so your
> feedback and questions will help shape the document and hopefully make
> it clearer :).
> 
> Thanks!
> 

I agree with the general principle of the main points, but I disagree with how "mandatory code review" is to be implemented (I support the general idea though).

Disallowing pushes to alioth is a bit on the heavy side IMO. Debian already has a mechanism for restricting non-experienced packagers from uploading - and that is the sponsorship process. Adding another barrier in the form of git restrictions only adds bureaucracy, and forces the newbies to find their own git hosting, which is not very supportive/friendly.

However, I can see how it would be useful to do reviews asynchronously in batches, instead of all-at-once during sponsorship. Perhaps we can have "reviewed" branches, that only "experienced reviewers" are allowed to update, and mandate that we only release things via `gbp buildpackage --git-debian-branch=reviewed`? In your document you already mandate using `--git-pbuilder` so adding another flag won't mess things up too much. (I am not sure how we are supposed to enforce these flags, but that applies to `--git-pbuilder` as well.)

X

-- 
GPG: 4096R/1318EFAC5FBBDBCE
git://github.com/infinity0/pubkeys.git

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 880 bytes
Desc: OpenPGP digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-go-maintainers/attachments/20140322/dce13dcf/attachment.sig>