Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 178176 - wxwidgets.eclass use the old fashion config tool
Summary: wxwidgets.eclass use the old fashion config tool
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo wxWidgets Herd
Depends on:
Reported: 2007-05-12 11:18 UTC by Cyrius
Modified: 2007-05-12 23:12 UTC (History)
0 users

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

Set-wxconfig replaced by wx-config (wxwidgets.eclass,3.84 KB, text/plain)
2007-05-12 11:20 UTC, Cyrius
Diff file to patch the current wxwidgets.class (wxwidgets.diff,1.36 KB, patch)
2007-05-12 12:07 UTC, Cyrius
Details | Diff
Patch fo supported wxGTK 2.8 (wxwidgets.eclass.patch,2.96 KB, patch)
2007-05-12 13:43 UTC, Cyrius
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Cyrius 2007-05-12 11:18:55 UTC
wxwidgets.eclass use the old fashion "set-wxconfig" tool and report the error :
Compile wxGTK with unicode USE flag.
But This error doesn't correspond to the real problem.

Reproducible: Always

Actual Results:  
When emerging pgadmin3, the ebuild call "need-wxwidgets unicode".
This fail in error with a bad error message.

Expected Results:  
Compile and run
Comment 1 Cyrius 2007-05-12 11:20:31 UTC
Created attachment 118990 [details]
Set-wxconfig replaced by wx-config

New proposal for wxwidgets.eclass
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-05-12 11:34:18 UTC
Not portage.
Comment 3 Jakub Moc (RETIRED) gentoo-dev 2007-05-12 11:36:09 UTC
Also, please attach a unified diff, instead of the whole eclass.
Comment 4 Cyrius 2007-05-12 12:07:53 UTC
Created attachment 118991 [details, diff]
Diff file to patch the current wxwidgets.class

Diff file to patch the current wxwidgets.class
Comment 5 Cyrius 2007-05-12 13:38:52 UTC
Comment on attachment 118991 [details, diff]
Diff file to patch the current wxwidgets.class

In fact, the problem come from wxGTK which is the 2.8 version
Comment 6 Cyrius 2007-05-12 13:43:05 UTC
This is needed to be able to use wxGTK 2.8
Comment 7 Cyrius 2007-05-12 13:43:56 UTC
Created attachment 119000 [details, diff]
Patch fo supported wxGTK 2.8

A more serious one
Comment 8 Ryan Hill (RETIRED) gentoo-dev 2007-05-12 20:27:56 UTC
thanks, but we have a completely rewritten eclass we're in the process of testing right now.  it'll be ready for the public pretty soon, so stay tuned.
Comment 9 Cyrius 2007-05-12 21:57:02 UTC
Comment 10 Mart Raudsepp gentoo-dev 2007-05-12 23:12:48 UTC
The premise of this patch is unfortunately completely wrong anyway.
There is nothing old in set-wxconfig - it is a bash function inside the very same eclass. wx-config is a configuration utility which we can NOT use inside portage installations as it is heavily dependent on which SLOT was last installed and so on. It also does not accept things like gtk-ansi as an argument (well it does, but spits out nonsense for the lib names then).
Thanks for the concern though and I hope this information helps for future patches from you in this area after the rework is in :)
Marking bug as closed.