Created attachment 37239 [details] patch for catalina.sh AFAICS the code in catalina.sh to rotate catalina.out has been withdrawn with 5.8.39. #53930 Changing the system's logrotate.conf to rotate catalina.out requires root permission. This might be not an option in some cases. I would like to suggest a new approach to rotate catalina.out. See attached patch. The idea is to define CATALINA_OUT_CMD (as for 8.5.38), but this time catalina.sh creates a named pipe on the fly, to be used instead of a plain file "catalina.sh". The code about catalina's PID is not changed, i.e. the systemd users should be happy with the new version. The default behavior of catalina.sh hasn't been changed (hopefully). Just a suggestion, of course. Keep on your good work
Created attachment 37240 [details] better patch
Please ignore the first patch. I picked the right file from the wrong directory.
The shell script has to be portable, "mknod -p" is not portable. This will require cases for various Unix systems. What about fifo with mkfifo?
Created attachment 37242 [details] even better patch Good suggestion. I highly appreciate your effort to keep portability. Attached is a new patch, using mkfifo. The comment about CATALINA_OUT_CMD and the indentation style have been adjusted, too.
I like the modified approach. I'll run some tests locally and if all is well, I'll apply this for the next release round.
Thanks for the patch. Fixed in: - master for 10.0.0-M6 onwards - 9.0.x for 9.0.36 onwards - 8.5.x for 8.5.56 onwards - 7.0.x for 7.0.105 onwards