Re: [PATCH 0/5] e2fsprogs: some small cleanups and fixes for journal code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jul 26, 2014 at 11:33:29AM +0400, Azat Khuzhin wrote:
> On Thu, Jul 10, 2014 at 06:35:03PM +0400, Azat Khuzhin wrote:
> > [PATCH 1/5] lib: add EXT2_ prefix for SUPERBLOCK_{OFFSET,SIZE}
> > [PATCH 2/5] journal: use consts instead of 1024 and add helper for
> > [PATCH 3/5] tune2fs: remove_journal_device(): use the correct block
> > [PATCH 4/5] tune2fs: update journal super block when changing UUID
> > [PATCH 5/5] tune2fs: update journal users while updating fs UUID
> 
> Hi Ted, any news on this?

What branch was this against?  The first patch scared me, since making
global changes to rename a constant is very likely to cause all sorts
of patch conflicts, especially when I have to juggle patches across
the maint and next branches.  So that's why I put it off.

I know there are some real bug fixes here, and it's helpful it those
come *first*, and make the minimal changes necessary, so I can apply
it to the maint branch and not have to worry about potential merge
conflicts.

In general, for global renames/cleanups, especially ones that don't
actually fix any bugs, and which are there mostly for aesthetic
reasons, I'd much prefer to hold off making them until after 1.43.0 is
released, and after I've a abandoned making any further releases
(except for truely catastrophic bugs) on the 1.42.x branch.

Cheers,

						- Ted

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux