blob: 40b4ca2d4b03baaddc62ed57f405837ce015118f [file] [log] [blame]
Code Guidelines
A few code guidelines to try to stick to, please comment if none of these make
sense, they are pretty basic and mostly apply to old code. However for people
who are looking at current code, they make take up bad habits that exist in the
current codebase.
Python Version
Python 2.6 is the minimum supported version, since it is the first version to
support Python 3 syntax. All exception handling should use Python 3 'except'
syntax, and the print function should be used instead of Python 2's print
statement (from __future__ import print_function).
Python and Bash should be the only hard dependencies. Any other dependencies,
including external Python modules that are not included with Python itself,
must be optionally enabled by run-time detection.
The current code uses tabs, not spaces. Keep whitespace usage consistent
between files. New files should use tabs. Space is sometimes used for
indentation in Python code. Tab stop should for this reason be set to 4.
Lines should typically not be longer than 80 characters; if they are an attempt
should be made to wrap them. Move code to the line below and indent once (\t).
(length, max_desc_len),
Do not do this:
(length, max_desc_len),
The mixing of tabs and spaces means other developers can't read what you did.
This is why the python peps state spaces over tabs; because with spaces the line
wrapping is always clear (but you cannot convert spaces as easily as tabwidth).
if foo != None
should be replaced with:
if foo is not None:
Is not does a reference comparison (address1 = address2 basically) and
the == forces a by value compare (with __eq__())
Dict Lookups
Try not to use has_key, you can use
if foo in dict
instead of if dict.has_key(foo)
Also don't do stuff like:
if foo in dict and dict[foo]:
Generally you can do two things here, if you are messing with defaults..
dict.get(foo, some_default)
will try to retrieve foo from dict, if there is a KeyError, will insert foo
into dict with the value of some_default. This method is preferred in cases where
you are messing with defaults:
except KeyError:
dict[foo] = default_value
The get call is nicer (compact) and faster (try,except are slow).
Don't use the format raise Exception, "string"
It will be removed in py3k.
raise KeyError("No key")
raise KeyError, "No key"
Import things one per line
import os
import time
import sys
import os,sys,time
When importing from a module, you may import more than 1 thing at a time.
from portage.module import foo, bar, baz
Multiline imports are ok (for now :))
Try to group system and package imports separately.
import os
import sys
import time
from portage.locks import lockfile
from portage.versions import vercmp
import os
import portage
import portage.util
import time
import sys
Try not to import large numbers of things into the namespace of a module.
I realize this is done all over the place in current code but it really makes it
a pain to do code reflection when the namespace is cluttered with identifiers
from other modules.
from portage import output
from portage.output import bold, create_color_func, darkgreen, \
green, nocolor, red, turquoise, yellow
The YES example imports the 'output' module into the current namespace.
The negative here is having to use output.COLOR all over the place instead of
just COLOR. However it means during introspection of the current namespace
'green','red', 'yellow', etc. will not show up.
The NO example just imports a set of functions from the output module. It is
somewhat annoying because the import line needs to be modified when functions
are needed and often unused functions are left in the import line until someone
comes along with a linter to clean up (does not happen often).
First update the NEWS and RELEASE-NOTES files and commit.
Second create a git tag for this release:
git tag v2.2.8
Then create the tarball and run the tests:
./ --changelog-rev v2.2.7 --tag --runtests 2.2.8
Make sure you have all supported python versions installed first
Version bump the ebuild and verify it can re-install itself:
emerge portage
emerge portage
Publish the results (no going back now):
- Push the new git tag
- Upload the tarball
- Commit the new ebuild version
Close the bugs blocking the tracker bug for this release.