Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 181789 - dev-db/qt-unixODBC-3.3.8: Ignored CFLAGS and LDFLAGS: /usr/qt/3/plugins/sqldrivers/libqsqlodbc.so
Summary: dev-db/qt-unixODBC-3.3.8: Ignored CFLAGS and LDFLAGS: /usr/qt/3/plugins/sqldr...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords: QAcanfix
Depends on:
Blocks:
 
Reported: 2007-06-12 16:44 UTC by Arfrever Frehtes Taifersar Arahesis (RETIRED)
Modified: 2008-03-27 08:00 UTC (History)
1 user (show)

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


Attachments
Patch (qt-unixODBC-FLAGS.patch,815 bytes, patch)
2007-06-12 18:31 UTC, Arfrever Frehtes Taifersar Arahesis (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 16:44:47 UTC
CFLAGS are ignored.

### List of files built with LDFLAGS ignored ###
/usr/qt/3/plugins/sqldrivers/libqsqlodbc.so
### End of list                              ###

Probably similar sedding like this in x11-libs/qt is needed.
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-06-12 17:48:21 UTC
OK, that was about enough.

1/ Use common sense when filing bugs (like, what about mailing gentoo-dev mailing list first and asking about the best course of action here?
2/ If you have a patch, submit it *upstream* unless this is a *Gentoo-specific* issue.
Comment 2 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 18:30:11 UTC
This bug report also concerns CFLAGS. Unfortunately you didn't see it.
http://devmanual.gentoo.org/ebuild-writing/functions/src_compile/building/index.html
and http://devmanual.gentoo.org/general-concepts/user-environment/index.html
say that CFLAGS should be respected.

I have patch so I'm reopening it.
Comment 3 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-06-12 18:31:33 UTC
Created attachment 121838 [details, diff]
Patch

Solution is copied from x11-libs/qt.
Comment 4 Piotr Jaroszyński (RETIRED) gentoo-dev 2007-06-28 14:11:41 UTC
Qt: I would love to see a comment from you.
Comment 5 Mark Loeser (RETIRED) gentoo-dev 2008-03-27 02:26:31 UTC
I'm not comfortable with applying this unless someone from the qt team comments.  Has anyone tried this out and made sure it works fine, and can get upstream to have a nicer solution for us? :)
Comment 6 Bo Ørsted Andresen (RETIRED) gentoo-dev 2008-03-27 08:00:53 UTC
It's obviously the right solution, given how it's the same tarball as qt itself from which the suggested solution was grabbed. Fixed in 3.3.8.