Bug 31499 - Add libhttp-tiny-perl 0.076 dependency for ES7
Summary: Add libhttp-tiny-perl 0.076 dependency for ES7
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Packaging (show other bugs)
Version: Main
Hardware: All All
: P1 - high blocker
Assignee: Mason James
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 25439
  Show dependency tree / graph
 
Reported: 2022-08-31 16:42 UTC by Mason James
Modified: 2023-06-08 22:26 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00,21.11, 22.05.05
Circulation function:

Attachments
Bug 31499: Add libhttp-tiny-perl 0.076 dependency for ES7 (738 bytes, patch)
2022-08-31 16:44 UTC, Mason James 		Details | Diff | Splinter Review
Bug 31499: Add libhttp-tiny-perl 0.076 dependency for ES7 (752 bytes, patch)
2022-08-31 17:26 UTC, Tomás Cohen Arazi (tcohen) 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mason James 2022-08-31 16:42:33 UTC 
this small patch allows ES7 to work on deb10
Comment 1 Mason James 2022-08-31 16:44:20 UTC 
Created attachment 140009 [details] [review]
Bug 31499: Add libhttp-tiny-perl 0.076 dependency for ES7

no testing info provided
Comment 3 Tomás Cohen Arazi (tcohen) 2022-08-31 17:26:18 UTC 
Created attachment 140012 [details] [review]
Bug 31499: Add libhttp-tiny-perl 0.076 dependency for ES7

no testing info provided

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Tomás Cohen Arazi (tcohen) 2022-08-31 17:27:42 UTC 
Mason, please clarify if this depends on something else, to aid rmaints. Also, please use the rel_*_candidate to let rmaints know they need to backport this.

And thanks, great job as usual.
Comment 5 Tomás Cohen Arazi (tcohen) 2022-08-31 17:37:32 UTC 
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 6 Marcel de Rooy 2022-09-13 12:13:59 UTC 
I reinstalled my dev setup (D10 container). And still found the version of HTTP/Tiny.pm under the required level.
0.70 < 0.76
Could this be an issue? I used http://debian.koha-community.org/koha-staging dev main buster
Comment 7 Marcel de Rooy 2022-09-13 12:16:34 UTC 
HTTP::Tiny                                    0.070 *           0.076                   Yes
Search::Elasticsearch                         6.00 *            6.80                    Yes
Comment 8 Marcel de Rooy 2022-09-13 12:39:21 UTC 
Okay, this triggered me sufficiently to move it to D11.
Also with staging.

And now I have:
perl koha_perl_deps.pl -u
                                              Installed         Required          Module is
Module Name                                   Version           Version            Required
--------------------------------------------------------------------------------------------

--------------------------------------------------------------------------------------------
Total modules reported: 0                      * Module is missing or requires an upgrade.

So, thats fine with me.

But could you please confirm that other people on D10 wont suffer from this one?
Comment 9 Lucas Gass (lukeg) 2022-09-20 19:37:37 UTC 
Backported to 22.05.x for 22.05.05
Comment 10 Mason James 2022-09-21 11:28:57 UTC 
(In reply to Lucas Gass from comment #9)
> Backported to 22.05.x for 22.05.05

hi Lucas
could you revert this patch for now?, lets leave it for master/main branch
Comment 11 Arthur Suzuki 2022-09-21 15:09:29 UTC 
Thanks!

Pushed to 21.11 for 21.11.12
Comment 12 Mason James 2022-09-21 17:16:40 UTC 
(In reply to Arthur Suzuki from comment #11)
> Thanks!
> 
> Pushed to 21.11 for 21.11.12

hi Arthur
could you revert this patch for now?, lets leave it for master/main branch
Comment 13 Lucas Gass (lukeg) 2022-09-28 14:51:33 UTC 
(In reply to Mason James from comment #10)
> (In reply to Lucas Gass from comment #9)
> > Backported to 22.05.x for 22.05.05
> 
> hi Lucas
> could you revert this patch for now?, lets leave it for master/main branch

Mason, yes. I have reverted this bugfix from the 22.05.x branch.
Comment 14 Arthur Suzuki 2022-10-03 21:08:39 UTC 
likewise, bug was reverted on the 21.11.x branch.
Comment 15 Victor Grousset/tuxayo 2022-10-13 15:10:31 UTC 
Not backporting to 21.05.x then. Marking resolved, nothing to document.