On Mon, Jul 30, 2012 at 05:45:32PM -0400, Theodore Ts'o wrote: > On Thu, Jun 28, 2012 at 12:43:56PM +1000, Tony Breeds wrote: > > > > Okay so I think you want somethign like: > > Yep! > > > What I did was remove dblist.o from my libext2fs.a and I now get: > > /home/tony/src/e2fsprogs/../e2fsprogs-root/lib/libext2fs.a(gen_bitmap64.o): In function `ext2fs_alloc_generic_bmap': > > /home/tony/src/e2fsprogs-build/lib/ext2fs/../../../e2fsprogs/lib/ext2fs/gen_bitmap64.c:112: undefined reference to `ext2fs_get_num_dirs' > > > > So it looks like dblist.o is being pulled in for ext2fs_get_num_dirs() > > in that case a EXT2FS_BMAP64_AUTODIR bmap type has been asked for. > > > > How would you feel about moving ext2fs_get_num_dirs from dblist.c to > > blknum.c? > > Sure, no problem. > > Take a look at this set of patches (see attached). With this and the > set of ext2fs functions that I understand yaboot is using, I think it > solves the concern that you have. This will be in the next branch of > the e2fsprogs. Thanks so much Ted for your help with my patches and the next set. I tried next + your 7 patches and yaboot only fails with errors for calloc and abort. Both of these I'm very happy to fix in yaboot. Looking forward to v1.42.6 :) Yours Tony
Attachment:
pgpsBZghxaivO.pgp
Description: PGP signature