Summary: | populate_db.pl is out of sync and must be removed | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Command-line Utilities | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | minor | ||
Priority: | P5 - low | CC: | 1joynelson, katrin.fischer, m.de.rooy, martin.renvoize, robin |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.05.00, 19.11.03
|
|
Bug Depends on: | 10337 | ||
Bug Blocks: | |||
Attachments: |
Bug 24397: Remove out of sync script populate_db.pl
Bug 24397: Remove out of sync script populate_db.pl Bug 24397: Remove out of sync script populate_db.pl |
Description
Jonathan Druart
2020-01-10 13:16:50 UTC
Created attachment 97199 [details] [review] Bug 24397: Remove out of sync script populate_db.pl populate_db.pl from koha-misc4dev must be used instead. This script existed before the creation of koha-misc4dev, it does no longer make sense to have it in Koha. Created attachment 97238 [details] [review] Bug 24397: Remove out of sync script populate_db.pl populate_db.pl from koha-misc4dev must be used instead. This script existed before the creation of koha-misc4dev, it does no longer make sense to have it in Koha. Signed-off-by: David Nind <david@davidnind.com> Removing it is easy and I understand the argument. But we could discuss if the statement "koha-misc4dev must be used instead" has full consensus. How do we make sure that it is in sync and that it meets our coding guidelines etc? It is easier to grep one git repo than two? The reason for putting it outside Koha probably relates to the tedious process of getting things signed off and pushed in Koha mainstream. Somehow we do not manage as community to get that more smoothly. But I still think we should address that and not just move code somewhere else. As an alternative approach, should we copy this script (or even more) at release time into the codebase? Formulate some procedure to sync them? Any opinions? @Katrin: could you include this in the weekly QA mail? > Any opinions?
> @Katrin: could you include this in the weekly QA mail?
Will do.
(In reply to Marcel de Rooy from comment #3) > Removing it is easy and I understand the argument. But we could discuss if > the statement "koha-misc4dev must be used instead" has full consensus. Maybe a bit of history was needed. I submitted koha:populate_db.pl to have an handy script, but then realized that our process (SO+QA+push) was too slow to get things pushed quickly. So I created misc4dev and copied the script there, along with a lot of other stuffs for Koha devs. The things is that koha-misc4dev is now used by the sandbox and jenkins (koha-testing-docker), and so we need a super fast reactivity when something is broken there, and we now it's broken as the CI is failing in that case. I am the one who usually fix it https://gitlab.com/koha-community/koha-misc4dev/commits/master Most of the time the change is minor and trivial. > How do we make sure that it is in sync and that it meets our coding > guidelines etc? It is easier to grep one git repo than two? Those scripts cannot be part of Koha, as I said previously we need a fast push and cannot wait for review. I ask for review when I am adding new enhancements (and in that cases it usually takes several weeks). > The reason for putting it outside Koha probably relates to the tedious > process of getting things signed off and pushed in Koha mainstream. Somehow > we do not manage as community to get that more smoothly. But I still think > we should address that and not just move code somewhere else. How? :) > As an alternative approach, should we copy this script (or even more) at > release time into the codebase? Formulate some procedure to sync them? Nope, the script will be broken during the release cycle, and so the script will become obsolete. As this is intended for developers and dev scripts tend to be able to do 'dangeorus things' I am fully ok with keeping them separate from the scripts in the release. It really is a Dev only script so I feel it indeed belongs in its own place. Happy for it to be removed from Koha's core repo. I think the Dev tools do get enough exposure living in misc4dev Created attachment 97705 [details] [review] Bug 24397: Remove out of sync script populate_db.pl populate_db.pl from koha-misc4dev must be used instead. This script existed before the creation of koha-misc4dev, it does no longer make sense to have it in Koha. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Nice work everyone! Pushed to master for 20.05 Pushed to 19.11.x for 19.11.03 |