You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/12/01 07:21:00 UTC
[jira] [Commented] (MNGSITE-501) Clarify JDK activation element
[ https://issues.apache.org/jira/browse/MNGSITE-501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17641714#comment-17641714 ]
ASF GitHub Bot commented on MNGSITE-501:
----------------------------------------
kwin commented on code in PR #346:
URL: https://github.com/apache/maven-site/pull/346#discussion_r1036763893
##########
content/apt/guides/introduction/introduction-to-profiles.apt:
##########
@@ -127,11 +129,12 @@ mvn groupId:artifactId:goal -P profile-1,profile-2,?profile-3
Profiles can be automatically triggered based on the detected state of
the build environment. These triggers are specified via an <<<\<activation\>>>>
section in the profile itself. Currently, this detection is limited to
- prefix-matching of the JDK version, the presence of a system property or
- the value of a system property. Here are some examples.
+ matching of the JDK version, matching the operating system,
Review Comment:
Fixed in https://github.com/apache/maven-site/pull/346/commits/d93b461cf74b40dc8fb8ca96d8c6f6538be810be.
##########
content/apt/guides/introduction/introduction-to-profiles.apt:
##########
@@ -127,11 +129,12 @@ mvn groupId:artifactId:goal -P profile-1,profile-2,?profile-3
Profiles can be automatically triggered based on the detected state of
the build environment. These triggers are specified via an <<<\<activation\>>>>
section in the profile itself. Currently, this detection is limited to
- prefix-matching of the JDK version, the presence of a system property or
- the value of a system property. Here are some examples.
+ matching of the JDK version, matching the operating system,
+ the presence of a system property or the value of a system property. Here are some examples.
Review Comment:
Fixed in https://github.com/apache/maven-site/pull/346/commits/d93b461cf74b40dc8fb8ca96d8c6f6538be810be.
> Clarify JDK activation element
> ------------------------------
>
> Key: MNGSITE-501
> URL: https://issues.apache.org/jira/browse/MNGSITE-501
> Project: Maven Project Web Site
> Issue Type: Improvement
> Reporter: Konrad Windszus
> Assignee: Konrad Windszus
> Priority: Major
>
> Both in [https://maven.apache.org/pom.html#Activation] and [https://maven.apache.org/guides/introduction/introduction-to-profiles.html#details-on-profile-activation] the {{jdk}} element needs clarification that not all values are interpreted as ranges (in contrast to what the referenced version range specification from enforcer maven plugin in [https://maven.apache.org/enforcer/enforcer-rules/versionRanges.html] says).
> Actually there are three different kind of values supported:
> # Version ranges (if the value starts with either "[" or "(")
> # A negated prefix (if the value starts with "!")
> # A (non-negated) prefix for all other cases
> This can be seen in [https://github.com/apache/maven/blob/001eef8452b38110a2557446aa81b8fb1ff35f92/maven-model-builder/src/main/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivator.java#L69]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)