[Catalyst-commits] r10415 - Catalyst-Devel/1.00/branches/helper_refactor/gsoc

[dhoss at dev.catalyst.perl.org]

Author: dhoss
Date: 2009-06-02 04:32:11 +0000 (Tue, 02 Jun 2009)
New Revision: 10415

Modified:
   Catalyst-Devel/1.00/branches/helper_refactor/gsoc/gsoc-devel-plan-5-23.txt
Log:
updated todo


Modified: Catalyst-Devel/1.00/branches/helper_refactor/gsoc/gsoc-devel-plan-5-23.txt
===================================================================
--- Catalyst-Devel/1.00/branches/helper_refactor/gsoc/gsoc-devel-plan-5-23.txt	2009-06-02 03:27:48 UTC (rev 10414)
+++ Catalyst-Devel/1.00/branches/helper_refactor/gsoc/gsoc-devel-plan-5-23.txt	2009-06-02 04:32:11 UTC (rev 10415)
@@ -104,7 +104,7 @@
 5-28
 03:04 < dhoss> SO. official next step: 1) #done trim crap out 2) #done write test for how 
                helpers are invoking Catalyst::Helper 3)??? 4) profit!
-03:05 < t0m> 3) Re arrange files in sharedir so we can make the comversion guts 
+03:05 < t0m> 3)#done Re arrange files in sharedir so we can make the comversion guts 
              totally generic
 03:05 < t0m> 4) merge&profit
 03:05 < t0m> *conversion
@@ -118,5 +118,25 @@
                the one in topic
 03:06 < t0m> cause then the entire generate application becomes 'copy this 
              directory from X => Y, applying some file mangling rules)
+             
+=============================================================================
+04:26 < t0m> dhoss: erm, yeah - so the two things to do is: unit tests for the 
+             methods like get_file which we're not actually using any more..
+04:27 < t0m> (do that by making t/lib/ExampleHelper.pm - copy the TT one from 
+             CPAN or something, and then write some tests that get_file can 
+             successfully pull chunks out the __DATA__ segment
+04:28 < t0m> and rename everything in the sharedir to be named as-per where it 
+             ends up in your app, but suffixed by .tt or .bin
+04:29 < t0m> e.g. share/Makefile.PL.tt share/root/favicon.ico.bin 
+             share/lib/MyApp.pm.tt
+04:30 < t0m> the exact reasons for this nameing scheme become aparent post 
+             first merge, when we remove most of the code :)
+04:30 < t0m> but all of share/ needs to be in it's final resting place pre-merge
+04:31 < marcus> http://lumberjaph.net/blog/index.php/2009/05/30/catalystxdispatcherasgraph/ # <3 lazyweb
+04:31 < t0m> so tests the stuff we're no longer using in Helper.pm (e.g. 
+             get_file) still works. Rename share/ bits
+04:31 < t0m> marcus: Indeed :)
+04:31 < t0m> ^^ those two, then first merge.
 
 
+