It's funny but most of the suggestions except whitespaces and bash were how I commit to my local mercurial clone :)
I hope today or tomorrow I'll have the time to publish my repo online so you can see the changes there. About the indentation. I will not change the indentation of code that I don't touch but it is double work to keep this code not indented. I'm just saying that in the next release the whole file should share the same indentation. Does, linux-ha has a codding standard for that? Marain P.S. Thanks for your comments and suggestions. On Tuesday 23 February 2010 10:38:33 Dejan Muhamedagic wrote: > Hi, > > On Tue, Feb 23, 2010 at 09:31:08AM +0100, Florian Haas wrote: > > On 02/23/2010 12:37 AM, Marian Marinov wrote: > > > Florian, > > > I greatly appreciate the comments. > > > > > > About the bash/sh. I prefer bash since I can use some neat things like > > > REGEX directly. > > > > Yes, that's a bash 3 feature. Well I guess it's safe to assume that no > > one in their right mind will run an HA cluster on a box that comes with > > a pre-version-3 bash. > > I think that we should stick to /bin/sh. Switching to /bin/bash > after the RA has already been released can be considered a regression. > > > > replication_aware - this was a first think I think of, thanks for the > > > suggestion already implemented > > > passwd/password - fixed > > > whitespaces - sorry but the script was not very well indented and I > > > need it indented in order to work. I'll try to clean this up for you. I > > > understand what is the problem you are facing. > > > > Now I realize that discussions about whitespace are among the most > > pointless in our industry, but please do stick to the indentation style > > the original author adopted. The current upstream release is indented > > just fine -- when viewed in Emacs, which the original author apparently > > used. :) > > > > > I keep my stuff in git, currently preparing all of my project to be > > > published with it. I have a mercurial clone of the agents. And I'll > > > switch to it for the linux-ha projects, soon (hopefully by the middle > > > of next week). > > > > Yes, please do. I'm fully with you in preferring git by default, and I > > use it for everything where the choice is mine, but linux-ha and > > pacemaker both use Mercurial so you'll have to go with that. If I can, > > you can too. :) > > > > It would also be helpful if you could break up your patch into easily > > digestible, logically self-contained chunks, such as > > > > - Low: RA: mysql: clean up whitespace (if you must) > > Better not, as you already suggested above. > > > - Medium: RA: mysql: switch interpreter to bash > > But please don't :) > > > - High: RA: mysql: introduce replication RA parameters (add parameters, > > defaults, metadata, and add relevant checks to the validate op) > > - High: RA: mysql: add promote and demote operations (add the functions > > themselves, and relevant changes to metadata, and the usage string) > > - High: RA: mysql: add notify operation (idem) > > > > Mercurial queues can come in quite handy for that. Just as a suggestion. > > > > And, please persist. :) We will eventually get this done. > > > :) > > Thanks, > > Dejan > > > Cheers, > > Florian -- Best regards, Marian Marinov
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Linux-HA mailing list [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha See also: http://linux-ha.org/ReportingProblems
