You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by GitBox <gi...@apache.org> on 2022/05/17 16:47:13 UTC
[GitHub] [solr] madrob opened a new pull request, #867: SOLR-16191: Verify that ps supports -p
madrob opened a new pull request, #867:
URL: https://github.com/apache/solr/pull/867
https://issues.apache.org/jira/browse/SOLR-16191
@colvinco You may be interested in this as well
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org
[GitHub] [solr] magibney commented on a diff in pull request #867: SOLR-16191: Verify that ps supports -p
Posted by GitBox <gi...@apache.org>.
magibney commented on code in PR #867:
URL: https://github.com/apache/solr/pull/867#discussion_r875091635
##########
solr/bin/solr:
##########
@@ -61,6 +61,11 @@ if [ "${THIS_OS:0:6}" == "CYGWIN" ]; then
echo -e "This script does not support cygwin due to severe limitations and lack of adherence\nto BASH standards, such as lack of lsof, curl, and ps options.\n\nPlease use the native solr.cmd script on Windows!"
exit 1
fi
+# Alpine Linux BusyBox comes with a stripped down ps, make sure we have a fully featured one
+if ! ps -p $$ &> /dev/null ; then
Review Comment:
This gets the job done; a slight change here could also check that the output is as expected -- probably unnecessary, but can't hurt; e.g.:
```sh
if [ "$$" != "`ps -o pid='' -p $$`" ]; then
```
Redirecting stdout then becomes unnecessary because it's captured into the equality comparison; redirecting stderr could be done, but allowing any error messages to print through might actually be preferable, to give more system-specific context in addition to the error message we're introducing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org
[GitHub] [solr] madrob merged pull request #867: SOLR-16191: Verify that ps supports -p
Posted by GitBox <gi...@apache.org>.
madrob merged PR #867:
URL: https://github.com/apache/solr/pull/867
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org