[Development] Coin news

Tor Arne Vestbø tor.arne.vestbo at qt.io
Thu Oct 27 12:32:36 CEST 2016


On 25/10/2016 09:22, Jedrzej Nowacki wrote:
> I would like to also remind everyone, that intermodule API changes
> need to be merged in two steps. For example if you plan to change a
> method "void foo()" in qtbase to "void foo(const QString&)", that
> would be the right order of execution: 1. Commit a change to qtbase
> that _adds_  foo(const QString&) 2. Update all modules that use foo()
> to use foo(const QString&) 3. Wait until Qt5(dev branch) submodule
> update, that gathers changes from 1 and 2 4. Remove old foo() from
> qtbase If you do not follow we would have to revert your changes.

What happened to pinned sync.profile %dependencies? That seems like a 
much smoother workflow:

  1. pin qtbase sha in dependent module
  2. update qtbase, changing method in one go
  3. update dependent module to use new api, along with an unpin qtbase sha

This doesn't split the logical change in qtbase into two just to keep 
the world building.

tor arne



More information about the Development mailing list