Re: [PATCH 3/3] Improve parsing and pass the devel flag to loader through the command line

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/14/2010 07:02 PM, Brian C. Lane wrote:
> On 07/14/2010 07:36 AM, Martin Sivak wrote:
>> This gets rid of ugly checking for this in /proc/cmdline in loader and init checks if devel is actually a single word to avoid conflicts
>> ---
>>  loader/init.c   |   36 +++++++++++++++++++++++++++++++++---
>>  loader/loader.c |   28 +++++++++++++---------------
>>  2 files changed, 46 insertions(+), 18 deletions(-)

> I think you could simplify this by using strstr(develstart, " devel"),
> that would eliminate the need to strip spaces and do a strncmp on every
> word of the command line.
> 
> devel is always going to have a space in front of it, so you only have
> to confirm that it is a single word.

What if I put "devel" as first option into my boot options?

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@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list



[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux