applications/mhwd
3
0
Fork 0
Code Issues 2 Pull requests Projects Releases 3 Packages Wiki Activity Actions

mhwd: gpu: Don't spam warnings if wayland is detected #119

Closed
philm wants to merge 0 commits from unknown repository into master
pull from: unknown repository
merge into: applications:master
Conversation 3 Commits - Files changed -
philm commented 2017-12-18 03:15:30 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Created by: CertifiedBlyndGuy

We shouldn't care about xorg configuration files if that compositor isn't even in use. We should just alert the user that wayland support is experimental at the time given that nvidia has yet to support wayland properly

*Created by: CertifiedBlyndGuy* We shouldn't care about xorg configuration files if that compositor isn't even in use. We should just alert the user that wayland support is experimental at the time given that nvidia has yet to support wayland properly
philm commented 2017-12-18 03:25:07 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Created by: CertifiedBlyndGuy

Late night :)

*Created by: CertifiedBlyndGuy* Late night :)
philm commented 2017-12-20 20:55:59 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Created by: Tids

I dont think we can check just for /usr/share/wayland, because thats part of the wayland package and a dependency of gtk, mesa. So even users who will never use a wayland compositor could see that messag?

I dont thinks its even a wayland related thing. we simply dont ship xorg config files anymore for intel, radeon, amdgpu, nouveau because it's better to do not. So maybe we could just remove this whole part?

*Created by: Tids* I dont think we can check just for /usr/share/wayland, because thats part of the wayland package and a dependency of gtk, mesa. So even users who will never use a wayland compositor could see that messag? I dont thinks its even a wayland related thing. we simply dont ship xorg config files anymore for intel, radeon, amdgpu, nouveau because it's better to do not. So maybe we could just remove this whole part?
philm commented 2017-12-20 22:04:18 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Created by: CertifiedBlyndGuy

@Tids Sounds reasonable to me. I'll push a commit here tonight addressing this. mhwd still expects those files to be there for some reason so I'll kill the old code. Thanks for the heads up. I appreciate it

*Created by: CertifiedBlyndGuy* @Tids Sounds reasonable to me. I'll push a commit here tonight addressing this. mhwd still expects those files to be there for some reason so I'll kill the old code. Thanks for the heads up. I appreciate it

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
invalid

   
question

   
wontfix

   
wrong bugtracker
No labels
bug
duplicate
enhancement
invalid
question
wontfix
wrong bugtracker
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
applications/mhwd!119
No description provided.