View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001716 | AI | AI | public | 2009-11-01 17:18 | 2018-10-31 16:17 |
| Reporter | hoijui | Assigned To | gajop | ||
| Priority | low | Severity | feature | Reproducibility | N/A |
| Status | closed | Resolution | no change required | ||
| Summary | 0001716: Remove AIInfo.lua, AIOptions.lua | ||||
| Description | AIInfo.lua and AIOptions.lua are used by unitsync and spring.exe to find and identify a Skirmish AI and know what options it supports. The same info could be fetched from the AI library directly. the change would bring: - unitsync/spring could crash if any of the installed AI libs do something bad when they get querried for the info + cleaner file structure (eg, the shared libs could be alone in a dir on debian) + no more double versioning + no need for lua for a simple Skirmish AI as fetching of these infos is a really trivial thing, i do not think we will likely see problems with AIs taking down unitsync/spring this way. | ||||
| Tags | No tags attached. | ||||
|
|
That is great. I always was wonder about absurd of storing DLL version in external file. |
|
|
This is probably never going to happen. It's also probably simpler and safer to keep such config in actual config files rather than to depend on loading AI binaries directly. I don't think there's much "version" duplication going on atm. If this refers to "interfaceVersion" I don't think that ever changes either. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-11-01 17:18 | hoijui | New Issue | |
| 2010-04-25 19:00 | slogic | Note Added: 0004852 | |
| 2018-10-31 16:17 | gajop | Assigned To | => gajop |
| 2018-10-31 16:17 | gajop | Status | new => closed |
| 2018-10-31 16:17 | gajop | Resolution | open => no change required |
| 2018-10-31 16:17 | gajop | Note Added: 0019457 |
