Skip to content

(wip) linux support for parser#2338

Draft
ellieplayswow wants to merge 1 commit intoATTWoWAddon:masterfrom
ellieplayswow:feature/parser-linux-support
Draft

(wip) linux support for parser#2338
ellieplayswow wants to merge 1 commit intoATTWoWAddon:masterfrom
ellieplayswow:feature/parser-linux-support

Conversation

@ellieplayswow
Copy link
Contributor

chunking through this bit by bit, currently it builds and starts running via dotnet but fails -

PARSER STAGE: ContributorDataMerge...
FILE: ./DATAS/00 - DB/ObjectDB_Conditionals.lua
==== EXCEPTION ====
FILE: ./DATAS/00 - DB/ObjectDB_Conditionals.lua
ERROR: [string "chunk"]:5: attempt to index a nil value (field 'text')
   at NLua.Lua.ThrowExceptionFromError(Int32 oldTop)
   at NLua.Lua.DoString(String chunk, String chunkName)
   at ATT.Program.ParseLUAFile(Lua lua, String fileName) in /home/user/scratch/AllTheThings/.contrib/Source Code/Parser/Program.cs:line 1078
3:
4:local brightlyColoredEgg = ObjectDB[113768];
5:>
6:brightlyColoredEgg.icon = [[~_.asset("Holiday_noblegarden")]];
Press Enter once you have resolved the issue.

due to a mix of issues on my host machine i can't really run the parser, even with wine. opening this pr mostly so anyone can shout at me if this isn't wanted :) will leave it marked as a wip until it works fully and i'm happy enough with it

@NORPG
Copy link
Member

NORPG commented Mar 8, 2026

The current library depends on the .NET framework and needs to be migrated to .NET to achieve cross-platform compatibility.
However, previous migrations have encountered some problems, so we have been temporarily shelved, and there are currently no plans for further migration.

@NORPG NORPG requested review from DFortun81 and ImUnicke March 8, 2026 10:29
@NORPG
Copy link
Member

NORPG commented Mar 8, 2026

The main problem is that NLua (Lua parser for .NET) seems to have inconsistent behavior, and the root cause has not yet been found.

@NORPG
Copy link
Member

NORPG commented Mar 8, 2026

Currently, the NLua within the parser is compiled by ourself, not captured from the package; it's unclear whether this is a compilation-time setting issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants