Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 141729 - media-sound/lastfmplayer-1.0.0.1_p2113 fails to build
Summary: media-sound/lastfmplayer-1.0.0.1_p2113 fails to build
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Stefan Schweizer (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-25 11:59 UTC by Mehmet Kemal EROL
Modified: 2006-08-05 00:32 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
portage log file for the latest build (media-sound:lastfmplayer-1.0.0.1_p2113:20060725-151458.log,33.27 KB, text/plain)
2006-07-25 12:01 UTC, Mehmet Kemal EROL
Details
my `emerge --info' output (emerge --info,3.11 KB, text/plain)
2006-07-25 12:02 UTC, Mehmet Kemal EROL
Details
new crash-log for the current version (media-sound:lastfmplayer-1.0.0.1_p2113:20060726-233831.log,86.95 KB, text/plain)
2006-07-26 16:50 UTC, Mehmet Kemal EROL
Details
Patch to ebuild to add -fPIC on amd64 (lastfmplayer-ebuild.patch,763 bytes, patch)
2006-07-26 19:37 UTC, David Li
Details | Diff
modified ebuild with amd64-fpic.patch (lastfmplayer-1.0.0.1_p2113.ebuild,2.80 KB, text/plain)
2006-07-27 18:14 UTC, Mehmet Kemal EROL
Details
log for moded ebuild compile crash (media-sound:lastfmplayer-1.0.0.1_p2113:20060728-004436.log,92.43 KB, text/plain)
2006-07-27 18:16 UTC, Mehmet Kemal EROL
Details
Proposed ebuild (lastfmplayer-1.0.0.1_p2113.ebuild,2.84 KB, text/plain)
2006-07-27 22:14 UTC, David Li
Details
Patch for Makefile (makefile-fpic.patch,432 bytes, patch)
2006-07-27 22:19 UTC, David Li
Details | Diff
`let-it-be' log (media-sound:lastfmplayer-1.0.0.1_p2113:20060728-170301.log,85.83 KB, text/plain)
2006-07-28 10:34 UTC, Mehmet Kemal EROL
Details
Proposed ebuild (lastfmplayer-1.0.0.1_p2113.ebuild,2.75 KB, text/plain)
2006-07-28 12:44 UTC, David Li
Details
Patch for amd64 problems (lastfmplayer-amd64.patch,1.16 KB, patch)
2006-07-28 12:46 UTC, David Li
Details | Diff
`thats-it' log (media-sound:lastfmplayer-1.0.0.1_p2113:20060728-195827.log,193.26 KB, text/plain)
2006-07-28 13:36 UTC, Mehmet Kemal EROL
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mehmet Kemal EROL 2006-07-25 11:59:46 UTC
The latest update fails to build, unfortunately...
I will attach the log file and my env-info, in a moment...
Comment 1 Mehmet Kemal EROL 2006-07-25 12:01:10 UTC
Created attachment 92736 [details]
portage log file for the latest build
Comment 2 Mehmet Kemal EROL 2006-07-25 12:02:09 UTC
Created attachment 92738 [details]
my `emerge --info' output
Comment 3 David Li 2006-07-26 08:40:42 UTC
Looks to me like you have the wrong sources.
player.h and player.cpp don't even exist anymore.

Could you please try deleting /usr/portage/distfiles/svn-src/lastfmplayer
and trying again?
Comment 4 David Li 2006-07-26 08:44:11 UTC
This appears to be the result of upstream switching SVN repositories, which caused SVN to become confused.

It's weird though, because I didn't experiance this problem while writing the ebuild. Unless anyone else has this problem as well, this bug should be marked as invalid.
Comment 5 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-26 09:20:15 UTC
have seen it neither, so INVALID. Please remove your /usr/portage/svn-src/lastfm.. directory
Comment 6 Mehmet Kemal EROL 2006-07-26 16:48:45 UTC
I have no interest in being insistence ... but inspect my next attachment carefully, please!
Comment 7 Mehmet Kemal EROL 2006-07-26 16:50:37 UTC
Created attachment 92816 [details]
new crash-log for the current version
Comment 8 David Li 2006-07-26 19:37:51 UTC
Created attachment 92818 [details, diff]
Patch to ebuild to add -fPIC on amd64

Okay, this is a fPIC error. See http://www.gentoo.org/proj/en/base/amd64/howtos/index.xml?part=1&chap=3

Here's a proposed patch to the ebuild which will add the fPIC to C(XX)FLAGS on AMD64. This will slightly increase the executable size on AMD64 but will hopefully allow you to compile (I don't have a AMD64, that's what arch testing is for). This is to be considered a temporary workaround until the Last.FM guys get around to fixing this.
Comment 9 David Li 2006-07-26 19:43:03 UTC
Actually, I followed your advice and looked at the log closer. It appears you are using a static version of qt.

If that's the case, please ignore the previous attachment as it will not help. Then procced to get a non-static version of qt.

Oh and next time, please comment out MAKEOPTS before making a log because that really helps with reading the log ;D
Comment 10 Mehmet Kemal EROL 2006-07-27 08:34:28 UTC
(In reply to comment #9)
> Actually, I followed your advice and looked at the log closer. It appears you
> are using a static version of qt.

I don't think so...

> If that's the case, please ignore the previous attachment as it will not help.
> Then procced to get a non-static version of qt.

If you know a way of gathering the desired information about qt's health, please instruct me how to ... I am ready to co-operate ;)

> Oh and next time, please comment out MAKEOPTS before making a log because that
> really helps with reading the log ;D

This [!] is a `portage' feature [not a bug] ... please contact the dev's with your concern ;)


Comment 11 David Li 2006-07-27 14:46:56 UTC
>> This [!] is a `portage' feature [not a bug] ... please contact the dev's with
your concern ;)
What I mean was, it's a great feature when the package works correctly. However, it makes logs really difficult to read. So when making a log, it would be kind to temporarily turn off that flag.

Anways, could you please try applying the patch to the ebuild? If you don't know how, adding -fPIC to CFLAGS only for this emerge will effectively do the same thing.
Comment 12 Mehmet Kemal EROL 2006-07-27 18:12:21 UTC
(In reply to comment #11)

Please ... forgive me, but I couldn't resist ... my bug about this lousy logging behaviour is still open:

> What I mean was, it's a great feature when the package works correctly.
> However, it makes logs really difficult to read. So when making a log, it would
> be kind to temporarily turn off that flag.

What `flag', dear David?
 
> Anways, could you please try applying the patch to the ebuild? If you don't
> know how, adding -fPIC to CFLAGS only for this emerge will effectively do the
> same thing.

Done! ... To prove the state of my trial, I will attach the modified, i.e. patched ebuild ... and the latest log file. Please bear in mind that I have had previously cleaned out the local svn repository ... all, with no luck!


Comment 13 Mehmet Kemal EROL 2006-07-27 18:14:36 UTC
Created attachment 92884 [details]
modified ebuild with amd64-fpic.patch
Comment 14 Mehmet Kemal EROL 2006-07-27 18:16:40 UTC
Created attachment 92885 [details]
log for moded ebuild compile crash
Comment 15 David Li 2006-07-27 21:43:45 UTC
Arg... that's what happens when people try to release a package without a configure script. People's CFLAGS get completely ignored. Looks like the Makefile has to be patched manually.

Anyways, thanks for bearing out with me and still being willing to find a solution that works.
Comment 16 David Li 2006-07-27 22:14:35 UTC
Created attachment 92890 [details]
Proposed ebuild

Okay, this is a proposed ebuild which will apply a patch to the Makefile.
Comment 17 David Li 2006-07-27 22:19:12 UTC
Created attachment 92891 [details, diff]
Patch for Makefile

This patch will add -fPIC when compiling Loqqer.c which is where you appear to get stuck at the moment. Note that it's likely that the compile will get stuck at whatever the next file that requires fPIC.

It goes in the files folder.
Comment 18 Mehmet Kemal EROL 2006-07-28 10:33:28 UTC
Here we go, again...

1. Cleaned out svn locals
2. Copied patch for makefile to //files dir
3. Copied proposed ebuild over original
4. Run `emerge' ... NOPE!
5. See below attached log: `let-it-be'

You are a polite young man ... relax, please ... the previous version of our application works smooth, so there is no reason for any anxiety ... talk, whenever you want to `upstream' ... in the meantime I would propose that you might want to listen to some strong sound, like `Little Feat's `Trouble' ;)
Thank YOU, David!
Comment 19 Mehmet Kemal EROL 2006-07-28 10:34:30 UTC
Created attachment 92942 [details]
`let-it-be' log
Comment 20 David Li 2006-07-28 12:44:24 UTC
Created attachment 92947 [details]
Proposed ebuild

Okay, good news is you are clearly past the previous problem.

The bad news is that the problem you have. Apperantly someone at last.fm made a bad design decision to store a pointer address in a int. This works in x86, but it needs to be a long long in amd64.

Anyways, please delete the previous patch and ebuild and try this new one that should fix the latest problem.
Comment 21 David Li 2006-07-28 12:46:22 UTC
Created attachment 92948 [details, diff]
Patch for amd64 problems

lastfmplayer-amd64.patch

This should fix the latest problem.
Goes in the files folder.
Comment 22 Mehmet Kemal EROL 2006-07-28 13:35:39 UTC
Again ...

1. Cleaned out svn locals
2. Copied patch for makefile to //files dir
3. Copied proposed ebuild over original
4. Run `emerge' ... YEESSS!
5. See below attached log: `thats-it'

Hey Stefan, verliere diesen Jungen ja nicht aus den Augen ;)

Thank you, David ... it worked out in the end! I will attach the latest log for your inspection ... and do not forget to merge these changes into the cvs-tree ... the honour of closing this bug belongs to you, my young friend ;)
Comment 23 Mehmet Kemal EROL 2006-07-28 13:36:30 UTC
Created attachment 92953 [details]
`thats-it' log
Comment 24 David Li 2006-07-28 14:01:48 UTC
I'm glad to hear that the player built for you.

I'm not an offical gentoo developer and I thus I don't have cvs access nor can I close the bug. Those actions are up to Stefan Schweizer.

Btw, Stefan, the relevant attachements are 92947 (the ebuild) and 92947 (the patch).
Comment 25 Stefan Schweizer (RETIRED) gentoo-dev 2006-07-28 15:40:51 UTC
thanks I have applied the patch. Can you please take care of sending them upstream to the lastfm devs? It would actually also be nice to have the fPIC patch applied before the Makefile is being generated.
Comment 26 David Li 2006-07-28 17:03:45 UTC
Yeah, that would be a good idea. I'll go a head an inform upstream.
Comment 27 Guido 2006-08-05 00:32:37 UTC
In my experience the lastfmplayer will not build when multiple make jobs are allowed through MAKEOPTS=-jN in /etc/make.conf. This can be fixed by limiting the the jobs to one in the ebuild by adding the line MAKEOPTS="${MAKEOPTS} -j1".