Denis Migounov wrote:
> I have no problem staying with install-sfw, I'm just not sure if the 
> sccs id will get corrected, or what
> the consequences will be if it does not.
> 
> 2Mike: The previously ported version had the file install-cvs, I used wx 
> mv to rename it to install-sfw
> but it keeps showing "install-cvs" in the "ident" line. Will this be 
> corrected once it gets into the gate?

first, I actually delete almost all 'code review request' mails to this
alias so asking me questions inside them is not a real reliable way
to get an answer :)

but the answer is "I don't know". If it retains the keywords yes.
If it doesn't, then no - and it may not if it still thinks its
name is install-cvs. wx nits or keywords would complain if it doesn't
but perhaps if you go 'sccs get' it again it will show the new name.
Though personally I'd not bother renaming it in the first place.

        Mike


> 
> - Denis
> 
> Paul Cunningham wrote:
>>
>>
>> Denis Migounov wrote:
>>> Nope, doesn't help. I guess I'll have to rename it back to 
>>> install-cvs, I can't
>>> think of a way to make it work for install-sfw.
>>
>> personally I think you should keep it with filename install-sfw 
>> (ignoring the -cvs on the sccs id - it might correct its once it gets 
>> into the gate)
>>
>> paul
>>>
>>> - Denis
>>>
>>> Paul Cunningham wrote:
>>>>
>>>>
>>>> Denis Migounov wrote:
>>>>> Paul,
>>>>>
>>>>> I renamed install-cvs to install-sfw (to conform with the latest 
>>>>> SFW standards)
>>>>> with wx mv. Was it a wrong way to do it? How do I go about fixing it?
>>>>
>>>> My guess is that the sccs ident line has the wrong thing on it when 
>>>> its checked out. I don't know how teamware handles the move and the 
>>>> sccs ident stuff.
>>>>
>>>> I guess you can check-it-out again - check the raw sccs ident stuff 
>>>> is correct and then check-it-back-in again.
>>>>
>>>> paul
>>>>
>>>>> Thanks.
>>>>> - Denis
>>>>>
>>>>> Paul Cunningham wrote:
>>>>>> Denis,
>>>>>>
>>>>>> This looks good, but I just noticed ...
>>>>>>
>>>>>> in usr/src/cmd/cvs/install-sfw
>>>>>>   27 # ident "@(#)install-cvs        1.6     09/01/30 SMI"
>>>>>> install-cvs??? the file is install-sfw so is there something wrong 
>>>>>> with the sccs ident line in the checkedout file?
>>>>>>
>>>>>> Paul
>>>>>>
>>>>>> Denis Migounov wrote:
>>>>>>
>>>>>>>
>>>>>>> The new webrev is at: 
>>>>>>> http://cr.opensolaris.org/~dm223115/cvs.webrev/
>>>>>>>
>>>>>>
>>>>>>>
>>>>>>> P.S. Should I start a new thread to make it clear that this is 
>>>>>>> for version 1.11.23?
>>>>>>
>>>>>> I personally don't think it matters
>>>>>>
>>>>
>>
> _______________________________________________
> sfwnv-discuss mailing list
> sfwnv-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss
> 


Reply via email to