Commit c0f1e774 authored by Jason Tishler's avatar Jason Tishler

Patch #553702: Cygwin make install patch

This patch fixes make install for Cygwin. Specifically,
it reverts to the previous behavior:

o install libpython$(VERSION)$(SO) in $(BINDIR)
o install $(LDLIBRARY) in $(LIBPL)

It also begins to remove Cygwin's dependency on
$(DLLLIBRARY) which I hope to take advantage of
when I attempt to make Cygwin as similar as possible
to the other Unix platforms (in other patches).

I tested this patch under Red Hat Linux 7.1 without
any ill effects.

BTW, I'm not the happiest using the following
test for Cygwin:

test "$(SO)" = .dll

I'm willing to update the patch to use:

case "$(MACHDEP)" in cygwin*

instead, but IMO that will look uglier.
parent 56796f67
...@@ -559,11 +559,11 @@ altbininstall: $(BUILDPYTHON) ...@@ -559,11 +559,11 @@ altbininstall: $(BUILDPYTHON)
done done
$(INSTALL_PROGRAM) $(BUILDPYTHON) $(BINDIR)/python$(VERSION)$(EXE) $(INSTALL_PROGRAM) $(BUILDPYTHON) $(BINDIR)/python$(VERSION)$(EXE)
if test -f libpython$(VERSION)$(SO); then \ if test -f libpython$(VERSION)$(SO); then \
$(INSTALL_SHARED) libpython$(VERSION)$(SO) $(LIBDIR)/$(INSTSONAME); \ if test "$(SO)" = .dll; then \
else true; \ $(INSTALL_SHARED) libpython$(VERSION)$(SO) $(BINDIR); \
fi else \
if test -f "$(DLLLIBRARY)"; then \ $(INSTALL_SHARED) libpython$(VERSION)$(SO) $(LIBDIR)/$(INSTSONAME); \
$(INSTALL_SHARED) $(DLLLIBRARY) $(BINDIR); \ fi; \
else true; \ else true; \
fi fi
...@@ -690,8 +690,12 @@ libainstall: all ...@@ -690,8 +690,12 @@ libainstall: all
done done
@if test -d $(LIBRARY); then :; else \ @if test -d $(LIBRARY); then :; else \
if test "$(PYTHONFRAMEWORKDIR)" = no-framework; then \ if test "$(PYTHONFRAMEWORKDIR)" = no-framework; then \
$(INSTALL_DATA) $(LIBRARY) $(LIBPL)/$(LIBRARY) ; \ if test "$(SO)" = .dll; then \
$(RANLIB) $(LIBPL)/$(LIBRARY) ; \ $(INSTALL_DATA) $(LDLIBRARY) $(LIBPL) ; \
else \
$(INSTALL_DATA) $(LIBRARY) $(LIBPL)/$(LIBRARY) ; \
$(RANLIB) $(LIBPL)/$(LIBRARY) ; \
fi; \
else \ else \
echo Skip install of $(LIBRARY) - use make frameworkinstall; \ echo Skip install of $(LIBRARY) - use make frameworkinstall; \
fi; \ fi; \
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment