Re: What to do with branches after submitting patches?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 11, 2013 at 3:34 PM, Mandeep Sandhu
<mandeepsandhu.chd@xxxxxxxxx> wrote:
>
>> >
>> > Is it possible to use a github repository and just send a "please pull
>> > from git@xxxxxxxxxx...." message to the kernel mailinglist, or will I
>> > be beaten for this?
>> >
>> You cannot directly send a pull request until the patches are reviewed,
>> once the the driver maintainer and other folks in the ML are happy, the
>> maintainer will pull your patches and get it merged in mainline kernel.
>
>
> If using github, I think send pull requests is a way to initiate a review.
> It does not mean that that pull request will be merged as-is. As you point
> out, if there are review comments, the reviewee will address those comments
> and send another pull request till the reviewer is satisfied. After that the
> reviewer just has to 'merge' the latest pull request.
>

For reviewing patches on the mailing list you don't send pull request,
! patches needs
to be sent to ML to be reviewed !.

For example look at [1] which is pull request to Linus. The pull request will
just have the patch description(it doesnt have the diff), and the link
to pull the patches from. People dont have time to go to your link and
review the
patches there and reply on your pull request.
"So you need to send patches to ML for review and not pull request"

Thanks,
--Prabhakar Lad

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies




[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux