Copilot commented on code in PR #3826:
URL: https://github.com/apache/avro/pull/3826#discussion_r3448131056
##########
share/docker/Dockerfile:
##########
@@ -138,12 +146,15 @@ RUN mkdir tmp && cd tmp \
&& echo "extension=snappy.so" >
"/etc/php/${PHP8_VERSION}/cli/conf.d/10-snappy.ini" \
&& cd .. && rm -rf php-ext-snappy \
&& php -m \
- && apt-get -qqy clean
+ && rm -rf /tmp/lang/php
Review Comment:
`rm -rf /tmp/lang/php` runs while the working directory is still inside
`/tmp/lang/php` (you end up back in `/tmp/lang/php` after the `cd .. && rm -rf
php-ext-snappy`). Removing the current working directory typically fails with
`Device or resource busy` and will break the build. `cd` out of the directory
before deleting it.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
