My first post here, so Hi. I have (my first) module for cpan and to try and avoid some "obvious" newbie mistakes ... a couple of questions:
I'm using App::Cache, which creates a .$dir in the users File::HomeDir. Running the tests of App::Cache leaves ~/.app_cache_test lingering in the filesystem. For my module App::Cache would create ~/.parse_cpan_cached_test Should I (a) reporting the lingering dir as a bug (and offer a patch?) in App::Cache and move on (b) ignore the fact the tests leave directories lingering around (c) try and clean up after myself (d) something else? The problem with (c) is that App::Cache doesn't have a documented method for extracting the directory in question. It does have a (undocumented) $app_cache->directory which appears to be the right thing. That aside, I think my module is ready to upload. If anyone was inclined to do a quick (code) review and offer any feedback I would much appreciate it. I can offer you a beer or similar beverage of choice at yapc::eu in return. cpan bundle: http://sysmonblog.co.uk/tmp/Parse-CPAN-Cached-0.01.tar.gz git: http://github.com/minty/parse-cpan-cached/tree/master Any issues with the name(space)? A additional (slightly un-related) question - I can't find any specific advice on uploading Catalyst apps to CPAN. Is there any, or does one just bundle it in the same way? My app needs a mysql db setup so I'm not sure how to go about flagging that dependency and/or how to deal with the "mysqladmin create db_name" & granting user permissions, other than in a README. I'm using DBIx::Class, so I can deploy the schema in code, but the db setup less so. I'm specifically using MySQL fulltext indexes, so although sqlite has fulltext indexes they aren't compatible with what I need ... thus I believe sqlite ain't an option for testing. Thanks for your time, Murray. ps. It did occur that this module is perhaps too simple as to not warrant being on cpan. Fwiw, I have a catalyst app I want to release. I want to abstract this logic out for clarity reasons in this catalyst app. I'm inclined to try and make the catalyst app smaller by pulling out suitable modules onto cpan. Hence, this module.
