diff mbox

foomatic-filters: make location of enscript very clear

Message ID 1365072901-11719-1-git-send-email-gustavo@zacarias.com.ar
State Accepted
Commit b392cb4c743cce128c2c777f3b2116d546118ec6
Headers show

Commit Message

Gustavo Zacarias April 4, 2013, 10:55 a.m. UTC
Passing --with-file-converter=enscript isn't enough since the build
system will try the fallback options (a2ps, mpage, texttops) if it's not
found.
This has two bad effects: one of the tools leaking from the distribution
making foomatic-filters think everything will be the same on the target
(and failing when that's not the case), and failed builds when none is
found (enscript is installed in the target directory, the build system
doesn't look there).
So just avoid the test by stating explicitly where enscript will live in
the target. Fixes:
http://autobuild.buildroot.net/results/37ad3d2c5c35f00c644a2c572f1429bafa4b7cf4/

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/foomatic-filters/foomatic-filters.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Peter Korsgaard April 4, 2013, 8:57 p.m. UTC | #1
>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 Gustavo> Passing --with-file-converter=enscript isn't enough since the
 Gustavo> build system will try the fallback options (a2ps, mpage,
 Gustavo> texttops) if it's not found.  This has two bad effects: one of
 Gustavo> the tools leaking from the distribution making
 Gustavo> foomatic-filters think everything will be the same on the
 Gustavo> target (and failing when that's not the case), and failed
 Gustavo> builds when none is found (enscript is installed in the target
 Gustavo> directory, the build system doesn't look there).  So just
 Gustavo> avoid the test by stating explicitly where enscript will live
 Gustavo> in the target. Fixes:
 Gustavo> http://autobuild.buildroot.net/results/37ad3d2c5c35f00c644a2c572f1429bafa4b7cf4/

Committed, thanks.
diff mbox

Patch

diff --git a/package/foomatic-filters/foomatic-filters.mk b/package/foomatic-filters/foomatic-filters.mk
index ad7ab7a..6e13486 100644
--- a/package/foomatic-filters/foomatic-filters.mk
+++ b/package/foomatic-filters/foomatic-filters.mk
@@ -9,8 +9,8 @@  FOOMATIC_FILTERS_SITE = http://www.openprinting.org/download/foomatic
 FOOMATIC_FILTERS_LICENSE = GPLv2+
 FOOMATIC_FILTERS_LICENSE_FILES = COPYING
 FOOMATIC_FILTERS_DEPENDENCIES = cups libusb enscript
-
 FOOMATIC_FILTERS_CONF_OPT = --with-file-converter=enscript
+FOOMATIC_FILTERS_CONF_ENV = ac_cv_path_ENSCRIPT="/usr/bin/enscript"
 
 ifeq ($(BR2_PACKAGE_DBUS),y)
 FOOMATIC_FILTERS_CONF_OPT += --enable-dbus