qw3rtty / neix

neix - a RSS/Atom feed reader for your terminal.
GNU General Public License v3.0
200 stars 12 forks source link

Always determine the user's config path at runtime #19

Closed dhgutteridge closed 3 years ago

dhgutteridge commented 3 years ago

Use getFeedConfigPath() for the "-i" option in processArguments(), the same as is done for the "-e" option, and in main(). Avoids errors or confusion when the runtime environment is not the same as the build environment.

dhgutteridge commented 3 years ago

This derives from downstream issues noted with packaging neix for pkgsrc.org. That one spot will result in a search against the build environment's definition of $HOME, not whatever a user's $HOME is. Everywhere else you already check the latter.

BTW, I will probably be submitting more and/or commenting on other tickets you already have open. This is a quick one-liner I thought I'd send on right away. Thanks!

qw3rtty commented 3 years ago

Thanks for opening this pull-request! Good to now.

BTW, I will probably be submitting more and/or commenting on other tickets you already have open. This is a quick one-liner I thought I'd send on right away. Thanks! I really appreciate it! Thanks.