search

DavidGriffith / frotz

Infocom-style interactive fiction player for Unix and DOS (moved to https://gitlab.com/DavidGriffith/frotz)
GNU General Public License v2.0
209 stars 64 forks source link

Optionally look for configuration in $XDG_CONFIG_HOME #25

Closed escondida closed 8 years ago

escondida commented 8 years ago

Also check $HOME/.frotzrc, for backwards compatibility. reference: https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

escondida commented 8 years ago

Some things to note: I used snprintf to simplify configfile variable assignment while still allowing use of FILENAME_MAX to ensure valid variable length. I arbitrarily decided to use tabs here, because the preceding block uses spaces but the following block uses tabs.

escondida commented 8 years ago

Ah, didn't realize that git would automatically add new commits to the pull request. In that case...

The first of the new commits allow users/packagers to run "make DESTDIR=/some/dir" install, which is pretty standard, and is really useful for packaging. This changes nothing about the build if DESTDIR isn't specified on the command line.

The second simply leaves the decision of whether to strip the binary in the hands of the user/packager--you never know when debugging symbols might come in handy (-: