Hi, Dirk.
2010/6/14 Dirk Eddelbuettel <e...@debian.org>:
>
> On 14 June 2010 at 09:42, Nobuhiro Iwamatsu wrote:
> | -----BEGIN PGP SIGNED MESSAGE-----
> | Hash: SHA1
> |
> | Source: rcmdr
> | Version: 1.5-5-1
> | Severity: serious
> | Justification: FTBFS
> | tags: patch
> |
> | Hi,
> |
> | your package FTBFS.
> | I checked on sid in amd64, i386 and sh4.
> |
> |
> http://buildd.debian-ports.org/fetch.php?pkg=rcmdr&arch=sh4&ver=1.5-5-1&stamp=1276475550&file=log&as=raw
>
> According to https://buildd.debian.org/build.cgi?pkg=rcmdr it builds
> everywhere but mips and mipsel which seem to have tcl/tk issues.
>
> | - ------
> | ...
> | mkdir -p "."
> | dh_installdirs usr/lib/R/site-library
> | if test -f /usr/bin/xvfb-run; then \
> | xvfb-run \
> | R CMD INSTALL -l
> /build/buildd-rcmdr_1.5-5-1-sh4-JWLpfO/rcmdr-1.5-5/debian/r-cran-rcmdr/usr/lib/R/site-library
> --clean \
> | . ; \
> | else \
> | R CMD INSTALL -l
> /build/buildd-rcmdr_1.5-5-1-sh4-JWLpfO/rcmdr-1.5-5/debian/r-cran-rcmdr/usr/lib/R/site-library
> \
> | --clean . ;\
> | fi
> | xvfb-run: error: xauth command not found
> | make: *** [R_any_arch] Error 3
>
> Yes -- but note that there is an 'if'. Some packages need x11, for those I
> add xfvb (as well as fonts and, yes. xauth).
>
> Here we do not. The build depends is
>
> Build-Depends: debhelper (>= 5.0.0), r-base-dev (>= 2.11.0~20100409), cdbs,
> r-cran-car
>
> so we should have been in the else branch.
>
> To me, it looks like your build environment is at fault as it has a partial
> x11 setup.
I checked my buildd environment.
You are right.
Xvfb was installed in the environment that build rcmdr package.
Thanks !
>
> | ...
> | - ------
> |
> | I created a patch that fix this problem.
> | Could you check and apply this patch?
> |
> | Best regards,
> | Nobuhiro
> |
> | - - --
> | Nobuhiro Iwamatsu
> | GPG ID: 40AD1FA6
> | -----BEGIN PGP SIGNATURE-----
> | Version: GnuPG v1.4.10 (GNU/Linux)
> |
> | iEYEARECAAYFAkwVewwACgkQQSHHQzFw6+nfCACcCsNuSH9fjeOgmR8Hltp7Nea4
> | BW0AnjPZkE1v22P0Y3koGjVDNDKElLLN
> | =eX/y
> | -----END PGP SIGNATURE-----
> |
> | ----------------------------------------------------------------------
> | diff --git a/debian/control b/debian/control
> | index 9b2c326..e17fb94 100644
> | --- a/debian/control
> | +++ b/debian/control
> | @@ -2,7 +2,7 @@ Source: rcmdr
> | Section: gnu-r
> | Priority: optional
> | Maintainer: Dirk Eddelbuettel <e...@debian.org>
> | -Build-Depends: debhelper (>= 5.0.0), r-base-dev (>= 2.11.0~20100409),
> cdbs, r-cran-car
> | +Build-Depends: debhelper (>= 5.0.0), r-base-dev (>= 2.11.0~20100409),
> cdbs, r-cran-car, xauth
> | Standards-Version: 3.8.4
>
> That is not right. To take a package that depends on xvfb (like cairoDevice,
> many of the f* packages from Rmetrics or rggobi) then we have
>
> xvfb, xauth, xfonts-base
>
> so either all or nothing.
>
> What do you think?
Yes, this is correct you.
But I think that does not need this to this package.
Best regards,
Nobuhiro
--
Nobuhiro Iwamatsu
iwamatsu at {nigauri.org / debian.org}
GPG ID: 40AD1FA6
--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
