public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
* [rpms/perl-WWW-Curl] epel10: add back support for CURLOPT_RESOLV
@ 2026-08-02 10:05 Emmanuel Seyman
0 siblings, 0 replies; only message in thread
From: Emmanuel Seyman @ 2026-08-02 10:05 UTC (permalink / raw)
To: git-commits
A new commit has been pushed.
Repo : rpms/perl-WWW-Curl
Branch : epel10
Commit : 4ea98af7a7fd80dc738ecf12eb406bc3030a676a
Author : Emmanuel Seyman <emmanuel@seyman.fr>
Date : 2023-10-29T12:13:36+01:00
Stats : +55/-7 in 2 file(s)
URL : https://src.fedoraproject.org/rpms/perl-WWW-Curl/c/4ea98af7a7fd80dc738ecf12eb406bc3030a676a?branch=epel10
Log:
add back support for CURLOPT_RESOLV
---
diff --git a/WWW-Curl-4.17-add-back-CURLOPT_RESOLV-support.patch b/WWW-Curl-4.17-add-back-CURLOPT_RESOLV-support.patch
new file mode 100644
index 0000000..3c87dbf
--- /dev/null
+++ b/WWW-Curl-4.17-add-back-CURLOPT_RESOLV-support.patch
@@ -0,0 +1,47 @@
+commit 379a461fa726498dcc7bf68e446c473eb73e310f
+Author: Emmanuel Seyman <emmanuel@seyman.fr>
+Date: Sun Oct 29 11:40:19 2023 +0100
+
+ revert commit 499baf9c3689c23dfc49196570fb230076c935de
+
+ Commit 499baf9 added preprocessor conditionals based on whether
+ CURLOPT_RESOLVE is defined Unfortunately, CURLOPT_RESOLVE is an enum,
+ and NOT a preprocessor macro, so in effect this removed support for
+ CURLOPT_RESOLVE and results in a perl crash if the option is used.
+
+diff --git a/Curl.xs b/Curl.xs
+index cfa282d..8085b49 100644
+--- a/Curl.xs
++++ b/Curl.xs
+@@ -38,9 +38,7 @@ typedef enum {
+ SLIST_HTTPHEADER = 0,
+ SLIST_QUOTE,
+ SLIST_POSTQUOTE,
+-#ifdef CURLOPT_RESOLVE
+ SLIST_RESOLVE,
+-#endif
+ SLIST_LAST
+ } perl_curl_easy_slist_code;
+
+@@ -129,11 +127,9 @@ slist_index(int option)
+ case CURLOPT_POSTQUOTE:
+ return SLIST_POSTQUOTE;
+ break;
+-#ifdef CURLOPT_RESOLVE
+ case CURLOPT_RESOLVE:
+ return SLIST_RESOLVE;
+ break;
+-#endif
+ }
+ croak("Bad slist index requested\n");
+ return SLIST_LAST;
+@@ -745,9 +741,7 @@ curl_easy_setopt(self, option, value, push=0)
+ case CURLOPT_HTTPHEADER:
+ case CURLOPT_QUOTE:
+ case CURLOPT_POSTQUOTE:
+-#ifdef CURLOPT_RESOLVE
+ case CURLOPT_RESOLVE:
+-#endif
+ {
+ /* This is an option specifying a list, which we put in a curl_slist struct */
+ AV *array = (AV *)SvRV(value);
diff --git a/perl-WWW-Curl.spec b/perl-WWW-Curl.spec
index c63f858..a840d98 100644
--- a/perl-WWW-Curl.spec
+++ b/perl-WWW-Curl.spec
@@ -14,6 +14,8 @@ Patch2: WWW-Curl-4.17-Adapt-to-changes-in-cURL-7.69.0.patch
Patch3: WWW-Curl-4.17-Adapt-to-curl-7.87.0.patch
# Workound a bug in cURL 7.87.0, bug #2160057, CPAN RT#145992
Patch4: WWW-Curl-4.17-Work-around-a-macro-bug-in-curl-7.87.0.patch
+# Workound a bug in WWW::CURL 4.17, bug #2245689, GH #9
+Patch5: WWW-Curl-4.17-add-back-CURLOPT_RESOLV-support.patch
BuildRequires: findutils
BuildRequires: make
BuildRequires: perl-interpreter
@@ -38,16 +40,11 @@ BuildRequires: libcurl-devel
WWW::Curl is a Perl extension interface for libcurl.
%prep
-%setup -q -n WWW-Curl-%{version}
-%patch0 -p1
-%patch1 -p1
-%patch2 -p1
-%patch3 -p1
-%patch4 -p1
+%autosetup -n WWW-Curl-%{version}
rm -rf inc && sed -i -e '/^inc\//d' MANIFEST
%build
-perl Makefile.PL INSTALLDIRS=vendor OPTIMIZE="%{optflags}" NO_PACKLIST=1 NO_PERLLOCAL=1
+/usr/bin/perl Makefile.PL INSTALLDIRS=vendor OPTIMIZE="%{optflags}" NO_PACKLIST=1 NO_PERLLOCAL=1
%{make_build}
%install
@@ -77,6 +74,10 @@ make test
%{_mandir}/man3/*
%changelog
+* Sun Oct 29 2023 Emmanuel Seyman <emmanuel@seyman.fr> - 4.17-36
+- Add back support for CURLOPT_RESOLV
+- Use %%autosetup to apply all patches
+
* Fri Jul 21 2023 Fedora Release Engineering <releng@fedoraproject.org> - 4.17-35
- Rebuilt for https://fedoraproject.org/wiki/Fedora_39_Mass_Rebuild
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2026-08-02 10:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-08-02 10:05 [rpms/perl-WWW-Curl] epel10: add back support for CURLOPT_RESOLV Emmanuel Seyman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox