[Bug 14705] New: Allow .pl too in atomicupdate
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 Bug ID: 14705 Summary: Allow .pl too in atomicupdate Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt@gmail.com Reporter: m.de.rooy@rijksmuseum.nl QA Contact: testopia@bugs.koha-community.org Bug 13893 added perl files with the extension .perl. In practice you always tend to add a .pl file. I suggest to add it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 --- Comment #1 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 41777 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41777&action=edit Bug 14705: Allow .pl too in atomicupdate Bug 13893 added perl files with the extension .perl. In practice you always tend to add a .pl file. I suggest to add it. Test plan: [1] Add a .pl file in atomicupdate. [2] Try the update step in webinstaller. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|gmcharlt@gmail.com |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #41777|0 |1 is obsolete| | --- Comment #2 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 41778 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41778&action=edit Bug 14705: Allow .pl too in atomicupdate Bug 13893 added perl files with the extension .perl. In practice you always tend to add a .pl file. I suggest to add it. Test plan: [1] Add a .pl file in atomicupdate. [2] Try the update step in webinstaller. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- The point of using .perl instead of .pl was to avoid executing the .pl files existing in the atomicupdate directory before 13068 removed them (it has been useful recently, see bug 14820 comment 8). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14705 --- Comment #4 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- (In reply to Jonathan Druart from comment #3)
The point of using .perl instead of .pl was to avoid executing the .pl files existing in the atomicupdate directory before 13068 removed them (it has been useful recently, see bug 14820 comment 8).
So if we do not backport this, there is no problem. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org