Index: head/en_US.ISO8859-1/articles/committers-guide/article.xml =================================================================== --- head/en_US.ISO8859-1/articles/committers-guide/article.xml +++ head/en_US.ISO8859-1/articles/committers-guide/article.xml @@ -312,7 +312,7 @@ - Protect your private key and passphrase. If either the + Protect the private key and passphrase. If either the private key or passphrase may have been compromised or disclosed, immediately notify accounts@FreeBSD.org and revoke the key. @@ -361,12 +361,12 @@ &prompt.user; kpasswd - Unless you have used the Kerberos-authenticated services - of the &os;.org cluster previously, + Unless the Kerberos-authenticated services + of the &os;.org cluster was used previously, Client unknown will be shown. This error means that the ssh kpasswd.freebsd.org method shown above - must be used first to initialize your Kerberos account. + must be used first to initialize the Kerberos account. @@ -479,7 +479,7 @@ Subversion Primer - It is assumed that you are already familiar with the basic + New committers are assumed to already be familiar with the basic operation of Subversion. If not, start by reading the Subversion Book. @@ -558,11 +558,11 @@ is the path to the repository, and head is the subdirectory within the repository. - If your &os; login name is different from your login - name on your local machine, you must either include it in + If your &os; login name is different from the login + name used on the local machine, either include it in the URL (for example svn+ssh://jarjar@repo.freebsd.org/base/head), - or add an entry to your ~/.ssh/config + or add an entry to ~/.ssh/config in the form: Host repo.freebsd.org @@ -754,7 +754,7 @@ Help SVN has built in help documentation. - It can be accessed by typing the following command: + It can be accessed by typing: &prompt.user; svn help @@ -832,8 +832,7 @@ It is possible to anonymously check out the &os; repository with Subversion. This will give access to a read-only tree that can be updated, but not committed back - to the main repository. To do this, use the following - command: + to the main repository. To do this, use: &prompt.user; svn co https://svn.FreeBSD.org/base/head /usr/src @@ -886,7 +885,7 @@ &prompt.user; svn commit lib/libfetch usr/bin/fetch There is also a commit wrapper for the ports tree to - handle the properties and sanity checking your + handle the properties and sanity checking the changes: &prompt.user; /usr/ports/Tools/scripts/psvn commit @@ -1003,7 +1002,7 @@ &prompt.user; svn diff -r179453:179454 ROADMAP.txt It can also show all changes for a specific changeset. - The following will show what changes were made to the + This command shows what changes were made to the current directory and all subdirectories in changeset 179454: @@ -1026,8 +1025,7 @@ conflict, Subversion will remember which files have conflicts and refuse to commit any changes to those files until explicitly told that the conflicts have been resolved. - The simple, not yet deprecated procedure is the - following: + The simple, not yet deprecated procedure is: &prompt.user; svn resolved foo @@ -1123,7 +1121,7 @@ ~/freebsd with ROADMAP.txt and empty subdirectories, and nothing will happen when svn update - is executed on the subdirectories. However, the following + is executed on the subdirectories. However, this command will set the depth for head (in this case) to infinity, and fully populate it: @@ -1165,12 +1163,12 @@ - Branching is very fast. The following command would be + Branching is very fast. This command would be used to branch RELENG_8: &prompt.user; svn copy svn+ssh://repo.freebsd.org/base/head svn+ssh://repo.freebsd.org/base/stable/8 - This is equivalent to the following set of commands + This is equivalent to this set of commands which take minutes and hours as opposed to seconds, depending on your network connection: @@ -1215,7 +1213,7 @@ Instead, mergeinfo is explicitly propagated down the tree. For merging something into branch/foo/bar/, - the following rules apply: + these rules apply: @@ -1297,18 +1295,25 @@ stable/10 and Newer Starting with the stable/10 - branch, all merges should be + branch, all merges are merged to and committed from the root of the - branch. All merges should look like: + branch. All merges look like: &prompt.user; svn merge -c r123456 ^/head/ checkout &prompt.user; svn commit checkout - Note that checkout should - be a complete checkout of the branch to which the merge + Note that checkout + must be a complete checkout of the branch to which the merge occurs. + + Merges to releng/ branches must + always originate from the corresponding + stable/ branch. For example: + + &prompt.user; svn merge -c r123456 ^/stable/10 releng/10.0 + Selecting the Source and Target for <literal>stable/9</literal> and Older @@ -1319,8 +1324,8 @@ a complete checkout. This procedure proved extremely error-prone, with the convenience of partial checkouts for merges significantly outweighed by the complexity of - picking mergeinfo targets. The below describes this - now-obsoleted procedure, which should be used + picking mergeinfo targets. The procedure below describes this + now-obsoleted process, which should be used only for merges prior to stable/10. @@ -1329,8 +1334,8 @@ minimise property changes on unrelated directories. The rules for selecting the merge target (the - directory that you will merge the changes to) can be - summarized as follows: + directory where the changes are being merged to) can be + summarized: @@ -1347,29 +1352,29 @@ - Changes to kernel code should be merged to + Changes to kernel code are merged to sys/. For instance, a change to - the &man.ichwd.4; driver should be merged to + the &man.ichwd.4; driver is merged to sys/, not sys/dev/ichwd/. Likewise, a - change to the TCP/IP stack should be merged to + change to the TCP/IP stack is merged to sys/, not sys/netinet/. Changes to code under etc/ - should be merged at etc/, not + is merged at etc/, not below it. Changes to vendor code (code in contrib/, - crypto/ and so on) should be + crypto/ and so on) are merged to the directory where vendor imports happen. For instance, a change to - crypto/openssl/util/ should be + crypto/openssl/util/ is merged to crypto/openssl/. This is rarely an issue, however, since changes to vendor code are usually merged wholesale. @@ -1379,8 +1384,8 @@ Changes to userland programs should as a general rule be merged to the directory that contains the Makefile for that program. For instance, a change to - usr.bin/xlint/arch/i386/ should - be merged to + usr.bin/xlint/arch/i386/ + is merged to usr.bin/xlint/. @@ -1403,7 +1408,7 @@ - Changes to manual pages should be merged to + Changes to manual pages are merged to share/man/manN/, for the appropriate value of N. @@ -1411,14 +1416,14 @@ Other changes to share/ - should be merged to the appropriate subdirectory and + are merged to the appropriate subdirectory and not to share/ directly. Changes to a top-level file in the source tree such as UPDATING or - Makefile.inc1 should be merged + Makefile.inc1 are merged directly to that file rather than to the root of the whole tree. Yes, this is an exception to the first three rules. @@ -1429,19 +1434,19 @@ - If you need to merge changes to several places at once + If a merge changes several places at once (for instance, changing a kernel interface and every userland program that uses it), merge each target separately, then commit them together. For instance, if - you merge a revision that changed a kernel + merging a revision that changed a kernel API and updated all the userland bits - that used that API, you would merge the + that used that API, merge the kernel change to sys, and the userland bits to the appropriate userland directories, then commit all of these in one go. The source will almost invariably be the same as the - target. For instance, you will always merge + target. For instance, always merge stable/7/lib/libc/ from head/lib/libc/. The only exception would be when merging changes to code that has moved in @@ -1455,10 +1460,10 @@ Preparing the Merge Target Because of the mergeinfo propagation issues described - earlier, it is very important that you never merge changes - into a sparse working copy. You must always have a full - checkout of the branch you will merge into. For instance, - when merging from HEAD to 7, you must have a full checkout + earlier, it is very important to never merge changes + into a sparse working copy. Always use a full + checkout of the branch being merge into. For instance, + when merging from HEAD to 7, use a full checkout of stable/7: &prompt.user; cd stable/7 @@ -1490,7 +1495,7 @@ The Principles - Say you would like to merge: + For example, To merge: @@ -1542,7 +1547,7 @@ Practical Example - As a practical example, consider the following + As a practical example, consider this scenario. The changes to netmap.4 in r238987 are to be merged from CURRENT to 9-STABLE. The file resides in @@ -1645,7 +1650,7 @@ Vendor patches: these are patches that have been issued by the vendor, or that have been extracted from the vendor's version control system, which address - issues which in your opinion cannot wait until the + issues which cannot wait until the next vendor release. @@ -1660,7 +1665,7 @@ - Vendor patches should be committed to the vendor + Vendor patches must be committed to the vendor branch, and merged from there to head. If the patch addresses an issue in a new release that is currently being imported, it must not be @@ -1671,7 +1676,7 @@ - &os; patches should be committed directly to + &os; patches are committed directly to head. @@ -1793,7 +1798,7 @@ Now, the sources must be copied into dist and the svn add and - svn rm commands should be used as + svn rm commands are used as needed: &prompt.user; cd vendor/pf/pf-4.3 @@ -1817,8 +1822,8 @@ *. No other properties should exist on any file in the tree. - Committing is now possible, however it is good - practice to make sure that everything is OK by using the + Committing is now possible. However, it is good + practice to make sure that everything is okay by using the svn stat and svn diff commands. @@ -1826,7 +1831,7 @@ Tagging - Once committed, vendor releases should be tagged for + Once committed, vendor releases are tagged for future reference. The best and quickest way to do this is directly in the repository: @@ -1856,8 +1861,8 @@ &prompt.user; svn merge --accept=postpone svn+ssh://repo.freebsd.org/base/vendor/pf/dist . The --accept=postpone tells - Subversion that it should not complain because merge - conflicts will be taken care of manually. + Subversion not to complain about merge + conflicts as they will be handled manually. The cvs2svn changeover occurred @@ -1901,9 +1906,9 @@ The --no-diff-deleted tells Subversion not to complain about files that are in the - vendor tree but not in the main tree, i.e., things that + vendor tree but not in the main tree. Things that would have previously been removed before the vendor - import, like for example the vendor's makefiles + import, like the vendor's makefiles and configure scripts. Using CVS, once a file was off the @@ -1992,8 +1997,8 @@ It is important to ensure that the mergeinfo - is correct when reverting a file in order to permit - svn mergeinfo --show-revs=eligible to work as + is correct when reverting a file to permit + svn mergeinfo --eligible to work as expected. @@ -2044,18 +2049,19 @@ Setting up a <application>svnsync</application> Mirror - You probably do not want to do this unless there is a - good reason for it. Such reasons might be to support many - multiple local read-only client machines, or if your network + Avoid setting up a svnsync + mirror unless there is a very good reason for it. Such + reasons might be to support + multiple local read-only client machines, or if the network bandwidth is limited. Starting a fresh mirror from empty would take a very long time. Expect a minimum of 10 hours - for high speed connectivity. If you have international - links, expect this to take 4 to 10 times longer. + for high speed connectivity. If international links are + involved, expect this to take four to ten times longer. A far better option is to grab a seed file. It is large (~1GB) but will consume less network traffic and take less - time to fetch than a svnsync will. This is possible in one - of the following three ways: + time to fetch than a svnsync will. There are several ways + to do this: &prompt.user; rsync -va --partial --progress freefall:/home/peter/svnmirror-base-r179637.tbz2 . @@ -2063,15 +2069,15 @@ &prompt.user; fetch ftp://ftp.freebsd.org/pub/FreeBSD/development/subversion/svnmirror-base-r221445.tar.xz - Once you have the file, extract it to somewhere like + Extract the file to somewhere like home/svnmirror/base/. Then, update it, so that it fetches changes since the last revision in the archive: &prompt.user; svnsync sync file:///home/svnmirror/base - You can then set that up to run from &man.cron.8;, do - checkouts locally, set up a svnserve server for your local + Now, set that up to run from &man.cron.8;, do + checkouts locally, set up a svnserve server for local machines to talk to, etc. The seed mirror is set to fetch from @@ -2114,18 +2120,18 @@ while and is finally committed back to the original branch. During development code migration in one direction (from head to the branch only). No code is committed back to head - until the end. Once you commit back at the end, the branch - is dead (although you can have a new branch with the same - name after you delete the branch if you want). + until the end. Once the branch is commited back at the end, + it is dead (although a new branch with the same name can be + created after the dead one was deleted). As per http://people.freebsd.org/~peter/svn_notes.txt, work that is intended to be merged back into HEAD should be - in base/projects/. If you are doing - work that is beneficial to the &os; community in some way + in base/projects/. If the + work is beneficial to the &os; community in some way but not intended to be merged directly back into HEAD then the proper location is - base/user/your-name/. + base/user/username/. This page contains further details. @@ -2155,10 +2161,10 @@ Some Tips - In commit logs etc., rev 179872 should be + In commit logs etc., rev 179872 is spelled r179872 as per convention. - Speeding up svn is possible by adding the following to + Speeding up svn is possible by adding these entries to ~/.ssh/config: Host * @@ -2236,7 +2242,7 @@ - All src commits should go to + All src commits go to &os.current; first before being merged to &os.stable;. The &os.stable; branch must maintain ABI and API @@ -2421,20 +2427,19 @@ taken up permanent residence on freefall may get truncated without warning if space needs to be freed, so forward it - or read it and you will not lose it. + or save it elsewhere. Due to the severe load dealing with SPAM places on the - central mail servers that do the mailing list processing + central mail servers that do the mailing list processing, the front-end server does do some basic checks and will drop some messages based on these checks. At the moment proper DNS information for the connecting host is the only check in place but that may change. Some people blame - these checks for bouncing valid email. If you want these - checks turned off for your email you can place a file - named .spam_lover in your home - directory on freefall.FreeBSD.org - to disable the checks for your email. + these checks for bouncing valid email. To have these + checks turned off for your email, create a file + named ~/.spam_lover + on freefall.FreeBSD.org. @@ -2498,8 +2503,7 @@ how commit logs are formatted. As well as including an informative message with each - commit you may need to include some additional - information. + commit, some additional information may be needed. This information consists of one or more lines containing the key word or phrase, a colon, tabs for formatting, @@ -2526,7 +2530,7 @@ username on the &os; cluster. If the submitter is the maintainer of the port - to which you are committing, include "(maintainer)" + being committed, include "(maintainer)" after the email address. Avoid obfuscating the email address of the @@ -2627,9 +2631,9 @@ Commit Log for a Commit Based on a PR - You want to commit a change based on a PR submitted by - John Smith containing a patch. The end of the commit message - should look something like this. + The commit is based on a patch from a PR submitted by John + Smith. The commit message PR and + Submitted by fields are filled.. ... @@ -2640,8 +2644,8 @@ Commit Log for a Commit Needing Review - You want to change the virtual memory system. You have - posted patches to the appropriate mailing list (in this + The virtual memory system is being changed. After + posting patches to the appropriate mailing list (in this case, freebsd-arch) and the changes have been approved. @@ -2653,8 +2657,8 @@ Commit Log for a Commit Needing Approval - You want to commit a port. You have collaborated with - the listed MAINTAINER, who has told you to go ahead and + Commit a port, after working with + the listed MAINTAINER, who said to go ahead and commit. ... @@ -2669,7 +2673,7 @@ Commit Log for a Commit Bringing in Code from OpenBSD - You want to commit some code based on work done in the + Commiting some code based on work done in the OpenBSD project. ... @@ -2681,7 +2685,7 @@ Commit Log for a Change to &os.current; with a Planned Commit to &os.stable; to Follow at a Later Date. - You want to commit some code which will be merged from + Committing some code which will be merged from &os.current; into the &os.stable; branch after two weeks. @@ -2697,15 +2701,15 @@ month, months. - In many cases you may need to combine some of these. + It is often necessary to combine these. Consider the situation where a user has submitted a PR - containing code from the NetBSD project. You are looking at the - PR, but it is not an area of the tree you normally work in, so - you have decided to get the change reviewed by the + containing code from the NetBSD project. Looking at the PR, the + developer sees it is not an area of the tree they normally work + in, so they have the change reviewed by the arch mailing list. Since the change is - complex, you opt to MFC after one month to - allow adequate testing. + complex, the developer opts to MFC after one + month to allow adequate testing. The extra information to include in the commit would look something like @@ -2725,7 +2729,7 @@ Preferred License for New Files - Currently the &os; Project suggests and uses the following + The &os; Project suggests and uses this text as the preferred license scheme: /*- @@ -2854,27 +2858,27 @@ Developer Relations - If you are working directly on your own code or on code + When working directly on your own code or on code which is already well established as your responsibility, then there is probably little need to check with other committers - before jumping in with a commit. If you see a bug in an area of + before jumping in with a commit. Working on a bug in an area of the system which is clearly orphaned (and there are a few such - areas, to our shame), the same applies. If, however, you are - about to modify something which is clearly being actively + areas, to our shame), the same applies. Trying + to modify something which is clearly being actively maintained by someone else (and it is only by watching the repository-committers - mailing list that you can really get a feel for just what is and + mailing list that a developer can really get a feel for just what is and is not) then consider sending the change to them instead, just - as you would have before becoming a committer. For ports, you - should contact the listed MAINTAINER in the + as a developer would have before becoming a committer. For ports, + contact the listed MAINTAINER in the Makefile. For other parts of the - repository, if you are unsure who the active maintainer might - be, it may help to scan the revision history to see who has + repository, if it s not clear who the active maintainer + is, it may help to scan the revision history to see who has committed changes in the past. An example script that lists each person who has committed to a given file along with the number of commits each person has made can be found at on freefall at - ~eadler/bin/whodid. If your queries go + ~eadler/bin/whodid. If queries go unanswered or the committer otherwise indicates a lack of interest in the area affected, go ahead and commit it. @@ -2884,20 +2888,20 @@ output. - If you are unsure about a commit for any reason at all, have + If there is any doubt about a commit for any reason at all, have it reviewed by -hackers before committing. Better to have it flamed then and there rather than when it is - part of the repository. If you do happen to commit something - which results in controversy erupting, you may also wish to + part of the repository. If a commit does + results in controversy erupting, it may be advisable to consider backing the change out again until the matter is - settled. Remember – with a version control system we can + settled. Remember, with a version control system we can always change it back. - Do not impugn the intentions of someone you disagree with. - If they see a different solution to a problem than you, or even - a different problem, it is not because they are stupid, because + Do not impugn the intentions of others. + If they see a different solution to a problem, or even + a different problem, it is probably not because they are stupid, because they have questionable parentage, or because they are trying to - destroy your hard work, personal image, or &os;, but simply + destroy hard work, personal image, or &os;, but basically because they have a different outlook on the world. Different is good. @@ -2921,7 +2925,7 @@ If in Doubt... - When you are not sure about something, whether it be a + When unsure about something, whether it be a technical issue or a project convention be sure to ask. If you stay silent you will never make progress. @@ -2930,7 +2934,7 @@ person that knows the answer. This way everyone will be able to learn from the question and the answer. - For project specific or administrative questions you should + For project specific or administrative questions ask, in order: @@ -2943,7 +2947,7 @@ - Any team with a "hat", as they should give you a + Any team with a "hat", as they can give you a definitive answer. @@ -3096,8 +3100,8 @@ &a.svn-src-all.name;, &a.svn-ports-all.name; and &a.svn-doc-all.name; are the mailing lists that the version control system uses to send commit messages to. - You should never send email directly - to these lists. You should only send replies to this list + Never send email directly + to these lists. Only send replies to this list when they are short and are directly related to a commit. @@ -3113,7 +3117,7 @@ voting, announcements, etc. The &a.developers; is for the exclusive use of &os; - committers. In order to develop &os;, committers must + committers. To develop &os;, committers must have the ability to openly discuss matters that will be resolved before they are publicly announced. Frank discussions of work in progress are not suitable for open @@ -3187,14 +3191,13 @@ - Now you should be able to use &man.ssh-add.1; for - authentication once per session. This will prompt you for - your private key's pass phrase, and then store it in your - authentication agent (&man.ssh-agent.1;). If you no longer - wish to have your key stored in the agent, issuing - ssh-add -d will remove it. + Now &man.ssh-add.1; can be used for + authentication once per session. It prompts for + the private key's pass phrase, and then stores it in the + authentication agent (&man.ssh-agent.1;). Use ssh-add + -d to remove keys stored in the agent. - Test by doing something such as ssh + Test with a simple remote command: ssh freefall.FreeBSD.org ls /usr. For more information, see @@ -3312,7 +3315,7 @@ src/crypto, or src/sys/contrib trees without explicit approval from the respective - maintainer(s). + maintainers. @@ -3382,7 +3385,7 @@ are angry or otherwise behave in a manner which is likely to strike others as needlessly confrontational. First calm down, then think about how to communicate in the most - effective fashion for convincing the other person(s) that + effective fashion for convincing the other persons that your side of the argument is correct, do not just blow off some steam so you can feel better in the short term at the cost of a long-term flame war. Not only is this very bad @@ -3427,21 +3430,21 @@ Discuss any significant change before committing. - The repository is not where changes should be + The repository is not where changes are initially submitted for correctness or argued over, that - should happen first in the mailing lists or by use of the - Phabricator service and the commit should only happen once + happens first in the mailing lists or by use of the + Phabricator service. The commit will only happen once something resembling consensus has been reached. This - does not mean that you have to ask permission before + does not mean that permission is required before correcting every obvious syntax error or manual page - misspelling, simply that you should try to develop a feel + misspelling, just that it is good to develop a feel for when a proposed change is not quite such a no-brainer and requires some feedback first. People really do not mind sweeping changes if the result is something clearly better than what they had before, they just do not like being surprised by those changes. - The very best way of making sure that you are on the right - track is to have your code reviewed by one or more other + The very best way of making sure that things are on the right + track is to have code reviewed by one or more other committers. When in doubt, ask for review! @@ -3464,7 +3467,7 @@ maintainer need to be reviewed by at least one other maintainer. In cases where the maintainer-ship of something is not clear, - you can also look at the repository logs for the file(s) + look at the repository logs for the files in question and see if someone has been working recently or predominantly in that area. @@ -3546,15 +3549,15 @@ self-control, tempers are lost and angry words are exchanged. The best thing that can be done in such cases is to minimize the effects of this until everyone has - cooled back down. That means that you should not air your - angry words in public and you should not forward private + cooled back down. Do not air + angry words in public and do not forward private correspondence or other private communications to public mailing lists, mail aliases, instant messaging channels or social media sites. What people say one-to-one is often much less sugar-coated than what they would say in public, and such communications therefore have no place there - they only serve to inflame an already bad situation. If - the person sending you a flame-o-gram at least had the + the person sending a flame-o-gram at least had the grace to send it privately, then have the grace to keep it private yourself. If you feel you are being unfairly treated by another developer, and it is causing you @@ -3628,7 +3631,7 @@ src/crypto, and src/sys/contrib trees without explicit approval from the respective - maintainer(s). + maintainers. The trees mentioned above are for contributed software usually imported onto a vendor branch. Committing @@ -3640,9 +3643,10 @@ (or you are the maintainer), do not commit there! + Please note that this does not mean you should not try to improve the software in question; you are still more - than welcome to do so. Ideally, you should submit your + than welcome to do so. Ideally, submit your patches to the vendor. If your changes are &os;-specific, talk to the maintainer; they may be willing to apply them locally. But whatever you do, do @@ -3661,7 +3665,7 @@ recent release cycles and is truly no longer an &i386; centric operating system. In an effort to make it easier to keep &os; portable across the platforms we support, core has - developed the following mandate: + developed this mandate:
Our 32-bit reference platform is &arch.i386;, and our @@ -3724,7 +3728,7 @@ Deprecating Features When it is necessary to remove functionality from software - in the base system the following guidelines should be followed + in the base system, follow these guidelines whenever possible: @@ -3782,7 +3786,7 @@ Where there are security problems that affect many different operating systems, &os; frequently depends on - early access in order to be able to prepare advisories for + early access to be able to prepare advisories for coordinated release. Unless &os; developers can be trusted to maintain security, such early access will not be made available. The Security Officer is responsible @@ -3790,7 +3794,7 @@ vulnerabilities, and for timing the release of all advisories. He may request help under condition of confidentiality from any developer with relevant knowledge - in order to prepare security fixes. + to prepare security fixes. @@ -3817,7 +3821,7 @@ - Private communications should not be made + Private communications must not be made public without permission. Beyond the specific requirements above there is a @@ -3831,13 +3835,13 @@ Communications on project-only or restricted access - channels should be treated as private. + channels must be kept private. Similarly to personal communications, certain internal communications channels, including &os; Committer only mailing lists and restricted access IRC channels - should be considered as private communications. You need - permission in order to publish material from these + are considered private communications. Permission is + required to publish material from these sources. @@ -3993,7 +3997,7 @@ handbook. The basics for how to get a system running must be documented, although not necessarily for every single board or system a Tier 2 architecture supports. The supported hardware - list must exist and should be relatively recent. It should be + list must exist and be relatively recent. It should be integrated into the &os; documentation. Current Tier 2 platforms are &arch.arm;, &arch.arm64;, @@ -4010,9 +4014,9 @@ Tier 3 platforms are architectures in the early stages of development, for non-mainstream hardware platforms, or which are considered legacy systems unlikely to see broad future - use. Initial support for Tier 3 platforms should be worked on + use. Initial support for Tier 3 platforms is worked on in external SCM repositories. - The transition to &os;'s subversion should take place after + The transition to &os;'s subversion takes place after the platform boots multi-user on hardware; sharing via subversion is needed for wider exposure; and multiple developers are actively working on the platform. @@ -4096,12 +4100,12 @@ sequence: &prompt.root; make install -&prompt.root; make package -&prompt.root; make deinstall -&prompt.root; pkg add package you built above -&prompt.root; make deinstall -&prompt.root; make reinstall -&prompt.root; make package + &prompt.root; make package + &prompt.root; make deinstall + &prompt.root; pkg add package you built above + &prompt.root; make deinstall + &prompt.root; make reinstall + &prompt.root; make package The Porters @@ -4122,7 +4126,7 @@ Close the PR if the port came in as a PR. To close a PR, change the state to Issue - Resolved and the resolution as + Resolved and the resolution as Fixed. @@ -4148,12 +4152,12 @@ - The port's PKGNAME should appear in exactly + The port's PKGNAME appears in exactly one line in a recent INDEX file. - No other ports should contain any reference + No other ports contains any reference to the port's directory or PKGNAME in their Makefiles @@ -4320,8 +4324,8 @@ (English-only and internationalized version). Another example is when a port is moved from one - subdirectory to another, or when you want to change the - name of a directory because the author(s) renamed their + subdirectory to another, or when the name of a directory + must be changed because the authors renamed their software even though it is a descendant of a port already in a tree. @@ -4368,8 +4372,8 @@ PORTNAME instead of adding PKGNAMEPREFIX or PKGNAMESUFFIX, but this - should only be done when it is really needed - — e.g., using an existing port as the base + is only done when it is really needed + — for example, using an existing port as the base for a very similar program with a different name, or upgrading a port to a new upstream version which actually changes the distribution @@ -4378,8 +4382,8 @@ textproc/libxml2. In most cases, adding or changing PKGNAMEPREFIX or - PKGNAMESUFFIX should - suffice. + PKGNAMESUFFIX + suffices. @@ -4639,7 +4643,7 @@ Handbook. Once that procedure has been followed and the PR has been assigned to the &a.portmgr;, it is their decision whether or not to approve it. If they do, it - is their responsibility to do the following: + is their responsibility to: @@ -4680,7 +4684,7 @@ Change the port's CATEGORIES (this was the point of the exercise, remember?) The new - category should be listed + category is listed first. This will help to ensure that the PKGORIGIN is correct. @@ -4716,7 +4720,7 @@ &man.portupgrade.1; fail. To do this, use the - chkorigin.sh tool, as follows: + chkorigin.sh tool: env PORTSDIR=/path/to/ports sh -e @@ -4760,8 +4764,7 @@ - Update the documentation by modifying the - following: + Update the documentation by modifying: @@ -4810,8 +4813,8 @@ - This is much simpler than a physical category. You - only need to modify the following: + This is much simpler than a physical category. Only + a few modifications are needed: @@ -4890,9 +4893,9 @@ - When the checksum for a port's distfile is updated + When the checksum for a distribution file is updated due to the author updating the file without changing the - port's revision, the commit message should include a + port's revision, the commit message includes a summary of the relevant diffs between the original and new distfile to ensure that the distfile has not been corrupted or maliciously altered. If the current @@ -4904,7 +4907,6 @@ - How can an experimental test build of the ports tree @@ -4992,7 +4994,7 @@ linkend="conventions-everyone">Conventions - You should get your mentor to add you to the + Get your mentor to add you to the Additional Contributors (doc/en_US.ISO8859-1/articles/contributors/contrib.additional.xml), if you are not already listed there. @@ -5023,7 +5025,7 @@ As of December 12, 2012, &ga; was enabled on the &os; Project website to collect anonymized usage statistics regarding usage of the site. The information collected is - valuable to the &os; Documentation Project, in order to + valuable to the &os; Documentation Project, to identify various problems on the &os; website.