Skip to content

Commit

Permalink
Fix link to DefaultRepositorySystemSessionFactory
Browse files Browse the repository at this point in the history
  • Loading branch information
kwin authored Nov 4, 2024
1 parent 811240c commit 48f0e9c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/site/markdown/configuration.md
Original file line number Diff line number Diff line change
Expand Up @@ -167,4 +167,4 @@ From | To | With
## Set Configuration from Apache Maven

To set one of the configuration options from above just use system variables. As system variables only support String values the type conversion mentioned above needs to be leveraged.
Sometimes Maven uses different default values than the Maven Resolver itself or tries to extract certain values from the `settings.xml`. For details refer to <https://github.com/apache/maven/blob/master/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java>.
Sometimes Maven uses different default values than the Maven Resolver itself or tries to extract certain values from the `settings.xml`. For details refer to [`DefaultRepositorySystemSessionFactory`](https://github.com/apache/maven/blob/8005826df75c09ed530c31d636206309f5fab13b/impl/maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java).

2 comments on commit 48f0e9c

@cstamas
Copy link
Member

@cstamas cstamas commented on 48f0e9c Nov 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kwin This file is generated. If you have clean checkout, it will be overwritten.
You want to modify here instead:
https://github.com/apache/maven-resolver/blob/master/maven-resolver-tools/src/main/resources/configuration.md.vm#L68

@cstamas
Copy link
Member

@cstamas cstamas commented on 48f0e9c Nov 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kwin do you plan anything with this change? As this is a generated content, and will be (and is) overwritten on each build.

Please sign in to comment.