xfs
[Top] [All Lists]

Re: [PATCH 01/17] xfsprogs: use common code for multi-disk detection

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH 01/17] xfsprogs: use common code for multi-disk detection
From: Jan Tulak <jtulak@xxxxxxxxxx>
Date: Fri, 19 Jun 2015 07:51:50 -0400 (EDT)
Cc: Dave Chinner <dchinner@xxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20150619111022.GA5212@xxxxxxxxxxxxx>
References: <1434711726-13092-1-git-send-email-jtulak@xxxxxxxxxx> <1434711726-13092-2-git-send-email-jtulak@xxxxxxxxxx> <20150619111022.GA5212@xxxxxxxxxxxxx>
Thread-index: z8C3ajR26wFywTblnOuM+WpcUDCujQ==
Thread-topic: xfsprogs: use common code for multi-disk detection

----- Original Message -----
> From: "Christoph Hellwig" <hch@xxxxxxxxxxxxx>
> To: "Jan ??ul??k" <jtulak@xxxxxxxxxx>
> Cc: "Dave Chinner" <dchinner@xxxxxxxxxx>, xfs@xxxxxxxxxxx
> Sent: Friday, June 19, 2015 1:10:22 PM
> Subject: Re: [PATCH 01/17] xfsprogs: use common code for multi-disk detection
> 
> > index 70e43a0..3269ec3 100644
> > --- a/include/Makefile
> > +++ b/include/Makefile
> > @@ -26,9 +26,15 @@ QAHFILES = libxfs.h libxlog.h \
> >     xfs_inode.h \
> >     xfs_log_recover.h \
> >     xfs_metadump.h \
> > +   xfs_mkfs.h \
> >     xfs_mount.h \
> > +   xfs_quota_defs.h \
> > +   xfs_sb.h \
> > +   xfs_shared.h \
> >     xfs_trace.h \
> > -   xfs_trans.h
> > +   xfs_trans.h \
> > +   xfs_trans_resv.h \
> > +   xfs_trans_space.h
> 
> These shouldn't be added to the headers installed by make install-qa.
> 
> (which I'm going to send a patch to remove once Dave applied the
> build fix, btw)
> 
OK, noted.

> Also I'm not sure xfs_mkfs.h really is a good name for a global header.

I don't see any issue with it, but if there is a reason for a different name, I 
can change it. :-)

Cheers,
Jan

-- 
Jan Tulak
jtulak@xxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>