From ohauer@FreeBSD.ORG  Thu May 17 14:41:58 2012
Return-Path: <ohauer@FreeBSD.ORG>
Received: by hub.freebsd.org (Postfix, from userid 1234)
	id 4DEA9106567A; Thu, 17 May 2012 14:41:58 +0000 (UTC)
Message-Id: <20120517144158.4DEA9106567A@hub.freebsd.org>
Date: Thu, 17 May 2012 14:41:58 +0000 (UTC)
From: Olli Hauer <ohauer@FreeBSD.org>
To: <FreeBSD-gnats-submit@freebsd.org>
Cc: <lev@FreeBSD.org>
Subject: [patch] port devel/subversion update to 1.7.5 + fix pkg-plist (PR167564)
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         168012
>Category:       ports
>Synopsis:       [patch] port devel/subversion update to 1.7.5 + fix pkg-plist (PR167564)
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    lev
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 17 14:50:01 UTC 2012
>Closed-Date:    Sun May 20 10:46:11 UTC 2012
>Last-Modified:  Sun May 20 10:50:01 UTC 2012
>Originator:     Olli Hauer
>Release:        
>Organization:
>Environment:


>Description:
- update subversion to 1.7.5
- fix pkg-plist issue (PR167564)
- use libtool to install svnauthz-validate
- add ugly hack to remove unwanted files from ${DATADIR}
- add option to run subversion testsuite
  Note: The testsuite will fail completly until PR156076 will be committed.
        Even with a fixed py-sqlite testsuite will fail since some tests
        are not bullet prove written but gives a good overview.
        (Maybe remove OPTION knob and run test with make -DWITH_TEST)

PR167564 can be closed after commit the patch


Changelog:
==========
Version 1.7.5
(17 May 2012, from /branches/1.7.x)
http://svn.apache.org/repos/asf/subversion/tags/1.7.5

 User-visible changes:
  - Client- and server-side bugfixes:
    * http: report deleted-revision upon delete during update (r1327474)

  - Client-side bugfixes:
    * avoid potential segfault when canonicalizing properties (r1296369)
    * improve memory and file-handle management with externals (issue #4130)
    * serf: convert assertions to "MERGE failed" errors (r1302417)
    * fix undefined behaviour during multi-segment reverse merges (issue #4144)
    * fix potential use of already freed memory during diff operation (r1311935)
    * improve performance of scan for the working copy root (r1306334)
    * cmdline: fix segfault during 'svn diff' argument processing (r1311702)
    * fix regression from 1.6 in update with --depth option (issue #4136)
    * propset: avoid undefined behaviour in error path (r1325361)
    * reset sqlite statements, partly for sqlite-3.7.11 compat (r1328846, et al)
    * fix assertion during 'svn diff -r BASE:HEAD ^/trunk' (issue #4161)
    * notify upon 'update' just removing locks on files (r1329876)
    * neon: fix potential use of freed memory during commits (r1329388)
    * 'status --xml' doesn't show repository deletes correctly (issue #4167)
    * fix assert on svn:externals with drive letter on Windows (issue #4073)
    * fix 'svn update --depth=empty' against 1.4 servers (issue #4046)
    * handle missing svn:date reported by svnserve gracefully (r1306111)
    * fix merges which first add a subtree and then delete it (issue #4166)
    * fix a regression with checkout of file externals (issue #4087)
    * don't add spurious mergeinfo to subtrees in edge-case merge (issue #4169)
    * improve performance of status on large working copies (issue #4178)

  - Server-side bugfixes:
    * fix non-fatal FSFS corruption bug with concurrent commits (issue #4129)
    * mod_dav_svn: raise an error on MERGE of non-existent resource (r1298343)
    * mod_dav_svn: support compiling/running under httpd-2.4 (r1232267)
    * mod_dav_svn: forbid BDB repositories under httpd's event MPM (issue #4157)

  - Other tool improvements and bugfixes:
    * emacs support: updates to dsvn.el and vc-svn.el (r1200896, et al)

 Developer-visible changes:
  - General:
    * windows example distribution scripts: include svnrdump (r1295007)
    * fix running the test suite with jsvn (r1335555)

  - Bindings:
    * swig-py tests: avoid FAILs on APR hash order (r1296137, r1292248)
    * swig-rb tests: avoid FAILs on APR hash order (r1310535, r1310594)
    * swig-pl: Improved perl detection in gen-make.py (r1291797, r1291810)



>How-To-Repeat:
>Fix:


--- patch_subversion.txt begins here ---
Index: subversion/Makefile
===================================================================
RCS file: /home/pcvs/ports/devel/subversion/Makefile,v
retrieving revision 1.144
diff -u -r1.144 Makefile
--- subversion/Makefile	2 Apr 2012 15:16:52 -0000	1.144
+++ subversion/Makefile	17 May 2012 13:56:27 -0000
@@ -33,7 +33,8 @@
 		STATIC			"Build static version (no shared libs)" off \
 		GNOME_KEYRING		"Build with GNOME Keyring auth support" off \
 		KDE_KWALLET		"Build with KDE KWallet auth support" off \
-		BOOK			"Install the Subversion Book" off
+		BOOK			"Install the Subversion Book" off \
+		TEST			"Run subversion test suite" off
 #		ASVN			"Build and install Archive SVN (asvn)" off \
 
 .include "Makefile.common"
@@ -83,6 +84,11 @@
 PLIST_SUB+=	BOOK="@comment "
 .endif
 
+# testsuite will fail until PR156076 is committed
+.if defined(WITH_TEST)
+TEST_DEPENDS=	${PYTHON_SITELIBDIR}/_sqlite3.so:${PORTSDIR}/databases/py-sqlite3
+.endif
+
 #.if defined(WITH_ASVN)
 #RUN_DEPENDS+=	${LOCALBASE}/bin/gfind:${PORTSDIR}/misc/findutils:install
 #PLIST_SUB+=	ASVN=""
@@ -210,18 +216,11 @@
 	fi
 .endif
 
-post-build:
-.if defined(WITH_MOD_DAV_SVN) && defined(WITH_MOD_DONTDOTHAT)
-	cd ${WRKSRC}/tools/server-side/mod_dontdothat && \
-		${APXS} -c -I${WRKSRC}/subversion/include -L${WRKSRC}/subversion/libsvn_subr/.libs -lsvn_subr-1 mod_dontdothat.c
-.endif
-# subversion 1.7+ -- it is not optional
-#.if defined(WITH_SVNAUTHZ_VALIDATE)
-#	# place binary in ${WRKDIR}, else we install it twice
-#	cd ${WRKSRC}/tools/server-side && \
-#		${CC} svnauthz-validate.c -o ../../svnauthz-validate -I${WRKSRC}/subversion/include -I${APR_INCLUDE_DIR} \
-#		-L${WRKSRC}/subversion/libsvn_repos/.libs -L${WRKSRC}/subversion/libsvn_subr/.libs -lsvn_repos-1 -lsvn_subr-1
-#.endif
+# takes a long time, and will fail but gives maintainer a good oversight
+.if defined(WITH_TEST)
+test:
+	make check -C ${WRKSRC}
+.endif
 
 pre-install:
 .if defined(WITH_MOD_DAV_SVN)
@@ -235,17 +234,17 @@
 	${APXS} -e -S LIBEXECDIR=${PREFIX}/${APACHEMODDIR} -a -n dav_svn ${PREFIX}/${APACHEMODDIR}/mod_dav_svn.so
 	${APXS} -e -S LIBEXECDIR=${PREFIX}/${APACHEMODDIR} -a -n authz_svn ${PREFIX}/${APACHEMODDIR}/mod_authz_svn.so
 .if defined(WITH_MOD_DONTDOTHAT)
-	cd ${WRKSRC}/tools/server-side/mod_dontdothat && \
-		${APXS} -i -n dontdothat mod_dontdothat.la
 	${APXS} -e -S LIBEXECDIR=${PREFIX}/${APACHEMODDIR} -a -n dontdothat ${PREFIX}/${APACHEMODDIR}/mod_dontdothat.so
 .endif
 .endif
 .if defined(WITH_SVNAUTHZ_VALIDATE)
-	${INSTALL} ${COPY} ${_BINOWNGRP} -m ${BINMODE} ${WRKSRC}/tools/server-side/svnauthz-validate ${PREFIX}/bin
+	${LIBTOOL} --mode=install ${INSTALL} ${COPY} ${_BINOWNGRP} -m ${BINMODE} ${WRKSRC}/tools/server-side/svnauthz-validate ${PREFIX}/bin
 .endif
 	@${MKDIR} ${DATADIR}
 	cd ${WRKSRC}/tools ; \
-	  ${TAR} --exclude '*.in' --exclude '.libs' --exclude '*.o' --exclude '*.lo' -cf - * | ${TAR} -C ${DATADIR} -xf -
+	  ${TAR} --exclude '*.in' --exclude '.libs' --exclude '*.o' --exclude '*.lo' --exclude='*.slo' -cf - * | ${TAR} -C ${DATADIR} -xf -
+# ugly hack to remove libtool scripts ...
+	@${GREP} -R ${WRKSRC} ${DATADIR} | ${CUT} -d\: -f 1 | ${XARGS} ${RM} -f
 	@${CHMOD} -R a-st,o+rX ${DATADIR}
 .if !defined(NOPORTDOCS)
 	${MKDIR} ${DOCSDIR}
Index: subversion/Makefile.common
===================================================================
RCS file: /home/pcvs/ports/devel/subversion/Makefile.common,v
retrieving revision 1.61
diff -u -r1.61 Makefile.common
--- subversion/Makefile.common	2 Apr 2012 15:16:52 -0000	1.61
+++ subversion/Makefile.common	17 May 2012 13:56:27 -0000
@@ -5,8 +5,8 @@
 # $FreeBSD: ports/devel/subversion/Makefile.common,v 1.61 2012/04/02 15:16:52 lev Exp $
 
 PORTNAME=	subversion
-PORTVERSION=	1.7.4
-PORTREVISION?=	1
+PORTVERSION=	1.7.5
+#PORTREVISION?=	1
 CATEGORIES+=	devel
 MASTER_SITES=	${MASTER_SITE_APACHE:S/$/:main/} \
 		${MASTER_SITE_LOCAL:S/$/:book/}
@@ -68,6 +68,10 @@
 PLIST_SUB+=	BDB="@comment "
 .endif
 
+.if defined(WITH_MOD_DONTDOTHAT) && defined(WITHOUT_MOD_DAV_SVN)
+IGNORE=	to build mod_dontdothat option mod_dav_svn is required
+.endif
+
 .if defined(WITH_MOD_DAV_SVN)
 WITH_APACHE=	yes
 CONFIGURE_ARGS+=--with-apxs=${APXS}
Index: subversion/distinfo
===================================================================
RCS file: /home/pcvs/ports/devel/subversion/distinfo,v
retrieving revision 1.86
diff -u -r1.86 distinfo
--- subversion/distinfo	2 Apr 2012 15:16:52 -0000	1.86
+++ subversion/distinfo	17 May 2012 13:56:27 -0000
@@ -1,5 +1,5 @@
-SHA256 (subversion17/subversion-1.7.4.tar.bz2) = c4c736f5d94a93b5862d1d2452353146c8aeaae33ccec67649bba91f91ad506f
-SIZE (subversion17/subversion-1.7.4.tar.bz2) = 5978020
+SHA256 (subversion17/subversion-1.7.5.tar.bz2) = cb23ccd08b0c1f0c66fdcda773265e404d95f378493ba64b49dbaaf54ae44aec
+SIZE (subversion17/subversion-1.7.5.tar.bz2) = 6002955
 SHA256 (subversion17/svn-book-html-r4259.tar.bz2) = f02b64ad255b507325a960586225d721e086ec6f2a1843214202814e7fee3ac3
 SIZE (subversion17/svn-book-html-r4259.tar.bz2) = 467637
 SHA256 (subversion17/svn-book-r4259.pdf) = c091c598b2964f9692f3b3ffb7585c14f7d4e70793dd7e0fb862e30d9a98d289
Index: subversion/pkg-plist
===================================================================
RCS file: /home/pcvs/ports/devel/subversion/pkg-plist,v
retrieving revision 1.46
diff -u -r1.46 pkg-plist
--- subversion/pkg-plist	1 Apr 2012 18:45:05 -0000	1.46
+++ subversion/pkg-plist	17 May 2012 13:56:27 -0000
@@ -229,7 +229,6 @@
 %%DATADIR%%/client-side/svn-graph.pl
 %%DATADIR%%/client-side/svn-ssl-fingerprints.sh
 %%DATADIR%%/client-side/svn-viewspec.py
-%%DATADIR%%/client-side/svnmucc/svnmucc
 %%DATADIR%%/client-side/svnmucc/svnmucc.c
 %%DATADIR%%/client-side/svnmucc/svnmucc-test.py
 %%DATADIR%%/client-side/wcfind
@@ -272,7 +271,6 @@
 %%DATADIR%%/dev/svn-entries.el
 %%DATADIR%%/dev/svn-merge-revs.py
 %%DATADIR%%/dev/svnqlite3-dump
-%%DATADIR%%/dev/svnraisetreeconflict/svnraisetreeconflict
 %%DATADIR%%/dev/svnraisetreeconflict/main.c
 %%DATADIR%%/dev/trails.py
 %%DATADIR%%/dev/unix-build/Makefile.svn
@@ -289,11 +287,8 @@
 %%DATADIR%%/dev/windows-build/document-version.pl
 %%DATADIR%%/dev/windows-build/Makefile
 %%DATADIR%%/dev/windows-build/README
-%%DATADIR%%/diff/diff
 %%DATADIR%%/diff/diff.c
-%%DATADIR%%/diff/diff3
 %%DATADIR%%/diff/diff3.c
-%%DATADIR%%/diff/diff4
 %%DATADIR%%/diff/diff4.c
 %%DATADIR%%/dist/backport.pl
 %%DATADIR%%/dist/checksums.py
@@ -353,16 +348,11 @@
 %%DATADIR%%/po/po-update.sh
 %%DATADIR%%/server-side/mod_dontdothat/README
 %%DATADIR%%/server-side/mod_dontdothat/mod_dontdothat.c
-%%MOD_DONTDOTHAT%%%%DATADIR%%/server-side/mod_dontdothat/mod_dontdothat.la
-%%MOD_DONTDOTHAT%%%%DATADIR%%/server-side/mod_dontdothat/mod_dontdothat.slo
 %%DATADIR%%/server-side/fsfs-reshard.py
 %%DATADIR%%/server-side/svn-backup-dumps.py
-%%DATADIR%%/server-side/svn-populate-node-origins-index
 %%DATADIR%%/server-side/svn-populate-node-origins-index.c
-%%DATADIR%%/server-side/svn-rep-sharing-stats
 %%DATADIR%%/server-side/svn-rep-sharing-stats.c
 %%DATADIR%%/server-side/svn_server_log_parse.py
-%%DATADIR%%/server-side/svnauthz-validate
 %%DATADIR%%/server-side/svnauthz-validate.c
 %%DATADIR%%/server-side/test_svn_server_log_parse.py
 %%DATADIR%%/xslt/svnindex.css
@@ -371,7 +361,7 @@
 %%MOD_DAV_SVN%%%%APACHEMODDIR%%/mod_authz_svn.so
 %%MOD_DAV_SVN%%@exec %D/sbin/apxs -e -S LIBEXECDIR=%D/%%APACHEMODDIR%% -a -n dav_svn libexec/%%APACHEDIR%%/mod_dav_svn.so
 %%MOD_DAV_SVN%%@exec %D/sbin/apxs -e -S LIBEXECDIR=%D/%%APACHEMODDIR%% -a -n authz_svn libexec/%%APACHEDIR%%/mod_authz_svn.so
-%%MOD_DONTDOTHAT%%%%APACHEMODDIR%%/mod_dontdothat.so
+%%MOD_DAV_SVN%%%%APACHEMODDIR%%/mod_dontdothat.so
 %%MOD_DONTDOTHAT%%@exec %D/sbin/apxs -e -S LIBEXECDIR=%D/%%APACHEMODDIR%% -a -n dontdothat libexec/%%APACHEDIR%%/mod_dontdothat.so
 %%PORTDOCS%%%%BOOK%%@dirrm %%DOCSDIR%%/svn-book-html/images
 %%PORTDOCS%%%%BOOK%%@dirrm %%DOCSDIR%%/svn-book-html
--- patch_subversion.txt ends here ---


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->lev 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Thu May 17 14:50:20 UTC 2012 
Responsible-Changed-Why:  
Over to maintainer (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=168012 

From: Lev Serebryakov <lev@FreeBSD.org>
To: bug-followup@FreeBSD.org, ohauer@FreeBSD.org
Cc:  
Subject: Re: ports/168012: [patch] port devel/subversion update to 1.7.5 + fix pkg-plist (PR167564)
Date: Sat, 19 May 2012 19:06:57 +0400

 Hello, Bug-followup.
 
  I'm not sure abut this part of patch:
 
 -%%MOD_DONTDOTHAT%%%%APACHEMODDIR%%/mod_dontdothat.so
 +%%MOD_DAV_SVN%%%%APACHEMODDIR%%/mod_dontdothat.so
 
   Why  did you change condition for mod_dontdothat.so from its own one
  to MOD_DAV_SVN?
 
 -- 
 // Black Lion AKA Lev Serebryakov <lev@FreeBSD.org>
 

From: Olli Hauer <ohauer@FreeBSD.org>
To: bug-followup@FreeBSD.org
Cc: Lev Serebryakov <lev@FreeBSD.org>, ohauer@FreeBSD.org
Subject: Re: ports/168012: [patch] port devel/subversion update to 1.7.5 +
 fix pkg-plist (PR167564)
Date: Sun, 20 May 2012 11:08:01 +0200

 On 2012-05-19 17:06, Lev Serebryakov wrote:
 > Hello, Bug-followup.
 > 
 >  I'm not sure abut this part of patch:
 > 
 > -%%MOD_DONTDOTHAT%%%%APACHEMODDIR%%/mod_dontdothat.so
 > +%%MOD_DAV_SVN%%%%APACHEMODDIR%%/mod_dontdothat.so
 > 
 >   Why  did you change condition for mod_dontdothat.so from its own one
 >  to MOD_DAV_SVN?
 > 
 
 If DAV is enabled then mod_dontdothat.so will be also installed and thats the reason for the broken pkg-plist (PR167564)
 Instead patching build-output.mk and remove the part which installs mod_dontdothat it is easier to install but not activate the module.
 pkg-plist will take care about this and enables the module only if build with DONTDOTHAT
 
 
 snippets from build-outputs.mk
 
 ########################################
 # Section 6: Install-Group build targets
 ########################################
 
 apache-mod: subversion/mod_authz_svn/mod_authz_svn.la subversion/mod_dav_svn/mod_dav_svn.la tools/server-side/mod_dontdothat/mod_dontdothat.la
 
 ...
 
 ########################################
 # Section 7: Install-Group install targets
 ########################################
 
 install-mods-shared: subversion/mod_dav_svn/mod_dav_svn.la subversion/mod_authz_svn/mod_authz_svn.la tools/server-side/mod_dontdothat/mod_dontdothat.la
         cd subversion/mod_dav_svn ; $(MKDIR) "$(APACHE_LIBEXECDIR)" ; $(INSTALL_MOD_SHARED) -n dav_svn mod_dav_svn.la
         cd subversion/mod_authz_svn ; $(MKDIR) "$(APACHE_LIBEXECDIR)" ; $(INSTALL_MOD_SHARED) -n authz_svn mod_authz_svn.la
         cd tools/server-side/mod_dontdothat ; $(MKDIR) "$(APACHE_LIBEXECDIR)" ; $(INSTALL_MOD_SHARED) -n dontdothat mod_dontdothat.la
 
 --
 Regards,
 olli
State-Changed-From-To: open->closed 
State-Changed-By: lev 
State-Changed-When: Sun May 20 10:45:34 UTC 2012 
State-Changed-Why:  
Committed, thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=168012 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/168012: commit references a PR
Date: Sun, 20 May 2012 10:41:24 +0000 (UTC)

 lev         2012-05-20 10:41:14 UTC
 
   FreeBSD ports repository
 
   Modified files:
     devel/subversion     Makefile Makefile.common distinfo 
                          pkg-plist 
     devel/subversion/files build-outputs.mk 
   Log:
     (1) Update to 1.7.5
     (2) Fix pkg-plist problem
   
   PR:             ports/168012
   Submitted by:   Olli Hauer <ohauer@FreeBSD.org>
   
   Revision  Changes    Path
   1.145     +16 -17    ports/devel/subversion/Makefile
   1.62      +7 -3      ports/devel/subversion/Makefile.common
   1.87      +2 -2      ports/devel/subversion/distinfo
   1.8       +6 -6      ports/devel/subversion/files/build-outputs.mk
   1.47      +2 -12     ports/devel/subversion/pkg-plist
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
>Unformatted:
