[jifty-devel] Re: [Jifty-commit] r858 - jifty/trunk/lib/Jifty

[Jesse Vincent]

I believe this patch will break jifty app -name FooApp by hardcoding pathis in its default config file.

Best,
Jesse
-- This message was sent from my Treo. Please accept my apologies for its brevity and for any typos. 
-----Original Message-----
From: jifty-commit at lists.jifty.org
Date: Thursday, Apr 13, 2006 4:01 am
Subject: [Jifty-commit] r858 - jifty/trunk/lib/Jifty

Author: autrijus
Date: Thu Apr 13 04:01:02 2006
New Revision: 858

Modified:
   jifty/trunk/lib/Jifty/Config.pm

Log:
* Jifty::Config: when guessing initial config, always use absolute
  path instead of relative ones, as lighttpd does a implicit "chdir"
  into document root, rendering relative paths for e.g. PO files useless.

Modified: jifty/trunk/lib/Jifty/Config.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Config.pm	(original)
+++ jifty/trunk/lib/Jifty/Config.pm	Thu Apr 13 04:01:02 2006
@@ -208,6 +208,9 @@
     $app_class =~ s/-/::/g;
     my $db_name = lc $app_name;
     $db_name =~ s/-/_/g;
+
+    my $app_root = Jifty::Util->app_root;
+
     return {
         framework => {
             AdminMode        => 1,
@@ -229,15 +232,15 @@
             Mailer     => 'Sendmail',
             MailerArgs => [],
             L10N       => {
-                PoDir => 'share/po',
+                PoDir => "$app_root/share/po",
             },
             Web        => {
                 Port => '8888',
                 BaseURL => 'http://localhost',
-                SessionDir  => "var/session",
-                DataDir     => "var/mason",
-                StaticRoot   => "share/web/static",
-                TemplateRoot => "share/web/templates",
+                SessionDir  => "$app_root/var/session",
+                DataDir     => "$app_root/var/mason",
+                StaticRoot   => "$app_root/share/web/static",
+                TemplateRoot => "$app_root/share/web/templates",
                 ServeStaticFiles => 1,
                 MasonConfig => {
                     autoflush    => 0,
_______________________________________________
Jifty-commit mailing list
Jifty-commit at lists.jifty.org
http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-commit