martin-g commented on code in PR #3042:
URL: https://github.com/apache/avro/pull/3042#discussion_r1691212368
##########
lang/c++/build.sh:
##########
@@ -58,7 +58,7 @@ function do_doc() {
function do_dist() {
rm -rf $BUILD_CPP/
mkdir -p $BUILD_CPP
- cp -r api AUTHORS build.sh CMakeLists.txt ChangeLog \
+ cp -r include/avro AUTHORS build.sh CMakeLists.txt ChangeLog \
Review Comment:
```suggestion
cp -r include AUTHORS build.sh CMakeLists.txt ChangeLog \
```
I hope this is what is expected!
Now it will copy `include/` with all its subfolders.
With `cp -r include/avro ...` it will copy just `avro/` and put it in the
root of $BUILD_CPP. And I guess the missing `include/` will cause problems
--
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]
