Check for manpage generation support by default

Doom-Utils / deutex

WAD composer for Doom, Heretic, Hexen, and Strife

Other

61 stars 17 forks source link

Check for manpage generation support by default #65

Closed vilhelmgray closed 5 years ago

vilhelmgray commented 5 years ago

Searches for a2x by default, by only errors if --enable-man is explicitly given. This fixes Github issue #64.

chungy commented 5 years ago

Still breaks if a2x is missing and no parameters are specified to configure:

make[2]: Entering directory '/home/chungy/Games/doom/src/deutex/man'
a2x -f manpage deutex.txt
make[2]: a2x: Command not found
make[2]: *** [Makefile:484: deutex.6] Error 127
make[2]: Leaving directory '/home/chungy/Games/doom/src/deutex/man'
make[1]: *** [Makefile:360: all-recursive] Error 1
make[1]: Leaving directory '/home/chungy/Games/doom/src/deutex'
make: *** [Makefile:299: all] Error 2

vilhelmgray commented 5 years ago

I apologize, I was testing several variants last night and force-pushed a broken version. I've updated the pull request now to fix the error. Let me know if you have any further issues.

chungy commented 5 years ago

Thanks, I've tested all the combinations (with/without a2x and both and no arguments to configure), I'm satisfied with how it works now.