[GitHub] [solr] janhoy commented on pull request #1346: SOLR-16457 Add symbolic link version of solr.install.dir to security.policy

["janhoy (via GitHub)" <gi...@apache.org>]

janhoy commented on PR #1346:
URL: https://github.com/apache/solr/pull/1346#issuecomment-1424675324

   Found a bug while testing. Need to resolve `pwd -L` on the original script path, not on the already normalized physical `SOLR_TIP`. After fixing that it woks. I tested with:
   
   ```bash
   cd ~/git/solr/
   gw dev
   cd /tmp/
   # Create a symlink to solr to simulate how the installer does it, and the docker image
   ln -s ~/git/solr/solr/packaging/build/dev symdev
   cd symdev
   # add some lib folder to shared lib, using symlink path
   bin/solr -c -Dsolr.sharedLib=/tmp/symdev/modules/extraction/lib
   # Create collection with a config that contains /update/extract but no <lib> directives (hacked techproducts sample)
   bin/solr create -c tech -d /path/to/techproducts
   # Post a PDF, it will fail with classNotFound
   bin/post -c tech example/exampledocs/solr-word.pdf
   ```


-- 
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