-------- 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,
Qu
@@ -47,4 +49,3 @@ SEE ALSO
`mkfs.btrfs`(8),
`btrfs-scrub`(8),
`btrfs-rescue`(8)
-`btrfsck`(8)
Sorry to bother you but 'btrfs-scrub'/'btrfs-rescue' and 'btrfs-restore'
seems also metioning 'btrfsck' and may also needs to remove 'btrfsck'.
Thanks for catching them, I'll fix it up.
--
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
--
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
