Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove obsolete tools #97

Closed
kdave opened this issue Feb 15, 2018 · 2 comments
Closed

Remove obsolete tools #97

kdave opened this issue Feb 15, 2018 · 2 comments

Comments

@kdave
Copy link
Owner

@kdave kdave commented Feb 15, 2018

The standalone tools that have been obsoleted long ago and have a replacement in the main tool btrfs can be now removed.

  • remove sources
  • document removal in manual pages (btrfs(8) STANDALONE TOOLS)
  • update Makefile
  • btrfs-debug-tree
  • btrfs-zero-log
  • btrfs-show-super
  • btrfs-calc-size
@kdave kdave added this to the v4.16 milestone Feb 15, 2018
@kdave kdave added this to WIP in Enhancements Feb 27, 2018
@kdave kdave moved this from WIP to TODO in Enhancements Feb 27, 2018
@kdave kdave modified the milestones: v4.16, future Mar 21, 2018
@lorddoskias
Copy link

@lorddoskias lorddoskias commented Mar 22, 2018

Tentative patches sent to the mailing list.

@kdave kdave removed the free label Mar 22, 2018
kdave added a commit that referenced this issue Apr 9, 2018
There is already a replacement in the face of btrfs inspect-internal
dump-tree. And this command is just a simple wrapper around it. Just
remove it and adjust the show-blocks script to call the main btrfs
binary to achieve the same effect.

Informally deprecated since 4.4.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 9, 2018
Its function has been subsumed by "btrfs rescue zero-log". Remove its
source file and adjust make/tests soruces accordingly.

Deprecated since 4.0.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 9, 2018
Its function has been superseded by btrfs inspect-internal show-super.
Furthermore the tools is currently not built by default. Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 9, 2018
Its function has been superseded by btrfs inspect-internal tree-stats.
Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
@kdave kdave modified the milestones: future, v4.16.1 Apr 9, 2018
kdave added a commit that referenced this issue Apr 9, 2018
Its function has been superseded by btrfs inspect-internal show-super.
Furthermore the tools is currently not built by default. Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 9, 2018
Its function has been superseded by btrfs inspect-internal tree-stats.
Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
@kdave
Copy link
Owner Author

@kdave kdave commented Apr 9, 2018

V2 added to devel.

@kdave kdave closed this Apr 9, 2018
@kdave kdave removed the easy label Apr 9, 2018
kdave added a commit that referenced this issue Apr 11, 2018
There is already a replacement in the face of btrfs inspect-internal
dump-tree. And this command is just a simple wrapper around it. Just
remove it and adjust the show-blocks script to call the main btrfs
binary to achieve the same effect.

Informally deprecated since 4.4.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 11, 2018
Its function has been subsumed by "btrfs rescue zero-log". Remove its
source file and adjust make/tests soruces accordingly.

Deprecated since 4.0.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 11, 2018
Its function has been superseded by btrfs inspect-internal show-super.
Furthermore the tools is currently not built by default. Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 11, 2018
Its function has been superseded by btrfs inspect-internal tree-stats.
Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 24, 2018
There is already a replacement in the face of btrfs inspect-internal
dump-tree. And this command is just a simple wrapper around it. Just
remove it and adjust the show-blocks script to call the main btrfs
binary to achieve the same effect.

Informally deprecated since 4.4.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 24, 2018
Its function has been subsumed by "btrfs rescue zero-log". Remove its
source file and adjust make/tests soruces accordingly.

Deprecated since 4.0.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 24, 2018
Its function has been superseded by btrfs inspect-internal show-super.
Furthermore the tools is currently not built by default. Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
kdave added a commit that referenced this issue Apr 24, 2018
Its function has been superseded by btrfs inspect-internal tree-stats.
Just remove it.

Deprecated since 4.8.

Issue: #97
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
@kdave kdave added the review:ok label Oct 23, 2018
@kdave kdave removed this from TODO in Enhancements Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.