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
