Bug 41781 - Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1
Summary: Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoops...
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low critical
Assignee: Kyle M Hall (khall)
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on: 28833
Blocks:
  Show dependency treegraph
 
Reported: 2026-02-05 21:10 UTC by Kyle M Hall (khall)
Modified: 2026-02-06 14:22 UTC (History)
7 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
26.05.00
Circulation function:


Attachments
Bug 41781 - Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1 (1.13 KB, patch)
2026-02-05 21:16 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 41781 - Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1 (1.16 KB, patch)
2026-02-05 22:18 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 41781: Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1 (1.26 KB, patch)
2026-02-06 10:35 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 2026-02-05 21:10:45 UTC
If HoldsQueueParallelLoopsCount is set to a value greater than 1, build_holds_queue.pl will fail with the message:

Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205.
Comment 1 Kyle M Hall (khall) 2026-02-05 21:15:18 UTC
I suspect this has something to do with perl versions ( or debian versions ). It works on Debian 11.11, but not on 12.11 or 12.12. They have different versions of Perl as well: 5.32.1, 5.36.0, and 5.36.0 respectively.
Comment 2 Kyle M Hall (khall) 2026-02-05 21:16:25 UTC
Created attachment 192552 [details] [review]
Bug 41781 - Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1

If HoldsQueueParallelLoopsCount is set to a value greater than 1, build_holds_queue.pl will fail with the message:

Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205.

Test Plan:
1) Run misc/cronjobs/holds/build_holds_queue.pl
2) Note the error message
3) Run misc/cronjobs/holds/build_holds_queue.pl
4) Note the error message is gone!
Comment 3 Lucas Gass (lukeg) 2026-02-05 22:17:55 UTC
My test plan:

Test on D11 ( Bullseye )
1. KOHA_IMAGE=main-bullseye ktd up
2. Set HoldsQueueParallelLoopsCount > 1 
3. perl misc/cronjobs/holds/build_holds_queue.pl
4. Success 
5. Apply patch, restart_all
6. perl misc/cronjobs/holds/build_holds_queue.pl
7. Success 

Test on D12 ( Bookworm )
1. KOHA_IMAGE=main-bookworm ktd up
2. Set HoldsQueueParallelLoopsCount > 1 
3. perl misc/cronjobs/holds/build_holds_queue.pl
4. FAILS: Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205. 
5. Apply patch, restart_all
6. perl misc/cronjobs/holds/build_holds_queue.pl
7. Success 

Test on D13: ( Trixie )
1. KOHA_IMAGE=main-bookworm ktd up
2. Set HoldsQueueParallelLoopsCount > 1 
3. perl misc/cronjobs/holds/build_holds_queue.pl
4. FAILS: Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205. 
5. Apply patch, restart_all
6. perl misc/cronjobs/holds/build_holds_queue.pl
7. Success
Comment 4 Lucas Gass (lukeg) 2026-02-05 22:18:31 UTC
Created attachment 192575 [details] [review]
Bug 41781 - Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1

If HoldsQueueParallelLoopsCount is set to a value greater than 1, build_holds_queue.pl will fail with the message:

Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205.

Test Plan:
1) Run misc/cronjobs/holds/build_holds_queue.pl
2) Note the error message
3) Run misc/cronjobs/holds/build_holds_queue.pl
4) Note the error message is gone!

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 5 Marcel de Rooy 2026-02-06 10:35:48 UTC
Created attachment 192602 [details] [review]
Bug 41781: Holds queue builder ( build_holds_queue.pl ) fails if HoldsQueueParallelLoopsCount is greater than 1

If HoldsQueueParallelLoopsCount is set to a value greater than 1, build_holds_queue.pl will fail with the message:

Can't locate object method "new" via package "Parallel::ForkManager" (perhaps you forgot to load "Parallel::ForkManager"?) at /kohadevbox/koha/C4/HoldsQueue.pm line 205.

Test Plan:
1) Run misc/cronjobs/holds/build_holds_queue.pl
2) Note the error message
3) Run misc/cronjobs/holds/build_holds_queue.pl
4) Note the error message is gone!

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Marcel de Rooy 2026-02-06 10:36:03 UTC
        * Commit title does not start with 'Bug XXXXX: ' - 6ea33659d7
You should know ;)
Comment 7 Kyle M Hall (khall) 2026-02-06 11:38:18 UTC
(In reply to Marcel de Rooy from comment #6)
>         * Commit title does not start with 'Bug XXXXX: ' - 6ea33659d7
> You should know ;)

Indeed! I was in a bit of a rush! I'll get that fixed up!
Comment 8 Kyle M Hall (khall) 2026-02-06 11:44:15 UTC
(In reply to Kyle M Hall (khall) from comment #7)
> (In reply to Marcel de Rooy from comment #6)
> >         * Commit title does not start with 'Bug XXXXX: ' - 6ea33659d7
> > You should know ;)
> 
> Indeed! I was in a bit of a rush! I'll get that fixed up!

Didn't notice you already fixed it. Thanks!
Comment 9 Lucas Gass (lukeg) 2026-02-06 14:22:09 UTC
Pushed to main for 26.05.00. 

Thanks all!