class Snownews < Formula
  desc "Text mode RSS newsreader"
  homepage "https://kiza.eu/software/snownews"
  url "https://kiza.eu/media/software/snownews/snownews-1.5.12.tar.gz"
  sha256 "26dd96e9345d9cbc1c0c9470417080dd0c3eb31e7ea944f78f3302d7060ecb90"
  revision 1

  bottle do
    sha256 "111d01a8162376cf510ca58d4db0dc87c3edd171f782738d36ce8326b25741f6" => :sierra
    sha256 "03ac9fea075ea76b934c2ff5365b5e48295c6fbcb03f9c402332bbdf5f84690b" => :el_capitan
    sha256 "9d950bf2641410e4ddc6646eeaead2e49a5925b186a7d72fb207f11ceaaa0572" => :yosemite
    sha256 "50505095e31d0c0a0960cae1abd00e8900c64967c5ad81068de161c510e59afe" => :mavericks
  end

  option "without-nls", "Build without translations"

  depends_on "gettext" if build.with? "nls"
  depends_on "openssl"

  # Fix system openssl linking error on macOS.
  # Allow EXTRA_LDFLAGS to take precedence so we can link brewed openssl
  # instead of deprecated system openssl.
  # Upstream has been notified but has no public bug tracker
  patch :DATA

  def install
    args = ["--prefix=#{prefix}"]
    args << "--disable-nls" if build.without? "nls"

    system "./configure", *args
    # Must supply -lz because snownews configure relies on "xml2-config --libs" for
    # it, which doesn't work on OS X prior to 10.11
    system "make", "install", "EXTRA_LDFLAGS=#{ENV.ldflags} -L#{Formula["openssl"].opt_lib} -lz", "CC=#{ENV.cc}"
  end
end

__END__
diff --git a/configure b/configure
index a752cd6..74e61d7 100755
--- a/configure
+++ b/configure
@@ -13,7 +13,7 @@ chomp($xmlldflags);

 my $prefix = "/usr/local";
 my $cflags = "-Wall -Wno-format-y2k -O2 -DLOCALEPATH=\"\\\"\$(LOCALEPATH)\\\"\" -DOS=\\\"$os\\\" $xmlcflags \$(EXTRA_CFLAGS) ";
-my $ldflags = "-lncurses -lcrypto $xmlldflags \$(EXTRA_LDFLAGS) ";
+my $ldflags = "\$(EXTRA_LDFLAGS) -lncurses -lcrypto $xmlldflags ";

 my $use_nls = 1;
 parse_cmdl_line();