adoroszlai commented on PR #8037:
URL: https://github.com/apache/ozone/pull/8037#issuecomment-2742481381

   > It looks to me `ozone-dist` is misbehaving during `mvn clean install 
-Pdist -DskipTests -e -Dmaven.javadoc.skip=true`.
   > 
   > It looks to be interpreting some variables in `enable-ozone-plugin.sh` to 
an unwanted hard-coded path. The following is found in the script under 
`target/`
   
   Nice catch @smengcl.
   
   This is intentional (`filtering=true`), but sometimes surprising.  We use 
`maven-resources-plugin` to replace properties while copying these files.  
Unfortunately `file` is also a built-in property, so `${file}` needs to be used 
as `$file`, or renamed to something else, like `${f}` or `${arg}`.
   
   
https://github.com/apache/ozone/blob/e805c15c744471506a21656792b43c3bf70369ce/hadoop-ozone/dist/pom.xml#L154-L172
   
   BTW, some of these files look like are copied from Ranger as is.  In the 
long run we should get them from Ranger, instead of duplicating them in Ozone.  
That would also avoid this problem.


-- 
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-unsubscr...@ozone.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org

Reply via email to