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] AutoYaST X11 test #125

Closed
wants to merge 1 commit into from
Closed

[WIP] AutoYaST X11 test #125

wants to merge 1 commit into from

Conversation

lslezak
Copy link
Member

@lslezak lslezak commented Jul 14, 2017

@kobliha
Copy link
Member

kobliha commented Jul 14, 2017

This is going to break with SLE 15 where we might use either Wayland or X11

From my POV, we should try to set the graphical target if requested, if that fails (graphical target is not available) then we should at least set text-mode, and THEN inform the user.

@kobliha
Copy link
Member

kobliha commented Jul 14, 2017

But it's "good as a hotfix"

@lslezak
Copy link
Member Author

lslezak commented Jul 14, 2017

The "xorg-x11-server" assumption is used at more places, we need to grep whole YaST for the correct Wayland support. But that's out of scope for this SP3 hot fix.

@teclator
Copy link
Contributor

So the hotfix worked but it was never merged or released .. and the created Trello Card was archived.

But no one was complaining in the bug anymore... So what we would like to do with this PR @lslezak @kobliha

@lslezak
Copy link
Member Author

lslezak commented Apr 13, 2018

I'm not sure if we want it in SLE15-GA, it's not that urgent (you can fix your AY profile to either include the X11 packages or switch to the text mode default).

I'd postpone it for SLE15-SP1 (and add a Wayland support).

@schubi2
Copy link
Member

schubi2 commented Feb 8, 2019

This PR has be replaced by
#187

@schubi2 schubi2 closed this Feb 8, 2019
@dgdavid dgdavid deleted the ay_x11_test branch July 19, 2019 14:12
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

4 participants