On Thu, May 08, 2014 at 09:40:03AM +0800, Qu Wenruo wrote:
> -------- Original Message --------
> Subject: Re: [PATCH] btrfs-progs: doc: link btrfsck to btrfs-check
> From: David Sterba <dste...@suse.cz>
> To: Qu Wenruo <quwen...@cn.fujitsu.com>
> Date: 2014年04月18日 22:48
> >On Thu, Apr 17, 2014 at 08:47:28AM +0800, Qu Wenruo wrote:
> >>>@@ -73,6 +74,7 @@ install: install-man
> >>>  install-man: man
> >>>   $(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
> >>>   $(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
> >>>+  $(LNS) btrfs-check.txt $(DESTDIR)$(man8dir)
> >>Shouldn't the source of soft link be btrfs-check.8.gz. ?
> Forgot to mention that the dest is also wrong. This will make
> $(DESTDIR)$(man8dir)/btrfs-check.8.gz to be a
> infinite loop(pointing to it self).
> The correct one should be like the following:
> +       $(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)/btrfsck.8.gz

Thanks, it was reported & fixed a few days ago, though it's not in the
integration branch, lag is on my side.

https://patchwork.kernel.org/patch/4115501/
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to