Revised patch attached!


Hi Dan,
and thank you for the feedback.

>>  Adds support for nilfs2 to blkid.

>>  Enabled with CONFIG_FEATURE_VOLUMEID_NILFS.
> 
> Since this is nilfs2, not nilfs1, most (all?) of the references to nilfs
> should probably say nilfs2 instead.

Hmm, I was not so concerned about nilfs1, since it is mostly experimental.
The idea was rather to look into the future and not require a config
change when/if nilfs3 shows up. Much the same way as ext2/3/4 is
implemented. Maybe a long shot, but anyway...

I have done some minor changes to the patch, so it is less general, but
still leaves the door to the future open, see attachment.

>>  +    // The scondary superblock is not always used, so ignore it for now.
> 
> s/scondary/secondary/

Fixed in attached patch.

Thanx
/S-G

Attachment: busybox-1.20.1-blkid-nilfs.patch
Description: Binary data

_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to