OpenJazzNX icon indicating copy to clipboard operation
OpenJazzNX copied to clipboard

Merging with mainline OpenJazz?

Open vanfanel opened this issue 5 years ago • 11 comments

Hi, @KranKRival !

I also tried to port OpenJazz to SDL2 but ultimately failed. You seem to have succeeded and it OpenJazz really needs SDL2 support. Have you though about merging with upstream OpenJazz in https://github.com/AlisterT/openjazz ?

Maybe you are up to fixing the remaining bugs before merging?

Thanks!

vanfanel avatar Oct 17 '19 13:10 vanfanel

well 1st of all yes i need to fix all the bugs that left , later compile backport it to linux and see how all perform and than i could send the pr , as this project now compiles for the nintendo switch only

KranKRival avatar Oct 17 '19 14:10 KranKRival

thank you ill keep the issue open until farder changes

KranKRival avatar Oct 17 '19 14:10 KranKRival

alt text

KranKRival avatar Nov 03 '19 22:11 KranKRival

i have back ported it to linux with sdl2 fully working :)

KranKRival avatar Nov 03 '19 22:11 KranKRival

@KranKRival This is what OpenJazz needed AT LAST!! :) Will you please merge it upstream to https://github.com/AlisterT/openjazz ?

vanfanel avatar Nov 04 '19 16:11 vanfanel

i will send pr after ill finish final tests

KranKRival avatar Nov 04 '19 18:11 KranKRival

sorry to relive that dead thread, but is anyone still planning to do that ? It would be great to have it mainlined since OpenJazz original repo is still active.

rtissera avatar Nov 08 '21 18:11 rtissera

I have also remembered this today. SDL2 support in OpenJazz is much needed, yes.

vanfanel avatar Nov 08 '21 21:11 vanfanel

Depends on maybe n repo I did my job and ported it over

On Mon, Nov 8, 2021, 23:06 Manuel Alfayate Corchete < @.***> wrote:

I have also remembered this today. SDL2 support in OpenJazz is much needed, yes.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/KranKRival/OpenJazzNX/issues/1#issuecomment-963572703, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNC3EN4CBKON6OXU2WDTLLULA3U7ANCNFSM4JBZPRZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

KranKRival avatar Nov 09 '21 12:11 KranKRival

Depends on maybe n repo I did my job and ported it over On Mon, Nov 8, 2021, 23:06 Manuel Alfayate Corchete < @.***> wrote: I have also remembered this today. SDL2 support in OpenJazz is much needed, yes. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNC3EN4CBKON6OXU2WDTLLULA3U7ANCNFSM4JBZPRZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Did you do a Pull Request so it gets merged?

vanfanel avatar Nov 09 '21 12:11 vanfanel

No, I didn't. Its not cleaned and to do so some one need to clean the job I'm not going too I remember main stream wanted to do so I sadly have no time to afford to do so

On Tue, Nov 9, 2021, 14:19 Manuel Alfayate Corchete < @.***> wrote:

Depends on maybe n repo I did my job and ported it over … <#m_4265999660068929411_> On Mon, Nov 8, 2021, 23:06 Manuel Alfayate Corchete < @.***> wrote: I have also remembered this today. SDL2 support in OpenJazz is much needed, yes. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment) https://github.com/KranKRival/OpenJazzNX/issues/1#issuecomment-963572703>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNC3EN4CBKON6OXU2WDTLLULA3U7ANCNFSM4JBZPRZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

Did you do a Pull Request so it gets merged?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/KranKRival/OpenJazzNX/issues/1#issuecomment-964100286, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNC3EJRMMK6ECBAJEBJYVDULEGTVANCNFSM4JBZPRZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

KranKRival avatar Nov 09 '21 12:11 KranKRival