[Bug 431515] New: terminal definition in YaST2 bootloader allows arbitrary strings
https://bugzilla.novell.com/show_bug.cgi?id=431515 Summary: terminal definition in YaST2 bootloader allows arbitrary strings Product: openSUSE 11.1 Version: Beta 1 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: YaST2 AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: fs@novell.com QAContact: jsrain@novell.com CC: duwe@novell.com Found By: Documentation Under Boot Loader Installation -> Boot Loader Options there is a free text input filed "Terminal Definition" with which you can set the GRUB parameter terminal. Since only two values are allowed for terminal (serial, console) this should not be a free text field. The default, when terminal is not specified is console. In fact it only makes sense to set terminal, if parameters for the serail console are specified. Therefore I would suggest to completely remove this configuration option from YaST and set terminal to serial automatically, id serial connection parameters are specified. If you do not like this, make it a radio button with [] console [] serial and choose console by default -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 Andreas Jaeger <aj@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bnc-team-screening@forge.provo.novell.com |yast2-maintainers@suse.de -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 Stefan Hundhammer <sh@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|yast2-maintainers@suse.de |juhliarik@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 User jreidinger@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=431515#c1 Josef Reidinger <jreidinger@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jreidinger@novell.com --- Comment #1 from Josef Reidinger <jreidinger@novell.com> 2008-10-02 08:21:10 MDT --- jozo - Yes, I find, that you ignore this value in console append create. line 2296 file routines/misc.ycp -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 User jreidinger@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=431515#c2 --- Comment #2 from Josef Reidinger <jreidinger@novell.com> 2008-10-02 08:24:06 MDT --- I find, that I send it to you in metadata, so if we decide remove it, you must remove check for non-empty terminal string and I remove this key from global metadata. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 User juhliarik@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=431515#c3 Jozef Uhliarik <juhliarik@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Normal |Enhancement Status|NEW |ASSIGNED Priority|P5 - None |P4 - Low --- Comment #3 from Jozef Uhliarik <juhliarik@novell.com> 2008-10-03 05:59:25 MDT --- Josef to your comments: # 1 I just save value from section with key "console" I don't save global settings to /etc/sysconfig/bootloader # 2 I gree but IMO it should be in next release or in new UI for bootloader I change this Priority to Enhancement -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=431515 User jreidinger@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=431515#c4 --- Comment #4 from Josef Reidinger <jreidinger@novell.com> 2008-10-06 02:11:22 MDT --- Few notes for future - radiobutton is not good, because grub support both. Also grub support as key hercules. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com