FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Gentoo > Gentoo User

 
 
LinkBack Thread Tools
 
Old 07-07-2010, 10:56 AM
Ales Kozumplik
 
Default When in devel mode, do not catch tracebacks, we want the core file

Hi Martin,

On 07/07/2010 09:16 AM, Martin Sivak wrote:

- /* unless the user specifies that they want utf8 */
+ /* check for development mode early */


Omitted from the first patch, eh?


+
+ /* Restore terminal */
+ cfd = open("/dev/console", O_RDONLY);
+ tcsetattr(cfd, TCSANOW,&orig_cmode);
+ fcntl(cfd, F_SETFL, orig_flags);
+ close(cfd);
+


Is this to kill the rubbish terminal settings left by Newt?


+ /* check for development mode early */
+ if (g_file_get_contents("/proc/cmdline",&buf, NULL, NULL) == TRUE) {
+ if (strstr(buf, "devel")) {
+ isDevelMode = 1;
+ }
+ g_free(buf);
+ }


Two things about this chunk:
1) similar thing is happening in init.c already (your first patch).
Can't you move this away into a library somewhere?
2) this will break if another kernel argument or option containing
'devel' is used. like in "inexpertlydevelopedmode".


Ales

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 07-07-2010, 12:49 PM
Steffen Maier
 
Default When in devel mode, do not catch tracebacks, we want the core file

On 07/07/2010 12:56 PM, Ales Kozumplik wrote:
> On 07/07/2010 09:16 AM, Martin Sivak wrote:

>> + /* check for development mode early */
>> + if (g_file_get_contents("/proc/cmdline",&buf, NULL, NULL) == TRUE) {
>> + if (strstr(buf, "devel")) {
>> + isDevelMode = 1;
>> + }
>> + g_free(buf);
>> + }

> 2) this will break if another kernel argument or option containing
> 'devel' is used. like in "inexpertlydevelopedmode".

Good point. We already have such an issue with loglevel=:
https://bugzilla.redhat.com/show_bug.cgi?id=603136

Even though existing cmdline parsing in loader is somewhat fuzzy,
new options should probably try to avoid such ambiguities.

Steffen

Linux on System z Development

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294


_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 02:19 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright ©2007 - 2008, www.linux-archive.org