On Thu, Jan 24, 2013 at 03:09:53PM -0700, Goffredo Baroncelli wrote:
> On 01/24/2013 08:42 PM, Eric Sandeen wrote:
> > On 1/24/13 11:57 AM, Goffredo Baroncelli wrote:
> >> On 01/24/2013 10:23 AM, Stefan Behrens wrote:
> >>> On Wed, 23 Jan 2013 22:39:29 -0600, Eric Sandeen wrote:
> >>>> instead of renaming& keeping the btrfsctl.c copy
> >>>
> >>> There is a new momentum to improve the Btrfs-progs quality :)
> >>>
> >>> IMO, one step is to get rid of the legacy tools and sources. It
> >>> wastes time to maintain them and these old tools cause confusion.
> >>> btrfsctl.c, btrfs-vol.c and btrfs-show.c are not needed anymore.
> >>> Please correct me if there are plans to use these old tools in
> >>> future Linux distributions. The "btrfs" tool replaces the legacy
> >>> "btrfsctl", "btrfs-vol" and "btrfs-show" tools. Below, the usage
> >>> text of the old tools is quoted. All these tasks are also offered
> >>> in the "btrfs" tool, and this tool is the newer one.
> >>
> >> I fully agree: btrfsctl, btrfs-vol, btrfs-show are perfectly
> >> replaced by by btrfs. Moreover time to time the patches are more
> >> complex than the needing because exists these "legacy" programs.
> >>
> >> I checked the debian package, and to me seems that there is no need
> >> of {btrfsctl,btrfs-vol,btrfs-show}
> >
> > Hm, they are shipped in the Fedora package.
>
> The same is true for the debian package, but are these used in Fedora ?
>
> >
> > For backwards compat, could those be turned into shell scripts which
> > invoke the btrfs tool?
>
> I don't see any gain to maintains a script bash (which has to be written
> from scratch) instead of maintains the current C code.
>
> These programs were deprecated two years ago [1]. If some distribution
> need them, could maintain them as separate patch. But I think that the
> mainstream should remove.
I'd say that if SuSE or oracle depend on them we keep them. Otherwise,
I'm fine with removing them or just making the 3 line bash script.
-chris
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
