Bug 351034

Summary:	sys-apps/pcfclock does not respect LDFLAGS
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	RESOLVED WONTFIX
Severity:	QA	CC:	c1pher, esigra
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	331933
Attachments:	Build log Diff to pcfclock-0.44-r6.ebuild Correct diff file.

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2011-01-07 18:41:52 UTC

See attached build log.

(Please do not complain about the need for looking at the build log until you can provide an easy way to open bugs with the correct data picked out of a tinderbox log. Thanks.)

Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev

2011-01-07 18:42:07 UTC

Created attachment 259218 [details]
Build log

Comment 2 Stefan Briesenick (RETIRED) gentoo-dev

2011-01-26 21:47:40 UTC

I know that, but it's a little bit tricky to fix, because the bug is within linux-mod eclass which strips out LDFLAGS for compiling the kernel module, but forget to reactivate them for compiling other stuff.

perhaps you have a hint for me, how to fix it properly...

Comment 3 Dane Smith (RETIRED) gentoo-dev

2011-05-19 18:09:34 UTC

Created attachment 274001 [details]
Diff to pcfclock-0.44-r6.ebuild

This should take care of it. 

linux-mod will actually cd into the modules directory and run linux-mod_src_compile there with the stripped LDFLAGS. Then it will cd into the pcfdate directory and run emake normally there.

I hope this helps.

Comment 4 Dane Smith (RETIRED) gentoo-dev

2011-06-07 23:01:47 UTC

Created attachment 276211 [details]
Correct diff file.

Sorry. I just looked at this and saw I had 2 different diffs in the same file. Only one was right.

This is the one to apply.

Having said that, if I don't hear something soonish, I'm going to go ahead and revbump and commit the changes.

Comment 5 Sergey Popov gentoo-dev

2013-12-09 09:35:03 UTC

Package was removed from tree(see bug #489510)