Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/git-plugin
  Commit: a113d097349e7fa1a805c4ebec09c1e5d5e1dd60
      
https://github.com/jenkinsci/git-plugin/commit/a113d097349e7fa1a805c4ebec09c1e5d5e1dd60
  Author: Mark Waite <[email protected]>
  Date:   2018-01-02 (Tue, 02 Jan 2018)

  Changed paths:
    M src/test/java/hudson/plugins/git/GitPublisherTest.java

  Log Message:
  -----------
  Skip LOGNAME dependent test if LOGNAME not defined

Most non-Windows execution environments define the LOGNAME environment
variable.  When running the Jenkins Dockerfile-slim, it does not appear
to define that environment variable on the master.  Rather than create
complicated searching for another environment variable whose value is
valid as a branch name, we will skip this test if LOGNAME is not set.

LOGNAME is set in enough contexts (like ci.jenkins.io and developer
desktop machines and other test environments) that it is not a good use of
time to embed additional logic into the test for a relatively rare case.
Skip the test if the environment won't support the test.


  Commit: 4ed4871423c190c7db631b272429ed7cf7a47d59
      
https://github.com/jenkinsci/git-plugin/commit/4ed4871423c190c7db631b272429ed7cf7a47d59
  Author: Mark Waite <[email protected]>
  Date:   2018-01-02 (Tue, 02 Jan 2018)

  Changed paths:
    M src/test/java/hudson/plugins/git/GitPublisherTest.java

  Log Message:
  -----------
  Merge pull request #566 from MarkEWaite/do-not-require-LOGNAME-in-test

Skip LOGNAME dependent test if LOGNAME not defined


Compare: 
https://github.com/jenkinsci/git-plugin/compare/649a68a5f4b9...4ed4871423c1

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to