intel / icamerasrc

icamerasrc
24 stars 35 forks source link

README renamed to README.md breaks automake #4

Closed vicamo closed 1 year ago

vicamo commented 3 years ago
$ export CHROME_SLIM_CAMHAL=ON
$ ./autogen.sh 
autoreconf: Entering directory `.'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal --force 
autoreconf: configure.ac: tracing
autoreconf: running: libtoolize --copy --force
libtoolize: putting auxiliary files in '.'.
libtoolize: copying file './ltmain.sh'
libtoolize: Consider adding 'AC_CONFIG_MACRO_DIRS([m4])' to configure.ac,
libtoolize: and rerunning libtoolize and aclocal.
libtoolize: Consider adding '-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
autoreconf: running: /usr/bin/autoconf --force
autoreconf: running: /usr/bin/autoheader --force
autoreconf: running: automake --add-missing --copy --force-missing
configure.ac:67: installing './compile'
configure.ac:67: installing './config.guess'
configure.ac:67: installing './config.sub'
configure.ac:57: installing './install-sh'
configure.ac:57: installing './missing'
Makefile.am: installing './INSTALL'
Makefile.am: error: required file './README' not found
Makefile.am: installing './COPYING' using GNU General Public License v3 file
Makefile.am:     Consider adding the COPYING file to the version control system
Makefile.am:     for your code, to avoid questions about which license your project uses
src/Makefile.am: installing './depcomp'
autoreconf: automake failed with exit status: 1
autogen.sh failed

This can be easily fixed by a symlink to README.md.

brmarkus commented 3 years ago

Why adding a workaround instead of using the new filename for automake, too?

vicamo commented 3 years ago

Your patch for automake is welcome.