Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 456148 - app-editors/mp does not respect CXX
Summary: app-editors/mp does not respect CXX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: tc-directly
  Show dependency tree
 
Reported: 2013-02-08 11:04 UTC by Agostino Sarubbo
Modified: 2013-03-15 12:43 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2013-02-08 11:04:27 UTC
You receive this bug because this package does not respect my system's CC ( x86_64-pc-linux-gnu-gcc - /usr/bin/x86_64-pc-linux-gnu-gcc ) and calls directly gcc -/usr/bin/gcc
The possible solutions to fix this issue are:
1)Fix the buildsystem, if you can
2)inherit toolchain-funcs and use tc-export CC
3)inherit toolchain-funcs and use emake CC="$(tc-getCC)"


This is a strange buildsystem, we have only C files but it links with g++

g++ -march=native -O2 -g0 -pipe mp_core.o mpv_qt4.o mpv_curses.o  `cat config.ldflags` -o mp-5

we need to replace g++ with $(CC)
Comment 1 Agostino Sarubbo gentoo-dev 2013-02-08 11:07:00 UTC
(In reply to comment #0)
> This is a strange buildsystem, we have only C files but it links with g++
> 
> g++ -march=native -O2 -g0 -pipe mp_core.o mpv_qt4.o mpv_curses.o  `cat
> config.ldflags` -o mp-5
> 
> we need to replace g++ with $(CC)

My bad...we have one cpp file, so it should respect CXX too:

g++ -march=native -O2 -g0 -pipe -I. `cat config.cflags` -c mpv_qt4.cpp
Comment 2 Sergey Popov gentoo-dev 2013-03-15 12:43:39 UTC
+  15 Mar 2013; Sergey Popov <pinkbyte@gentoo.org> mp-5.2.2.ebuild:
+  Respect CXX, wrt bug #456148. Thanks to Agostino Sarubbo for discovering this
+  issue

It did not respect only CXX. I have dropped old version, so it's fixed only in 5.2.2.