[PR PATCH] nextcloud-client: update to 3.3.1

[PR PATCH] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1593 bytes --]

There is a new pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 3281 bytes --]

From d4d80eb8ceebd8c5b2da4d2a3828aa13be97fc50 Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Sat, 21 Aug 2021 18:52:38 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.1

---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             |  9 +++--
 2 files changed, 7 insertions(+), 38 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..9d604aec8286 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,10 @@
 # Template file for 'nextcloud-client'
+# XXX packaging status
+# XXX testing status
+# XXX crossbuild status
+# XXX runtime status
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.1
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -18,8 +22,9 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=c315bbbd64b6ec0b169cfc5b2a063099ae66eb2ecbc3e3443c3f0ed3ddb2980f
 
 build_options="dolphin"
 desc_option_dolphin="Build KDE dolphin support"

Re: [PR PATCH] [Updated] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

There is an updated pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 3491 bytes --]

From fcef971ebafd67f5b2a220463412e6d5a0614dff Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Sat, 21 Aug 2021 18:52:38 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.1

---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             |  7 ++--
 2 files changed, 5 insertions(+), 38 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..d63f1101ad67 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,6 @@
 # Template file for 'nextcloud-client'
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.1
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -18,8 +18,9 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=c315bbbd64b6ec0b169cfc5b2a063099ae66eb2ecbc3e3443c3f0ed3ddb2980f
 
 build_options="dolphin"
 desc_option_dolphin="Build KDE dolphin support"
@@ -34,6 +35,8 @@ fi
 subpackages="nextcloud-client-kwallet nextcloud-client-devel"
 subpackages+=" $(vopt_if dolphin 'nextcloud-client-dolphin')"
 
+make_check="ci-skip"
+
 if [ "$XBPS_CHECK_PKGS" ]; then
 	configure_args+=" -DUNIT_TESTING=1"
 	# argp.h is used only by testing and is not a requirement.

Re: [PR REVIEW] nextcloud-client: update to 3.3.1

[paper42]

[-- Attachment #1: Type: text/plain, Size: 237 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#discussion_r694256546

Comment:
move this to the first block, remove " and add a comment why this is necessary if you know

Re: [PR PATCH] [Updated] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

There is an updated pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 3258 bytes --]

From 908734dc8117316ab1cd1bd82049e42ba50a8a82 Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Tue, 24 Aug 2021 20:10:43 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.1

---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             |  7 ++--
 2 files changed, 5 insertions(+), 38 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..da896d1dff87 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,6 @@
 # Template file for 'nextcloud-client'
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.1
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -18,8 +18,11 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=c315bbbd64b6ec0b169cfc5b2a063099ae66eb2ecbc3e3443c3f0ed3ddb2980f
+# can't reproduce locally failing tests from CI
+make_check="ci-skip"
 
 build_options="dolphin"
 desc_option_dolphin="Build KDE dolphin support"

Re: [PR PATCH] [Updated] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

There is an updated pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 3256 bytes --]

From a15f2cb79b6a7c319802571021ab1928a70bcf80 Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Tue, 24 Aug 2021 20:10:43 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.1

---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             |  7 ++--
 2 files changed, 5 insertions(+), 38 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..1b1cbfe6cd35 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,6 @@
 # Template file for 'nextcloud-client'
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.1
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -18,8 +18,11 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=c315bbbd64b6ec0b169cfc5b2a063099ae66eb2ecbc3e3443c3f0ed3ddb2980f
+# can't reproduce locally failing tests from CI
+make_check=ci-skip
 
 build_options="dolphin"
 desc_option_dolphin="Build KDE dolphin support"

Re: [PR REVIEW] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 200 bytes --]

New review comment by yopito on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#discussion_r695087876

Comment:
thanks for the review. please find an updated version.

Re: [PR REVIEW] nextcloud-client: update to 3.3.1

[paper42]

[-- Attachment #1: Type: text/plain, Size: 252 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#discussion_r695135928

Comment:
`make_check=ci-skip` already means the tests are failing only in CI, so you don't need to leave a comment

Re: [PR REVIEW] nextcloud-client: update to 3.3.1

[paper42]

[-- Attachment #1: Type: text/plain, Size: 286 bytes --]

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#discussion_r695135928

Comment:
`make_check=ci-skip` already means the tests are failing only in CI, so you don't need to leave a comment if you don't have anything to add

Re: [PR PATCH] [Updated] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

There is an updated pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 3207 bytes --]

From b01550640d71382e7f166d7890b2df81406923fc Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Tue, 24 Aug 2021 20:10:43 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.1

---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             |  6 ++--
 2 files changed, 4 insertions(+), 38 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..cea415ded04d 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,6 @@
 # Template file for 'nextcloud-client'
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.1
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -18,8 +18,10 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=c315bbbd64b6ec0b169cfc5b2a063099ae66eb2ecbc3e3443c3f0ed3ddb2980f
+make_check=ci-skip
 
 build_options="dolphin"
 desc_option_dolphin="Build KDE dolphin support"

Re: nextcloud-client: update to 3.3.1

[q66]

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#issuecomment-905458924

Comment:
webengine is optional now, so add the option back as it was 

Re: nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 317 bytes --]

New comment by yopito on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#issuecomment-907628195

Comment:
> webengine is optional now, so add the option back as it was

nice catch, thanks for pointing this.

BTW, 3.3.2 is out: I keep this PR for 3.3.2, or do I open another one ?

Re: [PR PATCH] [Updated] nextcloud-client: update to 3.3.1

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1598 bytes --]

There is an updated pull request by yopito against master on the void-packages repository

https://github.com/yopito/void-packages nextcloud.331
https://github.com/void-linux/void-packages/pull/32619

nextcloud-client: update to 3.3.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl



A patch file from https://github.com/void-linux/void-packages/pull/32619.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-nextcloud.331-32619.patch --]
[-- Type: text/x-diff, Size: 4212 bytes --]

From e6a514cb05464d35e6bc65eaaabd606b20f100e3 Mon Sep 17 00:00:00 2001
From: yopito <pierre.bourgin@free.fr>
Date: Tue, 24 Aug 2021 20:10:43 +0200
Subject: [PATCH] nextcloud-client: update to 3.3.2, webengine is optional
 again

upstream officially supports build without webengine.
webengine option is disabled for known non-matching architectures.
---
 ...ariadic-macro-warning-only-for-clang.patch | 36 -------------------
 srcpkgs/nextcloud-client/template             | 17 ++++++---
 2 files changed, 13 insertions(+), 40 deletions(-)
 delete mode 100644 srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch

diff --git a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch b/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
deleted file mode 100644
index 5afd721f392c..000000000000
--- a/srcpkgs/nextcloud-client/patches/fix-clang-variadic-macro-warning-only-for-clang.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-source: https://github.com/nextcloud/desktop/pull/3132
-
-From 52951820b2c094bcff0ec1a36c3c89d1c08c9618 Mon Sep 17 00:00:00 2001
-From: yopito <pierre.bourgin@free.fr>
-Date: Wed, 14 Apr 2021 22:07:19 +0200
-Subject: [PATCH] clang's variadic macro warnings: only for clang
-
-following settings of cmake/modules/Warnings.cmake
----
- src/CMakeLists.txt | 10 ++++++----
- 1 file changed, 6 insertions(+), 4 deletions(-)
-
-diff --git src/CMakeLists.txt src/CMakeLists.txt
-index f29349582..6f3145331 100644
---- a/src/CMakeLists.txt
-+++ b/src/CMakeLists.txt
-@@ -30,10 +30,12 @@ if(NOT MSVC)
-     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_FORTIFY_SOURCE=2")
-   endif()
- 
--  # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
--  # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
--  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
--  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  if (CMAKE_CXX_COMPILER MATCHES "Clang")
-+    # Calling Qt's qCWarning(category, ...) with no params for "..." is a GNU
-+    # extension (C++11 §16.3/4 forbids them). Silence clang's warnings.
-+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-gnu-zero-variadic-macro-arguments")
-+  endif()
- endif()
- 
- if(WIN32)
--- 
-2.31.1
-
diff --git a/srcpkgs/nextcloud-client/template b/srcpkgs/nextcloud-client/template
index ff2199b915d5..e13e8c1b20d0 100644
--- a/srcpkgs/nextcloud-client/template
+++ b/srcpkgs/nextcloud-client/template
@@ -1,6 +1,6 @@
 # Template file for 'nextcloud-client'
 pkgname=nextcloud-client
-version=3.2.3
+version=3.3.2
 revision=1
 wrksrc="desktop-${version}"
 build_style=cmake
@@ -10,7 +10,7 @@ makedepends="qt5-tools-devel qt5-declarative-devel qt5-webchannel-devel
  qt5-location-devel qtkeychain-qt5-devel sqlite-devel libcloudproviders-devel
  qt5-quickcontrols2-devel qt5-websockets-devel qt5-svg-devel
  $(vopt_if dolphin 'extra-cmake-modules kio-devel')
- qt5-webengine-devel"
+ $(vopt_if webengine 'qt5-webengine-devel')"
 depends="qt5-graphicaleffects"
 checkdepends="cmocka-devel"
 conf_files="/etc/Nextcloud/sync-exclude.lst"
@@ -18,13 +18,22 @@ short_desc="NextCloud Desktop client"
 maintainer="yopito <pierre.bourgin@free.fr>"
 license="GPL-2.0-or-later"
 homepage="https://nextcloud.com/clients/"
+changelog="https://github.com/nextcloud/desktop/releases"
 distfiles="https://github.com/nextcloud/desktop/archive/v${version}.tar.gz"
-checksum=40dc6fe3aaa62851af3fa480f108cc7e3bfff64b6bea0602e9546359b3d2cba2
+checksum=6ea5ccc9579339f2d1652f26582dbdd8212dd5e0c28659dce7fe9ddece8a59c5
+make_check=ci-skip
 
-build_options="dolphin"
+build_options="dolphin webengine"
 desc_option_dolphin="Build KDE dolphin support"
+desc_option_webengine="Build Qt5 WebEngine support"
 build_options_default="dolphin"
 
+if [ "$XBPS_TARGET_ENDIAN" = "le" ]; then
+	if [ "$XBPS_WORDSIZE" = "$XBPS_TARGET_WORDSIZE" ]; then
+		build_options_default+=" webengine"
+	fi
+fi
+
 if [ "$CROSS_BUILD" ]; then
 	hostmakedepends+=" qt5-qmake qt5-host-tools qt5-tools"
 	# provides desktoptojson

Re: nextcloud-client: update to 3.3.2

[yopito]

[-- Attachment #1: Type: text/plain, Size: 201 bytes --]

New comment by yopito on void-packages repository

https://github.com/void-linux/void-packages/pull/32619#issuecomment-907647595

Comment:
update to 3.3.2.
runtime test with webengine disabled: fine.

Re: [PR PATCH] [Closed]: nextcloud-client: update to 3.3.2

[yopito]

[-- Attachment #1: Type: text/plain, Size: 1438 bytes --]

There's a closed pull request on the void-packages repository

nextcloud-client: update to 3.3.2
https://github.com/void-linux/void-packages/pull/32619

Description:
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-musl)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl