diff options
| author | Larry Rosenman <ler@FreeBSD.org> | 2017-04-18 01:37:21 +0000 |
|---|---|---|
| committer | Larry Rosenman <ler@FreeBSD.org> | 2017-04-18 01:37:21 +0000 |
| commit | 54c9fec60a835b776febc73f3723df1e988879ba (patch) | |
| tree | 5dd074c0e39babc0d18dc6297355ee5eda753166 /textproc/apache-solr/files/patch-bin_solr | |
| parent | Fix py-cec build after update of libcec. (diff) | |
Stock apache-solr scripts use lsof during startup to check if Solr started properly.
When deploying Apache Solr in Jail lsof needs access to /dev/mem which is not allowed by default. Lack of the access makes the startup script run in the loop for 3 minutes and then it assumes that it might have succeed.
Allowing access to /dev/mem makes running Solr in Jail a bit pointless.
Following patch changes the behaviour of Solr startup script to use sockstat instead of lsof to check if Solr has already started. This also makes the dependency on lsof obsolete.
While here, pet portlint.
PR: 218654
Submitted by: Wiktor Niesiobedzki <bsd@vink.pl>
Approved by: idefix@fechner.net (maintainer), adamw (mentor)
Differential Revision: https://reviews.freebsd.org/D10416
Notes
Notes:
svn path=/head/; revision=438757
Diffstat (limited to 'textproc/apache-solr/files/patch-bin_solr')
| -rw-r--r-- | textproc/apache-solr/files/patch-bin_solr | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/textproc/apache-solr/files/patch-bin_solr b/textproc/apache-solr/files/patch-bin_solr new file mode 100644 index 000000000000..30d7c090f210 --- /dev/null +++ b/textproc/apache-solr/files/patch-bin_solr @@ -0,0 +1,31 @@ +--- bin/solr.orig 2017-04-15 09:06:51 UTC ++++ bin/solr +@@ -1757,13 +1757,12 @@ function launch_solr() { + 1>"$SOLR_LOGS_DIR/solr-$SOLR_PORT-console.log" 2>&1 & echo $! > "$SOLR_PID_DIR/solr-$SOLR_PORT.pid" + + # no lsof on cygwin though +- if hash lsof 2>/dev/null ; then # hash returns true if lsof is on the path + echo -n "Waiting up to $SOLR_STOP_WAIT seconds to see Solr running on port $SOLR_PORT" + # Launch in a subshell to show the spinner + (loops=0 + while true + do +- running=`lsof -PniTCP:$SOLR_PORT -sTCP:LISTEN` ++ running=`sockstat -ls -P tcp -p $SOLR_PORT | grep LISTEN` + if [ -z "$running" ]; then + slept=$((loops * 2)) + if [ $slept -lt $SOLR_STOP_WAIT ]; then +@@ -1781,13 +1780,6 @@ function launch_solr() { + fi + done) & + spinner $! +- else +- echo -e "NOTE: Please install lsof as this script needs it to determine if Solr is listening on port $SOLR_PORT." +- sleep 10 +- SOLR_PID=`ps auxww | grep start\.jar | grep -w "\-Djetty\.port=$SOLR_PORT" | grep -v grep | awk '{print $2}' | sort -r` +- echo -e "\nStarted Solr server on port $SOLR_PORT (pid=$SOLR_PID). Happy searching!\n" +- return; +- fi + fi + } + |