Mike Sullivan wrote: > 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 :) I wrote to your primary address as well, but the message returned as undelivered for some reason. > > 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. That's what I'm thinking too, but Paul says "install-sfw" is better :-)
- Denis > > 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 >> >
