ValveSoftware/gamescope

Make KMS "rotation" prop optional for non-libliftoff codepath

Open

#252 ouverte le 3 sept. 2021

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C++ (355 forks)github user discovery
embeddedgood first issue

Métriques du dépôt

Stars
 (4 809 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Commit https://github.com/Plagman/gamescope/commit/8782dc2cd110ef169e7d5abf5e40c94763bf7a05 made the KMS "rotation" property mandatory. However some drivers don't support it: https://drmdb.emersion.fr/properties/4008636142/rotation

libliftoff can deal with this already: https://github.com/emersion/libliftoff/blob/bc0d0617acdd48e69272771237cad2684c07c901/plane.c#L245

However the "basic" codepath doesn't have this fallback.

  • If the "rotation" prop exists, set it.
  • If gamescope is rotated but the "rotation" prop doesn't exist, error out.
  • If gamescope is not rotated and the "rotation" prop doesn't exist, ignore.

Guide contributeur

Direction de recherche
Consultez le commit 8782dc2cd110ef169e7d5abf5e40c94763bf7a05 qui a rendu la propriété de rotation obligatoire. Examinez le fallback dans libliftoff dans plane.c ligne 245. Implémentez ensuite une logique conditionnelle similaire dans le chemin non libliftoff : si la propriété de rotation existe, définissez la ; si gamescope est tourné et que la propriété n'existe pas, générez une erreur ; sinon, ignorez.
Stack technique
cpp
Domaine
computer graphics
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65