[rt-commit] CVS commit: rt

tobiasb at fsck.com tobiasb at fsck.com
Fri May 5 10:01:22 EDT 2000


Module Name:	rt
Committed By:	tobiasb
Date:		Fri May  5 14:01:22 UTC 2000

Modified Files:
	rt/etc: config.pm

Log Message:
Added Log::Dispatch.  I also have added those comments, which I would
like comments on (:

# Most (if not all?) $RT:: global variables should be here.  I'd
# suggest putting session information in another Namespace (main:: or
# RT::main or maybe something like that).

#use strict;

#use vars qw/%SitePolicy $dirmode $transactionmode $DatabasePassword $rtname $domain $host $DatabaseHost $DatabaseUser $RT::DatabaseName $DatabaseType $user_passwd_min $MailAlias $WebrtImagePath $web_auth_mechanism $web_auth_cookies_allow_no_path $DefaultLocale $LocalePath $Nobody $Logger/;

I'm not sure if "use strict" breaks, but at least "use vars" breaks
bigtime as we are using it other places (like rtmux.pl).  I suggest
using another package name for the execution logic.  What do you
think?


To generate a diff of this commit:
cvs rdiff -u -r1.11.2.16 -r1.11.2.17 rt/etc/config.pm






More information about the Rt-commit mailing list