Mercurial: Difference between revisions
(→Mercurial configuration: Add missing color "gray". Minor changes of spelling and whitespace.) |
m (→Submitting patches: spelling error) |
||
Line 111: | Line 111: | ||
Patch submissions is done via the [https://savannah.gnu.org/bugs/?group=octave bug] | Patch submissions is done via the [https://savannah.gnu.org/bugs/?group=octave bug] | ||
or [https://savannah.gnu.org/patch/?group=octave patch] trackers. Either | or [https://savannah.gnu.org/patch/?group=octave patch] trackers. Either | ||
way, you can submit via two different methods: pull requests | way, you can submit via two different methods: pull requests or attaching | ||
a changeset file. | a changeset file. | ||
Revision as of 05:29, 10 May 2020
Mercurial (sometimes referred to as hg
)
is the source code management system currently used to develop
Octave.
Introduction to Mercurial
An introduction to Mercurial is completely outside the scope of this document. There are plenty of available documentation on the topic. Some recommendations are:
Contributing to Octave
The preferred method to contribute to Octave is with Mercurial changesets. Other forms of contributions (e.g., simple diff patches) are also acceptable, but they slow down the review process.
If you plan on contributing to Octave:
- See other Contribution guidelines
- Always include commit messages in changesets. Please follow the Octave commit message guidelines
- Follow the style guides for both Octave and C++ languages.
Getting the development sources
To clone the Octave repository:
hg clone http://www.octave.org/hg/octave octave
Octave packages like image, signal, control, etc. are not parts of Octave itself, they belong to the Octave Forge. Each package has its own repository, a list of which can be found here. You can clone them in a similar way, for example, to clone the signal package
hg clone http://hg.code.sf.net/p/octave/signal octave-signal
Mercurial configuration
You can use the following to start your hgrc
[ui] username = Your Name <your@email> [extensions] color = histedit = pager = rebase = strip = [pager] pager = LESS='FSRX' less attend = help, annotate, cat, diff, export, glog, log, outgoing, incoming [diff] showfunc = True [color] mode = terminfo ## Custom colors color.gray = 244 color.orange = 202 color.lightyellow = 191 color.darkorange = 220 color.brightyellow = 226 status.modified = magenta bold status.added = green bold status.removed = red bold status.deleted = cyan bold status.unknown = gray bold status.ignored = gray bold ## Colours for each label log.branch = cyan log.summary = lightyellow log.description = lightyellow log.bookmark = green log.tag = darkorange log.graph = blue ## Colors for each phase changeset.secret = blue bold changeset.draft = red bold changeset.public = orange desc.here = bold blue_background [bookmarks] track.current = True [alias] glog = log --graph top = log --graph -l
Submitting patches
When you do not have push permissions to the repository (you cannot add your changes using mercurial itself) and you have a modification to the current GNU Octave code, you have to generate a patch (or changeset) so developers with permissions can include them in the code. The overview of the process is as follows
- Change the code and test that your changes do work (write tests, that's the best!).
- Create the changeset (instructions below).
- Post your patch in the Patch tracker.
Patch submissions is done via the bug
or patch trackers. Either
way, you can submit via two different methods: pull requests or attaching
a changeset file.
Pull request
This is the cleaner way. You push your clone to public site, and ask on the bug tracker to pull a specific changeset from it. It makes special sense if you plan to send more patches in the future but requires to host the clone somewhere. Free mercurial repositories are available on bitbucket
Creating changesets files with hg
Simple way
- Update to the latest revision.
hg up
- Make your changes and save them.
- Commit your code. Mercurial will ask you for a commit message, which should follow the commit message guidelines.
hg ci
- Export the modifications. This creates a file (in the case of the example below, it is called mypatch.patch) that contains a description of the changes that you've made. Someone else can then apply the patch and end up with a repository that looks the same as yours.
hg export -r tip -o mypatch.patch
- Save the output to a file and upload it to the patch tracker. If your patch file is larger than the upload limit, you can compress it before uploading. Please use a free format!
Mercurial Tips for SoC students
This section is meant to provide tips for SOCIS or GSoC students working on new Octave features.
Students should publish their work as it progresses in a public repository merging regularly the main savannah repository to facilitate merging back their code at the end of the project.
Here are some useful hg commands that can be used to do this.
Getting started
- Clone the main Octave repository at savannah:
hg clone http://www.octave.org/hg/octave
- Create a new bookmark:
hg bookmark student-bookmark-name
- Make the bookmark visible in the public repo, assuming the public repo is at
public.server.org/octave
hg push --bookmark ssh://student@public.server.org/octave
Staying up-to-date with the main savannah repository
As the students development proceeds,
the savannah repository gets updated, too.
To avoid having the two branches diverging too much, which can
lead to conflicts when the final merge is done, students should
keep their public repo up-to-date with the recent changes,
the following commands can be used for this:
- Download new changes from the main line of development
hg pull http://www.octave.org/hg/octave
- Merge the main line of development into the feature branch
hg up -r student-bookmark-name
hg merge @
- Apply the change and publish it
hg commit -m "periodic merge of default branch into my branch"
hg push ssh://student@public.server.org/octave
Preparing for code reviews
At the time of the mid-term or final review (or whenever the mentor requires it) students should prepare their code for review and possibly inclusion into the main development branch. To this end students should:
- prepare a full log of their changes, listing files that have been touched
and including a summary of the purpose of those changes. If students have been following
the Commit message guidelines the following command will give a good starting point
hg log --style=changelog --no-merges --user student-name
this message should be edited so that- each touched file appears only once
- changes that were backed out should not be mentioned
The main purpose of this log is to make it easy, not only for the main mentor, but also for other developers who have not been closely following the progress of the project to quickly understand where to look at in the code to evaluate it, but it will also be used as the commit message for the merge changeset, so it should itself comply with the Commit message guidelines.
- prepare a merge changeset including all the code that should be submitted for review
- pull from the main repository
hg pull http://www.octave.org/hg/octave
- move to the top of the main line of development and merge in the feature branch
hg up -r @
hg merge student-bookmark-name
- create a changeset, export it and send to the mentor for review, remember to use the log created above as a commit message
hg commit
hg export @ > mid-term-review.changeset
the file mid-term-review.changeset can then be sent to the mailing list or posted to the patch tracker
- pull from the main repository
Mercurial Tips for SoC mentors
Will fill in this section after trying out the above procedure at least once with a student and actually pushing his changes to the main repo.