[ https://issues.jenkins-ci.org/browse/JENKINS-13599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=162121#comment-162121 ]
SCM/JIRA link daemon commented on JENKINS-13599: ------------------------------------------------ Code changed in jenkins User: Gregory Boissinot Path: src/main/java/hudson/plugins/doxygen/DoxygenDirectoryParser.java src/test/java/hudson/plugins/doxygen/DoxygenDirectoryParserTest.java http://jenkins-ci.org/commit/doxygen-plugin/82a31e40096a95b3aed8351e4902aab9c7377fa8 Log: Merge pull request #4 from albert-so/master [JENKINS-13599] Provided better implementation of DoxygenDirectoryParser#isDirectoryAbsolute(). Compare: https://github.com/jenkinsci/doxygen-plugin/compare/f14d86f...82a31e4 > DoxygenDirectoryParser#isDirectoryAbsolute() can get confused and cause the > doxygen OUTPUT_DIR not to be found > -------------------------------------------------------------------------------------------------------------- > > Key: JENKINS-13599 > URL: https://issues.jenkins-ci.org/browse/JENKINS-13599 > Project: Jenkins > Issue Type: Bug > Components: doxygen > Affects Versions: current > Reporter: Albert So > Assignee: Albert So > > We have a doxygen.conf file where OUTPUT_DIRECTORY = doc. The doxygen html > files were generated properly under ${workspace}/doc/html however the plugin > kept on saying it couldn't find the directory. > After tracing in to the code, I think I see the problem. > DoxygenDirectoryParser#isDirectoryAbsolute() checks to see if the passed in > path is absolute by seeing if the path's parent exists. > If we pass in "doc" you would expext isDirectoryAbsolute() to return false. > However, in our environment, ${user.dir}/doc actually does exist. > From http://docs.oracle.com/javase/6/docs/api/java/io/File.html, "By default > the classes in the java.io package always resolve relative pathnames against > the current user directory. This directory is named by the system property > user.dir, and is typically the directory in which the Java virtual machine > was invoked." > For Unix environments, I'm thinking we can just check to see if the path > starts with a "/" to see if a directory is absolute. However, for Windows > environments, the check will have to include checking for drive letter names > and UNC paths. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jenkins-ci.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira