Well I have already tried to push it upstream, but received no response. Also, as I already said, it's specific to non-GNU builds, and OS X is pretty unique in using Clang by default. This looks like the best solution for now. I've made the recommended stylistic changes as you've suggested, however.

On 16 Feb 2014, at 18:30, Mike McQuaid <notifications@github.com> wrote:

> In Library/Formula/tpl.rb:
> 
> > +
> > +class Tpl < Formula
> > +  homepage "http://troydhanson.github.io/tpl/"
> > +  url "https://github.com/troydhanson/tpl/archive/v1.6.tar.gz"
> > +  sha1 "b7d16e9bcda16d86a5dd2d0af0ab90f7e85aa050"
> > +  head "https://github.com/troydhanson/tpl.git"
> > +
> > +  option 'with-tests', 'Verify the build using the test suite.'
> > +
> > +  depends_on :autoconf
> > +  depends_on :automake
> > +  depends_on :libtool
> > +
> > +  def install
> > +    inreplace 'configure.ac' do |s|
> > +      s.gsub! "AM_INIT_AUTOMAKE", "AM_INIT_AUTOMAKE([foreign])"
> What's this doing? Would still be good if you can make an issue upstream about this and link it in the commit (along with an explanation of why it's needed). Also: inreplace "configure.ac", "AM_INIT_AUTOMAKE", "AM_INIT_AUTOMAKE([foreign])".
> 
> —
> Reply to this email directly or view it on GitHub.
>