Every time you run translate script it makes temporary files on /tmp This files are not deleted, and over time it takes a lot of space.
Created attachment 27110 [details] [review] Bug 12081 - tmpl_process3.pl - delete temp files This patch enable deletion of temp files used by tmpl_process3.pl. Just removed coments on existing code To test: 1. Do a count of files on /tmp ( ls /tmp | wc -l ) 2. Update preferred language 3. Count again, new files on /tmp 4. Apply the patch 5. Update again, check, no new files
Created attachment 27137 [details] [review] [SIGNED OFF] Bug 12081 - tmpl_process3.pl - delete temp files This patch enable deletion of temp files used by tmpl_process3.pl. Just removed coments on existing code To test: 1. Do a count of files on /tmp ( ls /tmp | wc -l ) 2. Update preferred language 3. Count again, new files on /tmp 4. Apply the patch 5. Update again, check, no new files Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: I watched what temp files were actually in /tmp to make sure other processes didn't magically increase/decrease the number. $ perl translate update {lang code} generated 10 temporary files for me (2x5 po files). After removing those ten files, and applying the patch, no other files were generated.
Created attachment 27166 [details] [review] Bug 12081 - tmpl_process3.pl - delete temp files This patch enable deletion of temp files used by tmpl_process3.pl. Just removed coments on existing code To test: 1. Do a count of files on /tmp ( ls /tmp | wc -l ) 2. Update preferred language 3. Count again, new files on /tmp 4. Apply the patch 5. Update again, check, no new files Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: I watched what temp files were actually in /tmp to make sure other processes didn't magically increase/decrease the number. $ perl translate update {lang code} generated 10 temporary files for me (2x5 po files). After removing those ten files, and applying the patch, no other files were generated. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> These lines has been commented by commit a399dcefad193fc21ef2dc1fe31d07686ab2da46 without any apparent good reason.
In my opinion, this is a bug, not an enhancement.
Comment on attachment 27137 [details] [review] [SIGNED OFF] Bug 12081 - tmpl_process3.pl - delete temp files Obsolted old patch :)
Pushed to master. Thanks, Bernardo!
Pushed to 3.14.x, will be in 3.14.10
Picked for 3.12.15!