I wrote a script to read in a conf.php file and convert it to use a $conf array. I don't think we have a place for utility scripts like this right now. Where should it go? https://github.com/alexdean/ganglia-stuff/blob/master/reformat_conf.php
After you use this tool, but before all code is updated to use $conf, we can maintain backwards-compatibility with existing code with something like this: https://gist.github.com/843349 I've taken a quick tour through my local ganglia using a converted conf.php and not seen any obvious problems, but of course the non-obvious ones are probably there somewhere. alex On Feb 24, 2011, at 7:01 AM, Jesse Becker wrote: > +1 to that too > > On Wed, Feb 23, 2011 at 23:49, Bernard Li <bern...@vanhpc.org> wrote: >> +1 from me as well. >> >> I guess we should probably check it into both monitor-web-2.0 and trunk. >> >> Cheers, >> >> Bernard >> >> On Wed, Feb 23, 2011 at 7:35 PM, Jesse Becker <haw...@gmail.com> wrote: >>> +1 >>> >>> On Wed, Feb 23, 2011 at 21:27, Alex Dean <a...@crackpot.org> wrote: >>>> One of my gripes with the current PHP frontend code is how hard it can be >>>> to recall where which variables are configuration, which come from user >>>> input, and which are just local variables. As one step toward fixing this >>>> issue, I think it would be nice to place all configuration values (mainly >>>> in conf.php currently) into a $conf array. The benefit is that it's >>>> immediately clear in any code which uses these values that you're dealing >>>> with configuration values. There's no danger of name collisions with >>>> other variables. >>>> >>>> I'm just wondering if others feel the same way, and would support a change >>>> like this. It's pretty straighforward to do, but would obviously touch a >>>> lot of different code. Before I go ahead with making all those changes, I >>>> guess I'd just like to know if there are any huge objections out there to >>>> this idea. Take a look, let me know what you think. I'd like to do >>>> something similar for user input as well, maybe $user? >>>> >>>> http://pastie.org/1600587 >>>> >>>> thanks, >>>> alex >>>> >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> Free Software Download: Index, Search & Analyze Logs and other IT data in >>>> Real-Time with Splunk. Collect, index and harness all the fast moving IT >>>> data >>>> generated by your applications, servers and devices whether physical, >>>> virtual >>>> or in the cloud. Deliver compliance at lower cost and gain new business >>>> insights. http://p.sf.net/sfu/splunk-dev2dev >>>> _______________________________________________ >>>> Ganglia-developers mailing list >>>> Ganglia-developers@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/ganglia-developers >>>> >>> >>> >>> >>> -- >>> Jesse Becker >>> >>> ------------------------------------------------------------------------------ >>> Free Software Download: Index, Search & Analyze Logs and other IT data in >>> Real-Time with Splunk. Collect, index and harness all the fast moving IT >>> data >>> generated by your applications, servers and devices whether physical, >>> virtual >>> or in the cloud. Deliver compliance at lower cost and gain new business >>> insights. http://p.sf.net/sfu/splunk-dev2dev >>> _______________________________________________ >>> Ganglia-developers mailing list >>> Ganglia-developers@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/ganglia-developers >>> >> > > > > -- > Jesse Becker > ------------------------------------------------------------------------------ Free Software Download: Index, Search & Analyze Logs and other IT data in Real-Time with Splunk. Collect, index and harness all the fast moving IT data generated by your applications, servers and devices whether physical, virtual or in the cloud. Deliver compliance at lower cost and gain new business insights. http://p.sf.net/sfu/splunk-dev2dev _______________________________________________ Ganglia-developers mailing list Ganglia-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ganglia-developers
