Re: github repo (Re: [PATCH v2 - AAF PCM plugin 0/7] Introduce AVTP Audio Format (AAF) plugin)

From: Takashi Iwai <tiwai@suse.de>
To: Jaroslav Kysela <perex@perex.cz>
Cc: Andre Guedes <andre.guedes@intel.com>,
	alsa-devel@alsa-project.org, liam.r.girdwood@intel.com,
	pierre-louis.bossart@linux.intel.com
Subject: Re: github repo (Re: [PATCH v2 - AAF PCM plugin 0/7] Introduce AVTP Audio Format (AAF) plugin)
Date: Thu, 25 Oct 2018 09:42:10 +0200	[thread overview]
Message-ID: <s5hefceo4kt.wl-tiwai@suse.de> (raw)
In-Reply-To: <c53bd808-c980-a7e5-523c-59d3b8522bee@perex.cz>

On Thu, 25 Oct 2018 09:36:54 +0200,
Jaroslav Kysela wrote:
> 
> Dne 25.10.2018 v 08:51 Takashi Iwai napsal(a):
> > On Thu, 25 Oct 2018 08:41:15 +0200,
> > Jaroslav Kysela wrote:
> >>
> >> Our repos are on github now, too : https://github.com/alsa-project
> > 
> > I've seen your recent actions, and wondered whether this is a
> > permanent move or a testing.  Will you keep hosting the same git repos
> > on git.alsa-project.org?
> 
> I will do a proper github announce later today. And, it's one of the
> question for the discussion. For me, it would be probably better to do a
> mirror (and allow the push requests only to one primary repository -
> github?).

I don't mind either way, but limiting the push to only the primary
repo makes sense.

IMO, a bigger question is whether we allow github PR (supposing moving
to github as the primary repo).  Usually the changes are submitted to
alsa-devel ML for review, so far.  But with github PR, it may be
missing, and may be bad for the workflow of people including me.

I'm not sure whether hooking up the PR notification to ML would work.

thanks,

Takashi