From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: Sending in patches
Date: Sat, 16 Jan 2016 14:53:51 +0000 [thread overview]
Message-ID: <1452956031.5665.107.camel@ipfire.org> (raw)
In-Reply-To: <DUB406-EAS163F3237DC6DE1AF244CD409CCE0@phx.gbl>
[-- Attachment #1: Type: text/plain, Size: 6180 bytes --]
Hello,
I held a talk at the last IPFire Developer Summit about why we are
doing it this way. This is currently being edited. I will CC Aaron in
this conversation to remind him to get the editing of this done.
It will not explain how to send in a patch (we had a short demo of this
at the event, but that is probably not good enough to keep in the
video), but the objective behind many things so I hope that everyone
will understand what to take care of.
Maybe Alex can make a HOWTO video like the firewall screencast he has
been publishing in the last few weeks. I CC'ed him, too.
Best,
-Michael
On Sat, 2016-01-16 at 15:48 +0100, Blago Culjak wrote:
> Well, I do have cluster with free space, I can setup a machine
> myself.
>
> But recording proccess of submiting a patch would be of great value.
>
> Poslano s mog Windows Phonea
> Šalje: Michael Tremer
> Poslano: 16.1.2016. 15:45
> Prima: R. W. Rodolico; development(a)lists.ipfire.org
> Predmet: Re: Sending in patches
>
> Hey Rod,
>
> this is a great offer. Thank you very much. I hope that someone will
> find this useful.
>
> On Fri, 2016-01-15 at 13:24 -0600, R. W. Rodolico wrote:
> > I understand that some people may need or want Windows as their
> base
> > machine and have little reason for having a separate Unix (Linux)
> > machine. Yet, in some cases, having access to such a machine for
> > development purposes could be useful.
> >
> > I run a series of servers at a NOC here in the US. If anyone is
> > interested, I'd be happy to build out a Linux or FreeBSD virtual
> for
> > this purpose. I could be a pure Command Line machine (what I
> normally
> > run) or I could put a very basic GUI (XFCE4) on it. You could
> choose
> > to access it via OpenVPN (much more secure) or direct public IP.
>
> Just wanted to say quickly that it will have to be a Linux. I am not
> sure if is possible in any way to build IPFire on a BSD. Our build
> system doesn't support this right now and I doubt that this is
> possible
> at all. So use Linux please.
>
> >
> > If anyone wants to try it, let me know. This is totally no
> pressure,
> > you can try the system, and if the environment is too alien for you
> > to
> > really get work done, then just bail out. But I'd be happy to set
> it
> > up for one or more developers to use.
> >
> > Feel free to respond on list, or write me directly.
> >
> > Rod
>
> -Michael
>
> >
> > On 01/15/2016 02:07 AM, IT Superhack wrote:
> > > Hello Michael,
> > >
> > > Michael Tremer:
> > > > Hi,
> > > >
> > > > so about half a week has passed since my initial email. I
> suppose
> > > > this is all that is coming and that the others are perfectly
> > > > satisfied with this process.
> > > >
> > > > So let's conclude:
> > > >
> > > > On Mon, 2016-01-11 at 21:40 +0100, Larsen wrote:
> > > > > On Sun, 10 Jan 2016 22:54:26 +0100, Michael Tremer
> > > > > <michael.tremer(a)ipfire.org> wrote:
> > > > >
> > > > > > I find this process with "git send-email" very easy and I
> am
> > > > > > not sure where the problems could be.
> > > > >
> > > > > I am using TortoiseGit with Windows 7. I don't know if this
> > > > > tool supports "git send-email" and I wouldn't want to use
> > > > > command line git (it is annoying to use more than one tool
> for
> > > > > the same job, IMHO).
> > > >
> > > >
> > > > I didn't know that this many people use Windows. The problem
> with
> > > > that is you cannot build the distribution. You can copy some
> > > > files back and forth, but never build.
> > > >
> > > > I consider this to be a huge disadvantage.
> > > >
> > > > Is this just by choice or did you not want to go through the
> > > > trouble setting up a Linux system?
> > > >
> > > > > Therefore, I create a patch file using TortoiseGit (I need a
> > > > > workflow reminder for this to get it right), then send the
> > > > > contents with Thunderbird where I have to remember to disable
> > > > > line wrap (also using a workflow reminder text).
> > > >
> > > > When ever you make personal notes, why not use the wiki for
> that
> > > > and share? Many other people might have the same questions.
> > > I recently wrote a page about sending in patches via a Mail User
> > > Agent. It can be found here:
> > > https://wiki.ipfire.org/devel/send-tb-patches (Thanks to Matthias
> > > Fischer here who sent me a good hint about disabling line
> -wrapping
> > > in TB.)
> > > >
> > > > > In other projects (using Github), IMHO it is much easier to
> > > > > push to one's own repository and create a merge request to
> the
> > > > > upstream repository.
> > > >
> > > >
> > > > We can't use GitHub. I have explained that at various occasions
> > > > and this thread wasn't started to discuss other tools. Other
> > > > things have been proposed, too, and the same goes for them.
> > > >
> > > > At the end the only problem I understand you are experiencing
> > > > here is actually emailing the patch files without your mail
> > > > agents not making garbage out of that. This can't be too hard
> to
> > > > get right.
> > > >
> > > > > Discussion can take place with that merge request.
> > > >
> > > > No, not at all. This process is actually well thought through
> > > > and designed to maximise participation of the users and make
> > > > everything easy for developers. And I actually think it does
> that
> > > > well.
> > > For me, sending in patches is not a very big issue since you only
> > > need to "set up" a system for sending in patches once.
> > >
> > > The problem is more the workflow after a patch has been sent in.
> > > Some of them seem to be dropped silently, while others are
> > > commented and then somehow forgotten. Perhaps I need to polish my
> > > english skills in order to add better descriptions to my
> patches...
> > > ;-)
> > > >
> > > > > In case you are open to using other tools in place of
> > > > > Patchwork, what
> > > > >
> > > > > about Gitlab?
> > > >
> > > > Patchwork is just a view to the patches. My actual repository
> for
> > > > that is my inbox.
> > > >
> > > > >
> > > > >
> > > > > Lars
> > > >
> > > > -Michael
> > > >
> > > Best regards, Timmothy Wilson
> > >
> > >
> >
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next parent reply other threads:[~2016-01-16 14:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <DUB406-EAS163F3237DC6DE1AF244CD409CCE0@phx.gbl>
2016-01-16 14:53 ` Michael Tremer [this message]
[not found] <DUB406-EAS128B6703A1A0778E8A258779CC90@phx.gbl>
2016-01-12 23:51 ` Michael Tremer
2016-01-13 4:39 ` Xaver4all
2016-01-15 1:29 ` Michael Tremer
2016-01-10 21:54 Michael Tremer
2016-01-11 20:40 ` Larsen
2016-01-15 1:25 ` Michael Tremer
2016-01-15 8:07 ` IT Superhack
2016-01-15 19:24 ` R. W. Rodolico
2016-01-16 14:45 ` Michael Tremer
2016-01-16 16:07 ` Matthias Fischer
2016-01-16 14:48 ` Michael Tremer
2016-01-16 18:07 ` Larsen
2016-01-17 19:17 ` Michael Tremer
2016-01-28 17:36 ` Larsen
2016-01-29 2:13 ` Michael Tremer
2016-01-12 18:08 ` Matthias Fischer
2016-01-15 1:28 ` Michael Tremer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1452956031.5665.107.camel@ipfire.org \
--to=michael.tremer@ipfire.org \
--cc=development@lists.ipfire.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox