Re: Source reindenting - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Source reindenting
Date
Msg-id AANLkTik0jT-+3M+0ksBH=qwABpap+gDD1tg58Ad=ShpY@mail.gmail.com
Whole thread Raw
In response to Re: Source reindenting  (Magnus Hagander <magnus@hagander.net>)
Responses Re: Source reindenting  (Magnus Hagander <magnus@hagander.net>)
List pgadmin-hackers
On Thu, Dec 30, 2010 at 10:02 AM, Magnus Hagander <magnus@hagander.net> wrote:
>> Not unless you add a VC++ build step to do it too...
>
> Ah, wasn't aware you actually committed from Windows. Thought you
> worked kind of like I do for the pg backend work which is do all
> windows stuff on a separate branch and then fold it into my local git
> repository and clean it up on linux before committing anything (which
> is where I clean up things like linebreaks and indention).

I keep my Windows and other repos entirely independent of one another.
Typically, I do commit new work from Windows though, as that's where I
mostly develop, and patches from OSX, as that's more convenient for
testing other people's work.

> There's an astyle package for windows. It's officially maintained.
> While I don't think it would be too easy to create a VS build step for
> it, it would probably be easy to create a .BAT file you could run
> prior to commit, if that would be enough? (Do you do your actual git
> work from inside VS as well? Daring!)

I don't use git from inside VC++ - but if you want astyle to be run
prior to every commit, then I'll want a batch file or something that's
easy to remember to call (note, if it's a batch file, it's easy to add
as a pre-build step - though it may be annoying if it changes
anything).

Or, we could just do it the PostgreSQL way - try to get it right
throughout the cycle, and then have a pgindent run prior to release to
be sure.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: Trac tickets
Next
From: Dave Page
Date:
Subject: Re: Email notification pgAgent