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

[WIP] Update SharpDX dependency to version 4.2.0 #277

Closed
wants to merge 1 commit into from
Closed

[WIP] Update SharpDX dependency to version 4.2.0 #277

wants to merge 1 commit into from

Conversation

Venthe
Copy link

@Venthe Venthe commented Nov 24, 2018

Update to latest SharpDX #34

It is not yet tested., give me a few days. Pull request made to signal that someone is working :)

@CLAassistant
Copy link

CLAassistant commented Nov 24, 2018

CLA assistant check
All committers have signed the CLA.

@xen2 xen2 changed the title Update SharpDX dependency to version 4.2.0 [WIP] Update SharpDX dependency to version 4.2.0 Nov 24, 2018
@xen2
Copy link
Member

xen2 commented Nov 24, 2018

Thanks! Added WIP in title to reflect current state.

@tebjan
Copy link
Member

tebjan commented Nov 24, 2018

@Venthe do you need a specific feature of sharpdx 4.2 or do you know if a bug of the current version that influences xenko? if you update to 4.2 it is forcing all other projects that work with xenko to update as well. i would appreciate to not have to do that just now. current version is well tested and works at the momemt and i don't know the implications of the upgrade and whether it causes problems or new bugs...

EDIT: @Venthe didn't want to sound negative towards the pull request, glad to see other devs enjoyng the engine. it was just critical thinking. if you have a tested update, go ahead. if it doesn't work on our end we can downgrade.

@Venthe
Copy link
Author

Venthe commented Nov 27, 2018

@tebjan no, it's okay. I had a spare time; so I'll probably confirm it's working state & will wait for the new version for the merge

@xen2
Copy link
Member

xen2 commented Jan 18, 2019

@tebjan Would it be OK to upgrade now for 3.1? Or you think better to keep version as low as possible?

@tebjan
Copy link
Member

tebjan commented Jan 19, 2019

if the upgrade doesn't break anything, i think it's safe to go for it. it might well be the last SharpDX version since alexandre droppend support for it. so everyone will end up at this version sooner or later.

for us it would be a bit more easy to stay on the current version since we have third party dependencies that also use that one. but i can do a PR there as well to move up to 4.2. so go ahead, we will manage...

@dfkeenan
Copy link
Contributor

It looks as if SharpDX has been updated to 4.2.0. @xen2 should this PR request be closed?

@xen2
Copy link
Member

xen2 commented Oct 20, 2019

Yes, thanks !
Updated to SharpDX 4.2.0 in latest 3.1 beta

@xen2 xen2 closed this Oct 20, 2019
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.

None yet

5 participants