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

Jump to bottom

Restrict dependencies between content-type #24

Open
TrueBrain opened this issue Apr 21, 2020 · 1 comment
Open

Restrict dependencies between content-type #24

TrueBrain opened this issue Apr 21, 2020 · 1 comment
Labels
enhancement New feature or request prio: high This issue should be fixed when we get the time

Comments

@TrueBrain
Copy link
Member

Currently you can make a heightmap depend on an AI, which makes very little sense.

The list should be like this:

  • ai, ai-library: only allow ai-library
  • base-graphics, base-music, base-sounds: no dependencies allowed
  • game-script, game-script-library: only allow game-script-library
  • heightmap, newgrf: no dependencies allowed
  • scenario: only allow newgrf, ai and game-script

Currently we have AIs that depend on a scenario. There is a case for this, but not sure it is a solid one.
@TrueBrain TrueBrain added enhancement New feature or request prio: high This issue should be fixed when we get the time labels Apr 21, 2020
@James103
Copy link

James103 commented Oct 9, 2021

  • newgrf: no dependencies allowed

This should be changed to "newgrf: only allow newgrf" as some NewGRF do not work properly without some other NewGRF.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request prio: high This issue should be fixed when we get the time
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TrueBrain @James103