Skip to content

Add libogc2 fork as an feature for ogc-rs#140

Open
FrictionlessPortals wants to merge 6 commits into
masterfrom
feat/libogc2
Open

Add libogc2 fork as an feature for ogc-rs#140
FrictionlessPortals wants to merge 6 commits into
masterfrom
feat/libogc2

Conversation

@FrictionlessPortals

@FrictionlessPortals FrictionlessPortals commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

This PR aims to add the libogc fork libogc2 as a feature for the ogc-rs library. This aims to resolve #136.

Open to other ways to allow this feature inclusion, at the moment, I have done this through splitting libogc into its own separate feature, and making it default.

@ProfElements

Copy link
Copy Markdown
Collaborator

I don't see any major issues with this. I am little worried since features aren't supposed to take away any functionality. I haven't looked at libogc2 enough to see if they change or remove functions. It will be a real headache if that is the case.

@FrictionlessPortals

Copy link
Copy Markdown
Contributor Author

I agree, the only missing function was SYS_Time, however I think this could be replaced easily. Everything else that has an API change is mainly type changes on parameters.

@FrictionlessPortals FrictionlessPortals added the enhancement New feature or request label Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

libogc2 feature?

2 participants