On 04/21/09 01:50 AM, jan damborsky wrote:
> Hi Steven,
>
>
> On 04/21/09 03:23, Steven Lau wrote:
>> Hi Jan,
>>
>> On live-fs-root :
>>
>> line 280:  you might need a space at the end inside the double quote 
>> string.
>
> Actually, it is not needed, since echo(1) adds space automatically to 
> separate
> printed arguments.
>
> Thanks for looking at this !
> Jan
Hi Jan,

Actually ... based on your original code ... you did need add a space 
because you put your "\" immediately after the strings.   But I realized 
that you already put a space before "\" on you new webrev, which could 
also fix the problem (because you echo "xxx" and a space) .... I am not 
sure echo(1) will adds space 'automatically' ....  Just for you info.

Thx,

-Steven

 280                         echo "Sparc wanboot configuration file is not"\
 281                             "available" > /dev/msglog
-----
 281                         echo "Sparc wanboot configuration file is not" \
 282                             "available" > /dev/msglog


>
>>
>> Thx,
>>
>> -Steven
>>
>>
>>
>> On 04/17/09 09:16 AM, jan damborsky wrote:
>>> Hi,
>>>
>>> could I please ask two people for reviewing changes for following 
>>> blocker ?
>>>
>>> 8130 mismatch between boot_archive and solaris*.zlib if more than 
>>> one service is created for Sparc
>>>
>>> The fix also addresses
>>>
>>> 8115 SPARC AI shouldn't use root-path(Option 17) for locating 
>>> solaris*.zlib archives
>>>
>>> Sundar, could I please ask you, if you might confirm/deny
>>> if it is fine to get rid of using 'Rootpath' option by
>>> Sparc AI client for locating solaris*.zlib archives ?
>>> Thank you. Just in case you check emails - Monday is fine ;-)
>>>
>>> webrev:
>>> http://cr.opensolaris.org/~dambi/bug-8130
>>>
>>> Thank you very much,
>>> Jan
>>>
>>>
>>> modules affected:
>>> -----------------
>>> * DC (Sparc AI manifest)
>>> * installadm tools
>>> * AI image (live-fs-root)
>>>
>>> testing done (full test procedures attached):
>>> ---------------------------------------------
>>> [1] AI image containing fix built using modified DC
>>>    - verified that microroot contains /usr/bin/mkdir
>>>
>>> [2] combinations of 'create-service' & 'create-client'
>>>    tested using installadm tools with fix (please see
>>>    attached test results for more details).
>>>
>>> [3] new AI image booted successfully without 'Rootpath'
>>>    option provided by DHCP server.
>>>
>>> known issues:
>>> -------------
>>> * fix will introduce incompatible change in Sparc AI image
>>>  and installadm tools - old Sparc AI images will no longer
>>>  work with new installadm tools, since they rely on Rootpath
>>>  DHCP option. x86 is not affected.
>>>
>>> solution:
>>>  Flag day will be sent out.
>>>
>>> ------------------------------------------------------------------------
>>>
>>> _______________________________________________
>>> caiman-discuss mailing list
>>> caiman-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://mail.opensolaris.org/pipermail/caiman-discuss/attachments/20090421/7b9d5698/attachment.html>

Reply via email to