[rt-commit] CVS commit: rt
Jesse
jesse at fsck.com
Fri May 5 10:11:16 EDT 2000
Why do you want them in a namespace other than RT:: ?
On Fri, May 05, 2000 at 10:01:22AM -0400, tobiasb at fsck.com wrote:
>
> 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
>
>
>
> _______________________________________________
> Rt-commit mailing list
> Rt-commit at lists.fsck.com
> http://lists.fsck.com/mailman/listinfo/rt-commit
>
--
jesse reed vincent -- jrvincent at wesleyan.edu -- jesse at fsck.com
pgp keyprint: 50 41 9C 03 D0 BC BC C8 2C B9 77 26 6F E1 EB 91
--------------------------------------------------------------
"Bother," said Pooh, "Eeyore, ready two photon torpedoes and lock
phasers on the Heffalump, Piglet, meet me in transporter room three"
More information about the Rt-commit
mailing list