[ https://issues.apache.org/jira/browse/SOLR-9760?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15664798#comment-15664798 ]
ASF GitHub Bot commented on SOLR-9760: -------------------------------------- GitHub user afscrome opened a pull request: https://github.com/apache/lucene-solr/pull/113 SOLR-9760 Avoid temporary files to determine java version Avoid creating a temporary file so that solr does not require permissions in the current working directory. You can merge this pull request into a Git repository by running: $ git pull https://github.com/afscrome/lucene-solr patch-1 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/lucene-solr/pull/113.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #113 ---- commit 5330ed23cfbb78650537a23d455ae92648b203dd Author: Alex Crome <afscr...@users.noreply.github.com> Date: 2016-11-14T19:37:26Z SOLR-9760 Avoid temporary files to determine java version Avoid creating a temporary file so that solr does not require permissions in the current working directory. ---- > solr.cmd on Windows requires modify permissions in the current directory > ------------------------------------------------------------------------ > > Key: SOLR-9760 > URL: https://issues.apache.org/jira/browse/SOLR-9760 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: Server > Affects Versions: 6.3 > Environment: Windows > Reporter: Alex Crome > > Currently starting solr fails if the user does not have permission to write > to the current directory. This is caused by the resolve_java_vendor function > writing a temporary file to the current directory (javares). > {code} > :resolve_java_vendor > set "JAVA_VENDOR=Oracle" > "%JAVA%" -version 2>&1 | findstr /i "IBM J9" > javares > set /p JAVA_VENDOR_OUT=<javares > del javares > if NOT "%JAVA_VENDOR_OUT%" == "" ( > set "JAVA_VENDOR=IBM J9" > ) > {code} > Rather than writing this temporary file to disk, The exit code of findstr can > be used to determine if there is a match. (0 == match, 1 == no match, 2 == > syntax error) > {code} > :resolve_java_vendor > "%JAVA%" -version 2>&1 | findstr /i "IBM J9" > nul > if %ERRORLEVEL% == 1 set "JAVA_VENDOR=Oracle" else set "JAVA_VENDOR=IBM J9" > {code} > By not writing this temp file, you can reduce the permissions solr needs. As > a work around until this is fixed, you can start solr in a directory that has > the required permissions, -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org