[GitHub] [solr] HoustonPutman edited a comment on pull request #491: SOLR-15867 Bring in build stuff from lucene

[GitBox <gi...@apache.org>]

HoustonPutman edited a comment on pull request #491:
URL: https://github.com/apache/solr/pull/491#issuecomment-1006051349


   @janhoy, there is an issue with the new distribution. I believe this "create a directory, then make a tarball" approach was used initially, however it caused the files to lose permissions (i.e. no executable flags that had been set). This was fixed when we went straight into a tarball from sources, setting the necessary permissions.
   
   I have checked with your new build and the tarball does not contain the executable bit for solr/bin, solr/docker/scripts, etc.
   
   Would it be possible to move that stuff back into `solr/packaging` as it was originally done? You can keep the signing in the new `distributions` sub-project (though calling it `release` or `artifacts` might be a bit more clear at that point...)


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