QUIP 5 | Choosing a Branch

QUIP:5
Title:Choosing a Branch
Version:24fd5bd9046562fd75e45675b8476cd00c4df264
Last-Modified:2020-05-15
Author:Jędrzej Nowacki, Friedemann Kleint
Status:Superseded
Type:Process
Created:2016-12-05
Post-History:https://lists.qt-project.org/pipermail/development/2017-January/028248.html
Superseded-By:QUIP 16

Choosing a Branch

Motivation

This QUIP offers guidelines on the right branch to submit a change to. It originated from the QtCS2016 Managing Qt Branches session.

Note 1: The QUIP applies to the normal course of development. Close to feature-freezes, branching and releases, the release team may impose tighter control.

Note 2: The guidelines apply to the Qt frameworks only. Other products (like Qt Creator) may choose a different approach.

Goal of the release branches (5.X.Y)

  • Preparation of the code base for a release undisturbed by activities in the stable branches

Goal of the long term support (LTS) branches (5.6 at the time of writing)

  • Support for 3 years after its initial release, extending the stable phase
  • Release roughly every 6 Months or when appropriate
  • Support slower moving OSes
  • Fix bugs in applications that were already shipped bundling the LTS and continue to build against the LTS
  • Have a low regression risk
  • Add new minor platforms - if and only if changes are minimal, like win8 => win8.1
  • Add support for newer toolchains - if and only if changes are small, like adding MSVC 2015
  • Documentation clarifications

We divide the LTS into 3 periods:

stable
Initially, the LTS branch is handled like any stable branch.
strict
This period starts when the one after next stable branch is created (for the 5.9 LTS, the one after next is 5.11). Submitting to the LTS branch directly is then no longer possible; changes must be submitted to the stable branch and undergo some testing first before being cherry-picked into the LTS branch. Upgrading the Qt libraries (within LTS) must not put installed applications at risk of breaking. Breaking existing bug workarounds is therefore not allowed, even if the bug is fixed.
very strict
(third year) We believe that there are no more P1 and P0 bugs.

Goal of the stable branches (5.x)

  • Bug fixes
  • Performance fixes
  • Documentation fixes
  • As a special case, pre-release stabilization immediately after branching off the dev branch

Goal of the dev branch:

  • New feature development (adding new API, classes, libraries or modules)
  • Refactoring
  • Risky changes, requiring longer testing period (moc, qmake...)
  • Completely new platforms (WinRT)

Approach

The table below lists typical code changes, and indicates the branch the change should be pushed to. In all cases, we need to estimate the impact of the change. For example, a fix for a performance problem may end up in a different branch depending on: how many users are affected, how big is the improvement, and what is the risk of regressions. The table shows the most stable branch (i.e. right-most), for every case. If a change is risky or it has lower impact, then it should be placed in a less stable (i.e. further to the left) branch.

Reason for Change Dev Stable LTS Release Notes
Strict Very Strict
Binary compatibility break         x  
Source compatibility break         x  
API review finding         x Initial release only
RTA/package test finding         x  
Security         x  
3rd party update       x   Very Strict only security/critical
Test improvement (reduce flakiness)       x    
Adapt to minor OS/ Compiler updates       x    
Memory leak     x      
Fix to feature added in LTS     x      
Undefined behavior     x      
Something that makes existing apps unstable     x      
Performance: significant fix improving O()     x      
Performance: Issue detected by profiling   x        
Performance: Issue related to best practices (like usage of reserve()) x          
Fix compiler warnings     x      
Suppression of false positives raised by tools   x       e.g. static code analysers
Documentation: small fixes (links, typos, warnings)       x    
Documentation: refactoring   x        
Bugs: Easily triggered crash       x    
Bugs: Hard to reproduce crash     x      
Bugs: Regression     x      
Bugs: Hard-to-reproduce regression noticed some time after release   x        
Bugs: Other   x        
New modules x          
New features x          
Major build system changes x          
Refactoring: tests   x        
Refactoring: non-test code x          
Refactoring: build system x          
Enabling new compiler features x          
Using new C++ features x         This includes mass updates

History

The dev / stable / release pattern originated in a discussion during initial development of Qt 5: Branching for Qt 5 repositories

Glossary

RTA
Release testing automation