Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify loading of minetest.conf in Readme.txt #5249

Closed
wants to merge 1 commit into from
Closed

Clarify loading of minetest.conf in Readme.txt #5249

wants to merge 1 commit into from

Conversation

Fixer-007
Copy link
Contributor

@Fixer-007 Fixer-007 commented Feb 16, 2017

Clarify in doc/README.txt:

- A run-in-place build will look for the configuration file in 
        $location_of_exe/../minetest.conf and also $location_of_exe/../../minetest.conf

Addresses #5245:

[23:33:23] celeron55: a run-in-place will look for the configuration file from $location_of_exe/../minetest.conf and also $location_of_exe/../../minetest.conf
[23:33:31] celeron55: a run-in-place build*
[23:34:24] celeron55: useful for eg. visual studio builds that append another directory before bin
[23:35:58] celeron55: or for collections of different minetests that still should use the same configuration

Yay, my first pull request.

@paramat paramat added the Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines label Feb 25, 2017
@paramat
Copy link
Contributor

paramat commented Mar 5, 2017

👍

@paramat
Copy link
Contributor

paramat commented Mar 5, 2017

0a6e53b

Note the second line was indented by the incorrect number of spaces, there was also a space after the first line, i fixed both on merge.

@paramat paramat closed this Mar 5, 2017
@Fixer-007 Fixer-007 deleted the Fixer-007-readme branch March 5, 2017 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Documentation One approval ✅ ◻️ Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants