|
From: Christian B. <pos...@cb...> - 2010-11-27 00:03:37
|
Hello, Am Freitag, 26. November 2010 schrieb Juan Carlos Gutiérrez Deliot: > On 25/11/2010 06:54 a.m., David Goodwin wrote: > > [Christian Boltz] > >> We don't have something like coding guidelines, and for historical > >> reasons we even have some files with 3 and some with 4 spaces as > >> tab replacement. (New files should use 4 spaces.) > > > > I'd rather : [...] I just summarized that on the under-utilized wiki @SF: https://sourceforge.net/apps/mediawiki/postfixadmin/index.php?title=Coding_guidelines Let's hope we can point lots of people there *g* BTW: If you want write access to the wiki, see https://sourceforge.net/apps/mediawiki/postfixadmin/index.php?title=Main_Page > When I said personal contact I meant contact via "direct" email or > maybe chat, It was funny, Christian made laugh a lot with my name > not sounding german. ;-) > @Christian: I couldn't find the command-line interface directory (not > even in google), just the cli files in the scripts folder and another > UserHandler.php file inside scripts/models-ext/. Is that the file you > meant for me to check? Yes, exactly. The commandline interface is quite new and unfortunately not mentioned in the documentation - and therefore more or less invisible for Google... The files in scripts/models-ext/ will hopefully be merged with those in model/ - but that should not stop you. The worst thing that can happen is that you have to change the include path ;-) If you want/need to change something inside scripts/models-ext/, please send a short mail before to avoid that we get a marge problem. @David: please see my other mail about merging the classes and comment there on the changes. > I'm feeling a bit dumb with this, all you said > sounds quite obvious but I'm still a bit loss here. I'd say you are on the right track ;-) Regards, Christian Boltz -- 'blah blahb lah' ... I have no idea where it's coming from, grepping rug and zmd sources does not have any matches. [from https://bugzilla.novell.com/show_bug.cgi?id=157405] |
