[ 
https://issues.apache.org/jira/browse/SCM-932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17963724#comment-17963724
 ] 

ASF GitHub Bot commented on SCM-932:
------------------------------------

jira-importer commented on issue #1160:
URL: https://github.com/apache/maven-scm/issues/1160#issuecomment-2964649293

   **[Alex 
Harui](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=aharui)** 
commented
   
   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 (Moved to GitHub Issues)
>          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
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> 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.20.10#820010)

Reply via email to