Skip to content

Commit

Permalink
In the altbininstall target, which is the first subtarget for "make i…
Browse files Browse the repository at this point in the history
…nstall",

if we are running in an OSX framework enabled build directory, test that
the framework infrastructure exists. This catches the very common
error of doing "make install" in stead of "make frameworkinstall".
  • Loading branch information
jackjansen committed Aug 6, 2002
1 parent 9c5b61b commit 3a451b1
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion Makefile.pre.in
Original file line number Diff line number Diff line change
Expand Up @@ -569,6 +569,12 @@ bininstall: altbininstall
# Install the interpreter with $(VERSION) affixed
# This goes into $(exec_prefix)
altbininstall: $(BUILDPYTHON)
@if test "$(PYTHONFRAMEWORKDIR)" != no-framework; then \
if test ! -f $(PYTHONFRAMEWORKINSTALLDIR)/Versions/Current/Resources/Info.plist; then \
echo 'Framework build: use "make frameworkinstall" in stead of "make install"'; \
exit 1; \
fi; \
fi
@for i in $(BINDIR) $(LIBDIR); \
do \
if test ! -d $$i; then \
Expand Down Expand Up @@ -770,7 +776,7 @@ sharedinstall:
# automatically set prefix to the location deep down in the framework, so we
# only have to cater for the structural bits of the framework.

frameworkinstall: install frameworkinfrastructureinstall
frameworkinstall: frameworkinfrastructureinstall install
FRAMEWORKFINALDEST=$(PYTHONFRAMEWORKINSTALLDIR)/Versions/$(VERSION)
frameworkinfrastructureinstall: $(LDLIBRARY)
@if test "$(PYTHONFRAMEWORKDIR)" = no-framework; then \
Expand Down

0 comments on commit 3a451b1

Please sign in to comment.