Summary: | 22600075.pl is not idempotent | ||
---|---|---|---|
Product: | Koha | Reporter: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Component: | Installation and upgrade (command-line installer) | Assignee: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | dcook, tomascohen |
Version: | Main | Keywords: | rel_22_11_candidate |
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
23.05.00,22.11.04
|
Circulation function: | |
Bug Depends on: | 31948 | ||
Bug Blocks: | |||
Attachments: |
Bug 33051: Make 22600075.pl idempotent
Bug 33051: Make 22600075.pl idempotent |
Description
Tomás Cohen Arazi (tcohen)
2023-02-23 10:49:49 UTC
What's the best way to fix this? Alter the atomicupdate file and backport? Created attachment 147225 [details] [review] Bug 33051: Make 22600075.pl idempotent As there is no check, this dbrev could cause trouble if run twice. It doesn't look like it is going to get backported to 22.05, but just in case... Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 147229 [details] [review] Bug 33051: Make 22600075.pl idempotent As there is no check, this dbrev could cause trouble if run twice. It doesn't look like it is going to get backported to 22.05, but just in case... Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Thx for the patch and quick QA everyone! Pushed to master for 23.05. Nice work everyone, thanks! Many hands makes light work, thankyou everyone! Pushed to 22.11.x for the next release |