On Fri, November 8, 2019 16:15, Adriano Barbosa wrote: > Em sex., 8 de nov. de 2019 ?s 09:32, Kirill Bychkov > <[email protected]> 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 >> > <[email protected]> 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. >> >> BTW is there some changelog of key differences between nextcloud >> and owncloud clients? Are they compatible? I'm still using >> owncloudserver and have no time to migrate to nextcloud. > > I used to use ownCloud long time ago and, to be honest, since then I > have zero information about ownCloud project. Some new features I know > Nextcloud guys implemented(ing): end2end encryption (never tested) and > flow login (disabled in the port). I guess Rafael is/was using > owncloudclient with a Nextcloud server. > >> >> >> >> >> Best >> >> >> >> Em sex., 8 de nov. de 2019 ?s 05:36, Kirill Bychkov >> >> <[email protected]> escreveu: >> >> > >> >> > On Fri, November 8, 2019 04:25, Adriano Barbosa wrote: >> >> > > Follow a diff with Stuart's suggestion on keeping GH_* variables. >> >> > > I can confirm it works well. >> >> > > >> >> > [...] >> >> > >> >> > Hi, >> >> > Some cleanup Rafael's version + some chunks of Adriano's diff: >> >> > - use DISTNAME = nextcloudclient-${V} >> >> > - drop PKGNAME >> >> > - drop WRKDIST >> >> > - regen distinfo >> >> > - drop MASTER_SITES >> >> > >> >> > Rafael's port and cleaned up both fails to build on my amd64 (log >> >> attached). >> >> > Original submission with 2.6.0 builds fine. >> >> > >> >> > >> >> > P.S. Adriano, please send full port tarballs instead of diffs while the >> >> port >> >> > is WIP. >> >> >> >> >> >> >> >> -- >> >> Adriano Barbosa >> > >> > >> > >> >> > >
nextcloudclient.tar.gz
Description: application/gzip
