This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [pushed] [PATCH V7 0/8] Intel(R) MPX register support


Tom,

Thanks for clarifying this, fort the next series we will add a remark on the patches to be squashed.

Best regards,
-Fred


-----Original Message-----
From: Tom Tromey [mailto:tromey@redhat.com] 
Sent: Friday, November 22, 2013 6:16 PM
To: Tedeschi, Walfred
Cc: Pedro Alves; Yao Qi; Mark Kettenis (mark.kettenis@xs4all.nl); gdb-patches@sourceware.org
Subject: Re: [pushed] [PATCH V7 0/8] Intel(R) MPX register support

>>>>> ">" == Tedeschi, Walfred <walfred.tedeschi@intel.com> writes:

>> BTW: Next patch we will squash i386/amd64/gdbserver. Should this 
>> three patches also be send for review as a single commit?

Usually it's best to make each patch work independently.  This is the ideal because then one doesn't need to do anything special when submitting or merging.

Of course, it isn't always possible.  Sometimes a patch is easier to read when split, but the split means that some sub-patch doesn't build.
In this case I think it is fine to note the issue in the email and then remember to squash the relevant patches at merge time.

Tom
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895
Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]