search

erkin / ponysay

Pony rewrite of cowsay.
GNU General Public License v3.0
1.22k stars 82 forks source link

Update readme to reflect configure changes please #258

Closed MrOnosa closed 7 years ago

MrOnosa commented 7 years ago

./configure --without-info does not work but ./setup.py --without-info --freedom=partial install worked just fine.

JotaRandom commented 7 years ago

verbatim from the README.md

Download or clone the project. In the terminal, cd into the ponysay directory and ./setup.py --freedom=partial install or python3 setup.py --freedom=partial install.

so which file exactly do you talk?

MrOnosa commented 7 years ago

I am discussing the --without-info configuration flag. Verbatim from the README.md

texinfo: Used for building the info manual (suppressable (sic) with ./configure --without-info).

However, this is no longer true. To suppress the info manual, one does: ./setup.py --without-info --freedom=partial install ./configure is now erroneous.

JotaRandom commented 7 years ago

--without-info use texinfo to build the info pages and is listed as dep on the dependency test ./dependency-test.sh also in the README.md in the section Dependencies > Package building dependencies mention texinfo too.

Maybe move the section Dependency above or within Installation on GNU/Linux.

JotaRandom commented 7 years ago

I move the dependency to the first; let me know if that make it clearer on what is need.

JotaRandom commented 7 years ago

I will close it since now the first paragraft show what is need.

also ./dependency-list.sh will show if youmeet all the deps, if it not with the defaukt flags and installing it then fill a bug aagainst the dep-check to get updated (and by extention the Readme.md)