Time Nick Message 07:26 marcelr hi #koha 07:55 alex_a Bonjour 08:15 cait1 good morning #koha 08:25 reiveune hello 09:11 ashimema morning 09:13 marcelr o/ 09:16 tuxayo hi all :) 09:17 magnuse \o/ 09:24 Joubu @later tell fridolin something wrong happened on 29483 09:24 huginn Joubu: The operation succeeded. 09:27 Joubu @later tell fridolin forget that 09:27 huginn Joubu: The operation succeeded. 09:52 Joubu @later tell mtj ping for bug 30084 09:52 huginn Joubu: The operation succeeded. 09:58 mtj hi Joubu, i think the patch for 30084 will be ok :) 10:02 Joubu mtj: from where will it get pulled? 10:03 Joubu we were assuming it was part of perl deps, but it looks like it's not true on (at least) D11 10:04 mtj aah, yes - you are correct 10:05 mtj ..we will need to explicitly declare it for deb11 (and others?) 10:06 mtj well spotted Joubu++ 10:06 Joubu mtj: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25920#c14 10:06 huginn Bug 25920: blocker, P1 - high, ---, mtj, Pushed to oldstable , Add liblocale-codes-perl package to fix ubuntu-stable (focal) 10:07 Joubu this is the original question 10:07 Joubu "why do we need it?" :D 10:07 Joubu I have to admit I've lost track 10:08 mtj looking now 10:10 Joubu mtj: we are using several Locale::* perl packages, but I don't think they are coming from this package 10:14 mtj hmmm, it looks likely the package is not needed at all 🤔 10:17 mtj aaah, liblocale-codes-perl package provides 'Locale::Language' module - which we do need 10:20 Joubu liblocale-po-perl: /usr/share/perl5/Locale/PO.pm 10:20 Joubu libintl-perl: /usr/share/perl5/Locale/Messages.pm 10:21 Joubu libtemplate-plugin-gettext-perl: /usr/share/perl5/Locale/XGettext/TT2.pm 10:22 Joubu liblocale-currency-format-perl: /usr/share/perl5/Locale/Currency/Format.pm 10:22 Joubu that's all what I see for Locale:: 10:23 Joubu @later tell davidnind Hi David, could you add a SO stamp on 29940? 10:23 huginn Joubu: The operation succeeded. 10:28 Joubu mtj: are you on it? 10:30 mtj Joubu: yep 10:33 mtj git blame says 'Locale::Language' has been a dep since 2010 10:46 mtj ping Joubu, can you run 'apt-cache policy liblocale-codes-perl' 10:47 mtj ..if you dont have liblocale-codes-perl installed - dont install it yet :) 10:52 pastebot Someone at 127.0.0.1 pasted "liblocale-codes-perl: Installe" (7 lines) at https://paste.koha-community.org/7845 10:53 Joubu mtj: ^^ 10:56 mtj Joubu: looks like locale::lang was included in the fisrt version of the Makefile d0cc4b29c7d 10:56 mtj from 2007 11:00 mtj Joubu: it was used by ./misc/translator/po/stats.pl - which is now deleted :) 11:02 mtj ..so we confirm that liblocale-codes-perl aka Locale::Language, is not required 11:03 mtj git reset --hard b695a10e 11:03 mtj $ grep -ir 'Locale::Lang' ./* 11:03 mtj ./misc/translator/po/stats.pl:use Locale::Language; 11:07 * mtj adds irc log to the bug 11:20 domm I have a question regarding C4::Heading::MARC21: https://github.com/Koha-Community/Koha/blob/master/C4/Heading/MARC21.pm#L49 and #167 define some hardcoded mappings which subfields to include/display eg in the popup window when selecting an authority. 11:20 domm But we have some data needed to properly identify the correct auth in a subfield that is not included in this mapping (i.e. 150g). 11:20 domm Would you accept additions to these mapping in upstream? Or do we have to patch our local instance? 11:20 domm Or could these mapping be moved into some config files? 12:17 alohabot 🎠ðŸ Koha '21.11.x' packages pushed to 'koha-staging' repo 🥟 🫠🡠12:17 koha-jenkins Yippee, build fixed! 12:17 koha-jenkins Project Koha_21.11_U_Stable build #37: FIXED in 34 min: https://jenkins.koha-community.org/job/Koha_21.11_U_Stable/37/ 12:25 koha-jenkins Project Koha_21.11_U20 build #31: SUCCESS in 42 min: https://jenkins.koha-community.org/job/Koha_21.11_U20/31/ 12:34 koha-jenkins Project Koha_21.11_D10 build #47: SUCCESS in 50 min: https://jenkins.koha-community.org/job/Koha_21.11_D10/47/ 12:46 koha-jenkins Project Koha_21.11_U18 build #35: SUCCESS in 1 hr 2 min: https://jenkins.koha-community.org/job/Koha_21.11_U18/35/ 12:51 koha-jenkins Project Koha_21.11_U21 build #30: SUCCESS in 33 min: https://jenkins.koha-community.org/job/Koha_21.11_U21/30/ 12:54 koha-jenkins Project Koha_21.11_D11 build #49: SUCCESS in 1 hr 11 min: https://jenkins.koha-community.org/job/Koha_21.11_D11/49/ 13:10 koha-jenkins Project Koha_21.11_D12 build #25: SUCCESS in 45 min: https://jenkins.koha-community.org/job/Koha_21.11_D12/25/ 13:16 davewood im still working with bin/cronjobs/overdue_notices.pl ... the USAGE of said script is kind of mysterious to me ... it says to pass non-single-character options with a single dash. e.g. overdue_notices.pl -date=2022-01-01 ... but when I read Getopt::Long metacpan that doesnt make sense. it should be --date not -date 13:17 davewood but overdue_notices.pl tells me to use -test or -email and also --frombranch or --triggered. 13:17 davewood is there a reason for this inconsistency or can I fix it? 13:18 alohabot 🎠🙉 Koha '21.11.x' packages pushed to 'koha-staging' repo 🥑 🥂 🬠13:26 koha-jenkins Project Koha_21.11_U_Stable build #38: SUCCESS in 34 min: https://jenkins.koha-community.org/job/Koha_21.11_U_Stable/38/ 13:30 cait1 davewood: we don't use the date option so not sure, but i could be a simple mistake to fix? 13:30 cait1 we use -t and define the overdue notices in the overdue notice tool 13:35 davewood cait1: thats what im trying to do. but the USAGE of Getopt::Long in this script is not correct inmy opinion 13:36 davewood it should be --csv, --date, etcetc 13:37 davewood im just using -date for debugging my cronjob options 13:45 koha-jenkins Project Koha_21.11_U20 build #32: SUCCESS in 58 min: https://jenkins.koha-community.org/job/Koha_21.11_U20/32/ 13:51 koha-jenkins Project Koha_21.11_D10 build #48: SUCCESS in 41 min: https://jenkins.koha-community.org/job/Koha_21.11_D10/48/ 13:52 koha-jenkins Project Koha_21.11_D9 build #32: SUCCESS in 1 hr 17 min: https://jenkins.koha-community.org/job/Koha_21.11_D9/32/ 14:00 davewood PR for overdue_notices.pl ... https://github.com/Koha-Community/Koha/commit/8a12f1d017bb56336a5075f7d96be2207120d9ce 14:02 davewood fixed the single/double dash inconsistency and added readable names for single char options -n => --nomail ... -v => --verbose 14:04 koha-jenkins Project Koha_21.11_U18 build #36: SUCCESS in 37 min: https://jenkins.koha-community.org/job/Koha_21.11_U18/36/ 14:04 Joubu davewood: github repo is a mirror, please provide a patch on our bug tracker: bugs.koha-community.org/ 14:05 koha-jenkins Project Koha_21.11_D11 build #50: SUCCESS in 1 hr 11 min: https://jenkins.koha-community.org/job/Koha_21.11_D11/50/ 14:12 davewood Joubu: done. 14:21 davewood Joubu: I just figured out how to get a patch file from a github PR. :) 14:24 Joubu I've pulled your repo and attached your patch to the bug report :) 14:34 koha-jenkins Project Koha_21.11_U20 build #33: SUCCESS in 42 min: https://jenkins.koha-community.org/job/Koha_21.11_U20/33/ 14:38 koha-jenkins Project Koha_21.11_U21 build #31: SUCCESS in 34 min: https://jenkins.koha-community.org/job/Koha_21.11_U21/31/ 14:43 kidclamp ashimema++ 14:43 kidclamp joubu++ 14:43 ashimema ? 14:46 koha-jenkins Project Koha_21.11_D12 build #26: SUCCESS in 1 hr 1 min: https://jenkins.koha-community.org/job/Koha_21.11_D12/26/ 14:47 kidclamp bug 30085 14:47 huginn Bug https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30085 enhancement, P5 - low, ---, nick, Passed QA , Improve performance of CanItemBeReserved 14:57 Joubu My ROI for a the full day of QA is null however, not very engouraging :D 15:08 koha-jenkins Project Koha_21.11_D9 build #33: SUCCESS in 1 hr 16 min: https://jenkins.koha-community.org/job/Koha_21.11_D9/33/ 15:13 koha-jenkins Project Koha_21.11_D11 build #51: UNSTABLE in 1 hr 7 min: https://jenkins.koha-community.org/job/Koha_21.11_D11/51/ 15:14 koha-jenkins Project Koha_21.11_U20 build #34: SUCCESS in 35 min: https://jenkins.koha-community.org/job/Koha_21.11_U20/34/ 15:19 koha-jenkins Project Koha_21.11_U_Stable build #39: SUCCESS in 45 min: https://jenkins.koha-community.org/job/Koha_21.11_U_Stable/39/ 15:37 reiveune bye 16:20 koha-jenkins Yippee, build fixed! 16:20 koha-jenkins Project Koha_21.11_D11 build #52: FIXED in 1 hr 7 min: https://jenkins.koha-community.org/job/Koha_21.11_D11/52/