On Wed 09/08/2017 21:28, Björn Ketelaars wrote:
> On Tue 08/08/2017 11:53, Giovanni Bechis wrote:
> > On 08/08/17 11:41, Stuart Henderson wrote:
> > > 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..
> > > 
> > ok then
> > 
> > >> 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.
> > >
> > make test fails iff borgbackup is installed, even with the old version
> 
> I am able to reproduce the issue; when borgbackup is installed make test will
> fail. Workaround is to uninstall borgbackup before running make test.
> 
> As my py.test-fu is limited I Googled a bit. Conclusion is that this is an
> actual feature of py.test [0]. Solution would be to use a virtualenv
> environment [1]. This is exactly what upstream does.
> 
> My proposal is to print a message when someone does make test. OK?
> 
> [0]
> https://stackoverflow.com/questions/12582503/py-test-test-discovery-failure-when-tests-in-different-directories-are-called
> [1]
> https://docs.pytest.org/en/latest/goodpractices.html#choosing-a-test-layout-import-rules
>

OK?


diff --git sysutils/borgbackup/Makefile sysutils/borgbackup/Makefile
index bb4f9e4ac66..9f67246e49b 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} \
@@ -40,6 +44,7 @@ post-install:
        ${INSTALL_MAN} ${WRKSRC}/docs/_build/man/borg.1 
${PREFIX}/man/man1/borg.1
 
 do-test: fake
+       @printf "\n*** NOTE: borgbackup fails all tests when it is already 
installed ***\n\n"
        cd ${WRKSRC}; \
                PATH=${WRKINST}/usr/local/bin:${PATH} \
                PYTHONPATH=${WRKSRC} \
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

Reply via email to