On Tue, Aug 10, 2021 at 10:15:49AM +0200, Christoph Hellwig wrote: > On Sat, Jul 31, 2021 at 06:58:29PM +0200, Christian Brauner wrote: > > From: Christian Brauner <christian.brauner@xxxxxxxxxx> > > > > - Use sequential option numbering for option parsing since we're only > > using longopts anyway and to easier correlate test parsing in the > > switch with the options specified in longopts. > > These sequential options are rally weird. Why not use getopt_long_only? Good point! Let me see if I can make that work. > > > > > - Introduce an explicit command line switch to runs the basic test > > suite. This prepares for the introduction of additional command line > > switches to run tests. > > - Update help output. > > - Use die() when logging test failure. > > Each of these changes really should be a separate patch. Ok, will split this up. Thanks for the reviews, much appreciated! Christian
