Re: [PATCH] Btrfs-progs: remove btrfsctl, btrfs-show and btrfs-vol from default build

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

 



On 1/25/13 5:57 AM, Gene Czarcinski wrote:
> On 06/08/2012 02:00 PM, Stefan Behrens wrote:
>> Remove btrfsctl, btrfs-show and btrfs-vol from "all" target of Makefile.

TBH, unless there are objections, I'd rather remove the source files as well.

Having unbuilt source code lying around is potentially even more confusing.

I'd at least move it to a deprecated/ subdir or something, but it'll only
fester and die there as well.  I'd just remove it, if it's time to do so.

Are they fully deprecated?  Is there anything these tools do that btrfs
does not?

-Eric

>> Signed-off-by: Stefan Behrens <sbehrens@xxxxxxxxxxxxxxxx>
>> ---
>>   Makefile |    2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Makefile b/Makefile
>> index aaf1381..79e7a56 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -18,7 +18,7 @@ bindir = $(prefix)/bin
>>   LIBS=-luuid
>>   RESTORE_LIBS=-lz
>>
>> -progs = btrfsctl mkfs.btrfs btrfs-debug-tree btrfs-show btrfs-vol btrfsck \
>> +progs = mkfs.btrfs btrfs-debug-tree btrfsck \
>>       btrfs btrfs-map-logical btrfs-image btrfs-zero-log btrfs-convert \
>>       btrfs-find-root btrfs-restore btrfstune
>>
>>
> I agree.  It is time to remove these old programs; they serve no useful purpose; and they complicate maintenance.
> 
> Gene
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux