Question about "Creating first patch" guide
The OutreachyfirstpatchSetup has been very helpful in setting up my computer to develop a patch to submit to the kernel overlords. I'm at the point where I've changed the kernel code, ran and test it, and see just my changes with "git diff". What has me a little confused is that before I actually create the patch file (to submit to the appropriate mailing list), it says to actually commit the change. Perhaps I don't understand how git handles commit (I primarily use svn), but it seems like actually committing the change is kind of presumptuous before even posting anything on the mailing list of those that control the git repository. What part am I not understanding? Thank you. Rob Groner
On Wed, Sep 23, 2015 at 07:22:08PM +0000, Rob Groner wrote:
The OutreachyfirstpatchSetup has been very helpful in setting up my computer to develop a patch to submit to the kernel overlords.
I’m at the point where I’ve changed the kernel code, ran and test it, and see just my changes with “git diff”. What has me a little confused is that before I actually create the patch file (to submit to the appropriate mailing list), it says to actually commit the change. Perhaps I don’t understand how git handles commit (I primarily use svn), but it seems like actually committing the change is kind of presumptuous before even posting anything on the mailing list of those that control the git repository.
What part am I not understanding?
You should create a branch and work on that, making a commit there, then it is trivial to turn that into a patch, as the tutorial suggests. With git, everything can be local, svn requires you to push your changes to the server, which is the big difference here. hope this helps, greg k-h
Helps very much, thank you! I think I read somewhere in that tutorial that posting your patch to kernel-newbie was a safe thing to do to get comments/criticism before submitting to the maintainer mailing list (linux-serial in this case). Is that true? Thanks again. Rob Groner
-----Original Message----- From: Greg KH [mailto:greg@kroah.com] Sent: Wednesday, September 23, 2015 3:44 PM To: Rob Groner <rgroner@RTD.com> Cc: kernelnewbies@kernelnewbies.org Subject: Re: Question about "Creating first patch" guide
On Wed, Sep 23, 2015 at 07:22:08PM +0000, Rob Groner wrote:
The OutreachyfirstpatchSetup has been very helpful in setting up my computer to develop a patch to submit to the kernel overlords.
I’m at the point where I’ve changed the kernel code, ran and test it, and see just my changes with “git diff”. What has me a little confused is that before I actually create the patch file (to submit to the appropriate mailing list), it says to actually commit the change. Perhaps I don’t understand how git handles commit (I primarily use svn), but it seems like actually committing the change is kind of presumptuous before even posting anything on the mailing list of those that control the git repository.
What part am I not understanding?
You should create a branch and work on that, making a commit there, then it is trivial to turn that into a patch, as the tutorial suggests.
With git, everything can be local, svn requires you to push your changes to the server, which is the big difference here.
hope this helps,
greg k-h
On Wed, Sep 23, 2015 at 08:01:19PM +0000, Rob Groner wrote:
Helps very much, thank you!
I think I read somewhere in that tutorial that posting your patch to kernel-newbie was a safe thing to do to get comments/criticism before submitting to the maintainer mailing list (linux-serial in this case). Is that true?
Sure, that works, especially as the maintainer of that subsystem is on both lists :) greg k-h
-----Original Message----- From: Greg KH [mailto:greg@kroah.com] Sent: Wednesday, September 23, 2015 3:44 PM To: Rob Groner <rgroner@RTD.com> Cc: kernelnewbies@kernelnewbies.org Subject: Re: Question about "Creating first patch" guide
On Wed, Sep 23, 2015 at 07:22:08PM +0000, Rob Groner wrote:
The OutreachyfirstpatchSetup has been very helpful in setting up my computer to develop a patch to submit to the kernel overlords.
I’m at the point where I’ve changed the kernel code, ran and test it, and see just my changes with “git diff”. What has me a little confused is that before I actually create the patch file (to submit to the appropriate mailing list), it says to actually commit the change. Perhaps I don’t understand how git handles commit (I primarily use svn), but it seems like actually committing the change is kind of presumptuous before even posting anything on the mailing list of those that control the git repository.
What part am I not understanding?
You should create a branch and work on that, making a commit there, then it is trivial to turn that into a patch, as the tutorial suggests.
With git, everything can be local, svn requires you to push your changes to the server, which is the big difference here.
hope this helps,
greg k-h
And sorry for the top-posting. I had no idea that was a thing until 5 minutes ago. Mailing lists are a completely new thing to me (at age 44+). Rob
I don't know whether my procedure to generate patch is right.Share it with you and you can comment it ,feed back to me! First, I change the document in the master branch, but don't commit to master branch. Then I use "git diff" to generate the patch. At last , I do a check with checkpatch.pl. If there is no error, I will post it to the according subsystem. If there are any wrong tips, point it out for me. Thank you. - mdongliang 2015-09-24 5:02 GMT+08:00 Rob Groner <rgroner@rtd.com>:
-----Original Message----- From: Greg KH [mailto:greg@kroah.com] Sent: Wednesday, September 23, 2015 3:44 PM To: Rob Groner <rgroner@RTD.com> Cc: kernelnewbies@kernelnewbies.org Subject: Re: Question about "Creating first patch" guide
On Wed, Sep 23, 2015 at 07:22:08PM +0000, Rob Groner wrote:
The OutreachyfirstpatchSetup has been very helpful in setting up my computer to develop a patch to submit to the kernel overlords.
I’m at the point where I’ve changed the kernel code, ran and test it, and see just my changes with “git diff”. What has me a little confused is that before I actually create the patch file (to submit to the appropriate mailing list), it says to actually commit the change. Perhaps I don’t understand how git handles commit (I primarily use svn), but it seems like actually committing the change is kind of presumptuous before even posting anything on the mailing list of those that control the git repository.
What part am I not understanding?
You should create a branch and work on that, making a commit there, then it is trivial to turn that into a patch, as the tutorial suggests.
With git, everything can be local, svn requires you to push your changes to the server, which is the big difference here.
hope this helps,
greg k-h
And sorry for the top-posting. I had no idea that was a thing until 5 minutes ago. Mailing lists are a completely new thing to me (at age 44+).
Rob
_______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
On Thu, Sep 24, 2015 at 8:51 AM, 慕冬亮 <mudongliangabcd@gmail.com> wrote:
I don't know whether my procedure to generate patch is right.Share it with you and you can comment it ,feed back to me!
First, I change the document in the master branch, but don't commit to master branch. Then I use "git diff" to generate the patch.
Why not commit? A simple way to do this would be. 1. Make changes as per requirements 2. Commit them 3. Use `git format-patch` to generate patches. 4. Check using checkpatch 5. `Use git send-email` to mail those patches to the required list. -- Regards, Karthik Nayak
2015-09-24 13:56 GMT+08:00 Karthik Nayak <karthik.188@gmail.com>:
On Thu, Sep 24, 2015 at 8:51 AM, 慕冬亮 <mudongliangabcd@gmail.com> wrote:
I don't know whether my procedure to generate patch is right.Share it with you and you can comment it ,feed back to me!
First, I change the document in the master branch, but don't commit to master branch. Then I use "git diff" to generate the patch.
Why not commit? A simple way to do this would be. Why do I not commit? Because I don't this easy method to generate patch file. I read SubmittingPatches, just to understand I can use git diff to get my patch. Then I don't search any advanced, good method. This shows that I am not familiar with git.
1. Make changes as per requirements 2. Commit them 3. Use `git format-patch` to generate patches.
"git format-patch" is great. The format is very nice to read.
4. Check using checkpatch 5. `Use git send-email` to mail those patches to the required list. But I don't find git "send-email" in my system. I test in git 2.1.4 and git 2.5.1 on my Debian. Search it , I find I lost a package git-email.
I still have much to learn.Thank you. - mudongliang
-- Regards, Karthik Nayak
On Thu, Sep 24, 2015 at 12:17 PM, 慕冬亮 <mudongliangabcd@gmail.com> wrote:
2015-09-24 13:56 GMT+08:00 Karthik Nayak <karthik.188@gmail.com>:
On Thu, Sep 24, 2015 at 8:51 AM, 慕冬亮 <mudongliangabcd@gmail.com> wrote:
I don't know whether my procedure to generate patch is right.Share it with you and you can comment it ,feed back to me!
First, I change the document in the master branch, but don't commit to master branch. Then I use "git diff" to generate the patch.
Why not commit? A simple way to do this would be. Why do I not commit? Because I don't this easy method to generate patch file. I read SubmittingPatches, just to understand I can use git diff to get my patch. Then I don't search any advanced, good method. This shows that I am not familiar with git.
I personally suggest having a different branch for each feature/bug you might want to work on and also committing these changes in their respective branches. Why use 'diff' when 'git format-patch' is solely meant to create patches? ;) -- Regards, Karthik Nayak
2015-09-23 16:22 GMT-03:00 Rob Groner <rgroner@rtd.com>:
What part am I not understanding?
Help [0] ? 0 - http://kernelnewbies.org/FirstKernelPatch Albino
participants (5)
-
Albino B Neto -
Greg KH -
Karthik Nayak -
Rob Groner -
慕冬亮