[Bug 20727] New: Replace usage of File::Spec->tmpdir with Koha::UploadedFile->temporary_directory
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Bug ID: 20727 Summary: Replace usage of File::Spec->tmpdir with Koha::UploadedFile->temporary_directory Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org As explained in bug 20428 use tmpdir can cause issues and it just makes sense to standardize our temp directory in a universal way. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |20428 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20428 [Bug 20428] MARC import fails on Debian Stretch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 75200 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75200&action=edit Bug 20727: Replace usage of File::Spec->tmpdir with Koha::UploadedFile->temporary_directory As explained in bug 20428 use tmpdir can cause issues and it just makes sense to standardize our temp directory in a universal way. Test Plan: 1) Apply this patch 2) Verify you can still log in and use Koha 3) Verify the web installer still works 4) Verify EDI module can still download files via FTP 5) Verify fines.pl still runs with -o option 6) prove t/db_dependent/Plugins.t 7) prove t/db_dependent/Sitemapper.t 8) prove t/db_dependent/Templates.t -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy@rijksmuseum.nl --- Comment #2 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- +my $dir = Koha::UploadedFile->temporary_directory; I think it should be moved to some more general module? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #3 from Kyle M Hall <kyle@bywatersolutions.com> --- I had thought about that as well. Where would you want to move it? Koha.pm? (In reply to Marcel de Rooy from comment #2)
+my $dir = Koha::UploadedFile->temporary_directory;
I think it should be moved to some more general module?
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #4 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Kyle M Hall from comment #3)
I had thought about that as well. Where would you want to move it? Koha.pm?
Yeah. Definitely better.. Note that I commented too on 20428 that imo we need a general koha-conf variable for an alternative tmp directory.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Marcel de Rooy from comment #4)
(In reply to Kyle M Hall from comment #3)
I had thought about that as well. Where would you want to move it? Koha.pm?
Yeah. Definitely better.. Note that I commented too on 20428 that imo we need a general koha-conf variable for an alternative tmp directory..
I'll update this patch after bug 20428 has been pushed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mirko@abunchofthings.net --- Comment #6 from Mirko Tietgen <mirko@abunchofthings.net> --- (In reply to Kyle M Hall from comment #5)
(In reply to Marcel de Rooy from comment #4)
(In reply to Kyle M Hall from comment #3)
I had thought about that as well. Where would you want to move it? Koha.pm?
Yeah. Definitely better.. Note that I commented too on 20428 that imo we need a general koha-conf variable for an alternative tmp directory..
I'll update this patch after bug 20428 has been pushed.
Ping, 20428 has been pushed :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 75626 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75626&action=edit Bug 20727: Move temporary_directory() to C4::Context -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |BLOCKED --- Comment #8 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Looking here now -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|BLOCKED |Signed Off Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75200|0 |1 is obsolete| | --- Comment #9 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 75667 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75667&action=edit Bug 20727: Replace usage of File::Spec->tmpdir with Koha::UploadedFile->temporary_directory As explained in bug 20428 use tmpdir can cause issues and it just makes sense to standardize our temp directory in a universal way. Test Plan: 1) Apply this patch 2) Verify you can still log in and use Koha 3) Verify the web installer still works 4) Verify EDI module can still download files via FTP 5) Verify fines.pl still runs with -o option 6) prove t/db_dependent/Plugins.t 7) prove t/db_dependent/Sitemapper.t 8) prove t/db_dependent/Templates.t Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75626|0 |1 is obsolete| | --- Comment #10 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 75668 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75668&action=edit Bug 20727: Move temporary_directory() to C4::Context Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #11 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 75669 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75669&action=edit Bug 20727: (QA follow-up) Fix Upload.t and Plugins.t Trivial changes. Adding C4::Context->temporary_directory to subsequent tmpdir call in Plugins. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #12 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 75670 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75670&action=edit Bug 20727: (QA follow-up) Remove a few use statements again We do no longer need "use Koha::UploadedFile" in a few places. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Replace usage of |Replace usage of |File::Spec->tmpdir with |File::Spec->tmpdir with |Koha::UploadedFile->tempora |C4::Context->temporary_dire |ry_directory |ctory -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #13 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Do not we need to update the occurrence in updatedatabase.pl as well? installer/data/mysql/updatedatabase.pl:my $tmp= File::Spec->tmpdir.'/koha_upload'; -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #14 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 75833 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75833&action=edit Bug 20727: Replace use of tmpdir in updatedatabase.pl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #15 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 75834 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75834&action=edit Bug 20727: Add use line for File::Spec in C4::Context, sort used modules -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75834|0 |1 is obsolete| | --- Comment #16 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 75835 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75835&action=edit Bug 20727: Add use line for File::Spec in C4::Context, sort used modules -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75667|0 |1 is obsolete| | Attachment #75668|0 |1 is obsolete| | Attachment #75669|0 |1 is obsolete| | Attachment #75670|0 |1 is obsolete| | Attachment #75833|0 |1 is obsolete| | Attachment #75835|0 |1 is obsolete| | --- Comment #17 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75923 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75923&action=edit Bug 20727: Replace usage of File::Spec->tmpdir with Koha::UploadedFile->temporary_directory As explained in bug 20428 use tmpdir can cause issues and it just makes sense to standardize our temp directory in a universal way. Test Plan: 1) Apply this patch 2) Verify you can still log in and use Koha 3) Verify the web installer still works 4) Verify EDI module can still download files via FTP 5) Verify fines.pl still runs with -o option 6) prove t/db_dependent/Plugins.t 7) prove t/db_dependent/Sitemapper.t 8) prove t/db_dependent/Templates.t Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #18 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75924 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75924&action=edit Bug 20727: Move temporary_directory() to C4::Context Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #19 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75925 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75925&action=edit Bug 20727: (QA follow-up) Fix Upload.t and Plugins.t Trivial changes. Adding C4::Context->temporary_directory to subsequent tmpdir call in Plugins. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #20 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75926 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75926&action=edit Bug 20727: (QA follow-up) Remove a few use statements again We do no longer need "use Koha::UploadedFile" in a few places. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #21 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75927 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75927&action=edit Bug 20727: Replace use of tmpdir in updatedatabase.pl Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #22 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 75928 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=75928&action=edit Bug 20727: Add use line for File::Spec in C4::Context, sort used modules Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |nick@bywatersolutions.com --- Comment #23 from Nick Clemens <nick@bywatersolutions.com> --- Awesome work all! Pushed to master for 18.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to Master |RESOLVED CC| |martin.renvoize@ptfs-europe | |.com --- Comment #24 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Architectural enhancement, not backporting for 18.05.x series. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #25 from David Cook <dcook@prosentient.com.au> --- When I run "perl -c C4/Auth.pm", I get the following: dcook@kohaprod:/kohawebs/dev/dcook/git> perl -c C4/Auth.pm Subroutine psgi_env redefined at C4/Auth.pm line 50. Subroutine safe_exit redefined at C4/Auth.pm line 52. Subroutine get_template_and_user redefined at C4/Auth.pm line 142. Subroutine _version_check redefined at C4/Auth.pm line 715. Subroutine _session_log redefined at C4/Auth.pm line 761. Subroutine _timeout_syspref redefined at C4/Auth.pm line 768. Subroutine checkauth redefined at C4/Auth.pm line 778. Subroutine check_api_auth redefined at C4/Auth.pm line 1387. Subroutine check_cookie_auth redefined at C4/Auth.pm line 1629. Bareword "C4::Context::temporary_directory" not allowed while "strict subs" in use at C4/Auth.pm line 1750. BEGIN not safe after errors--compilation aborted at C4/Auth.pm line 1928. If I change "C4::Context::temporary_directory" to "C4::Context::temporary_directory()", that error goes away and compilation is fine. I'm guessing no one else has experienced this? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #26 from David Cook <dcook@prosentient.com.au> --- When I run "perl -c C4/Auth.pm", I get the following: perl -c C4/Auth.pm Subroutine psgi_env redefined at C4/Auth.pm line 50. Subroutine safe_exit redefined at C4/Auth.pm line 52. Subroutine get_template_and_user redefined at C4/Auth.pm line 142. Subroutine _version_check redefined at C4/Auth.pm line 715. Subroutine _session_log redefined at C4/Auth.pm line 761. Subroutine _timeout_syspref redefined at C4/Auth.pm line 768. Subroutine checkauth redefined at C4/Auth.pm line 778. Subroutine check_api_auth redefined at C4/Auth.pm line 1387. Subroutine check_cookie_auth redefined at C4/Auth.pm line 1629. Bareword "C4::Context::temporary_directory" not allowed while "strict subs" in use at C4/Auth.pm line 1750. BEGIN not safe after errors--compilation aborted at C4/Auth.pm line 1928. If I change "C4::Context::temporary_directory" to "C4::Context::temporary_directory()", that error goes away and compilation is fine. I'm guessing no one else has experienced this? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #27 from David Cook <dcook@prosentient.com.au> --- (In reply to David Cook from comment #26)
I'm guessing no one else has experienced this?
For some reason, when I use the web app, I don't get any errors. I wonder if we're suppressing them unintentionally in whatever is loading C4::Auth? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #28 from David Cook <dcook@prosentient.com.au> --- I see C4::Context turns off the warnings for redefined functions when there's a HTTP_USER_AGENT environmental variable, so that explains why the warnings don't show up in the web server logs... I wonder if Modern::Perl is up to something here... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #29 from David Cook <dcook@prosentient.com.au> --- That's not it either as I have a newer CGI version than 4.08... HTTP_USER_AGENT=stuff perl -c C4/Auth.pm [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine psgi_env redefined at C4/Auth.pm line 50. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine safe_exit redefined at C4/Auth.pm line 52. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine get_template_and_user redefined at C4/Auth.pm line 142. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine _version_check redefined at C4/Auth.pm line 715. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine _session_log redefined at C4/Auth.pm line 761. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine _timeout_syspref redefined at C4/Auth.pm line 768. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine checkauth redefined at C4/Auth.pm line 778. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine check_api_auth redefined at C4/Auth.pm line 1387. [Thu Oct 18 11:16:58 2018] Auth.pm: Subroutine check_cookie_auth redefined at C4/Auth.pm line 1629. Status: 500 Content-type: text/html <h1>Software error:</h1> <pre>Bareword "C4::Context::temporary_directory" not allowed while "strict subs" in use at C4/Auth.pm line 1750. BEGIN not safe after errors--compilation aborted at C4/Auth.pm line 1928. </pre> <p> For help, please send mail to this site's webmaster, giving this error message and the time and date of the error. </p> [Thu Oct 18 11:16:58 2018] Auth.pm: Bareword "C4::Context::temporary_directory" not allowed while "strict subs" in use at C4/Auth.pm line 1750. [Thu Oct 18 11:16:58 2018] Auth.pm: BEGIN not safe after errors--compilation aborted at C4/Auth.pm line 1928. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 --- Comment #30 from David Cook <dcook@prosentient.com.au> --- I'm guessing the redefined warnings come from some circular dependency... but that's a red herring since we'd never run C4::Auth on its own anyway. If I create a script and load "C4::Auth", the script compiles fine. In fact, if I remove the semi-colon from the end of "my $dir = C4::Context::temporary_directory", the Koha web app still loads just fine. But I get the following when I try to compile the module: perl -c "C4/Auth.pm" Subroutine psgi_env redefined at C4/Auth.pm line 50. Subroutine safe_exit redefined at C4/Auth.pm line 52. Subroutine get_template_and_user redefined at C4/Auth.pm line 142. Subroutine _version_check redefined at C4/Auth.pm line 715. Subroutine _session_log redefined at C4/Auth.pm line 761. Subroutine _timeout_syspref redefined at C4/Auth.pm line 768. Subroutine checkauth redefined at C4/Auth.pm line 778. Subroutine check_api_auth redefined at C4/Auth.pm line 1387. Subroutine check_cookie_auth redefined at C4/Auth.pm line 1629. syntax error at C4/Auth.pm line 1751, near "C4::Context::temporary_directory my " Global symbol "$instance" requires explicit package name at C4/Auth.pm line 1751. Global symbol "$dir" requires explicit package name at C4/Auth.pm line 1752. Global symbol "$instance" requires explicit package name at C4/Auth.pm line 1752. BEGIN not safe after errors--compilation aborted at C4/Auth.pm line 1928. Interesting... I always thought that modules were compiled when they were loaded using "use". Not even the BEGIN{} block in the module is run when loaded using "use", which seems contradictory to experience... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20727 Bug 20727 depends on bug 20428, which changed state. Bug 20428 Summary: MARC import fails on Debian Stretch https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20428 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Stable |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org