[GitHub] [flink] rmetzger commented on a change in pull request #14805: [FLINK-13996][docs] Update instructions on building with Maven 3.3.x

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

rmetzger commented on a change in pull request #14805:
URL: https://github.com/apache/flink/pull/14805#discussion_r566852095



##########
File path: docs/flinkDev/building.md
##########
@@ -105,20 +105,25 @@ Flink [shades away](https://maven.apache.org/plugins/maven-shade-plugin/) some o
 
 The dependency shading mechanism was recently changed in Maven and requires users to build Flink slightly differently, depending on their Maven version:
 
-**Maven 3.1.x and 3.2.x**
+**Maven and 3.2.x**
 It is sufficient to call `mvn clean install -DskipTests` in the root directory of Flink code base.
 
 **Maven 3.3.x**
-The build has to be done in two steps: First in the base directory, then in the distribution project:
+The build has to be done in two steps: First in the base directory, then in shaded modules, such as the distribution and the filesystems:
 
 {% highlight bash %}
 mvn clean install -DskipTests
 cd flink-dist
 mvn clean install
+cd ../flink-filesystems/flink-s3-fs-presto/

Review comment:
       ah, damn, you are right.




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

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