Page 3 of 3

Vegastrike Engine GitHub Cleanup

Posted: Sun Feb 09, 2020 11:12 am
by pyramid
Thought that would be smoother.
It seems that the taose for was created in 2017, however during merging of code I noticed that some changes from 2013 (e.g. svn r13606) were not available in the taose build.
This means that I may be reverting back to the original github branch and discarding the taose imported and merged one, at least as soon as I can bring the original master to build on my system.

Re: Vegastrike Engine GitHub Cleanup

Posted: Mon Feb 10, 2020 3:48 pm
by pyramid
Cleanup and merge from svn are now completed and the code compiles on modern systems (kernel 5.x, g++ 9.x).

Re: Vegastrike move to GitHub - Proposal

Posted: Wed Feb 12, 2020 4:45 pm
by crome
Hi,

first of all, thanks for merging my PR. I have noticed, however, that I am no longer showing up in the master-branch as contributer for my PR. The changes in the affected file are all still there but the file itself show "0 Contributors". I am not demanding that you build me an altar and sacrifice a goat in my honour but I would appreciate it if my contributions are attributed to me :-). Besides, if the project is to be resurrected and gain a user base then bug/feature reports may also start coming in from 3rd parties. In that case it would be advantageous to know who is already familiar with which code section and assign tasks accordingly.

PS: stephentuggy's contributions also do not show up in the master-branch, not sure if his changes also persisted.

Re: Vegastrike move to GitHub - Proposal

Posted: Mon Feb 17, 2020 3:14 pm
by pyramid
Ouch, that was me.
Your commit is still there, just in a branch I created before merging the missing svn changes https://github.com/vegastrike/Vega-Stri ... .5.2-alpha. Dunno why they were not included when I merged the code again into master (since I was temporarily using taose's code as master). Something must have gone wrong. stephentuggy contributions do not compile under c++11, so may need to be reworked again. And klauss mentioned that the bug was already fixed by him (didn't test though) so when you feel your changes are still necessary after merging his code from svn, I could add you as contributor.