RFR: 868: Skara should ignore .gitconfig and /etc/gitconfig [v3]
Erik Helin
ehelin at openjdk.java.net
Thu Feb 4 14:41:56 UTC 2021
On Wed, 3 Feb 2021 11:18:06 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> Skara is already ignoring .hgrc for mercurial, but the corresponding solution for git was never put in place.
>>
>> I also fixed a problem where an error condition would cause an infinite loop, which I ran into during testing.
>
> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>
> Add unit test to verify that we block user/system configs.
Looks good, just a minor comment inline.
process/src/main/java/org/openjdk/skara/process/Process.java line 70:
> 68: }
> 69:
> 70: public Description environMap(Map<String, String> keyValueMap) {
I would have just overloaded `environ`, as in `public Description environ(Map<String, String> env)`
-------------
Marked as reviewed by ehelin (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/997
More information about the skara-dev
mailing list