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