[postgis-devel] Astyle Bot

[Paragon Corporation]

Nicklas,
Hmm I forgot about that one keeping astyle separate from logic commit.  Once
a week sounds reasonable to me.  
 
Chris et. al ,
You think we could schedule say once a week a job to do an svn astyle
commit?  As Nicklas says, it will probably be a lot of files in the
beginning but few
each successive time.
 
I amended the commit guidelines to hyperlink to our below astyle guidelines.
 
I still need to put in Pierre's recommendation changes.

  _____  

From: postgis-devel-bounces at postgis.refractions.net
[mailto:postgis-devel-bounces at postgis.refractions.net] On Behalf Of Nicklas
Avén
Sent: Saturday, May 14, 2011 2:53 AM
To: PostGIS Development Discussion
Subject: Re: [postgis-devel] Astyle Bot



Hallo 

----- Original message ----- 
> I'm not sure if this is the 4th or 5th time this has come up... 
> 

sorry 
I must have missed 2 or 3 of them. 


> This causes future commits to potentially fail, and there is NO WAY to 
> update your client because it thinks that everything is happily the same 
> updated version as is in the repo. 
> 

Ok I see th problem. 



> We could do an automatic process to go in after an update and checkout 
> any changed files, restyle them, and commit them, but then our SVN 
> history is full of "restyle" commits. This causes confusion when looking 
> for "who/what broke the build?" and generally just seems messy to me... 
> comparably messy to just having an irregularly styled codebase. 

i am not sure I agree here. according to our style guidelines: 
http://trac.osgeo.org/postgis/browser/trunk/STYLE 
we are supposed to do the style-committ separerately anyway. 
But I agree that it would be messy to do it after each commit. 

Maybe a middleway would be to run it once a week automatically with a fixed
easy to see message. i don't think that commit will mess up the timeline too
much and the code will not be too messed up in one week if we try to keep
the style manually. 

/nicklas 






> 
> What we could do is put a pre-commit hook in SVN that runs astyle and 
> looks for changes - if there are any, then the commit is rejected, and 
> you have to run astyle yourself before committing. However, due to the 
> apparent lack of compatibility between versions of astyle, this doesn't 
> seem to work either. Perhaps we could include a version of astyle or a 
> similar script inside postgis? Similar to including autoconf-generated 
> tools. Then we can ensure that the same formatting is being run on both 
> the SVN server and all clients. I don't know whether this is possible 
> with astyle or if there is another tool that could do it? 
> 
> Not to start a holy war but I don't really have a problem with 
> formatting in general. Compared to Mapserver, the PostGIS code looks 
> great! But then, more developers == more differences. So anyways I'm 
> pretty indifferent about the "solution" to this "problem". 
> 
> Chris 
> 
> 
> On 13/05/2011 1:22 PM, Paragon Corporation wrote: 
> > +1 
> > 
> > Strk and Nicklas, 
> > But I thought we had this discussion and putting in an SVN hook or a 
> > smart Astyle bot was only in our dreams?  Is it possible? 
> > 
> > Thanks 
> > Regina 
> > 
> > On Fri, May 13, 2011 at 12:30:06PM +0200, Nicklas Avén wrote: 
> > 
> > > We run a lot of different jobs building documentation, building 
> > > postgis for regresstest and so on. 
> > > 
> > > Why is it a problem to run astyle somewhere in the chain too? 
> > +1 on this, let's have bots cleanup the code for us. 
> > 
> > --strk; 
> > 
> > 
> > _______________________________________________ 
> > postgis-devel mailing list 
> > postgis-devel at postgis.refractions.net 
> > http://postgis.refractions.net/mailman/listinfo/postgis-devel 
> 
> _______________________________________________ 
> postgis-devel mailing list 
> postgis-devel at postgis.refractions.net 
> http://postgis.refractions.net/mailman/listinfo/postgis-devel 
> 



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/postgis-devel/attachments/20110514/50996cb7/attachment.html>