]> source.dussan.org Git - jgit.git/commit
Add TypedConfigGetter.getPath() 56/169856/2
authorThomas Wolf <thomas.wolf@paranor.ch>
Thu, 24 Sep 2020 19:34:29 +0000 (21:34 +0200)
committerMatthias Sohn <matthias.sohn@sap.com>
Fri, 25 Sep 2020 22:40:10 +0000 (18:40 -0400)
commit6f41458d8a3fc436569520974b1072dfbcf7347b
tree8041eb296577c52722b8dca4fe1d16c55bcacbd7
parent597a6bc7ecc98d78be44c605bb9ff5baf95e1ead
Add TypedConfigGetter.getPath()

This enables EGit to override with a lenient variant that logs the
problem and continues with the default value. EGit needs this because
otherwise a corrupt core.excludesFile entry can render the whole UI
unusable (until the git config is fixed) because any use of the
WorkingTreeIterator will throw an InvalidPathException.

This is not a problem on OS X, where all characters are allowed in
file names. But on Windows some characters are forbidden... see bug
567296. The message of the InvalidPathException is not helpful since
it doesn't point to the origin of the problem. EGit can log a much
better message indicating the offending config file and entry in the
Eclipse error log, where the user can see it.

Bug: 567309
Change-Id: I4e57afa715ff3aaa52cd04b5733f69e53af5b1e0
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
org.eclipse.jgit/.settings/.api_filters [new file with mode: 0644]
org.eclipse.jgit/src/org/eclipse/jgit/lib/Config.java
org.eclipse.jgit/src/org/eclipse/jgit/lib/TypedConfigGetter.java
org.eclipse.jgit/src/org/eclipse/jgit/treewalk/WorkingTreeIterator.java