[drupal-devel] [task] Performance: fix all include paths
Issue status update for http://drupal.org/node/30122 Post a follow up: http://drupal.org/project/comments/add/30122 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: Anonymous Reported by: m3avrck Updated by: m3avrck Status: patch (code needs review) Attachment: http://drupal.org/files/issues/drupal_2.patch (9.68 KB) This patch fixes all of the outstanding Drupal include paths by placing the './' in front of the include path. This brings all of the Drupal core files up to date with the boostrap.inc which made first use of this performance gain. In simple testing, I've have noticed much faster load times with Drupal on my slow machine. Definetly makes a big difference! m3avrck
Issue status update for http://drupal.org/node/30122 Post a follow up: http://drupal.org/project/comments/add/30122 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: Anonymous Reported by: m3avrck Updated by: m3avrck Status: patch (code needs review) Attachment: http://drupal.org/files/issues/drupal_3.patch (8.94 KB) Ok updated patch and coding styles according to comments from IRC. m3avrck Previous comments: ------------------------------------------------------------------------ Wed, 31 Aug 2005 18:44:38 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_2.patch (9.68 KB) This patch fixes all of the outstanding Drupal include paths by placing the './' in front of the include path. This brings all of the Drupal core files up to date with the boostrap.inc which made first use of this performance gain. In simple testing, I've have noticed much faster load times with Drupal on my slow machine. Definetly makes a big difference!
Issue status update for http://drupal.org/node/30122 Post a follow up: http://drupal.org/project/comments/add/30122 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: Anonymous Reported by: m3avrck Updated by: Dries -Status: patch (ready to be committed) +Status: patch (code needs work) Patch no longer applies cleanly. Should be easy to fix though. Dries Previous comments: ------------------------------------------------------------------------ Wed, 31 Aug 2005 18:44:38 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_2.patch (9.68 KB) This patch fixes all of the outstanding Drupal include paths by placing the './' in front of the include path. This brings all of the Drupal core files up to date with the boostrap.inc which made first use of this performance gain. In simple testing, I've have noticed much faster load times with Drupal on my slow machine. Definetly makes a big difference! ------------------------------------------------------------------------ Wed, 31 Aug 2005 19:18:16 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_3.patch (8.94 KB) Ok updated patch and coding styles according to comments from IRC.
Issue status update for http://drupal.org/node/30122 Post a follow up: http://drupal.org/project/comments/add/30122 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: Anonymous Reported by: m3avrck Updated by: m3avrck -Status: patch (code needs work) +Status: patch (ready to be committed) Attachment: http://drupal.org/files/issues/drupal_5.patch (10.62 KB) Rerolled against head. m3avrck Previous comments: ------------------------------------------------------------------------ Wed, 31 Aug 2005 18:44:38 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_2.patch (9.68 KB) This patch fixes all of the outstanding Drupal include paths by placing the './' in front of the include path. This brings all of the Drupal core files up to date with the boostrap.inc which made first use of this performance gain. In simple testing, I've have noticed much faster load times with Drupal on my slow machine. Definetly makes a big difference! ------------------------------------------------------------------------ Wed, 31 Aug 2005 19:18:16 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_3.patch (8.94 KB) Ok updated patch and coding styles according to comments from IRC. ------------------------------------------------------------------------ Wed, 07 Sep 2005 20:38:46 +0000 : Dries Patch no longer applies cleanly. Should be easy to fix though.
Issue status update for http://drupal.org/node/30122 Post a follow up: http://drupal.org/project/comments/add/30122 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: Anonymous Reported by: m3avrck Updated by: m3avrck Status: patch (ready to be committed) Attachment: http://drupal.org/files/issues/drupal_6.patch (8.9 KB) One more time, bug fixed and settings.php removed. m3avrck Previous comments: ------------------------------------------------------------------------ Wed, 31 Aug 2005 18:44:38 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_2.patch (9.68 KB) This patch fixes all of the outstanding Drupal include paths by placing the './' in front of the include path. This brings all of the Drupal core files up to date with the boostrap.inc which made first use of this performance gain. In simple testing, I've have noticed much faster load times with Drupal on my slow machine. Definetly makes a big difference! ------------------------------------------------------------------------ Wed, 31 Aug 2005 19:18:16 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_3.patch (8.94 KB) Ok updated patch and coding styles according to comments from IRC. ------------------------------------------------------------------------ Wed, 07 Sep 2005 20:38:46 +0000 : Dries Patch no longer applies cleanly. Should be easy to fix though. ------------------------------------------------------------------------ Wed, 07 Sep 2005 21:02:38 +0000 : m3avrck Attachment: http://drupal.org/files/issues/drupal_5.patch (10.62 KB) Rerolled against head.
participants (2)
-
Dries -
m3avrck