Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Liblastfm build fails #12886

Closed
t-unit opened this issue Jun 16, 2012 · 6 comments
Closed

Liblastfm build fails #12886

t-unit opened this issue Jun 16, 2012 · 6 comments

Comments

@t-unit
Copy link

t-unit commented Jun 16, 2012

https://gist.github.com/2941714
https://gist.github.com/2941718
@2bits
Copy link
Contributor

2bits commented Jun 17, 2012

This was fixed in a recent commit. Do you have HOMEBREW_USE_CLANG set?

@t-unit
Copy link
Author

t-unit commented Jun 17, 2012

I do not have HOMEBREW_USE_CLANG set.

But using brew install --use-llvm liblastfm fixed the problem!

@t-unit t-unit closed this as completed Jun 17, 2012
@2bits
Copy link
Contributor

2bits commented Jun 17, 2012

I asked because your gist showed that you had run brew update and had a more recent commit, dc7b620, than the one that fixed this formula, 35d1b8f. So it should have just worked. I can't figure out why it didn't at this point, but I'm glad it's working.

@jacknagel
Copy link
Contributor

It's because "liblastfm" and "lastfmlib" are two different formulae; we need a patch or fails_with :clang on this one too unless @mxcl feels like fixing it.

@jacknagel jacknagel reopened this Jun 17, 2012
@2bits
Copy link
Contributor

2bits commented Jun 17, 2012

Heh, thanks for spotting that. Dyslexia on a grand scale there. This formula also needs some work because it uses a hard-coded gcc to link but was using clang to compile. I'll take a look at it later.

2bits pushed a commit to 2bits/homebrew that referenced this issue Jun 20, 2012
liblastfm: add fails_with :clang
5d0aadf 
Liblastfm-0.3.3 fails to compile with clang.
Add a `fails_with :clang` block.
Add the upstream bug report in the comments.

Fixes Homebrew#12886
@2bits 2bits mentioned this issue Jun 20, 2012
Closed
@adamv adamv closed this as completed in 432d67a Jun 23, 2012
@adamv
Copy link
Contributor

adamv commented Jun 23, 2012

Pulled; though if this software isn't being maintained then...

eproxus pushed a commit to eproxus/homebrew that referenced this issue Jul 18, 2012
@eproxus
liblastfm: add fails_with :clang
d50b28b 
Liblastfm-0.3.3 fails to compile with clang.
Add a `fails_with :clang` block.
Add the upstream bug report in the comments.

Fixes Homebrew#12886

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
Sharpie pushed a commit to Sharpie/homebrew that referenced this issue Sep 12, 2012
@adamv
liblastfm: add fails_with :clang
8410d0a 
Liblastfm-0.3.3 fails to compile with clang.
Add a `fails_with :clang` block.
Add the upstream bug report in the comments.

Fixes Homebrew#12886

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this issue Dec 17, 2012
@adamv
liblastfm: add fails_with :clang
b51fc55 
Liblastfm-0.3.3 fails to compile with clang.
Add a `fails_with :clang` block.
Add the upstream bug report in the comments.

Fixes Homebrew#12886

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

liblastfm: add fails_with :clang 2bits/homebrew
4 participants
@adamv @t-unit @jacknagel @2bits