Show quoted text>>>>> On Mon, 23 Jul 2007 16:15:55 -0400, "Michael G Schwern via RT" <bug-CPAN@rt.cpan.org> said:
> Martin Thurn via RT wrote:
>> The configuration question regarding build_source_where, as well as the
>> CPAN pod itself, incorrectly implies that the user can choose to NOT
>> keep sources. Please delete the phrase "(if we do)" from CPAN.pm, and
>> please delete the phrase "Unless you are accessing the CPAN via the
>> filesystem directly" from FirstTime.pm
> I believe the intention there is to communicate that if you're accessing a
> local CPAN mirror via file URLs (example: file:///var/local/CPAN_mirror) then
> it will not store a copy of the source tarball in sources. My observations
> show that it doesn't. It does, however, create a directory for the author in
> authors/id/
Show quoted text > The wording could be made clearer. Also the author directory probably
> shouldn't be created.
I think I managed to improve wording:
--- lib/CPAN/FirstTime.pm (revision 2880)
+++ lib/CPAN/FirstTime.pm (local)
@@ -1072,9 +1072,9 @@
keep_source_where => qq{
-Unless you are accessing the CPAN via the filesystem directly CPAN.pm
-needs to keep the source files it downloads somewhere. Please supply a
-directory where the downloaded files are to be kept.},
+Unless you are accessing the CPAN on your filesystem via a file: URL,
+CPAN.pm needs to keep the source files it downloads somewhere. Please
+supply a directory where the downloaded files are to be kept.},
build_cache_intro => qq{
I'm reluctant to fix the unneeded creation of the directory but made a
note in the code to fix it next time I have something to change in
that area. (Rev. 2069 in SVN)
Thanks!
--
andreas
