-
-
Notifications
You must be signed in to change notification settings - Fork 11.4k
tidy-html5 4.9.25 #39010
tidy-html5 4.9.25 #39010
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
class TidyHtml5 < Formula | ||
homepage "http://www.html-tidy.org/" | ||
url "https://github.com/htacg/tidy-html5/archive/4.9.26.tar.gz" | ||
sha256 "28674745db53b6ef1aa4b8466e6e231915dcd596672ec40515d0ab53ee0c33f6" | ||
|
||
depends_on "cmake" => :build | ||
|
||
def install | ||
cd "build/cmake" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it necessary to change directory here? What happens if just run in the root of the repo? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. actually it is, according to tidy's docs. Can confirm: removing that line will cause a build failure. def install
- cd "build/cmake"
- system "cmake", "../..", *std_cmake_args
+ system "cmake", *std_cmake_args
system "make"
system "make", "install"
end
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Weird, thanks. |
||
system "cmake", "../..", *std_cmake_args | ||
system "make" | ||
system "make", "install" | ||
end | ||
|
||
test do | ||
output = pipe_output(bin/"tidy5 -q", "<!doctype html><title></title>") | ||
assert_match /^<!DOCTYPE html>/, output | ||
assert_match /HTML Tidy for HTML5/, output | ||
end | ||
end | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can a test be added to do something more substantial than e.g. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a
sha256
declared.