You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2020/12/29 01:23:48 UTC
[GitHub] [maven-archetype] dhutchison opened a new pull request #61: [ARCHETYPE-612] Fixes to required property sort approach
dhutchison opened a new pull request #61:
URL: https://github.com/apache/maven-archetype/pull/61
Fix for [ARCHETYPE-612](https://issues.apache.org/jira/browse/ARCHETYPE-612).
This implementation will now respect the required property order defined by the archetype author,
unless they have interdependencies which are not ordered correctly. If properties are detected to be in the wrong order, it will use a (slightly simplistic) graph traversal approach to find a working sort order.
This removes the previous behaviour where it would try to sort based on dependencies, while aiming for alphabetic property order with fixed core property order.
----------------------------------------------------------------
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
[GitHub] [maven-archetype] dhutchison commented on pull request #61: [ARCHETYPE-612] Fixes to required property sort approach
Posted by GitBox <gi...@apache.org>.
dhutchison commented on pull request #61:
URL: https://github.com/apache/maven-archetype/pull/61#issuecomment-765907113
The only way I could find was to get people to specify properties on the command line instead of using the interactive prompts.
----------------------------------------------------------------
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
[GitHub] [maven-archetype] jaredculp commented on pull request #61: [ARCHETYPE-612] Fixes to required property sort approach
Posted by GitBox <gi...@apache.org>.
jaredculp commented on pull request #61:
URL: https://github.com/apache/maven-archetype/pull/61#issuecomment-765685735
Is there a way to work around this issue in the meantime?
----------------------------------------------------------------
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
[GitHub] [maven-archetype] jaredculp commented on pull request #61: [ARCHETYPE-612] Fixes to required property sort approach
Posted by GitBox <gi...@apache.org>.
jaredculp commented on pull request #61:
URL: https://github.com/apache/maven-archetype/pull/61#issuecomment-766919691
Yeah we basically decided to write a shell script as a wrapper which does all the parsing and then passes the values to the archetype in batch mode.
----------------------------------------------------------------
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
[GitHub] [maven-archetype] jaredculp commented on pull request #61: [ARCHETYPE-612] Fixes to required property sort approach
Posted by GitBox <gi...@apache.org>.
jaredculp commented on pull request #61:
URL: https://github.com/apache/maven-archetype/pull/61#issuecomment-766919691
Yeah we basically decided to write a shell script as a wrapper which does all the parsing and then passes the values to the archetype in batch mode.
----------------------------------------------------------------
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