Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libffi 0.7.0 on docs.rs failed to build #10

Closed
ghost opened this issue Aug 17, 2019 · 4 comments
Closed

libffi 0.7.0 on docs.rs failed to build #10

ghost opened this issue Aug 17, 2019 · 4 comments

Comments

@ghost
Copy link

ghost commented Aug 17, 2019

Hi, I'm not sure if this is an issue with libffi-rs/libffi-sys-rs or with docs.rs, but the current page for libffi-rs 0.7.0 on docs.rs says it failed to build. You can see the build output here, although the interesting bits in the output have \n escaped, so I attempted to fix the output:

make[2]: Entering directory \'/home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/x86_64-unknown-linux-gnu\'
restore=: && backupdir=\".am$$\" && \\
am__cwd=`pwd` && CDPATH=\"${ZSH_VERSION+.}:\" && cd .. && \\
rm -rf $backupdir && mkdir $backupdir && \\
if (/bin/bash /home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/missing makeinfo  --version) >/dev/null 2>&1; then \\
  for f in ../doc/libffi.info ../doc/libffi.info-[0-9] ../doc/libffi.info-[0-9][0-9] ../doc/libffi.i[0-9] ../doc/libffi.i[0-9][0-9]; do \\
    if test -f $f; then mv $f $backupdir; restore=mv; else :; fi; \\
  done; \\
else :; fi && \\
cd \"$am__cwd\"; \\
if /bin/bash /home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/missing makeinfo    -I doc -I ../doc \\
 -o ../doc/libffi.info ../doc/libffi.texi; \\
then \\
  rc=0; \\
  CDPATH=\"${ZSH_VERSION+.}:\" && cd ..; \\
else \\
  rc=$?; \\
  CDPATH=\"${ZSH_VERSION+.}:\" && cd .. && \\
  $restore $backupdir/* `echo \"./../doc/libffi.info\" | sed \'s|[^/]*$||\'`; \\
fi; \\
rm -rf $backupdir; exit $rc
Makefile:1419: recipe for target \'../doc/libffi.info\' failed
make[2]: Leaving directory \'/home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/x86_64-unknown-linux-gnu\'
Makefile:1604: recipe for target \'install-recursive\' failed
make[1]: Leaving directory \'/home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/x86_64-unknown-linux-gnu\'
Makefile:3153: recipe for target \'install\' failed
--- stderr
autoreconf: Entering directory `.\'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal --force -I m4
autoreconf: configure.ac: tracing
autoreconf: running: libtoolize --copy --force
autoreconf: running: /usr/bin/autoconf --force
autoreconf: running: /usr/bin/autoheader --force
autoreconf: running: automake --add-missing --copy --force-missing
configure.ac:31: installing \'./compile\'
configure.ac:8: installing \'./config.guess\'
configure.ac:8: installing \'./config.sub\'
configure.ac:19: installing \'./install-sh\'
configure.ac:19: installing \'./missing\'
Makefile.am: installing \'./depcomp\'
Makefile.am:56: installing \'./mdate-sh\'
autoreconf: Leaving directory `.\'
/home/cratesfyi/cratesfyi/debug/build/libffi-sys-7b10269bbdd70d1a/out/libffi-build/missing: line 81: makeinfo: command not found
WARNING: \'makeinfo\' is missing on your system.
         You should only need it if you modified a \'.texi\' file, or
         any other file indirectly affecting the aspect of the manual.
         You might want to install the Texinfo package:
         <http://www.gnu.org/software/texinfo/>
         The spurious makeinfo call might also be the consequence of
         using a buggy \'make\' (AIX, DU, IRIX), in which case you might
         want to install GNU make:
         <http://www.gnu.org/software/make/>
make[2]: *** [../doc/libffi.info] Error 127
make[1]: *** [install-recursive] Error 1
make: *** [install] Error 2
thread \'main\' panicked at \'Building libffi\', /home/cratesfyi/.cargo/registry/src/github.com-1ecc6299db9ec823/libffi-sys-0.7.0/build.rs:41:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

Unfortunately I don't have much experience with building C projects/libffi, or using make/texinfo, so I can't help figure out what's going on here. The docs for libffi-rs 0.6.4 appear to have compiled correctly, and the changelog entry for 0.7.0 doesn't mention any big API changes, so in the mean time people can just use the docs from 0.6.4.

@tov
Copy link
Owner

tov commented Aug 29, 2019

Thanks. I think this might be because of an unsatisfied dependency on texinfo. I know that was a thing in the past, and I had .travis.yaml installing that package. But I thought someone managed to remove the dependency. I’ll look into it…

@tov
Copy link
Owner

tov commented Aug 29, 2019

Hmm, this is still there, though:

addons:
  apt:
    packages:
      - texinfo

@tov
Copy link
Owner

tov commented Aug 29, 2019

So I guess the question is can we either 1) not depend on texinfo or 2) get docs.rs to make it available.

@tov
Copy link
Owner

tov commented Aug 29, 2019

This actually belongs here: tov/libffi-sys-rs#24

@tov tov closed this as completed Aug 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant