[Development] Suggestion to add labels when changing API

Adam Treat adam.treat at qt.io
Fri Dec 8 20:46:28 CET 2017



On 12/08/2017 09:20 AM, Konstantin Tokarev wrote:
>
> 08.12.2017, 17:14, "Tor Arne Vestbø" <tor.arne.vestbo at qt.io>:
>> On 08/12/2017 14:54, Sergio Ahumada wrote:
>>>   an old concept that used to pin the sha1 of repo/module's dependency, eg
>>>
>>>   http://code.qt.io/cgit/qt/qtdeclarative.git/commit/sync.profile?id=ab6b6b7c7ab544d347d59b7eefad403837d94012
>>>
>>>   that was replaced in, eg,
>>>   http://code.qt.io/cgit/qt/qtdeclarative.git/commit/sync.profile?id=9bfe3324f7fa94e1f272c35bcb943daa2669edba
>>>
>>>   for
>>>
>>>   http://code.qt.io/cgit/qt/qt5.git/tree/.gitmodules
>> Ouch, what was the rationale for making this change?
> IIRC, it was done to have all things in one place. Integration is strongly affected by used qt5 commit, which determines provisioning configuration and platform configs, so it's logical to use its submodule references as well.

Again, this all boils down to the commits across submodules are not 
atomic. The very least that needs to be done is to make it atomic from 
the CI perspective if not atomic to all those doing git fetch and rebuild.



More information about the Development mailing list