- Remove "first-installed-win" logic for automatically setting

  ${PYTHON_DEFAULT_VERSION}, this generates conflicting packages.

- Create symbolic links as PEP 394 [1] suggests. ${PYTHON_DEFAULT_VERSION}
  will create python and python${MAJOR_VERSION} links.  In current default,
  lang/python27 will create: python -> python2 -> python2.7

- Introduce ${PYTHON3_DEFAULT_VERSION}, which will handle bin/python3 link.
  At this point, lang/python33 will create python3 -> python3.3

- Minor cleanups
  * Trim Makefile headers
  * Remove ${OSVERSION} detection for xz, whihc is done by USE_XZ

[1] http://www.python.org/dev/peps/pep-0394/

2 files changed, 14 insertions, 17 deletions

diff --git a/lang/python33/Makefile b/lang/python33/Makefile
index 59cea232f033..d30fcbb9b3e0 100644
--- a/lang/python33/Makefile
+++ b/lang/python33/Makefile
@@ -1,13 +1,8 @@
-# New ports collection makefile for:    python31
-# Date created:         2009-06-28
-# Whom:                 Li-Wen Hsu <lwhsu@FreeBSD.org>
-#
 # $FreeBSD$
-#
 
 PORTNAME=	python33
 PORTVERSION=	3.3.0
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	lang python ipv6
 MASTER_SITES=	${PYTHON_MASTER_SITES}
 MASTER_SITE_SUBDIR=	${PYTHON_MASTER_SITE_SUBDIR}
@@ -61,19 +56,19 @@ NLS_DESC=	Enable Gettext support for the locale module
 
 .include <bsd.port.pre.mk>
 
-.if ${OSVERSION} < 800505
-BUILD_DEPENDS+=	xz>0:${PORTSDIR}/archivers/xz	# Needed for lzma support
-LDFLAGS+=	"-L${LOCALBASE}/lib"
-CFLAGS+=	"-I${LOCALBASE}/include"
-.endif
-
-.if ${PYTHON_VERSION} == ${PYTHON_DEFAULT_VERSION} || !exists(${LOCALBASE}/bin/python)
+.if ${PYTHON_VERSION} == ${PYTHON_DEFAULT_VERSION}
 MLINKS=		${PYTHON_VERSION}.1 python.1
 PLIST_SUB+=	IF_DEFAULT=""
 .else
 PLIST_SUB+=	IF_DEFAULT="@comment "
 .endif
 
+.if ${PYTHON_VERSION} == ${PYTHON3_DEFAULT_VERSION}
+PLIST_SUB+=	IF_DEFAULT_MAJOR=""
+.else
+PLIST_SUB+=	IF_DEFAULT_MAJOR="@comment "
+.endif
+
 .if ${PORT_OPTIONS:MNLS}
 USE_GETTEXT=	yes
 # XXX do not set any LDFLAGS or CFLAGS - this causes pyexpat to fail building
@@ -212,16 +207,18 @@ post-install:
 	${INSTALL_MAN} ${PYTHON_WRKSRC}/Misc/python.man \
 		${MANPREFIX}/man/man1/${PYTHON_VERSION}.1
 
-.if ${PYTHON_VERSION} == ${PYTHON_DEFAULT_VERSION} || !exists(${LOCALBASE}/bin/python)
+.if ${PYTHON_VERSION} == ${PYTHON_DEFAULT_VERSION}
 	for f in ${BIN_FILES}; do \
 		TARGET=`${ECHO_CMD} $$f | ${SED} -E ${BINLINKS_SUB}`; \
-		cd ${PREFIX}/bin && ${LN} -f $$TARGET $$f; \
+		cd ${PREFIX}/bin && ${LN} -sf $$TARGET $$f; \
 	done
 .endif
 
+.if ${PYTHON_VERSION} == ${PYTHON3_DEFAULT_VERSION}
 .for file in python python-config
 	cd ${LOCALBASE}/bin && ${LN} -sf ${file:S/python/${PYTHON_VERSION}/} ${file:S/python/python${PYTHON_MAJOR_VER}/}
 .endfor
+.endif
 
 .if !defined(NOPORTDATA)
 	@${MKDIR} ${DATADIR}
diff --git a/lang/python33/pkg-plist b/lang/python33/pkg-plist
index 69b4f7adbaad..dc24792618c5 100644
--- a/lang/python33/pkg-plist
+++ b/lang/python33/pkg-plist
@@ -5,10 +5,10 @@ bin/idle3-%%PYVER%%
 %%IF_DEFAULT%%bin/pydoc3
 bin/pydoc3-%%PYVER%%
 %%IF_DEFAULT%%bin/python
-bin/python%%PYMAJOR%%
+%%IF_DEFAULT_MAJOR%%bin/python%%PYMAJOR%%
 bin/python%%PYVER%%
 %%IF_DEFAULT%%bin/python-config
-bin/python%%PYMAJOR%%-config
+%%IF_DEFAULT_MAJOR%%bin/python%%PYMAJOR%%-config
 bin/python%%PYVER%%-config
 %%IF_DEFAULT%%bin/python-shared
 bin/python-shared%%PYVER%%