Skip to content

Conversation

@gucio321
Copy link
Collaborator

@gucio321 gucio321 commented Dec 1, 2022

like @eliasdaler suggested, thirdparty/glfw should be a submodule like cimgui and cimplot

Upstream: https://github.com/glfw/glfw
Version/Revision: (current) master

fix #39

@gucio321
Copy link
Collaborator Author

gucio321 commented Dec 1, 2022

strange... It seems that cimgui/cimplot submodules has been changed in my commits, but cant imagine why 😄
git-submodules seems to be a bit tricky...

@eliasdaler
Copy link
Contributor

Try going inside git submodule dir and checking out to the same commits as in master now.
Also let’s move all submodules to the thirdparty dir for consistency.

@gucio321
Copy link
Collaborator Author

gucio321 commented Dec 2, 2022

oh, thanx, it works!

@eliasdaler
Copy link
Contributor

Looks good. Now the only thing left is to move cimgui and cimplot to "thirdparty".

@gucio321
Copy link
Collaborator Author

gucio321 commented Dec 2, 2022

Well @eliasdaler but i'd do that in a separate pr since it's not related to glfw and needs a bit more changes in justfile and workflows

@eliasdaler
Copy link
Contributor

eliasdaler commented Dec 2, 2022

Okay, let's do it another PR. Merging.
Thanks!

@eliasdaler eliasdaler merged commit cc697fa into AllenDang:main Dec 2, 2022
gucio321 added a commit to gucio321/cimgui-go that referenced this pull request Dec 12, 2022
eliasdaler pushed a commit that referenced this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make glfw a git submodule too

2 participants