Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 436168 - dev-lang/vala should use virtual/yacc
Summary: dev-lang/vala should use virtual/yacc
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-25 05:24 UTC by Matt Turner
Modified: 2012-10-06 09:24 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 Matt Turner gentoo-dev 2012-09-25 05:24:46 UTC
vala for some reason has || ( sys-devel/bison dev-util/byacc dev-util/yacc ) in DEPEND instead of virtual/yacc.

The virtual doesn't have byacc in it, but I question why the vala ebuild author would explicitly list byacc.
Comment 1 Alexandre Rostovtsev (RETIRED) gentoo-dev 2012-09-25 08:41:25 UTC
> I question why the vala ebuild author would explicitly list byacc.

Instead of questioning the ebuild author, you could unpack the tarball and read vala's configure.ac:

AC_CHECK_PROGS(YACC, 'bison -y' byacc yacc, :)
if test "$YACC" = :; then
        AC_MSG_ERROR([bison not found but required])
fi

I suppose we could switch "|| ( sys-devel/bison dev-util/byacc dev-util/yacc )" to "|| ( virtual/yacc dev-util/byacc )", but I am worried about the possibility of other yacc implementations being added to virtual/yacc, when vala only checks specifically for these three cases.
Comment 2 Gilles Dartiguelongue (RETIRED) gentoo-dev 2012-09-28 07:19:13 UTC
Don't worry, I've been working with yacc (as a user) and I don't see someone suddenly starting a new yacc project :)

On the other hand, bison & all should (and most likely already do, like bison) install a symlink to yacc so the test will succeed here, whatever is installed.
Comment 3 Pacho Ramos gentoo-dev 2012-10-06 09:24:13 UTC
+  06 Oct 2012; Pacho Ramos <pacho@gentoo.org> vala-0.10.4-r2.ebuild,
+  vala-0.12.1-r1.ebuild, vala-0.14.2-r2.ebuild, vala-0.16.1-r2.ebuild,
+  vala-0.18.0.ebuild:
+  Move to virtual/yacc, #436168 by Matt Turner.
+