Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Sam Hartman
Hi. In my latest bits from the DPL, I said that I was going to start a couple of consensus-forming discussions on debian-devel surrounding Git and packaging. These will run similarly to the Debhelper/Dh discussion. The issues are more complicated, and so I want to spread things out a bit. In t

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Jonathan Carter
On 2019/08/14 20:02, Sam Hartman wrote: > If you're going to set up a repository for Debian packaging on Salsa, > you need to either: > > 1) turn off merge requests That would be quite horrible IMHO, this is the de facto method that young (let's say under 35 years old) developers use to submit im

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Sam Hartman
> "Jonathan" == Jonathan Carter writes: Jonathan> On 2019/08/14 20:02, Sam Hartman wrote: >> If you're going to set up a repository for Debian packaging on >> Salsa, you need to either: >> >> 1) turn off merge requests Jonathan> That would be quite horrible IMHO, thi

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Jonas Smedegaard
Quoting Jonathan Carter (2019-08-14 20:25:05) > On 2019/08/14 20:02, Sam Hartman wrote: > > If you're going to set up a repository for Debian packaging on Salsa, > > you need to either: > > > > 1) turn off merge requests > > That would be quite horrible IMHO, this is the de facto method that > yo

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Jonathan Carter
On 2019/08/14 20:36, Sam Hartman wrote: > And yet, consider the following cases: > > * You are simply mirroring from somewhere else to Salsa > > * You buy Sean's argument that MRs aren't stable enough to capture > discussion long-term and/or we're unlikely to port forward the > database of me

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-14 Thread Holger Levsen
On Wed, Aug 14, 2019 at 09:08:44PM +0200, Jonas Smedegaard wrote: > I systematically turn off Gitlab MR support for projects I am involved > in, because I am not confortable and efficient using it myself, it is what helps me is having a note with this line: git config alias.mr '!sh -c "git fetc

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Philip Hands
Jonathan Carter writes: > On 2019/08/14 20:02, Sam Hartman wrote: >> If you're going to set up a repository for Debian packaging on Salsa, >> you need to either: >> >> 1) turn off merge requests > > That would be quite horrible IMHO, this is the de facto method that > young (let's say under 35 ye

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Sam Hartman
> "Philip" == Philip Hands writes: Philip> Jonathan Carter writes: >> On 2019/08/14 20:02, Sam Hartman wrote: >>> If you're going to set up a repository for Debian packaging on >>> Salsa, you need to either: >>> >>> 1) turn off merge requests >> >> That woul

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Adam Borowski
On Wed, Aug 14, 2019 at 10:32:29PM +, Holger Levsen wrote: > On Wed, Aug 14, 2019 at 09:08:44PM +0200, Jonas Smedegaard wrote: > > I systematically turn off Gitlab MR support for projects I am involved > > in, because I am not confortable and efficient using it myself, it is > > what helps m

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Holger Levsen
On Thu, Aug 15, 2019 at 07:02:37PM +0200, Adam Borowski wrote: > You do want --global for the alias, though -- that removes the need to > repeat the config for every machine:checkout pair, requiring it just once > per machine. thanks for this as well! :) -- cheers, Holger -

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Vincent Bernat
❦ 14 août 2019 22:32 +00, Holger Levsen : >> I systematically turn off Gitlab MR support for projects I am involved >> in, because I am not confortable and efficient using it myself, it is > > what helps me is having a note with this line: > > git config alias.mr '!sh -c "git fetch $1 > merge-r

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread gregor herrmann
On Wed, 14 Aug 2019 20:25:05 +0200, Jonathan Carter wrote: > On 2019/08/14 20:02, Sam Hartman wrote: > > If you're going to set up a repository for Debian packaging on Salsa, > > you need to either: > > 1) turn off merge requests > That would be quite horrible IMHO, this is the de facto method tha

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-15 Thread Russ Allbery
Jonathan Carter writes: > The Debian QA DDPO pages will show you whether you have MRs on the same > page where you see how many open bugs, RC bugs, lintian errors, etc you > have. This makes it super easy to notice MRs when doing routine checks > of your general package health overview. Is there

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-16 Thread David Prévot
Hi, Le 15/08/2019 à 16:38, gregor herrmann a écrit : > On Wed, 14 Aug 2019 20:25:05 +0200, Jonathan Carter wrote: >> The Debian QA DDPO pages will show you whether you have MRs […] > I don't see any MRs there You need to look for (or grep) the exclamation marks (yay, discrete). > https://qa.deb

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-16 Thread Raphael Hertzog
On Thu, 15 Aug 2019, Russ Allbery wrote: > Jonathan Carter writes: > > > The Debian QA DDPO pages will show you whether you have MRs on the same > > page where you see how many open bugs, RC bugs, lintian errors, etc you > > have. This makes it super easy to notice MRs when doing routine checks >

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-16 Thread Raphael Hertzog
On Wed, 14 Aug 2019, Sam Hartman wrote: > Regardless of anything they are doing with Git, maintainers of a package > are expected to process patches sent to the BTS. Yes. > You cannot respond to a patch telling someone they need to file a merge > request to have it considered. I would not hesita

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-26 Thread Bernd Zeimetz
Hi Sam, > BTS for Patches > === > > Regardless of anything they are doing with Git, maintainers of a package > are expected to process patches sent to the BTS. You cannot respond to > a patch telling someone they need to file a merge request to have it > considered. > Today, the BTS

Re: Git Packaging Round 1: Hopefully Easy Stuff

2019-08-26 Thread Sam Hartman
> "Bernd" == Bernd Zeimetz writes: Bernd> I agree with that, although I think that we should tell bug Bernd> reporters that - if the package is maintained on salsa (or Bernd> maybe github, gitlab, ...), then we prefer merge/pull Bernd> requests. [...] Bernd> 3. If a pack