`brew untap homebrew/foo` chokes w/ error message #17

Closed
chdiza opened this Issue Apr 2, 2013 · 4 comments

Comments

Projects
None yet
2 participants
@chdiza
Contributor

chdiza commented Apr 2, 2013

Previously I had succesfully done brew tap homebrew/dupes.

Now doing brew untap homebrew/dupes gives the following:

Error: undefined method `count' for #<Array:0x38c5e8>
Please report this bug:
    https://github.com/mistydemeo/tigerbrew/wiki/troubleshooting
/usr/local/Library/Homebrew/cmd/untap.rb:24:in `untap'
/usr/local/Library/brew.rb:81:in `send'
/usr/local/Library/brew.rb:81

This is on 10.5.8PPC. Other taps fail to untap too, with the same error message.

@mistydemeo

This comment has been minimized.

Show comment
Hide comment
@mistydemeo

mistydemeo Apr 2, 2013

Owner

Looks like I may have to backport Array#count for Leopard too.

What's your system ruby version? /usr/bin/ruby --version

Owner

mistydemeo commented Apr 2, 2013

Looks like I may have to backport Array#count for Leopard too.

What's your system ruby version? /usr/bin/ruby --version

@chdiza

This comment has been minimized.

Show comment
Hide comment
@chdiza

chdiza Apr 2, 2013

Contributor

/usr/bin/ruby --version yields:

ruby 1.8.6 (2009-06-08 patchlevel 369) [universal-darwin9.0]

Contributor

chdiza commented Apr 2, 2013

/usr/bin/ruby --version yields:

ruby 1.8.6 (2009-06-08 patchlevel 369) [universal-darwin9.0]

@mistydemeo

This comment has been minimized.

Show comment
Hide comment
@mistydemeo

mistydemeo Apr 2, 2013

Owner

Thanks, looks like Array#count was introduced in 1.8.7.

The method is only used in two places in Homebrew so I just changed it to #length. This will be fixed when I merge in Homebrew updates this evening.

Owner

mistydemeo commented Apr 2, 2013

Thanks, looks like Array#count was introduced in 1.8.7.

The method is only used in two places in Homebrew so I just changed it to #length. This will be fixed when I merge in Homebrew updates this evening.

@mistydemeo

This comment has been minimized.

Show comment
Hide comment
@mistydemeo

mistydemeo Apr 3, 2013

Owner

If you brew update, this should be fixed now! Let me know if you have any more trouble.

Owner

mistydemeo commented Apr 3, 2013

If you brew update, this should be fixed now! Let me know if you have any more trouble.

@mistydemeo mistydemeo closed this Apr 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment