Gradle copy to mods task

[mnukka]

No description provided.

[jaghaimo]

Please use the default file: gradle.properties (gitignored). That way you can access the properties directly, and can remove file reading etc. Do also add a gradle.properties.dist with an example of correctly set path.

Additionally, please make the task run after build has finished, but instead of throwing skip the copy if the directory is not set (could do with a warning message).

[mnukka]

Please use the default file: gradle.properties (gitignored). That way you can access the properties directly, and can remove file reading etc.

Are you sure about this? The reason I did not use "gradle.properties" is because this is the default file used by gradle for various shared settings that developers may want to define. It is generally not used for local settings but shared settings.

I can do it. The need for any shared settings might not be necessary in this project.

[mnukka]

Additionally, please make the task run after build has finished, but instead of throwing skip the copy if the directory is not set (could do with a warning message).

Can you elaborate the point of building if you have nothing to copy? You can run gradle build if you are just interested in the jar file.

[jaghaimo]

Additionally, please make the task run after build has finished, but instead of throwing skip the copy if the directory is not set (could do with a warning message).

Can you elaborate the point of building if you have nothing to copy? You can run gradle build if you are just interested in the jar file.

I thought of redefining build as build + (conditional) copy. Perhaps that's a wrong approach. Keeping throw is fine, just point the user to dist file.

[jaghaimo]

Please use the default file: gradle.properties (gitignored). That way you can access the properties directly, and can remove file reading etc.

Are you sure about this? The reason I did not use "gradle.properties" is because this is the default file used by gradle for various shared settings that developers may want to define. It is generally not used for local settings but shared settings.

I can do it. The need for any shared settings might not be necessary in this project.

Yes please. If for some reason we do end up needing shared properties then we'll revert this and go with custom (local) variation.

[jaghaimo]

Happy with this but would be great if you could make it more generic:
gradle.properties:

game.dir = C:\\Something\\Starsector
game.executable = starsector.exe (or starsector.sh for Linux)

With another task to start the game (depends on copyMod) and copyMod changed so the mod name is not hardcoded (taken from mod_info.json instead).