582644 – =x11-misc/gpaste-3.18.3[vala] should DEPEND on >=dev-lang/vala:0.30[vapigen]

Bug 582644 - =x11-misc/gpaste-3.18.3[vala] should DEPEND on >=dev-lang/vala:0.30[vapigen]

Summary: =x11-misc/gpaste-3.18.3[vala] should DEPEND on >=dev-lang/vala:0.30[vapigen]

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Leho Kraav (:macmaN @lkraav)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2016-05-10 12:57 UTC by Martin Sekera
Modified:	2016-12-18 16:38 UTC (History)
CC List:	2 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Martin Sekera 2016-05-10 12:57:59 UTC

GPaste 3.18.3 raised the required vala version from 0.14 to 0.30, so the current DEPEND no longer applies:

vala? (
	|| (
		dev-lang/vala:0.30[vapigen(+)]
		dev-lang/vala:0.28[vapigen(+)]
		dev-lang/vala:0.26[vapigen(+)]
		dev-lang/vala:0.24[vapigen(+)]
		dev-lang/vala:0.22[vapigen(+)]
	)
)

It fails to configure with anything lower than 0.30:

(...)
configure: error: vapigen >= 0.30 not found

Comment 1 Coacher 2016-05-11 02:40:52 UTC

Please attach the complete build.log and the output of `emerge --info =x11-misc/gpaste-3.18.3'.

Comment 2 Leho Kraav (:macmaN @lkraav) 2016-05-11 06:57:12 UTC

This is legit. I will file a PR.

Comment 3 Leho Kraav (:macmaN @lkraav) 2016-05-11 07:13:36 UTC

Actually, not sure how to proceed. This is what I see in my ebuild copy

DEPEND="${CDEPEND}
    dev-util/intltool
    virtual/pkgconfig
    vala? ( $(vala_depend) )
"

There's no specific version list. What now?

Comment 4 Martin Sekera 2016-05-11 11:37:15 UTC

According to [1] the ebuild should set:

VALA_MIN_API_VERSION="0.30"

before inheriting vala.eclass (and the _MAX_ as well if there's a maximum version required upstream).

[1] https://devmanual.gentoo.org/eclass-reference/vala.eclass/index.html

Comment 5 Leho Kraav (:macmaN @lkraav) 2016-06-29 18:48:17 UTC

https://github.com/gentoo/gentoo/pull/1794

Comment 6 Leho Kraav (:macmaN @lkraav) 2016-12-18 16:38:29 UTC

Fixed back in July.