Chris and Otavio,

 

            Thanks for your review of my patch.

            Owe to Mart’s patient help on how to use git and how to send the 
patch, I can give you the best format you guys can easily review. Special 
thanks for Mart.

            Now that the first patch is ok, I’ll put the other 6 patches to the 
xorg-driver-geode mail list today.

            Any feedback is welcome. The “Picky” for patch is necessary I 
think, so feel free to give me the suggestion. I understand that :-)

 

Thanks

Frank

 

________________________________

From: [email protected] 
[mailto:[email protected]] On Behalf 
Of Otavio Salvador
Sent: 2010年6月10日 1:39
To: Chris Ball
Cc: Huang, FrankR; [email protected]
Subject: Re: [Xorg-driver-geode] Patches for rendering

 

Hello,

On Wed, Jun 9, 2010 at 11:23 AM, Chris Ball <[email protected]> wrote:

Hi,


  > On Wed, Jun 9, 2010 at 7:52 AM, Chris Ball <[email protected]> wrote:
  >> It's better to e-mail patches inline rather than attached, one
  >> e-mail per patch, so that it's easy to comment on specific parts
  >> of them.

  > Please, no.  Having to extract inline patches to commit to GIT is
  > a real PITA.

Sorry, I still disagree -- the Xorg community rejects attached patches
in general, so we shouldn't be recommending them here.  Inline patches
are much easier to review line-by-line.

As a compromise, if it helps, I'm happy to volunteer to do the
importing of Frank's patches into git.  It's just two commands
for me; save the whole mail into a file, and git-am thatfile.


Same hassle for me; so I offer help if need.

-- 
Otavio Salvador                  O.S. Systems
E-mail: [email protected]  http://www.ossystems.com.br
Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br

_______________________________________________
Xorg-driver-geode mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-driver-geode

Reply via email to