Package Details: cpr 1.11.1-4

Git Clone URL: https://aur.archlinux.org/cpr.git (read-only, click to copy)
Package Base: cpr
Description: C++ Requests: Curl for People, a spiritual port of Python Requests.
Upstream URL: https://github.com/libcpr/cpr
Licenses: MIT
Provides: libcpr.so
Submitter: zlima12
Maintainer: zlima12 (horror-proton, FabioLolix)
Last Packager: zlima12
Votes: 5
Popularity: 0.22
First Submitted: 2021-04-22 20:19 (UTC)
Last Updated: 2025-02-18 06:48 (UTC)

Latest Comments

1 2 Next › Last »

zlima12 commented on 2025-02-18 06:54 (UTC)

I've confirmed that the issue is upstream and I'll create an issue there soon. In the mean time, I've pushed a temporary fix that avoids testing the problematic value, so the package should build fine now.

zlima12 commented on 2025-02-17 23:11 (UTC)

It seems as though the issue was introduced with curl 8.12, since downgrading to 8.11.1-3 causes the errors to go away. I'll look into it a bit further.

FabioLolix commented on 2025-02-17 18:12 (UTC)

Hi, I have made some tests using both AUR pkgbuilds and with mine just refreshed https://github.com/FabioLolix/PKGBUILD-AUR_fix/tree/master/zz_tests

I confirm the issue, cpr-git instead don't enable them

About my tests; of the four variations (stable, git and bundled, unbundled curl+gtest) have found that only git_bundled pass all tests while other other fail st least 3 tests

With a quick look don't seem to be already reported https://github.com/libcpr/cpr/issues

From memory tests where already failing when I send the patch to zlima12 on 2024/10 (that's why they weren't enabled) but I didn't in it further

About this commit https://aur.archlinux.org/cgit/aur.git/commit/?h=cpr&id=345c8e77d1b194980057e9a36968b24c98f61211 honestly I don't the solution

Meanwhile --nocheck options exist for makepkg

PatrikModesto commented on 2025-02-17 17:29 (UTC)

Hi, something must have changed recently, because I'm unable to reinstall cpr, it fails 3 unittests.

The following tests FAILED:
     1 - cpr_get_tests (Failed)
     3 - cpr_session_tests (Failed)
     6 - cpr_head_tests (Failed)

BowDown097 commented on 2024-10-15 12:47 (UTC)

Thanks for the update! Everything works again.

FabioLolix commented on 2024-10-14 16:50 (UTC)

Thanks for updating!

horror-proton commented on 2024-10-14 16:19 (UTC) (edited on 2024-10-14 16:39 (UTC) by horror-proton)

Does the latest commit resolve this issue? A -DBUILD_SHARED_LIBS=ON was added and the package should now contain shared library instead of a static one. Patch from @FabioLolix has been applied.

FabioLolix commented on 2024-10-14 16:18 (UTC) (edited on 2024-10-14 16:18 (UTC) by FabioLolix)

Confirming the issue, I have already send John a patch (btw horror-proton is not reachable by email, not even using git log here) and fixed a couple of other things

(1/1) Arming ConditionNeedsUpdate...
Checking PKGBUILD
PKGBUILD (cpr) W: Non-unique source name (1.11.0.tar.gz). Use a unique filename.
Checking cpr-1.11.0-1-x86_64.pkg.tar.zst
cpr E: Uncommon license identifiers such as 'MIT' require license files below /usr/share/licenses/cpr/ or switching to common license identifiers. Found 0/1 required license files.
cpr W: Dependency included, but may not be needed ('curl')
==> Running checkpkg
  -> Checking packages
usr/lib/libcpr.so                         | usr/lib/libcpr.a
usr/lib/libcpr.so.1                       <
usr/lib/libcpr.so.1.10.3                      <
==> WARNING: Sonames differ in cpr!
libcpr.so=1-64                            <

my pkgbuild: https://github.com/FabioLolix/AURCOPY/blob/main/cpr/PKGBUILD

revision diff: https://github.com/FabioLolix/AURCOPY/commit/1708ce42c03729a7b3667f1bb86acff175cc607e

BowDown097 commented on 2024-10-14 15:15 (UTC) (edited on 2024-10-14 15:17 (UTC) by BowDown097)

CMake package is completely broken since the update to 1.11.0:

  • Link interface contains OpenSSL, but the target is never imported at any point. This problem can be fixed by manually adding find_package(OpenSSL).
  • Linking the library no longer works and has to be done manually.

I believe this breakage may have to do with the library switching from shared to static by default with this update, but I'm not 100% sure.

zlima12 commented on 2023-04-02 02:13 (UTC)

horror-proton: I have added you as a co-maintainer. Sorry for my inactivity, I should have updated the package long ago. The change you mentioned below seems reasonable to me; please feel free to add it.