Re: [PATCH v2 00/15] xfstests: remove IRIX support

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

 



On Thu, Jul 20, 2017 at 09:38:38PM -0700, Eric Biggers wrote:
> Hello,
> 
> This is v2 of the patchset to remove IRIX support from xfstests.  For
> some time xfstests has de facto not supported IRIX anymore.  The final
> release of IRIX was over 10 years ago and it seems no one has been
> running the latest xfstests releases on any IRIX system, as at the very
> least xfstests now assumes the existence of some util-linux binaries.
> There are very likely other examples of this bitrot as well.  Also,
> xfsprogs has already dropped IRIX support.
> 
> Therefore, this patchset simplifies things by removing IRIX support from
> the xfstests tests, build system, and documentation.  It also ports the
> useful IRIX-specific tests to Linux.  Of course, people will still find
> more cruft later to remove, but this should be a good start.
> 
> To get around the mailing list message size limit, the series is
> formatted with --irreversible-delete.  Therefore it can't be applied
> directly.  It can be retrieved from git here:
> https://github.com/ebiggers/xfstests.git, branch remove_irix_support_v2.

Great work! Thank you so much for doing this! I pulled & merged your
branch into a test branch and am running some tests now, overall it
looks good to me.

Though, as you mentioned, there're some places that could do further
cleanups (e.g. I'm wondering if we can kill _supported_os completely and
remove it from 'new' template, and some $HOSTOS and $PKG_PLATFORM checks
could be cleaned up too), I agree this is a great start. Thanks again!

Eryu
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux