Summary: | [PATCH] fix fortune out-of-tree build | ||||||
---|---|---|---|---|---|---|---|
Product: | Base System | Reporter: | George Reid <greid> | ||||
Component: | bin | Assignee: | freebsd-bugs (Nobody) <bugs> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Only Me | ||||||
Priority: | Normal | ||||||
Version: | 5.0-CURRENT | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
George Reid
2001-02-20 16:50:04 UTC
On Tue, 20 Feb 2001, George Reid wrote:
> >Description:
> games/fortune out-of-tree build is broken.
>
> make tries to include ${CURDIR}../../Makefile.inc which exists in
> src/games in-tree. This prevents the program from compiling out-of-tree.
> ...
> >Fix:
> Index: fortune/Makefile
> ===================================================================
> RCS file: /usr/home/ncvs/src/games/fortune/fortune/Makefile,v
> retrieving revision 1.3
> diff -u -r1.3 Makefile
> --- fortune/Makefile 1996/09/05 17:13:50 1.3
> +++ fortune/Makefile 2001/02/20 16:54:08
> @@ -6,5 +6,5 @@
> DPADD= ${LIBCOMPAT}
> LDADD= -lcompat
>
> -.include "${.CURDIR}/../../Makefile.inc"
> +.include "${.CURDIR}/../Makefile.inc"
> .include <bsd.prog.mk>
> ...
I just committed an equivalent "fix" (remove these includes; your version
of them is duplicated in <bsd.prog.mk>).
But this is not really right. The Makefile.inc at the next level up
(still in the fortune tree) does the old version of the includes
(include from 2 levels up, at the top of the games tree), except it
does the include conditionally. It should include from 1 level up
for technical reasons (so that it doesn't know too much about the
depth of the subdirs) and do the include unconditionally so that
security-related macros are defined, but including it unconditionally
would bring back your out-of-tree problem. I'm not even sure how
to test for its existence. `.include' has an undocumented search
path, which IIRC begins with the directory of the file containing the
include statement and ends with ${.CURDIR}. I'm not sure if this
magic does/should apply to exists().
Bruce
State Changed From-To: open->closed Seems Bruce fixed this one already. |