tags 508832 + patch
thanks

(Cc'ing the maintainers of pccts and antlr)

pccts (/usr/bin/antlr) appears to be an older version of antlr, in C.
New versions of antlr are written in Java and installed as /usr/bin/runantlr.

Unfortunately, gpt looks for antlr in $PATH before looking for runantlr, so
it preferentially picks up the older version, which is in fact too old
for gpt.

Potential fixes seem to be:

* run ./configure with ANTLR_BIN=/usr/bin/runantlr as per the attached
  patch, forcing the use of the correct antlr (I can confirm that this does
  build correctly, even with pccts installed)

* Build-Conflict with pccts

(Side point: if the Java antlr is strictly better than the C one, perhaps it
would be better for pccts to install /usr/bin/antlr1 and let "the real antlr"
be /usr/bin/antlr?)

Regards,
    Simon
--- gpt-1.0.3/debian/rules	2008-12-16 00:24:27.000000000 +0000
+++ gpt-1.0.3/debian/rules	2008-12-16 00:24:14.000000000 +0000
@@ -3,5 +3,6 @@
 include /usr/share/cdbs/1/rules/debhelper.mk
 include /usr/share/cdbs/1/class/autotools.mk
 
+DEB_CONFIGURE_SCRIPT_ENV += ANTLR_BIN=/usr/bin/runantlr
 DEB_CONFIGURE_EXTRA_FLAGS += --with-antlr=/usr/share/antlr \
                              --enable-install-devel

Attachment: signature.asc
Description: Digital signature

Reply via email to