search

the3dfxdude / 7kaa

Seven Kingdoms: Ancient Adversaries - Go to the main source repository at https://sourceforge.net/projects/skfans/ for source code and builds
https://7kfans.com
Other
253 stars 71 forks source link

Wasm #241

Closed nintyfan closed 2 years ago

nintyfan commented 2 years ago

Now it works. There is need to add shiny look&feel to shell and maybe other changes, but the game working with only build system changes and disabling code related to multiplayer (it not are included in build process now).

To build made:

emconfigure ./autogen.sh emconfigure ./configure.sh --enable-emscripten --disable-networking emmake make

Next, put it on your server or run via emrun or something.

nintyfan commented 2 years ago

I will try to resolve conflicts later.

the3dfxdude commented 2 years ago

Please see my comment on the previous pull request https://github.com/the3dfxdude/7kaa/pull/239

the3dfxdude commented 2 years ago

The comment I had on pull #239 applies to this pull as I cannot accept the flag changes in the way you provided for enet. configure needs to be agnostic to emscripten for enabling/disabling enet.

On the part of setting the options to build the webpage and change how the main loop works; those appear to simply be changes that can be applied via environment variables for the compiler. It is normal to set environment variables necessary for your platform. I think emscripten should just be treated as a cross compiler. Then I think it's just a matter of telling configure what to do, which in that case, you don't need a commit change for that.

You could rework the request correctly to show what needs to be done again, but if all that's needed is to set CFLAGS and LDFLAGS, then there really isn't a need to run that through autoconf. But still, I don't see a need for an emscripten build, because the browsers/engines that it would run on, we already are running on those platforms.