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

Add the boot option inst.xtimeout #1241

Merged
merged 1 commit into from
Nov 28, 2017

Conversation

poncovka
Copy link
Contributor

Added a new boot option inst.xtimeout to set the timeout in seconds
for starting X.

(cherry-picked from a commit 08afa5c)

@poncovka poncovka added the master Please, use the `f39` label instead. label Nov 22, 2017
@poncovka
Copy link
Contributor Author

Resolves the issue https://github.com/rhinstaller/anaconda/issues/1237

Copy link
Member

@jkonecny12 jkonecny12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one nitpick. Looks good to me otherwise.

@@ -107,6 +107,9 @@ the password needs to be 6 to 8 characters long (limitation of the VNC protocol)
xdriver
Use DRIVER as the X driver to use during installation as well as on the installed system.

xtimeout
Specify the timeout in seconds for starting X.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add X server to be more precise.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

inst.xtimeout
^^^^^^^^^^^^^

Specify the timeout in seconds for starting X.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@@ -48,7 +48,7 @@
from gi.repository import GLib

from pyanaconda.flags import flags
from pyanaconda.constants import DRACUT_SHUTDOWN_EJECT, TRANSLATIONS_UPDATE_DIR, UNSUPPORTED_HW, IPMI_ABORTED
from pyanaconda.constants import DRACUT_SHUTDOWN_EJECT, TRANSLATIONS_UPDATE_DIR, UNSUPPORTED_HW, IPMI_ABORTED, X_TIMEOUT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One last small thing, could you please wrap this line to 99 characters before merge. Thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@jkonecny12
Copy link
Member

It looks even better now :).

Added a new boot option inst.xtimeout to set the timeout in seconds
for starting X server.

(cherry-picked from a commit 08afa5c)
@poncovka poncovka merged commit 1695311 into rhinstaller:master Nov 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master Please, use the `f39` label instead.
2 participants