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

Allow overrides to be declared in a file (rather than just via the overrides directory) #813

Open
Cervator opened this issue Jan 10, 2014 · 0 comments
Labels
Topic: Architecture Requests, Issues and Changes related to software architecture, programming patterns, etc.

Comments

@Cervator
Copy link
Member

Cervator commented Jan 10, 2014

As mentioned in #763 (which could very well use a similar strategy as this)

Currently textures can be overridden by putting an identically named texture in an "overrides" directory in a different module. While that's easy it isn't always ideal

Example: LASR holds assets for L&S, which includes a few overrides - yet other modules (like Portals) that has a dependency on LASR (to use the models) thus also get the overridden textures automatically

Instead we could use a module definition file (module.txt?) to map overrides that could be sourced from a dependent module (LASR) rather than the module doing the overriding (L&S)

IMHO it may still be convenient to support the overrides directory as well, akin to the "auto" blockTiles directory - better to eventually use a more comprehensive approach but it works right away for easy progress!

Edit: We may also want to look at module loading order vs. overrides at some point

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic: Architecture Requests, Issues and Changes related to software architecture, programming patterns, etc.
Projects
None yet
Development

No branches or pull requests

1 participant