[
https://issues.apache.org/jira/browse/SCM-932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17078955#comment-17078955
]
Alex Harui commented on SCM-932:
--------------------------------
I am not a Maven expert, but I agree it isn't compliant with the standard
conventions as I understand them. However, it looks like Maven does allow some
variations: you can change the output and source directory names, for example.
In this case, I think the patch allows for a similar variation: instead of
requiring one released project per repo, it allows for some small projects to
be collected in a repo. One of the projects in our code base contains only two
.java files (the other has 8 .java files) so having each project in its own
repo seems like too much overhead, and complicates the non-Maven legacy build
process we want to continue to support.
Also, I thought of this as a mere bug fix in JGit. On my local machine I added
a similar test to gitexe's GitCheckInCommandTest and it worked there. It seems
to only be a problem in JGit.
I will try to get the patches into a PR by the weekend.
> JGit does not commit files in fileset based in subfolder
> --------------------------------------------------------
>
> Key: SCM-932
> URL: https://issues.apache.org/jira/browse/SCM-932
> Project: Maven SCM
> Issue Type: Bug
> Components: maven-scm-provider-jgit
> Affects Versions: 1.11.2
> Reporter: Alex Harui
> Priority: Major
> Attachments: SCM-932-Test.patch, SCM-932.patch
>
>
> If `mvn release` is run in a subfolder, the changed files are relative to the
> subfolder and not the repo root and `maven-scm-provider-jgit` does not
> compute the correct path and thus the files are not added and committed.
> For example, a git repo is created at `repo-root`
> {code:java}
> repo-root/pom.xml
> repo-root/unparented-submodule/pom.xml {code}
> The user does:
> {code:java}
> cd repo-root/unparented-submodule
> mvn release:prepare {code}
> A `git status` will show that `unparented-submodule/pom.xml` is changed and
> not staged for commit. That's because the fileset's basedir is
> `unpainted-submodule` and the relative path is just `pom.xml`.
> `maven-scm-provider-jgit` 1.12.1 tries to stage 'pom.xml' not
> `unparented-submodule/pom.xml` and since repo-root/pom.xml has not changed,
> nothing is in the commit and `unparented-submodule/pom.xml` remains un-staged.
>
>
>
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
