Re: nextcloudclient port

On 16 Nov 07:13, Adriano Barbosa wrote:
> Thank you all for all testing and suggestions.
> Updated tarball attached. ok for importing (I can't do it)
>
> Em sáb., 16 de nov. de 2019 às 05:53, Giovanni Bechis
> <giovanni@paclan.it> escreveu:
> >
> > On 11/16/19 8:49 AM, Rafael Sadowski wrote:
> > > On Fri Nov 15, 2019 at 07:47:32PM +0300, Kirill Bychkov wrote:
> > >> On Fri, November 15, 2019 19:37, Giovanni Bechis wrote:
> > >>> On 11/15/19 4:08 PM, Kirill Bychkov wrote:
> > >>>> On Fri, November 8, 2019 16:15, Adriano Barbosa wrote:
> > >>>>> Em sex., 8 de nov. de 2019 ?s 09:32, Kirill Bychkov
> > >>>>> <kirby@linklevel.net> escreveu:
> > >>>>>>
> > >>>>>> On Fri, November 8, 2019 12:32, Adriano Barbosa wrote:
> > >>>>>>> By the way, Kirill's suggestions are ok for me. Just compiled it
> > >>>>>>> without problems.
> > >>>>>>>
> > >>>>>>> Best
> > >>>>>>>
> > >>>>>>> Em sex., 8 de nov. de 2019 ?s 06:03, Adriano Barbosa
> > >>>>>>> <barbosa.aob@gmail.com> escreveu:
> > >>>>>>>>
> > >>>>>>>> Hi Kirill,
> > >>>>>>>> thank you for your test and suggestions.
> > >>>>>>>> error: use of undeclared identifier 'EVP_PKEY_CTX_set_rsa_oaep_md' is
> > >>>>>>>> related to LibreSSL. Are you running -current?
> > >>>>>>>> My original port had a patch for it, but its not necessary anymore in
> > >>>>>>>> -current.
> > >>>>>>
> > >>>>>> My -current was two weeks old, sorry. Now it builds fine.
> > >>>>>> Not tested yet.
> > >>>>>>
> > >>>>>> owncloudclient has 40 test and one is failing (FolderWatcherTest).
> > >>>>>> nextcloudclient has 35 tests and four of them are failing:
> > >>>>>> 12 - UtilityTest (Failed)
> > >>>>>> 19 - AllFilesDeletedTest (Failed)
> > >>>>>> 33 - check_vio_ext (Failed)
> > >>>>>> 35 - check_encoding_functions (Failed)
> > >>>>>> All of them passes in owncloudclient.
> > >>>>>
> > >>>>> I'm trying to learn as much as I can from this porting process and
> > >>>>> tests using cmake/cmocka are part of my ignorance. I found 25 binaries
> > >>>>> related to tests in ${WRKSRC}/nextcloudclient-2.6.1/build-amd64/bin
> > >>>>> after building the port. None of them fails, including
> > >>>>> AllFilesDeletedTest and UtilityTest, log attached, didn't find the
> > >>>>> other two tests you pointed out. I would be very happy if you could
> > >>>>> point out something for me to read about it.
> > >>>>
> > >>>> We can deal with the tests later. Nextcloudclient works fine for me
> > >>>> with my owncloud server. It even works better than owncloudclient
> > >>>> I'm maintaining. It syncs all my files without errors. With
> > >>>> owncloudclient I have to add more and more files to exclude list
> > >>>> because it fails.
> > >>>> Attached is updated tarball with more tweaks:
> > >>>> - add COMPILER and comment because of c++14
> > >>>> - pre-configure -> post-extract
> > >>>> - saet SYSCONF_INSTALL_DIR similar to owncloudclient
> > >>>> - fix typos in post-extract comment
> > >>>> - use post-install target for dance with global exclude list (same
> > >>>> as in owncloudclient)
> > >>>> Changes for config file were needed because previous version of port
> > >>>> errors immediately after finishing setup wizard complaining that global
> > >>>> exclude file couldn't be found.
> > >>>> Currently it works for me. I want to test fresh install to be sure it
> > >>>> will work fine.
> > >>>>
> > >>> Bogus WANTLIB: inotify.2 (/usr/local/lib/libnextcloudsync.so.0.0) (NOT
> > >>> REACHABLE)
> > >>
> > >> That's normal for all ports linking with inotify.
> > >>
> > >>>
> > >>> at runtime I have the error:
> > >>> could not read system exclude file
> > >>>
> > >>> doas mkdir /etc/Nextcloud && doas cp
> > >>> /usr/obj/ports/nextcloudclient-2.6.1/desktop-2.6.1/sync-exclude.lst
> > >>> /etc/Nextcloud
> > >>> fixes the problem.
> > >>
> > >> Thanks for testing. So the last tweak we need is to add a couple of
> > >> @sample lines.
> > >> Updated tarball attached.
> > >>
> > >>>
> > >
> > > Small tweaks:
> > >
> > > - Fix spaces, tabs
> > > - Remove COMPILER because it comes from qt5
> > >
> > > Optional: I would prefer "desktop sync client for Nextcloud" as COMMENT.
> > >
> > > With this two tweaks + optional comment, OK rsadowski@ to import it.
> > >
> > ok giovanni@ with all tweaks, I would prefer rsadowsky@ comment as well
> >
> > > --- Makefile.orig Fri Nov 15 22:05:27 2019
> > > +++ Makefile Fri Nov 15 22:11:04 2019
> > > @@ -1,13 +1,13 @@
> > > # $OpenBSD: Makefile.template,v 1.85 2019/09/09 19:19:05 kmos Exp $ #
> > >
> > > -COMMENT = nextcloud sync client
> > > +COMMENT = desktop sync client for Nextcloud
> > >
> > > V = 2.6.1
> > > DISTNAME = nextcloudclient-${V}
> > >
> > > -GH_ACCOUNT = nextcloud
> > > -GH_PROJECT = desktop
> > > -GH_TAGNAME = v${V}
> > > +GH_ACCOUNT = nextcloud
> > > +GH_PROJECT = desktop
> > > +GH_TAGNAME = v${V}
> > >
> > > CATEGORIES = net
> > >
> > > @@ -27,9 +27,6 @@ WANTLIB += sqlite3 ssl z lib/inotify/inotify
> > >
> > > MODULES = devel/cmake \
> > > x11/qt5
> > > -
> > > -# C++14
> > > -COMPILER = base-clang ports-gcc
> > >
> > > BUILD_DEPENDS = devel/gettext,-tools
> > >
> > >
> >
>
>
> --
> Adriano Barbosa

Hi all,

use it now since two weeks and works like a charm for me.

Thanks for your effort.

With kind regards

Uwe

--