Re: Diffing submodule does not yield complete logs for merge commits

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

 



On Tue, May 19, 2015 at 5:44 AM, Heiko Voigt <hvoigt@xxxxxxxxxx> wrote:
> On Mon, May 18, 2015 at 10:06:32AM -0500, Robert Dailey wrote:
>> Unfortunately I find it unintuitive and counter productive to perform
>> inline patches or do anything on a mailing list. Especially on
>> Windows, it's a pain to setup git to effectively do this. Also I read
>> mailing lists through Gmail which does not offer a proper monospace
>> font view or syntax coloring to effectively review patches and
>> comments pertaining to them.
>
> Are you sure you are not overestimating the effort it takes to send
> patches inline? Once you've got your user agent correctly setup its just
> a matter of copy and paste instead of attaching the patch. On Windows I
> would probably use Thunderbird which has a section in the format-patch
> documentation how to configure it. Compared to the effort you probably
> spent on writing your patch isn't this bit of extra effort neglectable?
> And your patch is almost done. It just needs some tests and maybe a few
> rounds on the mailinglist after that.
>
>> Since I am not willing to properly follow your process, I will
>> withdraw my patch. However it is here if someone else wishes to take
>> it over. Really wish you guys used github's amazing features but I
>> understand that Linus has already made his decision in that matter.
>
> It not just Linus decision it is also a matter of many people are used
> to this workflow. AFAIR there have been many discussions and tries about
> using other tools. Email has many advantages which a webinterface does
> not provide. It is simply less effort that one person adjusts to this
> workflow instead of changing many peoples working workflow.
>
>> I'm sorry I couldn't be more agreeable on the matter. Thanks for the
>> time you spent reviewing my patch.
>
> If you are really this fixed in your workflow that would be too bad.

How do you send your patches inline? Do you use git send-email? I have
tried that and it is horrible to setup. Do you just copy/paste the
patch inline in your compose window?

It would be much simpler to fork Git, create a branch, make my change,
and initiate a pull request. I can get email notifications on comments
to my PR diff and address them with subsequent pushes to my branch
(which would also automatically update the code review). Turn around
times for collaborating on a change are much quicker via Github pull
requests.

I am willing to review the typical workflow for contributing via git
on mailing lists but I haven't seen any informative reading material
on this. I just find using command line to email patches and dealing
with other issues not worth the trouble. Lack of syntax highlighting,
lack of monospace font, the fact that I'm basically forced to install
mail client software just to contribute a single git patch.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]