Автоматизация сборочного процесса + поддержка работы в Linux by ololoshka2871 · Pull Request #1 · OakwoodTable/CyberBiology3

ololoshka2871 · 2022-08-02T17:09:14Z

Увидел ваш ролик на ютубе, давно хотел сделать нечто подобное.
Оказалось что у вас сборка только в Visual Studio, хотя все зависимости доступны и в Linux, немного переписал код и добавил сборку при помощи CMake.

golxzn · 2022-08-06T04:38:41Z

@ololoshka2871, Я поддерживаю твое стремление перевести этот проект на CMake + submodules, однако почему бы не подтягивать и SDL заодно?

Хотя, будем надеяться, что этот ПР вообще включат хотя бы как отдельную ветку)

golxzn · 2022-08-06T04:40:10Z

 }


+#ifndef UNIX


I would like to use #if !defined(UNIX) :idk:

golxzn · 2022-08-06T04:53:29Z

 					if (ImGui::Selectable(filenamesPartial[i], &selectedFilenames[i]))
 					{
-						ZeroMemory(selectedFilenames, sizeof(selectedFilenames));
+						memset(selectedFilenames, 0, sizeof(selectedFilenames));


OH, It's so painful 😢
If we'll change selectedFilenames's type to std::array, there'll fill method for this stuff

Just fast fix 🥲

OakwoodTable · 2022-08-06T10:50:06Z

Здравствуйте, рад что вам понравился мой ролик. Также приятно знать что кому-то интересна программа. Плохо умею работать с этим сайтом, что мне сейчас нужно делать? Я внес изменения в код с тех пор, но сюда пока не выкладывал, скажите может быть мы оставим вашу ветку как отдельную и вы будете её вести. Никогда не работаю в Linux. Да и честно говоря вообще планировал забросить код после ещё одного видео, так как хочу написать новую программу на CLR + DirectX. Посоветуйте как лучше.

ololoshka2871 · 2022-08-07T05:53:51Z

@OakwoodTable Извините, я сам не особо силен в интерфейсе гитхаба, но в теории вам нужно согласиться принять этот запрос, но оформить его как новую ветку, например с именем cmake-build или вроде того.
Кроме того вы можете передать мне ваши изменения и я попробую их интегрировать чтобы у нас все сложилось вместе.

Я еще хотел поработать над автоматическим выделением количества рабочих потоков равным количеству ядер процессора. Ну если будет время.

@grpilot Я не имел опыта сборки SDL2 в Windows, просто погуглил и нашел готовый собранный дистрибутив, и оно заработало. Можно попробовать и submodul.

ololoshka2871

Добавил SDL2 в качестве субмодуля, теперь внешних зависимостей нет.

qwertilus · 2022-08-13T17:21:26Z


 	//----------------------------------------------------------------------------------------------
-	//������� ��� ������������� ��������, ���������� � ����� ������� tick(), ���� �� ������������
+	//������� ��� ������������� ��������, ���������� � ����� ������� tick(), ���� �� ������������


у вас тут беды с utf-8....

qwertilus

все офигенно работает, тестил на Ubuntu 20.04 LTS + CLion

proller · 2022-08-18T18:40:54Z

может быть мы оставим вашу ветку как отдельную и вы будете её вести.

Пожалуйста не плодите долгоживущие ветки, сборка под любый ос с использованием cmake и без должна работать в основной ветке.

edombek · 2022-08-23T06:22:23Z

не работает в ArchLinux:

X Error of failed request:  GLXBadDrawable
  Major opcode of failed request:  152 (GLX)
  Minor opcode of failed request:  5 (X_GLXMakeCurrent)
  Serial number of failed request:  229
  Current serial number in output stream:  229

qwertilus · 2022-08-23T14:20:41Z

не работает в ArchLinux:

export MESA_GL_VERSION_OVERRIDE=4.5

5 сек гугла https://forum.winehq.org/viewtopic.php?f=8&t=34889

edombek · 2022-08-23T14:56:19Z

MESA_GL_VERSION_OVERRIDE=4.5

неа, непомогает:

$ MESA_GL_VERSION_OVERRIDE=4.5 ./CyberBiology3 
X Error of failed request:  GLXBadDrawable
  Major opcode of failed request:  152 (GLX)
  Minor opcode of failed request:  5 (X_GLXMakeCurrent)
  Serial number of failed request:  229
  Current serial number in output stream:  229

justuser-31 · 2022-08-29T20:01:03Z

Что-то не могу собрать в Void Linux.Поможете?

При запуске:
git submodule update --init --recursive
Выдаёт:
fatal: not a git repository (or any of the parent directories): .git
(Не критично,я и руками могу натаскать библеотек,но не приятно)

Далее.При запуске:
mkdir build && cd build
cmake ..

Выдаёт:

CMake Error at libs/CMakeLists.txt:5 (add_subdirectory):
The source directory
/home/void/Desktop/CyberBiology3-main/libs/SDL
does not contain a CMakeLists.txt file.
CMake Error at libs/CMakeLists.txt:9 (add_subdirectory):
The source directory
/home/void/Desktop/CyberBiology3-main/libs/SDL_ttf
does not contain a CMakeLists.txt file.
-- Configuring incomplete, errors occurred!
See also "/home/void/Desktop/CyberBiology3-main/build/CMakeFiles/CMakeOutput.log".

0x8BADFOOD · 2022-08-31T04:06:52Z

Посмотрите, как организована сборка проектов SDL для разных платформ(Win, MacOS, Linux) на примере популярных проектов. На пример тут: https://github.com/cxong/cdogs-sdl

qwertilus · 2022-09-02T14:05:38Z

"/home/void/Desktop/CyberBiology3-main/build/CMakeFiles/CMakeOutput.log".

думаю, стоит скинуть содержимое логов

justuser-31 · 2022-09-02T16:18:00Z

думаю, стоит скинуть содержимое логов

Вот - https://pastebin.com/qzd4gBrk

willbasky · 2022-09-03T06:40:25Z

Спасибо за инструкцию, на Manjaro собралось и запускается. (Бинарник по умолчанию сохраняется сюда build/source/CyberBiology3)
@ololoshka2871 @OakwoodTable есть в проекте настройки, чтобы подкрутить масштабирование шрифтов? На большом экране 2K мелковато. И окно не разворачивается на весь экран.

ololoshka2871 · 2022-09-03T13:02:37Z

Смерджил последние изменения от @OakwoodTable

marmusevich · 2022-09-09T04:32:35Z

залейте/смержите пожалуйста, или в отдельном бранче, или подскажите как забрать изменения гитом, без установки гитхаба

Rai220 · 2022-11-11T16:11:20Z

Автору большое спасибо за программу и видео, очень интересно!
ololoshka2871 спасибо за адаптацию сборки.
Попробовал на маке - сборка проходит, правда при запуске Segmentation fault. Впрочем, чего я ждал :)

SergeiNA · 2022-12-04T22:59:56Z

Здравствуйте, планируется ли какой-то мердж?

SergeiNA · 2022-12-04T23:02:25Z

@ololoshka2871 cмерджил куда? Пишет это=(
This branch is 12 commits ahead, 17 commits behind OakwoodTable:main.

SergeiNA · 2022-12-05T00:27:29Z

@Rai220 получилось понять в чем дело?

ololoshka2871 · 2022-12-29T10:42:51Z

Смерджил последние изменения.
Пришлось сделать SDL_GL_SetAttribute(SDL_GL_DEPTH_SIZE, 24); в source/SDL.cpp:22 иначе на рабочем компе на встройке не запускалось.

OakwoodTable · 2022-12-29T20:48:22Z

Честно говоря я бы принял этот пул реквест, если бы у вас не была изменена структура файлов проекта и была бы моя версия CMakeLists.txt. Не могли бы вы убрать эти изменения, или это не обсуждается?:) Также для чего у вас используется SDL_tff?

ololoshka2871 - wsl and others added 4 commits August 1, 2022 17:50

Linux build

d4f8518

build in linux

59b8362

Cmake + MSVC

18a2f20

reorganise sources

73df26e

golxzn reviewed Aug 6, 2022

View reviewed changes

Comment thread source/Main.cpp Outdated

}

#ifndef UNIX

golxzn Aug 6, 2022

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to use #if !defined(UNIX) :idk:

golxzn reviewed Aug 6, 2022

View reviewed changes

ololoshka2871 - wsl and others added 4 commits August 11, 2022 15:06

Add SDL2 as submodule

2d65722

fix windows build - sdl now static library, don\t copy .dll

2f96f85

Replace cmake UNIX macro with _WIN32 macro from C-precprocessor

4991a53

Update readmes

133e309

ololoshka2871 commented Aug 11, 2022

View reviewed changes

qwertilus reviewed Aug 13, 2022

View reviewed changes

qwertilus approved these changes Aug 13, 2022

View reviewed changes

ololoshka2871 - wsl and others added 4 commits September 3, 2022 15:05

merge with upstream

1bf2ab0

resolve build issues

e4954b6

fix windows build

9581e53

Remove unused library imgui_sdl

2131dae

ololoshka2871 and others added 2 commits December 29, 2022 13:13

merge upstream

e5037aa

fix missing include <string>

528b9bf

Conversation

ololoshka2871 commented Aug 2, 2022

Uh oh!

golxzn commented Aug 6, 2022

Uh oh!

golxzn Aug 6, 2022

Choose a reason for hiding this comment

Uh oh!

golxzn Aug 6, 2022

Choose a reason for hiding this comment

Uh oh!

ololoshka2871 Aug 7, 2022

Choose a reason for hiding this comment

Uh oh!

OakwoodTable commented Aug 6, 2022

Uh oh!

ololoshka2871 commented Aug 7, 2022

Uh oh!

ololoshka2871 left a comment

Choose a reason for hiding this comment

Uh oh!

qwertilus Aug 13, 2022

Choose a reason for hiding this comment

Uh oh!

qwertilus left a comment

Choose a reason for hiding this comment

Uh oh!

proller commented Aug 18, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

edombek commented Aug 23, 2022

Uh oh!

qwertilus commented Aug 23, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

edombek commented Aug 23, 2022

Uh oh!

justuser-31 commented Aug 29, 2022

Uh oh!

0x8BADFOOD commented Aug 31, 2022

Uh oh!

qwertilus commented Sep 2, 2022

Uh oh!

justuser-31 commented Sep 2, 2022

Uh oh!

willbasky commented Sep 3, 2022

Uh oh!

ololoshka2871 commented Sep 3, 2022

Uh oh!

marmusevich commented Sep 9, 2022

Uh oh!

Rai220 commented Nov 11, 2022

Uh oh!

SergeiNA commented Dec 4, 2022

Uh oh!

SergeiNA commented Dec 4, 2022

Uh oh!

SergeiNA commented Dec 5, 2022

Uh oh!

ololoshka2871 commented Dec 29, 2022

Uh oh!

OakwoodTable commented Dec 29, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

proller commented Aug 18, 2022 •

edited

Loading

qwertilus commented Aug 23, 2022 •

edited

Loading