Skip Menu |

This queue is for tickets about the Search-Kinosearch CPAN distribution.

Report information
The Basics
Id: 13775
Status: resolved
Priority: 0/
Queue: Search-Kinosearch

People
Owner: CREAMYG [...] cpan.org
Requestors: qef [...] ungwe.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.03_02
Fixed in: (no value)



Subject: temp_directory must be on same partition as index
Hi, It seems that the -temp_directory option to Search::Kinosearch::Kindexer->new() has to be a path on the same partition as the -mainpath directory. This caught me out when I left it to the default of the current directory while putting the index on a different partition in case it got too big. It failed with this message: Couldn't move directory 'kinotemp_HKdy/subkindex0' to '/blah/index/subkindex0': Is a directory at index.pl line 49 The line number refers to the "$kindexer->write_kindex;" call in my 'index.pl' script. It's probably not a big deal to make this work, since as things are it forces you to use a more efficient place for temp files, but I think the documentation should point this out prominently, both where -temp_directory is pointed out and probably in the synopsis where the indexer object is created. hth, geoff
The ultimate solution to this is probably to change the default behavior to comply with expected practice -- put the temp directory wherever File::Temp says it should go. The option to override the temp directory location for the purpose of performance gains can be documented. For now, I think I'll make a short documentation patch and whip up a bugfix release that also fixes a couple other problems. A major change in the API is coming, and a change in the temp directory behavior should happen at the same time. Thanks, -- Marvin Humphrey