[dev] Send event reminders and daily agendas for different domains
Gonçalo Queirós
goncalo.queiros at portugalmail.net
Mon May 30 14:54:35 UTC 2011
On 05/25/2011 03:00 PM, Gonçalo Queirós wrote:
>
> On 05/25/2011 02:48 PM, Gonçalo Queirós wrote:
>>> Yes, this is the right direction. I would strip cli_ from the option
>>> name though, because it could potentially be users for different
>>> things than CLI scripts. And I would rather use an environment
>>> variable (SERVER_NAME) than a command line argument for the actual
>>> script, because you can run several several scripts under the same
>>> environment, or one script under several environments, without
>>> changing the script calls. Finally, this should of course be used by
>>> all CLI scripts.
>>>
>>> Jan.
>>>
>> If i set the SERVER_NAME environment variable, then the scripts from CLI
>> wouldn't run because they would fail on Horde_Cli::runningFromCLI(),
>> that's why i used a command line argument.
>>
> Nvm this..i didn't interpreted this as "shell environment variable"
>
Fixed the previous patch proposal.
What do you think Jan?
--
Gonçalo Queirós
Eng. Software
*m.* 913918777
*Portugalmail* | plataformas de inovação
*w.* http://www.portugalmail.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: horde.patch
Type: text/x-patch
Size: 1430 bytes
Desc: not available
URL: <http://lists.horde.org/archives/dev/attachments/20110530/93a84101/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: registry.patch
Type: text/x-patch
Size: 859 bytes
Desc: not available
URL: <http://lists.horde.org/archives/dev/attachments/20110530/93a84101/attachment-0001.bin>
More information about the dev
mailing list