tags 481505 patch thanks -=| Niko Tyni, Sat, May 17, 2008 at 02:01:32PM +0300 |=- > FWIW, the attached trivial patch changes the backend to GDBM_File > while still passing all the tests. This fixes the sparse file problem, > but it's probably not a viable alternative for upstream because > gdbm linkage is optional while sdbm comes with Perl. And it's still > architecture-dependent. > > I suppose an architecture-independent alternative would be to use > nstore_fd and fd_retrieve from Storable, which means that the whole > database is slurped in for just one lookup. This is probably an > unacceptable regression. > > Better suggestions welcome.
I can't think of a solution that kills all rabits: 1. doesn't rely on shipping a 400M file 2. is architecture-independent 3. efficiant lookups, i.e. do not read the whole database for a single lookup Right now, MARC::Charset only satisfies 3. The patch proposed also fullfils 1, which is a great improvement for space-constrained systems. GDBM_File is already shipped in the perl package, so at least on Debian the impact is zero. I am going to prepare a patched package and upload unless there are objections. Ah, I'll also forward the report upstream, noting the patched Debian package. -- dam JabberID: [EMAIL PROTECTED]
signature.asc
Description: Digital signature