This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 243707 - Error executing command tidy
Summary: Error executing command tidy
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EPF (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Onno van der Straaten CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-11 01:07 EDT by Onno van der Straaten CLA
Modified: 2023-02-16 16:33 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Onno van der Straaten CLA 2008-08-11 01:07:17 EDT
Some users have reported that on edit of a page, there is an error message in the browser:

Error executing command tidy -m -config "D:/My Documents/My EclipseSDK Workspaces/epfproject2/EPFWiki/config/tidy.cfg" "D:/My Documents/My EclipseSDK Workspaces/epfproject2/EPFWiki/public/development_wikis/openup/openup_basic/guidances/concepts/core_principle_balance,_ssG6MMvpEdqukPpotm3DYg.html_EPFWIKI_co.html":
Comment 1 Onno van der Straaten CLA 2008-08-11 01:55:09 EDT
I improved part documentation on Tidy install because a similar problem can occur if Tidy is not installled. In the Development Guide it was part of the step http://wiki.eclipse.org/index.php/EPF_Wiki_Development_Guide#Install_Ruby_and_Ruby_Gems. This is confusing so I have made this a separate step http://wiki.eclipse.org/EPF_Wiki_Development_Guide#Install_HTML_Tidy.

In this case it is not the problem. The problem is that the Tidy command reports back that the command failed when there are only warnings? See in method tidy_file in generic.rb. raise "Error executing command #{cmdline}: #{$?}" if !tidy_success. I can run the command from the command line without problems, there are only warnings. 

A workaround is to remove the raise statement and ignore the Tidy exit status. So in the method tidy_file the last statement, the raise statement can be removed.


Comment 2 Onno van der Straaten CLA 2008-08-11 14:06:39 EDT
Tidy exit status is 0 for success, 1 for warnings and 2 for errors. Changed tidy_file so that it tests that the exit status is not 2.
Comment 3 Onno van der Straaten CLA 2010-12-02 09:46:32 EST
OK