Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Sergio Ahumada
On 08.12.2017 20:52, Adam Treat wrote: As another alternative I'm trying to figure out if Gitlab's CI system has a way to deal with this: https://forum.gitlab.com/t/trigger-ci-build-if-any-dependent-git-repo-is-updated/9725 They do have ways to kick off integration jobs based on git trig

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Konstantin Tokarev
08.12.2017, 22:53, "Adam Treat" : > On 12/08/2017 12:47 PM, Sergio Ahumada wrote: >>  On 08.12.2017 16:50, Oswald Buddenhagen wrote: >>>  On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote:  On 08.12.2017 15:42, Adam Treat wrote: >  Relying upon qt5 submodule pins is the prob

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Adam Treat
On 12/08/2017 12:47 PM, Sergio Ahumada wrote: On 08.12.2017 16:50, Oswald Buddenhagen wrote: On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote: On 08.12.2017 15:42, Adam Treat wrote: Relying upon qt5 submodule pins is the problem. The underlying issue is atomicity of commits. O

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Adam Treat
On 12/08/2017 09:20 AM, Konstantin Tokarev wrote: 08.12.2017, 17:14, "Tor Arne Vestbø" : 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=ab6b6b7c7ab544

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Sergio Ahumada
On 08.12.2017 16:50, Oswald Buddenhagen wrote: On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote: On 08.12.2017 15:42, Adam Treat wrote: Relying upon qt5 submodule pins is the problem. The underlying issue is atomicity of commits. Oswald is right. We need to have a way to provide

Re: [Development] Qbs 1.10 released

2017-12-08 Thread NIkolai Marchenko
can you please clarify this passage with a link to an example? > It is now possible to add file tags to generated artifacts by setting the new fileTags property in a group that has a fileTagsFilter. This allows you to “append” to the list of file tags assigned to artifacts created by a rule that i

Re: [Development] Qbs 1.10 released

2017-12-08 Thread Jake Petroules
Hi everyone! Following our successful Qt, Qt Creator, and Qt 3D Studio releases yesterday, we have released qbs 1.10 today. Find all the relevant information in the blog post at http://blog.qt.io/blog/2017/12/07/qbs-1-10-released/ > On Dec 7, 2017, at 3:29 AM, Jani Heikkinen wrote: > > Hi eve

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Konstantin Tokarev
08.12.2017, 18:50, "Oswald Buddenhagen" : > On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote: >>  On 08.12.2017 15:42, Adam Treat wrote: >>  > Relying upon qt5 submodule pins is the problem. The underlying issue is >>  > atomicity of commits. Oswald is right. >>  > >>  > We need to

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Oswald Buddenhagen
On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote: > On 08.12.2017 15:42, Adam Treat wrote: > > Relying upon qt5 submodule pins is the problem. The underlying issue is > > atomicity of commits. Oswald is right. > > > > We need to have a way to provide atomic commits across modules at

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Sergio Ahumada
On 08.12.2017 15:42, Adam Treat wrote: Relying upon qt5 submodule pins is the problem. The underlying issue is atomicity of commits. Oswald is right. We need to have a way to provide atomic commits across modules at least the CI should see these as atomic and integrate accordingly. what ab

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Adam Treat
Relying upon qt5 submodule pins is the problem. The underlying issue is atomicity of commits. Oswald is right. We need to have a way to provide atomic commits across modules at least the CI should see these as atomic and integrate accordingly. On 12/08/2017 09:20 AM, Konstantin Tokarev wrote:

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Konstantin Tokarev
08.12.2017, 17:14, "Tor Arne Vestbø" : > 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 r

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Tor Arne Vestbø
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.g

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Sergio Ahumada
On 08.12.2017 14:23, Edward Welbourne wrote: On 07/12/2017 17:22, Liang Qi wrote: The changes that are important to be merged up before other changes should have a special tag, such as API_CHANGE in the commit message. Then the script used to do the merges could stop and/or warn about commits wi

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Edward Welbourne
On 07/12/2017 17:22, Liang Qi wrote: >> The changes that are important to be merged up before other changes >> should have a special tag, such as API_CHANGE in the commit >> message. Then the script used to do the merges could stop and/or warn >> about commits with this tag in the message, which wo

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Tor Arne Vestbø
On 07/12/2017 17:22, Liang Qi wrote: The changes that are important to be merged up before other changes should have a special tag, such as API_CHANGE in the commit message. Then the script used to do the merges could stop and/or warn about commits with this tag in the message, which would mak

Re: [Development] Suggestion to add labels when changing API

2017-12-08 Thread Oswald Buddenhagen
On Thu, Dec 07, 2017 at 10:28:36PM +, Liang Qi wrote: > I really don’t think switch back to a monolithic repo is a good idea… > > But if I understand atomic commits across submodules as successful > integrations in qt5, I think it has some meaning to me. Here are a few tasks > which I think

Re: [Development] QML and Qt Quick versioning of our modules

2017-12-08 Thread Shawn Rutledge
> On 7 Dec 2017, at 16:42, Robin Burchell wrote: > > On Thu, Dec 7, 2017, at 02:53 PM, Frederik Gladhorn wrote: > >> 2) Make the minor version import optional and we pick the lastest. This >> should >> be optional to prevent the name clashes described above and shifts the >> risk to >> the us

[Development] Provisioning problems with macOS currently

2017-12-08 Thread Tony Sarajärvi
Hi You might have noticed provisionings failing with "" messages etc. Further investigations will show you that command line tools are needed in macOS. We have a script for that, but we've had problems getting it in. That's our P0 currently. Until we get https://codereview.qt-project.org/#/c/2