Skip to content

Commit

Permalink
media-sound/forked-daapd: Use antlr 3.5 instead of earlier version
Browse files Browse the repository at this point in the history
My included patch has already been accepted upstream.

Package-Manager: portage-2.2.26
  • Loading branch information
chewi committed Dec 6, 2015
1 parent 9ee199e commit 57d7b60
Show file tree
Hide file tree
Showing 2 changed files with 88 additions and 4 deletions.
81 changes: 81 additions & 0 deletions media-sound/forked-daapd/files/antlr-3.5.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
From 7cf35d1e8bb8dedb710e4a60bada2739d60306d0 Mon Sep 17 00:00:00 2001
From: James Le Cuirot <[email protected]>
Date: Mon, 5 Oct 2015 23:10:45 +0100
Subject: [PATCH 1/2] Allow ANTLR 3.5 to work

This involves a harmless backwards-compatible adjustment to the
grammar. I can't explain why this is necessary but this might help.

http://stackoverflow.com/questions/20057063/follow-set-in-is-undefined-in-generated-parser
---
src/RSP.g | 34 +++++++++++++++++-----------------
1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/RSP.g b/src/RSP.g
index 435de16..6a27536 100644
--- a/src/RSP.g
+++ b/src/RSP.g
@@ -43,43 +43,43 @@ strcrit : FIELD strop STR -> ^(strop FIELD STR)
| FIELD NOT strop STR -> ^(NOT ^(strop FIELD STR))
;

-strop : EQUAL
- | INCLUDES
- | STARTSW
- | ENDSW
+strop : equal=EQUAL
+ | includes=INCLUDES
+ | startsw=STARTSW
+ | endsw=ENDSW
;

intcrit : FIELD intop INT -> ^(intop FIELD INT)
| FIELD NOT intop INT -> ^(NOT ^(intop FIELD INT))
;

-intop : EQUAL
- | LESS
- | GREATER
- | LTE
- | GTE
+intop : equal=EQUAL
+ | less=LESS
+ | greater=GREATER
+ | lte=LTE
+ | gte=GTE
;

datecrit: FIELD dateop datespec -> ^(dateop FIELD datespec)
;

-dateop : BEFORE
- | AFTER
+dateop : before=BEFORE
+ | after=AFTER
;

datespec: dateref
| INT dateintval dateop dateref -> ^(dateop dateref INT dateintval)
;

-dateref : DATE
- | TODAY
+dateref : date=DATE
+ | today=TODAY
;

dateintval
- : DAY
- | WEEK
- | MONTH
- | YEAR
+ : day=DAY
+ | week=WEEK
+ | month=MONTH
+ | year=YEAR
;

QUOTE : '"';
--
2.4.3

Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ IUSE="alsa flac itunes lastfm mpd musepack"
# Note: mpd support appears to be standalone, e.g. --enable-mpd doesn't
# result in additional linkage.
RDEPEND="
>=dev-libs/antlr-c-3.1.3
dev-db/sqlite:3
dev-libs/antlr-c:0
dev-libs/confuse
dev-libs/libevent
dev-libs/libgcrypt:0
Expand All @@ -38,8 +38,7 @@ RDEPEND="
"

DEPEND="
dev-java/antlr:3
virtual/jre
dev-java/antlr:3.5
${RDEPEND}
"

Expand All @@ -49,13 +48,17 @@ pkg_setup() {
}

src_prepare() {
# Required until upstream accepts https://github.com/ejurgensen/forked-daapd/pull/179
# Fixed in 23.3.
epatch "${FILESDIR}/${P}-fix-arg-enable.patch"

# https://github.com/ejurgensen/forked-daapd/pull/185
epatch "${FILESDIR}/antlr-3.5.patch"

eautoreconf
}

src_configure() {
ac_cv_prog_ANTLR=antlr3.5 \
econf \
--with-alsa \
$(use_enable flac) \
Expand Down

0 comments on commit 57d7b60

Please sign in to comment.