Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 677886 - net-libs/libproxy - add USE=vala to control vapi files
Summary: net-libs/libproxy - add USE=vala to control vapi files
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Freedesktop bugs
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2019-02-13 17:02 UTC by Daniel Campello
Modified: 2019-02-13 18:42 UTC (History)
1 user (show)

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 Daniel Campello 2019-02-13 17:02:44 UTC
We should have the following USE flag to control vapi files as it is done with app-arch/gcab here https://bugs.gentoo.org/677454

18c18
< IUSE="gnome kde mono networkmanager perl python spidermonkey test webkit"
---
> IUSE="gnome kde mono networkmanager perl python spidermonkey test vala webkit"
62,64c62
< 		# WITH_VALA just copies the .vapi file over and needs no deps,
< 		# hence always enable it unconditionally
< 		'-DWITH_VALA=ON'
---
> 		"-DWITH_VALA=$(usex vala)"
Comment 1 Mart Raudsepp gentoo-dev 2019-02-13 17:57:21 UTC
Nothing has changed regarding the comment you are removing. Vala is not required for it, so we just let it copy it. USE=vala is not for controlling the copying of an already available *.vapi file, but about building the VAPI file from scratch and thus needing dev-lang/vala at runtime, or for other reasons (actually linking against vala, needing the compiler when the sources aren't pregenerated in tarball for an optional piece of code, etc)
Comment 2 Daniel Campello 2019-02-13 18:42:08 UTC
According to https://packages.gentoo.org/useflags/vala the use flag controls the bindings which iiuc are the .vapi files. if USE=-vala is for the whole system why should we be copying this files along. Also upstream defaults to off: https://github.com/libproxy/libproxy/blob/master/INSTALL#L117