@Xeno https://github.com/XenoAmess/gop-platform-maven-structure-demo/issues/1
T On Thu, Oct 16, 2025 at 3:40 PM Xeno Amess <[email protected]> wrote: > > @Guillaume Nodet > https://github.com/XenoAmess/gop-platform-maven-structure-demo.git > > Guillaume Nodet <[email protected]> 于2025年10月16日周四 13:45写道： > > > That's the case anymore with Maven 4. > > The consumer POM is installed to your local repository and does not contain > > CI-friendly versions anymore. > > > > Can you set up a small project to reproduce the problem ? I don't really > > understand... > > > > Le jeu. 16 oct. 2025 à 05:05, Xeno Amess <[email protected]> a écrit : > > > > > seems nisse can also not handle this situation > > > > > > ---------------- > > > > > > I know if I add this to maven 4 it can work > > > > > > Index: > > > > > > > > impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java > > > IDEA additional info: > > > Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP > > > <+>UTF-8 > > > =================================================================== > > > diff --git > > > > > > > > a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java > > > > > > > > b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java > > > --- > > > > > > > > a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java > > > (revision 5a33f4b24cf97bc7df5ac467b41d90c150571889) > > > +++ > > > > > > > > b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultModelBuilder.java > > > (date 1760427656771) > > > @@ -1219,6 +1219,11 @@ > > > String groupId = parent.getGroupId(); > > > String artifactId = parent.getArtifactId(); > > > String version = parent.getVersion(); > > > + > > > + for (Map.Entry<String, String> entry : > > > childModel.getProperties().entrySet()) { > > > + version = version.replace("${" + entry.getKey() + "}", > > > entry.getValue()); > > > + } > > > + > > > String classifier = parent instanceof Mixin ? ((Mixin) > > > parent).getClassifier() : null; > > > String extension = parent instanceof Mixin ? ((Mixin) > > > parent).getExtension() : null; > > > > > > > > > but I shouldn't, because I read > > > > > > > > https://cwiki.apache.org/confluence/display/MAVENOLD/Automatic+Parent+Versioning > > > , > > > according to it: > > > > > > When maven installs or deploys an artifact the pom is copied from the > > > source directory to the local or remote repository without modification. > > So > > > a pom that looks like > > > <project> > > > ... > > > <groupId>org.apache.maven</groupId> > > > <artifactId>demo</groupId> > > > <version>${myVersion}</version> > > > </project> > > > and where myVersion is set to 1.0.0 will be deployed to > > > /org/apache/maven/demo/1.0.0/demo-1.0.0.pom and will still contain > > > ${myVersion} as the "value" of the version element. This could lead to > > the > > > odd case of a dependency being specified with 1.0.0 as the version and > > > myVersion never being specified, or worse being set to a value other than > > > 1.0.0. If one were to assume that item 1 above was the be fixed then > > this > > > issue must also be addressed. If a project is deployed with its parent > > > version (or any of the other parent elements) set to variables then it is > > > impossible to guarantee that the build will always be reproducable since > > > the parent version variable would have to be set either in a settings > > file > > > or from the command line, neither of which can be guaranteed from one > > build > > > to the next. Worse, any project that has a dependency on such a project > > > would also have to define the variable in a settings file or on the > > command > > > line. > > > > > > I think it be correct for parent version as well, so I have to modify the > > > flatten plugin as well, thus the deployed pom be flattened. > > > Though I have to find a way to avoid abusion of this, means deploy a > > > missing-parent-version pom to central and cause unreproducable.... > > > So maybe the best choice for me is enjoy the modified maven version > > > locally by myself...for this repo at least needn't be deployed; it just > > > need to build a docker image and we use that image in our production... > > > > > > Tamás Cservenák <[email protected]> 于2025年10月16日周四 01:57写道： > > > > > > > Howdy, > > > > > > > > And not to mention, how Nisse works with Maven 3 as well, without the > > > > need for the flatten plugin. > > > > When extension is present, version can contain ANY expression, the > > > > only requirement is to HAVE VALUE. > > > > Also, Nisse (automatically) collects properties that need inlining > > > > (user can extend configure more) and inlines them. > > > > > > > > https://github.com/maveniverse/nisse > > > > > > > > Thanks > > > > T > > > > > > > > On Wed, Oct 15, 2025 at 6:45 AM Guillaume Nodet <[email protected]> > > > wrote: > > > > > > > > > > Can't you use Maven 4, where CI-friendly properties aren't restricted > > > > > anymore and where the version for the whole reactor can be set by a > > > > > property, or even using an extension ? > > > > > JLine does that now : the property is defined by the nisse extension, > > > > > automatically computed from the git history, and defined in a single > > > > place, > > > > > in the root pom. > > > > > The children just inherit this version. > > > > > https://github.com/jline/jline3/blob/master/pom.xml#L16 > > > > > > > > > > Le mar. 14 oct. 2025 à 11:24, Xeno Amess <[email protected]> a > > > écrit : > > > > > > > > > > > I found out even if I extend the CI-Friendly versions can't I > > find a > > > > good > > > > > > way to implement what I want, means declaring parent version from > > > > > > properties. > > > > > > If I really want it I have to modify maven to allow the pom be > > read, > > > > and > > > > > > modify flatten to flatten the version in parent, then add a > > validator > > > > for > > > > > > deploying to avoid some people pass a dynamic parent version to > > > > remote... > > > > > > > > > > > > Christoph Läubrich <[email protected]> 于2025年10月14日周二 17:06写道： > > > > > > > > > > > > > Mavens CI-Friendly versions are extensible, see for example: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/eclipse-tycho/tycho/blob/main/tycho-build/src/main/java/org/eclipse/tycho/build/TychoCiFriendlyVersions.java > > > > > > > > > > > > > > but you need to know them by name due to how they work, so there > > > > can't > > > > > > > be a simple prefix. > > > > > > > > > > > > > > > > > > > > > Am 14.10.25 um 11:02 schrieb Xeno Amess: > > > > > > > > findout I have to modify both maven and flatten-plugin to allow > > > > modify > > > > > > > the > > > > > > > > parent, and it be hard to forbid misuse to let dynamic parent > > > > version > > > > > > by > > > > > > > > argline > > > > > > > > possibly I should find another idea to achieve this... > > > > > > > > > > > > > > > > Xeno Amess <[email protected]> 于2025年10月11日周六 17:01写道： > > > > > > > > > > > > > > > >> according to the codes we only have 3 of these properties. > > > > > > > >> > > > > > > > >> private static final String SHA1_PROPERTY = "sha1"; > > > > > > > >> > > > > > > > >> private static final String CHANGELIST_PROPERTY = > > "changelist"; > > > > > > > >> > > > > > > > >> private static final String REVISION_PROPERTY = "revision"; > > > > > > > >> > > > > > > > >> > > > > > > > >> should we add a new set of them (likely a prefix like > > > > ci_friendly_* ) > > > > > > > >> > > > > > > > >> ---------- > > > > > > > >> > > > > > > > >> the reason I want it is I have several repos who both > > inherited > > > > from a > > > > > > > pom > > > > > > > >> of a version, and use a same version number elsewhere. > > > > > > > >> > > > > > > > >> for example > > > > > > > >> > > > > > > > >> we have > > > > > > > >> > > > > > > > >> <parent> > > > > > > > >> <groupId>org.springframework.boot</groupId> > > > > > > > >> <artifactId>spring-boot-starter-parent</artifactId> > > > > > > > >> <version>3.5.6</version> > > > > > > > >> <relativePath></relativePath> > > > > > > > >> </parent> > > > > > > > >> > > > > > > > >> and same time > > > > > > > >> > > > > > > > >> <properties> > > > > > > > >> <java.version>21</java.version> > > > > > > > >> <springboot.version>3.5.6</springboot.version> > > > > > > > >> <revision>0.0.1</revision> > > > > > > > >> > > > > <native.maven.plugin.version>0.11.0</native.maven.plugin.version> > > > > > > > >> <graalpy.version>25.0.0</graalpy.version> > > > > > > > >> <antlr.version>3.5.3</antlr.version> > > > > > > > >> <antlr4.version>4.13.2</antlr4.version> > > > > > > > >> <antlr4-c3.version>1.2.0x</antlr4-c3.version> > > > > > > > >> <grpc.version>1.76.0</grpc.version> > > > > > > > >> <protobuf.version>4.32.1</protobuf.version> > > > > > > > >> > > > > > > <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> > > > > > > > >> <gop-root>${maven.multiModuleProjectDirectory}</gop-root> > > > > > > > >> <maven.compiler.proc>full</maven.compiler.proc> > > > > > > > >> > > > > <surefire-report-phase>prepare-package</surefire-report-phase> > > > > > > > >> <os-maven-plugin.version>1.7.1</os-maven-plugin.version> > > > > > > > >> > > > > > > > > > > <protobuf-maven-plugin.version>0.6.1</protobuf-maven-plugin.version> > > > > > > > >> <replacer.version>1.5.3</replacer.version> > > > > > > > >> </properties> > > > > > > > >> > > > > > > > >> and > > > > > > > >> > > > > > > > >> <dependency> > > > > > > > >> <groupId>org.springframework.boot</groupId> > > > > > > > >> > > > > > > > > > > > > > <artifactId>spring-boot-starter-oauth2-authorization-server</artifactId> > > > > > > > >> <version>${springboot.version}</version> > > > > > > > >> </dependency> > > > > > > > >> > > > > > > > >> so seems it be better to let the version in parent use a same > > > > > > propertie > > > > > > > >> with the dependency. > > > > > > > >> so we have to use a ci friendly property > > > > > > > >> but we already use revision for the project version, and > > > > > > sha/changelist > > > > > > > >> seems not quite suitable > > > > > > > >> so maybe we shall add rule to make every of ci_friendly_* > > > > property be > > > > > > > >> ci_friendly property? > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > > To unsubscribe, e-mail: [email protected] > > > > > > > For additional commands, e-mail: [email protected] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > ------------------------ > > > > > Guillaume Nodet > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [email protected] > > > > For additional commands, e-mail: [email protected] > > > > > > > > > > > > > > > > > -- > > ------------------------ > > Guillaume Nodet > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
