Re: [PATCH] btrfs-progs: have restore set atime/mtime

2015-04-17 Thread Dan Merillat
On Fri, Apr 17, 2015 at 7:54 AM, Noah Massey noah.mas...@gmail.com wrote: On Thu, Apr 16, 2015 at 7:33 PM, Dan Merillat dan.meril...@gmail.com wrote: The inode is already found, use the data and make restore friendlier. Signed-off-by: Dan Merillat dan.meril...@gmail.com --- cmds-restore.c |

Re: [PATCH] btrfs-progs: have restore set atime/mtime

2015-04-16 Thread Duncan
Dan Merillat posted on Thu, 16 Apr 2015 19:33:46 -0400 as excerpted: The inode is already found, use the data and make restore friendlier. Unless things have changed recently, restore doesn't even restore user/ group ownership, let alone permissions. IOW, atime/mtime are the least of the

Re: [PATCH] btrfs-progs: have restore set atime/mtime

2015-04-16 Thread Duncan
On Thu, 16 Apr 2015 22:19:46 -0400 Dan Merillat dan.meril...@gmail.com wrote: [Reordered to standard list quote/reply-in-context order.] On Thu, Apr 16, 2015 at 9:09 PM, Duncan 1i5t5.dun...@cox.net wrote: Dan Merillat posted on Thu, 16 Apr 2015 19:33:46 -0400 as excerpted: The inode is

[PATCH] btrfs-progs: have restore set atime/mtime

2015-04-16 Thread Dan Merillat
The inode is already found, use the data and make restore friendlier. Signed-off-by: Dan Merillat dan.meril...@gmail.com --- cmds-restore.c | 12 1 file changed, 12 insertions(+) diff --git a/cmds-restore.c b/cmds-restore.c index d2fc951..95ac487 100644 --- a/cmds-restore.c +++

Re: [PATCH] btrfs-progs: have restore set atime/mtime

2015-04-16 Thread Dan Merillat
That's not a bad idea. In my case it was all owned by the same user (media storage) so the only thing of interest was the timestamps. I can whip up a patch to do that as well. On Thu, Apr 16, 2015 at 9:09 PM, Duncan 1i5t5.dun...@cox.net wrote: Dan Merillat posted on Thu, 16 Apr 2015 19:33:46

Re: [PATCH] btrfs-progs: have restore set atime/mtime

2015-04-16 Thread Dan Merillat
I think thunderbird ate that patch, sorry. I didn't make it conditional - there's really no reason to not restore the information. I was actually surprised that it didn't restore before this patch. If it looks good I'll resend without the word-wrapping. -- To unsubscribe from this list: send