Skip to content

Commit

Permalink
moreutils: remove options
Browse files Browse the repository at this point in the history
  • Loading branch information
fxcoudert committed Jan 5, 2019
1 parent a6f64c9 commit 60c72ba
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions Formula/moreutils.rb
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,11 @@ class Moreutils < Formula
sha256 "9f2e15ade4e996988704728fa9d3b9263a1337b33847ba68afb8863a3293162d" => :el_capitan
end

option "without-parallel", "Build without the 'parallel' tool."
option "without-errno", "Build without the 'errno' tool, for compatibility with 'pwntools'."
option "without-ts", "Build without the 'ts' tool, for compatibility with 'task-spooler'."

depends_on "docbook-xsl" => :build

conflicts_with "parallel", :because => "Both install a `parallel` executable." if build.with? "parallel"
conflicts_with "pwntools", :because => "Both install an `errno` executable." if build.with? "errno"
conflicts_with "task-spooler", :because => "Both install a `ts` executable." if build.with? "ts"
conflicts_with "parallel", :because => "Both install a `parallel` executable."
conflicts_with "pwntools", :because => "Both install an `errno` executable."
conflicts_with "task-spooler", :because => "Both install a `ts` executable."

resource "Time::Duration" do
url "https://cpan.metacpan.org/authors/id/N/NE/NEILB/Time-Duration-1.20.tar.gz"
Expand Down Expand Up @@ -50,12 +46,6 @@ def install
inreplace "Makefile" do |s|
s.gsub! "/usr/share/xml/docbook/stylesheet/docbook-xsl",
"#{Formula["docbook-xsl"].opt_prefix}/docbook-xsl"
%w[parallel errno ts].each do |util|
next if build.with? util
s.gsub! /^BINS=.*\K#{util}/, "", false
s.gsub! /^MANS=.*\K#{util}\.1/, ""
s.gsub! /^PERLSCRIPTS=.*\K#{util}/, "", false
end
end
system "make", "all"
system "make", "check"
Expand Down

2 comments on commit 60c72ba

@Julian
Copy link
Contributor

@Julian Julian commented on 60c72ba Jan 24, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this done?

@SMillerDev
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.