Feature #838867: Add SDL2 Emscripten client to autotools build - Freeciv - HostedRedmine by Planio

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...

Send by e-mail

Feature #838867

Task #774803: Emscripten clients for web.

Add SDL2 Emscripten client to autotools build

Added by Zoltán Žarkov almost 3 years ago. Updated over 2 years ago.

Status:

New

Priority:

Normal

Assignee:

Category:

Sprint/Milestone:

3.1.0

Start date:

Due date:

% Done:

100%

Estimated time:

(Total: 0.00 h)

Description

This patch builds under the original travis ci, and is the bare minimum for a successful build of wasm sdl2 client. Other patches will be needed to make it actually runnable- I did not want to make this one unmanageably large.

0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch (23.3 KB) 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch	master	Zoltán Žarkov, 2019-09-28 03:02 AM
0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch (23.3 KB) 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch	master	Zoltán Žarkov, 2019-09-28 03:06 AM
0003-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch (22 KB) 0003-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch	S2_6	Zoltán Žarkov, 2019-09-28 03:59 AM
0002-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch (23.3 KB) 0002-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch	S3_0	Zoltán Žarkov, 2019-09-28 03:59 AM

Subtasks

Related issues

History

#1 Updated by Zoltán Žarkov almost 3 years ago

File 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch added

#2 Updated by Zoltán Žarkov almost 3 years ago

File deleted (~~0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch~~)

#3 Updated by Zoltán Žarkov almost 3 years ago

File 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch added

Master

#4 Updated by Zoltán Žarkov almost 3 years ago

File 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch 0001-Update-the-build-system-and-add-macros-for-SDL2-Emsc.patch added

Fixed a redundant case statement in sound.m4

#5 Updated by Zoltán Žarkov almost 3 years ago

I made it for the other branches. CI builds here https://gitlab.com/zekoz/freeciv-emscripten/pipelines/85125846 and https://gitlab.com/zekoz/freeciv-emscripten/pipelines/85125346

#6 Updated by Zoltán Žarkov almost 3 years ago

Sprint/Milestone set to 2.6.1

#7 Updated by Zoltán Žarkov almost 3 years ago

Blocks Feature #839110: Add Qt Emscripten client to autotools build added

#8 Updated by Zoltán Žarkov almost 3 years ago

Assignee set to Sveinung Kvilhaugsvik

May I request your review for this?

#9 Updated by Sveinung Kvilhaugsvik over 2 years ago

Assignee deleted (~~Sveinung Kvilhaugsvik~~)

Thank you for the reminder, Zoltán. I'm unfortunately not competent to review this patch. I am however interested in learning. If I find time to learn before someone else reviews your patch I'll have another look.

Potentially stupid question: Why is (parts of) the SDL 2 client the bare minimum? Wouldn't libfreeciv have been smaller and easier? Is it more difficult to test a C library compiled to wasm by calling it from JavaScript than I think it is?