Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ogre2.1 branch is long time update #219

Open
36yang opened this issue Apr 7, 2021 · 13 comments
Open

Ogre2.1 branch is long time update #219

36yang opened this issue Apr 7, 2021 · 13 comments

Comments

@36yang
Copy link

36yang commented Apr 7, 2021

Can you update the ogre2.1 branch

@36yang
Copy link
Author

36yang commented Apr 7, 2021

There are too many errors compile with Ogre2.2。 need support

@Altren
Copy link
Contributor

Altren commented Apr 7, 2021

I'm not familiar with Ogre2, so the only was I can help you is by telling what was the branch purpose. Someone implemented ogre2 render system and created a branch for it, but since it never was finished it haven't got into master, thus it is outdated now. The plan was to add Ogre2 render system in master, and keep it as well as Ogre1.
May be @al2950 , @PieroV or scrawl (hm, scrawl user name is gone) can help you with that.

@al2950
Copy link

al2950 commented Apr 7, 2021

Hi

The issue is Ogre 2+ has changed significantly between minor releases making it very difficult to create a single plugin that supports 2.0, 2.1, 2.2 & 2.3, and so this branch has gone a bit stale. Ogre 2+ has become more stable in the last year or so, so it might be worth revisiting this and creating a new branch and starting again making it support Ogre 2.2++. I have a my own version of mygui that compiles with Ogre 2.2 but I would not say its fully functional.

@Altren
Copy link
Contributor

Altren commented Apr 7, 2021

Would it be OK to support only latest Ogre2 version? Having more that one Ogre2 render system sounds crazy and adds too much hussle to maintain that.
Or if API is not changing too rapidly may be we can have one version with a few #if OGRE_VERSION checks, that makes it work with all 2.2+ versions.

@sercero
Copy link
Contributor

sercero commented Apr 8, 2021

Also, it might make sense to have Ogre2 as an additional render system instead of a separate branch.

@al2950
Copy link

al2950 commented Apr 8, 2021

Would it be OK to support only latest Ogre2 version? Having more that one Ogre2 render system sounds crazy and adds too much hussle to maintain that.
Or if API is not changing too rapidly may be we can have one version with a few #if OGRE_VERSION checks, that makes it work with all 2.2+ versions.

Yes... But please note I started to write code to support all Ogre 2.x versions with #if OGRE_VERSION but it got too messy, so it would have to just be 2.2+

Also, it might make sense to have Ogre2 as an additional render system instead of a separate branch.

This branch does actually create an additional render system

@36yang
Copy link
Author

36yang commented Apr 9, 2021

Yes, Only Support the lasted 2.2+ is OK, I can wait for it.

@cryham
Copy link

cryham commented Feb 18, 2022

It would be great to have official working MyGUI with latest Ogre 2.2+.

@cryham
Copy link

cryham commented Feb 26, 2022

I'm trying to get MyGUI working with Ogre 2.3 basing on latest fork from xrgo for Ogre 2.2
https://github.com/yoyTeam/mygui/commits/ogre2.1
Here is the Ogre topic, and my latest runtime error:
https://forums.ogre3d.org/viewtopic.php?p=552319#p552319
Edit: It works now.

@cryham
Copy link

cryham commented Mar 18, 2023

Just for info, I have a working fork of MyGUI for latest Ogre-Next 3.0:
https://github.com/cryham/mygui-next/tree/ogre3
(Windows CMake not fixed yet)
Ogre topic with details here: https://forums.ogre3d.org/viewtopic.php?t=96874

@Altren
Copy link
Contributor

Altren commented Mar 21, 2023

I'm willing to accept MR, that would be compatible with master. I recommend rebasing to it, because there is at least one major feature, related to render systems (custom shaders support), that also require some extra code in your branch. Ogre 2 branch was very old.

@cryham
Copy link

cryham commented Aug 10, 2023

Yeah, after this commit e19a011 it will be rather impossible to upmerge changes (to any forks).
I think the only way would be to apply fork changes (like our for Ogre-Next) again to the new master of MyGui. Probably means more work. (I still am very far from considering doing it, busy with other stuff).

@Apteryks
Copy link
Contributor

Apteryks commented Oct 29, 2024

@cryham hello! I'm currently packaging Stunt Rally 3 for GNU Guix, and having a single MyGUI package would be nice. In case it's fallen into cracks, I'd like to encourage you to consider upstreaming your changes again... it should mean less work down the road.

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

No branches or pull requests

6 participants