Re: [PATCH general-docs] update DevGuide for Mercurial
Mats Lidell <[email protected]> Sat, 18 Jul 2015 01:07:20 +0200
| Newsgroups | gmane.emacs.xemacs.patches |
|---|---|
| Organization | The XEmacs Project |
| Message-ID | <[email protected]> |
>>>>> Mike Kupfer <[email protected]> writes: Now my comments on the other points. > - I left in some references to CVS, especially in text that talks about > email addresses and the release process, since I wasn't sure about the > current process. In the patch you have kept cvs-manager but on our web page we refer to MercurialAdministrator which links or [email protected]! (This refers to access rights for the xemacs repository. Isn't that what the cvs mail address was about. Access rights?) But you mention release process. Do I need to read the dev guide to find the cvs parts you have kept? (reasonable I guess ;-) > - I left in the @cindex tags that refer to CVS. Should those just be > changed from "CVS" to "Mercurial"? Yes I think so. > - There was some email back in May about CNAMEs and Bitbucket--are any > changes needed to the DevGuide to deal with that? I don't think so (but I did not understand the practical implications of that CNAME issue to be honest.) > - I didn't change the process rule in Section 5.10 that says "(t)he > log message should refer unambiguously to the 'COMMIT' message, > preferably via the RFC 2822 message ID". This is something we don't > seem to be doing, so should that text remain? No. I think you should remove it. It is good if the dev guide tries to be close to what we actually are doing. Yours -- %% Mats