pcp
[Top] [All Lists]

Re: [pcp] pcp updates: merge kenj (pmlogextract, various data type fixes

To: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>, Mark Goodwin <mgoodwin@xxxxxxxxxx>
Subject: Re: [pcp] pcp updates: merge kenj (pmlogextract, various data type fixes and logrewrite, linux_proc fixes, debian packaging issue, pmdaproc) + Lukas (qa, fedora spec) + nathans (pmdalinux fixes, pmdaproc, qa, build fixes) + Martins (types in linux_proc)
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Sun, 31 May 2015 18:33:37 -0400 (EDT)
Cc: pcp <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <00a601d09b1d$3a46cde0$aed469a0$@internode.on.net>
References: <55629EFA.5030403@xxxxxxxxxx> <00a601d09b1d$3a46cde0$aed469a0$@internode.on.net>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: AQLlC3QqsLWVIZwVmpoXKWPP3RlYyptsICcw5jJ7tpI=
Thread-topic: pcp updates: merge kenj (pmlogextract, various data type fixes and logrewrite, linux_proc fixes, debian packaging issue, pmdaproc) + Lukas (qa, fedora spec) + nathans (pmdalinux fixes, pmdaproc, qa, build fixes) + Martins (types in linux_proc)

----- Original Message -----
> [...]
> Perhaps we need a discussion about our maintainer workflow for pull requests
> from github.

I think this was a one-off, oddly setup repo/account - we pull from folks
(like martins, pevans) on github frequently without this kind of problem.

> [...]
> I have no clue if the "EC2 Default User" is specific to the way
> "marcliangma" committed on github or is generic to any commit made directly
> on github.

I think its more the former than the latter - only thing I can suggest here
(other than asking the submitter to fix it up before pulling it in), would
be to pull and then use "git commit --amend --author=xxx" before publishing
your tree.

cheers.

--
Nathan

<Prev in Thread] Current Thread [Next in Thread>