Hartmut/paradiso
2
0
Fork 3
Code Issues Pull requests Projects Releases 2 Packages Wiki Activity

Probleme mit Windows #1

New issue
Closed
opened 2023-07-03 17:34:00 +02:00 by Timeplex · 3 comments
Timeplex commented 2023-07-03 17:34:00 +02:00
Contributor
Copy link

Es gibt 3 Probleme mit der aktuellen Version, wenn man ParadiSO auf Windows ausführen möchte:

  1. Das clang-format enthält (für zumindestens VS) unerlaubte Zeichen: ---

  2. u_int32_t Context::get_error() const { return impl_->get_error(); } (lib/stc/context.cpp:87)
    u_int32_t existiert unter Windows nicht und muss mit uint32_t ersetzt werden.

  3. Unter Windows ist max() ein vordefiniertes Makro welches vorher neu oder undefinrt werden muss.

#undef max

    uint64_t change_count{std::numeric_limits<uint64_t>::max()};
    uint64_t change_count_texture{std::numeric_limits<uint64_t>::max()};

(lib/src/renderer.cpp:40-41)

Ich habe diese Änderung einmal durchgeführt und anschließend getestet ((Win 64, Win 32) x (VS, VSC)). Ob die Änderungen auch noch so auf Linux ausführbar sind, ist mir leider unbekannt.

Link zum Fork

Es gibt 3 Probleme mit der aktuellen Version, wenn man ParadiSO auf Windows ausführen möchte: 1. Das clang-format enthält (für zumindestens VS) unerlaubte Zeichen: `---` 2. `u_int32_t Context::get_error() const { return impl_->get_error(); }` (lib/stc/context.cpp:87) `u_int32_t` existiert unter Windows nicht und muss mit `uint32_t` ersetzt werden. 3. Unter Windows ist max() ein vordefiniertes Makro welches vorher neu oder undefinrt werden muss. `#undef max` ``` uint64_t change_count{std::numeric_limits<uint64_t>::max()}; uint64_t change_count_texture{std::numeric_limits<uint64_t>::max()}; ``` (lib/src/renderer.cpp:40-41) Ich habe diese Änderung einmal durchgeführt und anschließend getestet ((Win 64, Win 32) x (VS, VSC)). Ob die Änderungen auch noch so auf Linux ausführbar sind, ist mir leider unbekannt. Link zum [Fork](https://code.technotecture.net/Timeplex/paradiso-win)
Timeplex changed title from Probleme mit windows to Probleme mit Windows 2023-07-03 17:39:33 +02:00
Hartmut commented 2023-07-03 18:10:16 +02:00
Owner
Copy link

Sehr gut. Vielen Dank! Welcher Compiler ist das? MSVC? Ich kann ihren Fork zwar manuell pullen aber Sie können mir auch einen Pull-Request schicken. Melden Sie sich mal auf der Hochschul-Email - ich suche noch einen Tutor 👍

Sehr gut. Vielen Dank! Welcher Compiler ist das? MSVC? Ich kann ihren Fork zwar manuell `pullen` aber Sie können mir auch einen Pull-Request schicken. Melden Sie sich mal auf der Hochschul-Email - ich suche noch einen Tutor 👍
Timeplex commented 2023-07-03 18:43:46 +02:00
Author
Contributor
Copy link

Ja, cmake verwendet bei mir MSVC als Compiler:

//CXX compiler
CMAKE_CXX_COMPILER:STRING=D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe

Der Pull-Request wurde gestellt. #2

Ja, cmake verwendet bei mir MSVC als Compiler: ``` //CXX compiler CMAKE_CXX_COMPILER:STRING=D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe ``` Der Pull-Request wurde gestellt. #2
Hartmut commented 2023-07-03 19:58:42 +02:00
Owner
Copy link

Merge war gut. Weiterer fix mit 227d4a8648. Danke für den PR!

Merge war gut. Weiterer fix mit 227d4a8648. Danke für den PR!
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
v0.0.3
v0.0.2
v0.0.1
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Hartmut/paradiso#1
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?