Re: [qubes-devel] Packaging 3rd-party software

2016-12-11 Thread Andrew David Wong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2016-12-09 17:59, Marek Marczykowski-Górecki wrote: > [...] I'm slightly leaning towards having it separate, but enabled > by default. But I'd like to hear other opinions. > IMO, separate sounds fine. > Extra thought: maybe builder-github shoul

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Sat, Dec 10, 2016 at 10:47:02AM -0500, Jean-Philippe Ouellet wrote: > On Sat, Dec 10, 2016 at 9:59 AM, Marek Marczykowski-Górecki > wrote: > > A `git diff` could be used to avoid this, but still it makes much sense > > to have it always fast-forw

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Jean-Philippe Ouellet
On Sat, Dec 10, 2016 at 9:59 AM, Marek Marczykowski-Górecki wrote: > A `git diff` could be used to avoid this, but still it makes much sense > to have it always fast-forward. This does not catch someone rewriting old commit messages to hide things. AFAIK there is no better way than manually check

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Sat, Dec 10, 2016 at 09:35:33AM -0500, Jean-Philippe Ouellet wrote: > On Sat, Dec 10, 2016 at 9:12 AM, Marek Marczykowski-Górecki > wrote: > >> The purpose of the github PRs here would be to improve the workflow of > >> a maintainer telling the w

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Jean-Philippe Ouellet
On Sat, Dec 10, 2016 at 9:35 AM, Jean-Philippe Ouellet wrote: > On Sat, Dec 10, 2016 at 9:12 AM, Marek Marczykowski-Górecki > wrote: >> This is the workflow we started in this repository and probably will >> implement in others too (especially using PR, then pushing to -staging to >> always go th

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Jean-Philippe Ouellet
On Sat, Dec 10, 2016 at 9:12 AM, Marek Marczykowski-Górecki wrote: >> The purpose of the github PRs here would be to improve the workflow of >> a maintainer telling the world "Hey, here's a set of complete and >> locally-tested changes, please review." with two goals: >> 1) Help ensure things get

Re: [qubes-devel] Packaging 3rd-party software

2016-12-10 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Sat, Dec 10, 2016 at 12:05:07AM -0500, Jean-Philippe Ouellet wrote: > (Re-ordered the quoted email for more readable reply) > > On Fri, Dec 9, 2016 at 11:06 PM, Marek Marczykowski-Górecki > wrote: > > Depending only on github repo permissions is

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Jean-Philippe Ouellet
On Sat, Dec 10, 2016 at 12:05 AM, Jean-Philippe Ouellet wrote: > IMO something like this should already be the workflow for reviewing > PRs to core (non -contrib) code. (Perhaps it is, idk.) I am actually quite curious what your reviewing workflow currently does look like. -- You received this

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Jean-Philippe Ouellet
I guess more succinctly: I think signed tags triggering builds makes a lot of sense. I think signed tags triggering code reviews does not. I believe that a workflow involving PRs would be preferable because: - Random comments on commits are IMO not an effective way to keep track of issues. - PRs

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Jean-Philippe Ouellet
(Re-ordered the quoted email for more readable reply) On Fri, Dec 9, 2016 at 11:06 PM, Marek Marczykowski-Górecki wrote: > Depending only on github repo permissions is against the rule of not > trusting the infrastructure. I definitely agree, and on the "here's what we actually download and comp

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Fri, Dec 09, 2016 at 10:46:25PM -0500, Jean-Philippe Ouellet wrote: > On Fri, Dec 9, 2016 at 8:59 PM, Marek Marczykowski-Górecki wrote: > > I think handling contributions there can be easily automated with > > https://github.com/QubesOS/qubes-issu

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Jean-Philippe Ouellet
On Fri, Dec 9, 2016 at 8:59 PM, Marek Marczykowski-Górecki wrote: > I think handling contributions there can be easily automated with > https://github.com/QubesOS/qubes-issues/issues/1818 > So, the only actual work needed, will be reviewing changes, then adding > a signed tag. Everything else will

Re: [qubes-devel] Packaging 3rd-party software

2016-12-09 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Fri, Oct 14, 2016 at 07:33:59PM -0700, Andrew David Wong wrote: > On 2016-10-14 06:54, Wojtek Porczyk wrote: > > On Thu, Oct 13, 2016 at 02:19:35PM -0700, Andrew David Wong wrote: > >> On 2016-10-13 13:22, Marek Marczykowski-Górecki wrote: > >>> O

Re: [qubes-devel] Packaging 3rd-party software

2016-10-24 Thread Holger Levsen
On Thu, Oct 13, 2016 at 01:34:13PM +0200, Wojtek Porczyk wrote: > > Option 4 sounds fine to me. > Second that. same here, even though I'm late to this party. :) Regarding the name, maybe something more general, like QubesOS-3rdparty? I fear that QubesOs-Apps could become too wrong / misnomed too

Re: [qubes-devel] Packaging 3rd-party software

2016-10-14 Thread Andrew David Wong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2016-10-14 06:54, Wojtek Porczyk wrote: > On Thu, Oct 13, 2016 at 02:19:35PM -0700, Andrew David Wong wrote: >> On 2016-10-13 13:22, Marek Marczykowski-Górecki wrote: >>> On Thu, Oct 13, 2016 at 11:30:39AM -0700, Andrew David Wong wrote: If w

Re: [qubes-devel] Packaging 3rd-party software

2016-10-14 Thread Manuel Amador (Rudd-O)
On 10/14/2016 01:54 PM, Wojtek Porczyk wrote: > > Just a thought: can't we invite those original authors to this new > organisation? They could upload the code directly, but we'd still have > to tag > the code for builder to verify. They'll have a nice logo on their GitHub > profile as a token of r

Re: [qubes-devel] Packaging 3rd-party software

2016-10-14 Thread Manuel Amador (Rudd-O)
On 10/13/2016 09:19 PM, Andrew David Wong wrote: > > I think it's fine to choose option 4 here, in line with your preference. > Since it would be a dedicated GitHub organization, we could simply > preserve the upstream names. Or, if there are organizational benefits > to prefixing (e.g., to sort th

Re: [qubes-devel] Packaging 3rd-party software

2016-10-14 Thread Wojtek Porczyk
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Thu, Oct 13, 2016 at 02:19:35PM -0700, Andrew David Wong wrote: > On 2016-10-13 13:22, Marek Marczykowski-Górecki wrote: > > On Thu, Oct 13, 2016 at 11:30:39AM -0700, Andrew David Wong wrote: > >> If we want to allow authors to retain control (on

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Andrew David Wong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2016-10-13 13:22, Marek Marczykowski-Górecki wrote: > On Thu, Oct 13, 2016 at 11:30:39AM -0700, Andrew David Wong wrote: >> On 2016-10-13 10:19, Marek Marczykowski-Górecki wrote: >>> On Thu, Oct 13, 2016 at 02:21:24PM +, Manuel Amador (Rudd-O)

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Thu, Oct 13, 2016 at 11:30:39AM -0700, Andrew David Wong wrote: > On 2016-10-13 10:19, Marek Marczykowski-Górecki wrote: > > On Thu, Oct 13, 2016 at 02:21:24PM +, Manuel Amador (Rudd-O) wrote: > >> On 10/12/2016 10:16 PM, Marek Marczykowski-Gó

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Andrew David Wong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2016-10-13 10:19, Marek Marczykowski-Górecki wrote: > On Thu, Oct 13, 2016 at 02:21:24PM +, Manuel Amador (Rudd-O) wrote: >> On 10/12/2016 10:16 PM, Marek Marczykowski-Górecki wrote: >>> Hi, >>> >>> [...packagin 3rd party software...] >>> Any

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Marek Marczykowski-Górecki
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Thu, Oct 13, 2016 at 02:21:24PM +, Manuel Amador (Rudd-O) wrote: > On 10/12/2016 10:16 PM, Marek Marczykowski-Górecki wrote: > > Hi, > > > > [...packagin 3rd party software...] > > Any thoughts? > > I think it depends on whether the 3rd party

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Manuel Amador (Rudd-O)
On 10/12/2016 10:16 PM, Marek Marczykowski-Górecki wrote: > Hi, > > [...packagin 3rd party software...] > Any thoughts? I think it depends on whether the 3rd party software is meant to be upstreamed into Qubes OS. For example, in the case of my tools, I would like them to be upstreamed, therefore

Re: [qubes-devel] Packaging 3rd-party software

2016-10-13 Thread Wojtek Porczyk
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On Wed, Oct 12, 2016 at 10:23:55PM -0700, Andrew David Wong wrote: > On 2016-10-12 15:16, Marek Marczykowski-Górecki wrote: > > Hi, > > > > Currently most of Qubes OS repositories are about Qubes-specific code as > > almost all other components we p

Re: [qubes-devel] Packaging 3rd-party software

2016-10-12 Thread Andrew David Wong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 2016-10-12 15:16, Marek Marczykowski-Górecki wrote: > Hi, > > Currently most of Qubes OS repositories are about Qubes-specific code as > almost all other components we pull from upstream distribution(s). This > works well with our current reposit