On 2017/08/08 08:54, Giovanni Bechis wrote:
> On 08/06/17 15:46, Björn Ketelaars wrote:
> > Please find enclosed a diff for bringing sysutils/borgbackup to the latest
> > version, which fixes a couple of bugs. Changelog can be found at
> > https://github.com/borgbackup/borg/blob/1.0.11/docs/changes.rst#version-1011-2017-07-21
> > 
> > Upstream has made some changes to setup.py with as result that some source
> > files (*.c and *.pyx) are installed as well. For now I prefer following
> > upstream, and make these files part of the final port. Any other opinions?
> > 
> I do not like it that much, I would prefer to remove them before installing.

I'd generally prefer to follow upstream for this too..

> Other than that all regression tests are failing, backup is working, I have 
> not tested sysutils/borgmatic, do you ?

Seems ok here .. I'm using the updated diff below which adds RUN_DEPENDS
to BUILD_DEPENDS, which might be important.

old version:
======= 555 passed, 71 skipped, 2 xfailed, 1 warnings in 277.10 seconds ========

new version:
======= 590 passed, 82 skipped, 2 xfailed, 1 warnings in 298.59 seconds ========


----- Forwarded message from Björn Ketelaars <bjorn.ketela...@hydroxide.nl> 
-----

From: Björn Ketelaars <bjorn.ketela...@hydroxide.nl>
Date: Mon, 7 Aug 2017 02:13:43 +0200
To: Stuart Henderson <s...@spacehopper.org>
User-Agent: Mutt/1.8.3 (2017-05-23)
Subject: Re: [update] borgbackup 1.0.11

On Sun 06/08/2017 21:49, Stuart Henderson wrote:
> On 2017/08/06 15:46, Björn Ketelaars wrote:
> > Please find enclosed a diff for bringing sysutils/borgbackup to the latest
> > version, which fixes a couple of bugs. Changelog can be found at
> > https://github.com/borgbackup/borg/blob/1.0.11/docs/changes.rst#version-1011-2017-07-21
> > 
> > Upstream has made some changes to setup.py with as result that some source
> > files (*.c and *.pyx) are installed as well. For now I prefer following
> > upstream, and make these files part of the final port. Any other opinions?
> 
> Sounds reasonable.
> 
> What do you think about adding ${RUN_DEPENDS} to BUILD_DEPENDS, to avoid
> most of the "failed to import module" whining when processing docs?
> 

New diff included, which addresses your suggestion. Thanks!


diff --git sysutils/borgbackup/Makefile sysutils/borgbackup/Makefile
index bb4f9e4ac66..99982fafa0f 100644
--- sysutils/borgbackup/Makefile
+++ sysutils/borgbackup/Makefile
@@ -2,7 +2,7 @@
 
 COMMENT =              deduplicating backup program
 
-MODPY_EGG_VERSION =    1.0.10
+MODPY_EGG_VERSION =    1.0.11
 DISTNAME =             borgbackup-${MODPY_EGG_VERSION}
 
 MODPY_PI =             Yes
@@ -17,11 +17,15 @@ PERMIT_PACKAGE_CDROM =      Yes
 MODULES =              lang/python
 
 MODPY_VERSION =                ${MODPY_DEFAULT_VERSION_3}
-WANTLIB +=             ${MODPY_WANTLIB} crypto pthread lz4
 
-LIB_DEPENDS =          ${MODPY_LIB_DEPENDS} archivers/lz4
-BUILD_DEPENDS +=       devel/py-setuptools_scm${MODPY_FLAVOR} \
+WANTLIB +=             ${MODPY_WANTLIB} crypto lz4 pthread
+
+LIB_DEPENDS +=         ${MODPY_LIB_DEPENDS} archivers/lz4
+
+BUILD_DEPENDS +=       ${RUN_DEPENDS} \
+                       devel/py-setuptools_scm${MODPY_FLAVOR} \
                        textproc/py-sphinx${MODPY_FLAVOR}
+
 RUN_DEPENDS +=         net/py-msgpack${MODPY_FLAVOR}
 
 TEST_DEPENDS +=                ${RUN_DEPENDS} \
diff --git sysutils/borgbackup/distinfo sysutils/borgbackup/distinfo
index d59ca0edbae..efa07b0453b 100644
--- sysutils/borgbackup/distinfo
+++ sysutils/borgbackup/distinfo
@@ -1,2 +1,2 @@
-SHA256 (borgbackup-1.0.10.tar.gz) = 
l48o8USSQAUhnamJsbQ2oysnNaNMbQ3BW7ChzP+tWek=
-SIZE (borgbackup-1.0.10.tar.gz) = 563363
+SHA256 (borgbackup-1.0.11.tar.gz) = 
/G79LRlFRiVTLg3vrJDNjoc3Zl4QrlntsVNW7lqZ0pE=
+SIZE (borgbackup-1.0.11.tar.gz) = 603381
diff --git sysutils/borgbackup/pkg/PLIST sysutils/borgbackup/pkg/PLIST
index 7a7484318f0..005571f11f4 100644
--- sysutils/borgbackup/pkg/PLIST
+++ sysutils/borgbackup/pkg/PLIST
@@ -23,14 +23,24 @@ 
lib/python${MODPY_VERSION}/site-packages/borg/${MODPY_PYCACHE}repository.${MODPY
 
lib/python${MODPY_VERSION}/site-packages/borg/${MODPY_PYCACHE}shellpattern.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/borg/${MODPY_PYCACHE}upgrader.${MODPY_PYC_MAGIC_TAG}pyc
 
lib/python${MODPY_VERSION}/site-packages/borg/${MODPY_PYCACHE}xattr.${MODPY_PYC_MAGIC_TAG}pyc
+lib/python${MODPY_VERSION}/site-packages/borg/_chunker.c
+lib/python${MODPY_VERSION}/site-packages/borg/_hashindex.c
 lib/python${MODPY_VERSION}/site-packages/borg/_version.py
 lib/python${MODPY_VERSION}/site-packages/borg/archive.py
 lib/python${MODPY_VERSION}/site-packages/borg/archiver.py
 lib/python${MODPY_VERSION}/site-packages/borg/cache.py
+lib/python${MODPY_VERSION}/site-packages/borg/chunker.c
+lib/python${MODPY_VERSION}/site-packages/borg/chunker.pyx
 lib/python${MODPY_VERSION}/site-packages/borg/chunker.so
+lib/python${MODPY_VERSION}/site-packages/borg/compress.c
+lib/python${MODPY_VERSION}/site-packages/borg/compress.pyx
 lib/python${MODPY_VERSION}/site-packages/borg/compress.so
+lib/python${MODPY_VERSION}/site-packages/borg/crypto.c
+lib/python${MODPY_VERSION}/site-packages/borg/crypto.pyx
 lib/python${MODPY_VERSION}/site-packages/borg/crypto.so
 lib/python${MODPY_VERSION}/site-packages/borg/fuse.py
+lib/python${MODPY_VERSION}/site-packages/borg/hashindex.c
+lib/python${MODPY_VERSION}/site-packages/borg/hashindex.pyx
 lib/python${MODPY_VERSION}/site-packages/borg/hashindex.so
 lib/python${MODPY_VERSION}/site-packages/borg/helpers.py
 lib/python${MODPY_VERSION}/site-packages/borg/key.py
@@ -40,6 +50,12 @@ lib/python${MODPY_VERSION}/site-packages/borg/logger.py
 lib/python${MODPY_VERSION}/site-packages/borg/lrucache.py
 lib/python${MODPY_VERSION}/site-packages/borg/paperkey.html
 lib/python${MODPY_VERSION}/site-packages/borg/platform.py
+lib/python${MODPY_VERSION}/site-packages/borg/platform_darwin.c
+lib/python${MODPY_VERSION}/site-packages/borg/platform_darwin.pyx
+lib/python${MODPY_VERSION}/site-packages/borg/platform_freebsd.c
+lib/python${MODPY_VERSION}/site-packages/borg/platform_freebsd.pyx
+lib/python${MODPY_VERSION}/site-packages/borg/platform_linux.c
+lib/python${MODPY_VERSION}/site-packages/borg/platform_linux.pyx
 lib/python${MODPY_VERSION}/site-packages/borg/remote.py
 lib/python${MODPY_VERSION}/site-packages/borg/repository.py
 lib/python${MODPY_VERSION}/site-packages/borg/shellpattern.py
@@ -66,6 +82,7 @@ 
lib/python${MODPY_VERSION}/site-packages/borg/testsuite/${MODPY_PYCACHE}upgrader
 
lib/python${MODPY_VERSION}/site-packages/borg/testsuite/${MODPY_PYCACHE}xattr.${MODPY_PYC_MAGIC_TAG}pyc
 lib/python${MODPY_VERSION}/site-packages/borg/testsuite/archive.py
 lib/python${MODPY_VERSION}/site-packages/borg/testsuite/archiver.py
+lib/python${MODPY_VERSION}/site-packages/borg/testsuite/attic.tar.gz
 lib/python${MODPY_VERSION}/site-packages/borg/testsuite/benchmark.py
 lib/python${MODPY_VERSION}/site-packages/borg/testsuite/chunker.py
 lib/python${MODPY_VERSION}/site-packages/borg/testsuite/compress.py

----- End forwarded message -----

Reply via email to