search

tbeu / matio

MATLAB MAT File I/O Library
https://matio.sourceforge.io
BSD 2-Clause "Simplified" License
334 stars 97 forks source link

Add 'autoconf' to 'Quick Build Guide' #63

Closed NiklasEi closed 7 years ago

NiklasEi commented 7 years ago

For someone strictly following the 'Quick Build Guide' running './configure' will produce 'bash: ./configure: No such file or directory'. To spare others the same problem 'autoconf' was added to the guide which will produce the missing configure script with the given 'configure.ac'.

I did not change anything on line 42 but GitHub automatically changed the encoding of the file from ISO-8859-1 to UTF-8, which I guess is leading GitHub to mark the line as changed.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 80.779% when pulling 8631251255ee8e6c4a424a1b2145b0e0459163e4 on NiklasEi:NiklasEi-patch-1 into db86364da8d143c2aaea983384a8af4a8ecb720d on tbeu:master.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 80.779% when pulling 8631251255ee8e6c4a424a1b2145b0e0459163e4 on NiklasEi:NiklasEi-patch-1 into db86364da8d143c2aaea983384a8af4a8ecb720d on tbeu:master.

tbeu commented 7 years ago

Note, there is a difference between building the lib from the repo or building from the distribution. README is meant for the distribution, where configure is available.

tbeu commented 7 years ago

I fixed the encoding issue in both READMEs though. Thanks for reporting it.

NiklasEi commented 7 years ago

Sure, no problem.