Skip to content

Sync controller mappings DB with SDL community repo #107962

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

Merged
merged 1 commit into from
Jun 25, 2025

Conversation

geowarin
Copy link
Contributor

@geowarin geowarin commented Jun 25, 2025

Synced with: mdqinc/SDL_GameControllerDB@1849728

After #106218 was merged, I was surprised that my DualSense Edge controller still wouldn't work properly under linux.

Turns out updating gamecontrollerdb.txt fixes the problem.

It looks like it wasn't updated for a while.

I'm not very familiar with this, shouldn't this be updated regularly?
Maybe updating gamecontrollerdb.txt can be included in the script that updates the SDL code?

@geowarin geowarin requested a review from a team as a code owner June 25, 2025 03:15
@AThousandShips AThousandShips changed the title update gamecontrollerdb.txt Update gamecontrollerdb.txt Jun 25, 2025
@AThousandShips AThousandShips added this to the 4.x milestone Jun 25, 2025
@geowarin
Copy link
Contributor Author

geowarin commented Jun 25, 2025

I think an argument can be made that this is a bugfix.

Personally, I'd like this to be included in 4.5, I think it goes hand in hand with the inclusion of SDL v3.
It also makes sense to include this for testing at the same time.

IMHO, people are going to expect outstanding issues with controllers fixed by this move and may be disappointed if that's not the case.

It may also add support for newer controllers from 8bitdo, Nintendo (switch 2), Hori, etc.

@AThousandShips
Copy link
Member

Previous updates to this have been marked as enhancements, simply following that pattern

@akien-mga
Copy link
Member

Could you update the commit message to clarify which commit of SDL_GameControllerDB you're syncing with? See how it's done usually: #91108.

@AThousandShips AThousandShips modified the milestones: 4.x, 4.5 Jun 25, 2025
@geowarin geowarin force-pushed the update-controller-db branch from 2f24962 to 9b455aa Compare June 25, 2025 10:40
@geowarin geowarin force-pushed the update-controller-db branch from 9b455aa to 88c086d Compare June 25, 2025 10:42
@AThousandShips AThousandShips changed the title Update gamecontrollerdb.txt Sync controller mappings DB with SDL community repo Jun 25, 2025
@Repiteo Repiteo merged commit 3071b98 into godotengine:master Jun 25, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Jun 25, 2025

Thanks!

@geowarin geowarin deleted the update-controller-db branch June 25, 2025 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants