Bubbus / ACF-Missiles

Racked munitions for ACF
MIT License
24 stars 13 forks source link

Include beam_netvars.lua to fix issue with Wiremod update #131

Closed CheezusChrust closed 5 years ago

CheezusChrust commented 5 years ago

This fix will stop ACF missiles from breaking entirely for servers using the Github version of Wiremod (and, likely soon, the workshop version) due to removing support for beam_netvars.lua in this update.

Fixes #128

In the future, it may be a better alternative to remove the dependency on beam_netvars.lua but for now, this will fix it.

TwistedTail commented 5 years ago

Wouldn't it be a better idea to just get rid of beam_netvars.lua entirely? It's already doing pretty much the same thing as any [Get/Set]NW* function while only being used on two files.

CheezusChrust commented 5 years ago

Yeah, that's the better way to do it, this is a quick fix I'm using on my server because at least right now I can't be bothered to go through and remove the dependency on the beam_netvars file. If someone else wants to create a different PR to do that I'll close this one and let them fix it more properly.