Re: [Mesa-dev] patches (Re: undefined C++ GLSL symbol error)

2012-02-02 Thread Pekka Paalanen
On Wed, 1 Feb 2012 06:14:40 -0800 (PST)
Jose Fonseca  wrote:

> - Original Message -
> > On Wed, 1 Feb 2012 05:03:21 -0800 (PST)
> > Jose Fonseca  wrote:
> > 
> > > I wonder if people have use any tools to facility applying patches
> > > posted to mesa, as for me locate, save, and applying them takes a
> > > lot of time, and more often than not fails due to white space
> > > munging. I barely can keep up with the review requests anyway.
> > > 
> > 
> > If a patch is malformatted to begin with, I can't help it. But, if
> > your email client happens to support running external commands on
> > mail
> > files, I have a script that applies them to a given local git repo,
> > launching a terminal window for it and optionally running it though
> > the
> > kernel checkpatch.pl. On any errors, it drops you to a shell.
> > 
> > Personally I use claws-mail, and could share my setup, if
> > anyone is interested in homebrewn solution for the save & apply part.
> 
> I use webmail (mostly due to namely calendar integration, quick search of all 
> email, easy access anywhere, etc), which makes this difficult.  So I probably 
> would need to tag and/or move the mail messages w/ the patches to an IMAP 
> folder, and then have the fetchmail + apply + etc on the script which I'd run 
> from a git checkout.
> 
> I wouldn't mind take a look to your script.

Attached. The scripts rely on me selecting the emails in order, and
then clicking a menu item that runs the UI script with the email file
names as arguments.


Thanks,
pq



claws-git-am.bash
Description: Binary data


claws-git-am-UI.bash
Description: Binary data
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] patches (Re: undefined C++ GLSL symbol error)

2012-02-01 Thread Jose Fonseca
- Original Message -
> On Wed, 1 Feb 2012 05:03:21 -0800 (PST)
> Jose Fonseca  wrote:
> 
> > I wonder if people have use any tools to facility applying patches
> > posted to mesa, as for me locate, save, and applying them takes a
> > lot of time, and more often than not fails due to white space
> > munging. I barely can keep up with the review requests anyway.
> > 
> 
> If a patch is malformatted to begin with, I can't help it. But, if
> your email client happens to support running external commands on
> mail
> files, I have a script that applies them to a given local git repo,
> launching a terminal window for it and optionally running it though
> the
> kernel checkpatch.pl. On any errors, it drops you to a shell.
> 
> Personally I use claws-mail, and could share my setup, if
> anyone is interested in homebrewn solution for the save & apply part.

I use webmail (mostly due to namely calendar integration, quick search of all 
email, easy access anywhere, etc), which makes this difficult.  So I probably 
would need to tag and/or move the mail messages w/ the patches to an IMAP 
folder, and then have the fetchmail + apply + etc on the script which I'd run 
from a git checkout.

I wouldn't mind take a look to your script.

Jose
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] patches (Re: undefined C++ GLSL symbol error)

2012-02-01 Thread Pekka Paalanen
On Wed, 1 Feb 2012 05:03:21 -0800 (PST)
Jose Fonseca  wrote:

> I wonder if people have use any tools to facility applying patches posted to 
> mesa, as for me locate, save, and applying them takes a lot of time, and more 
> often than not fails due to white space munging. I barely can keep up with 
> the review requests anyway.
> 

If a patch is malformatted to begin with, I can't help it. But, if
your email client happens to support running external commands on mail
files, I have a script that applies them to a given local git repo,
launching a terminal window for it and optionally running it though the
kernel checkpatch.pl. On any errors, it drops you to a shell.

Personally I use claws-mail, and could share my setup, if
anyone is interested in homebrewn solution for the save & apply part.


Thanks,
pq
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev