[PATCH 2/2] xfstests: some refinements on "make depend"
Alex Elder
aelder at sgi.com
Thu Mar 10 16:09:36 CST 2011
On Thu, 2011-03-10 at 12:26 -0500, Christoph Hellwig wrote:
> On Fri, Mar 04, 2011 at 01:37:41PM -0600, Alex Elder wrote:
> > Make it so "make depend" is a generic target, like "make clean".
> >
> > Each Makefile has a "depend" target that indicates whether making
> > dependencies means creating ".dep" or creating ".ltdep" (or, I
> > suppose, both, though none do that right now). Both files get
> > created even if there are no CFILES to scan (to ensure the target
> > up-to-date). The "default" target now depends on "depend" (there is
> > no "ltdepend" any more).
> >
> > Remove the "depend" and "ltdepend" definitions from the "buildrules"
> > file; only the actual generated files (".dep" and ".ltdep") remain
> > as generic targets. The "depend' target is still defined as phony.
> >
> > Do a shell trick when expanding the value of CFILES, to avoid a
> > problem that occurs if it is created by "make" by concatentating two
> > empty strings. The problem was that in that case CFILES will
> > contain a space, and that wasn't getting treated as empty as
> > desired.
> >
> > Make the rule for tool/lib dependencies more generic, to reflect the
> > general desire that "lib" subdirectories need to be built before
> > things in the "tool" subdirectories.
>
> Can we instead port over the generic dependency generation from
> xfsprogs/xfsdump?
My intention was to work toward unifying all the user space
stuff, basically in line with what you suggest.
This is part of a flurry of changes related to the xfstests
dmapi subtree. One of the things I needed to do was populate
a set of new Makefiles there, and I guess in looking into
that I found I didn't like the way the dependencies were done,
and the changes I made to address that ended up in this patch.
I'll hold off on this patch and will reconsider it once
I get the more substantive dmapi changes in.
-Alex
More information about the xfs
mailing list