Difference between revisions of "Source branches"

From Second Life Wiki
Jump to navigation Jump to search
(→‎Flow of changes between branches: Linden Lab Repository Strategy)
(→‎Branches mirroring (snapshots of) Linden Lab's internal SVN: linden/branches/2010 stuff (exported Viewer 2 sources))
Line 13: Line 13:
** '''feature branch''' a major new feature gets developed here before turning into various first looks (snapshots of this branch), or getting eventually merged into "trunk" and no longer maintained as a feature branch.
** '''feature branch''' a major new feature gets developed here before turning into various first looks (snapshots of this branch), or getting eventually merged into "trunk" and no longer maintained as a feature branch.
**  '''Numbered branch''' (currently [http://svn.secondlife.com/svn/linden/branches/viewer_1-23 linden/branches/'''viewer_1-23''']) - this is a branch of "trunk" where RC snapshots come from, and where final releases are checked in.  Small fixes can be checked in here if they fix regressions.
**  '''Numbered branch''' (currently [http://svn.secondlife.com/svn/linden/branches/viewer_1-23 linden/branches/'''viewer_1-23''']) - this is a branch of "trunk" where RC snapshots come from, and where final releases are checked in.  Small fixes can be checked in here if they fix regressions.
** [http://svn.secondlife.com/svn/linden/branches/2010/ linden/branches/'''2010'''] exports of Viewer 2 sources from internal [[Mercurial]] repositories. See [[Linden Lab Repository Strategy]].
*** [http://svn.secondlife.com/svn/linden/branches/2010/viewer-external linden/branches/2010/'''viewer-external'''] The internal "viewer-public" mercurial repository gets exported to this branch. The export process removes files that aren't freely distributable and re-brands the sources from "Second Life" to "Snowglobe" to avoid undeliberate trade mark breaches. Though, as Snowglobe-specific modifications (features, bug-fixes) are missing, these are '''not''' the official Snowglobe viewer sources. This branch will become the upstream of the Snowglobe 2 branches (see [[#Snowglobe 2 series|below]]).
*** '''Numbered tag''' (e.g. [http://svn.secondlife.com/svn/linden/branches/2010/viewer_2-0_beta4 linden/branches/2010/'''viewer_2-0_beta4''']) - export of sources corresponding to a specific binary download.


=== Branches for development together with open source community and/or external partners ===
=== Branches for development together with open source community and/or external partners ===

Revision as of 04:25, 1 April 2010

Current repository structure

KBnote.png Note: The current state of the repository should be temporary, so these explanations might be outdated by the time you view them. They are based on the situation as of 2009-04-25 (2010-04-01 for the Snowglobe related branches)

At the moment, Linden Lab's only public SVN repository is http://svn.secondlife.com/svn/linden/. (More might be added in the future, if they aren't superseded by the initiative to move things to Mercurial by then.) Some parts of this repository mirror Linden Lab's internal SVN repository structure, while others are dedicated to open source development and cooperation with external partners.

Viewer source branches

Branches mirroring (snapshots of) Linden Lab's internal SVN

  • linden/trunk ("release" in earlier revisions) - the main branch of development. This is often referred to as "head", "default" or "main" in other SCMs. Linden Lab is rather conservative about what gets checked into trunk, though, preferring that development work mainly happens on branches (either a dedicated feature branch, or one of the "maintenance" branches for the small stuff, and only gets merged into trunk once QA has happened). Developers aspire to keep "trunk" compatible with the agni (main) grid. All branches occasionally pick up changes from trunk, and are occasionally merged back into trunk.
  • linden/branches/*There are several major branches at all times, along with many temporary development branches:
    • linden/branches/maint-* - bug fixes and small features that don't warrant their own branch get checked into the matching maint-* branch. Changes from maintenance are periodically merged up to release after QA has tested a build from the branch. Many source code contributions from the community land in this branch first. There are actually several maintenance branches internally, UI/Server/Viewer/Rendering. The public ones are:
    • feature branch a major new feature gets developed here before turning into various first looks (snapshots of this branch), or getting eventually merged into "trunk" and no longer maintained as a feature branch.
    • Numbered branch (currently linden/branches/viewer_1-23) - this is a branch of "trunk" where RC snapshots come from, and where final releases are checked in. Small fixes can be checked in here if they fix regressions.
    • linden/branches/2010 exports of Viewer 2 sources from internal Mercurial repositories. See Linden Lab Repository Strategy.
      • linden/branches/2010/viewer-external The internal "viewer-public" mercurial repository gets exported to this branch. The export process removes files that aren't freely distributable and re-brands the sources from "Second Life" to "Snowglobe" to avoid undeliberate trade mark breaches. Though, as Snowglobe-specific modifications (features, bug-fixes) are missing, these are not the official Snowglobe viewer sources. This branch will become the upstream of the Snowglobe 2 branches (see below).
      • Numbered tag (e.g. linden/branches/2010/viewer_2-0_beta4) - export of sources corresponding to a specific binary download.

Branches for development together with open source community and/or external partners

Branches in the sandbox and projects areas won't be overwritten by automated exports from Linden Lab's internal SVN repository.

Snowglobe project

The Snowglobe project produces an open source Second Life-compatible viewer, built jointly by the open source community and Linden Lab.

Snowglobe 1 series
Snowglobe 2 series

Non-Viewer branches

  • linden/lindenlib/* home of 3rd party libraries for which Linden Lab has become either the primary maintainer or the primary source. At the moment only libndofdev (Win/Mac, originally from 3DConnexion)

Non-branch stuff in the repository


KBcaution.png Important: The following sections are probably out of date and should be updated by someone who knows the details of the current state.

Flow of changes between branches

KBtip2.png Tip: For the flow between Linden Lab's internal Mercurial repositories and the public Subversion branches, see Linden Lab Repository Strategy

The public version control repository provided by Linden Lab has many of the same branches that Linden Lab's internal repository has.

http://svn.secondlife.com/trac/linden/browser/branches

Branches.png

Linden Lab does feature development on temporary branches. As those features are validated by the QA group, they are merged into the "release" branch. Three kinds of releases are possible:

  • Server-only - may occur with or without downtime
  • Viewer-only - optional viewers
  • Viewer and Server (with dependencies) - required viewer updates. This case is avoided whenever possible.

Release Candidates

When a release is about to happen, a release candidate (RC) branch is created for work on that release (e.g. "Branch_1-13-2"). In the rare case of a client/server update with dependencies, pending changes which would break compatibility with the main grid and require downtime and new viewers for an update are merged into this release candidate branch.

Following this point, only stability work is done on the branch - developers fix any new bugs above a certain priority that are found by QA.

For server-side changes, the beta grid is updated frequently as the stabilization occurs. Once the code is finalized, it is deployed to the main grid. After that, any stability work in the RC branch gets merged back into "release", and the process repeats for the next major release. Additional patches may be made to the RC branch and deployed to the grid.

For viewer-side changes, once the code is deemed stable it is released as Release Candidate viewer. Any bug reports which are new to this version are triaged. As fixes are made, additional RC viewer builds may be made and released. Once no more bugs have passed triage, the final RC becomes the next released viewer.


The intended process is:

  1. developers commit changes that have passed QA into the code trunk
  2. at appropriate times we freeze (branch off) from the trunk to start stabilizing a version and call it an RC
  3. we stabilize the RC branch via iterations of internal QA, fixes, public releases, fixes; meanwhile (1) continues unblocked
  4. when deemed "good enough" then the RC is released.


We triage changes going into an RC. When the RC is first frozen, the bar is roughly: regression fixes, bug fix for new features, fit-and-finish on new features, internationalization/localization changes, security, performance, text-only UI changes. We try and raise the bar with each RC iteration.

The purpose is to get a "we think it's ready" version to the public for use on the service. Some times we're closer to being ready out of the gate than others. Apart from changes that meet the triage bar, the feature set should be frozen so that as an RC is iterated on the risk is constantly being reduced.

There are humans in the triage loop, and yes, we may say "you know what, that change is low risk and high impact, and we'd prefer to have it out in the RC and next release, and not wait another month for the next go-round of the cycle".

Basically, almost all development work happens in maintenance or feature branches. Those merge into the trunk (named "release", a source of potential confusion). Periodically an RC is branched off the trunk. So something can be marked "fixed internally" on JIRA when it's in a maintenance branch but, until it merges into the trunk it's not going to be in an RC.

This is best shown with a picture. It is of course overly simplified. The flow is left to right. This diagram *just* shows how a fix that goes in while 1.18.6 RCx is out may still not make it into 1.19.0 and is not a complete picture at all. The point marked 'fix foo submitted' is actually the point it will often get marked as 'fixed internally' or 'fix pending', and when it starts going through internal QA.

Merging.jpg

There are always exceptions. Some work is done right in the RC branch to fix regressions. Some work doesn't affect production code so can go right into the trunk. But the goal is to have nothing enter the trunk that hasn't passed QA.

We internally track this by tagging the items with an actual Fix Version only at the point where the change has merged into the trunk - which is the soonest point you can predict what version it will be in. We do this by tracking the maintenance and feature branches as they go in and doing bulk updates. Therefore, Status = Resolved, Resolution = Fixed, Fix Version = 1.19.0 indicates that not only is it fixed but it's merged into the trunk. We'd create a Version 1.19.1 as soon as 1.19.0 is frozen (branched) so that the next fixes to merge into the trunk can be marked appropriately.

Ideally we would sweep through the JIRAs that have PJIRA counterparts and do the same thing. We're still trying to get tools/processes into place for this, and additional people to help. There's also the overhead that suddenly 1.19.1 appears in PJIRA before we're even ready to really start talking in detail about 1.19.0, so suddenly a gazillion questions come our way. But that's why this is fun.

Maintenance

"Maintenance" is now subdivided into four branches in order to distribute and organize maintenance responsibilities between development teams. (new as of August 2007)

Maintenance Branches

  • maint-ui
    • UI specific maintenance tasks
  • maint-render
    • Render specific maintenance tasks
  • maint-viewer
    • Other viewer only maintenance tasks
    • e.g. viewer crashes, audio, misc
  • maintenance
    • Tasks that involve server-side changes
    • Tasks that affect shared viewer / server libraries
    • Tasks that in any way affect messaging