X-Original-To: alpine-devel@lists.alpinelinux.org
Received: from mx1.tetrasec.net (mx1.tetrasec.net [74.117.190.25])
	by lists.alpinelinux.org (Postfix) with ESMTP id 0B0095C4C6A
	for <alpine-devel@lists.alpinelinux.org>; Thu, 27 Jul 2017 10:32:06 +0000 (GMT)
Received: from mx1.tetrasec.net (mail.local [127.0.0.1])
	by mx1.tetrasec.net (Postfix) with ESMTP id C24149E22F1;
	Thu, 27 Jul 2017 10:32:05 +0000 (GMT)
Received: from ncopa-desktop.copa.dup.pw (15.63.200.37.customer.cdi.no [37.200.63.15])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	(Authenticated sender: n@tanael.org)
	by mx1.tetrasec.net (Postfix) with ESMTPSA id 96B4A9E2154;
	Thu, 27 Jul 2017 10:32:03 +0000 (GMT)
Date: Thu, 27 Jul 2017 12:31:59 +0200
From: Natanael Copa <ncopa@alpinelinux.org>
To: Jakub Jirutka <jakub@jirutka.cz>
Cc: Alpine-devel <alpine-devel@lists.alpinelinux.org>, Jean-Louis Fuchs
 <ganwell@fangorn.ch>
Subject: Re: [alpine-devel] =?ISO-8859-1?B?QWTpbGll?= on Alpine, two months
 later
Message-ID: <20170727123159.015956f7@ncopa-desktop.copa.dup.pw>
In-Reply-To: <5301B93A-A3AA-4AD3-8191-EBA542A6C3D3@jirutka.cz>
References: <5975E0C0.1030008@adelielinux.org>
	<57A1A151-0352-40DA-BC91-52980CEE99C3@shiz.me>
	<59761031.6000605@adelielinux.org>
	<20170724195040.GA18090@vimes.1042.ch>
	<5301B93A-A3AA-4AD3-8191-EBA542A6C3D3@jirutka.cz>
X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-alpine-linux-musl)
X-Mailinglist: alpine-devel
Precedence: list
List-Id: Alpine Development <alpine-devel.lists.alpinelinux.org>
List-Unsubscribe: 
	<mailto:alpine-devel+unsubscribe@lists.alpinelinux.org?subject=unsubscribe>
List-Post: <mailto:alpine-devel@lists.alpinelinux.org>
List-Help: <mailto:alpine-devel+help@lists.alpinelinux.org?subject=help>
List-Subscribe: 
	<mailto:alpine-devel+subscribe@lists.alpinelinux.org?subject=subscribe>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Mon, 24 Jul 2017 22:46:44 +0200
Jakub Jirutka <jakub@jirutka.cz> wrote:

> There*s no clear relation between related patches, *cause it*s just
> nearly impossible to reconstruct such information from separated
> patches in mail.

That depends on how you send them. Use --in-reply-to=3D<message-id> to
send the patch to a previous email thread.

> Patchwork is based on ancient workflow from CVS era.

No. It's based on the linux kernel development workflow, the people who
invented git in the first place.

>  Maintainer can work with it directly via git (see
> https://gist.github.com/piscisaureus/3342247), without leaving
> terminal. When the contributor push new changes to the pull request,
> maintainer can simply pull them via git and view in context, it*s
> just a branch.

This was great news for me. I have looked for something like that,
without installing hub to provide github specific functionality to the
cli.

> It*s really horrible for reviews.

Then you are using it wrong. Patchwork is not designed to make reviews.
It is designed to help maintainer to keep track of the state of the
patch, and to make sure that patches are not forgotten.

You do the review, commenting and interaction via email, using your
favorite email client.

Using emails has some benefits. For example, you can read and write
reviews while you are offline (on an airplane for example). You don't
need sign up to any service. (github has critical mass now days so you
cannot really be involved in open source without having github account
anyway - I'm not convinced that is a good thing)

> I understand why some ppl don*t like GitHub (it*s third-party
> commercial service after all)

Yes. I don't want be locked in to a service provider. I don't want make
it a requirement that you sell your soul to github (or facebook or
twitter or whatever) to be able to contribute to alpine.

I also find the github web interface sluggish at times. Some pages can
take various seconds to load. And if content or directory is to bit it
will simply not show anything at all.

The major problem with supporting both github and email submissions is
that different people may send same patch. It is inconvenient for users
that they need to check patchwork before making a github PR and it is
inconventient that you need to search github PRs before you `git
send-email`. I suppose that is the price we pay.

I cannot claim that I love patchwork, but I want make it possible for
people to `git send-email` patches.

-nc

PS. interestingly patchwork itself is hosted on github.



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---