Build SystemOrganization
ActivePublic

Recent Activity

Wed, Apr 17

sh-zam added a comment to D20558: Bug fix: find c++ stl using regex.

well.. I couldn't get it running as a toolchain and kept running into errors.

Wed, Apr 17, 4:23 PM · Build System, Frameworks
apol added a comment to D20558: Bug fix: find c++ stl using regex.

In my case if I do this: include (Android.cmake), then it won't be able to find the ECMAndroidDeployQt.cmake because I am using android-ndk's toolchain.

Wed, Apr 17, 4:08 PM · Build System, Frameworks
sh-zam closed D20558: Bug fix: find c++ stl using regex.
Wed, Apr 17, 3:57 PM · Build System, Frameworks
sh-zam added a comment to D20558: Bug fix: find c++ stl using regex.

What does it mean that "it expects to run as a toolchain"?

Wed, Apr 17, 3:55 PM · Build System, Frameworks
apol accepted D20558: Bug fix: find c++ stl using regex.
Wed, Apr 17, 3:16 PM · Build System, Frameworks
sh-zam added inline comments to D20558: Bug fix: find c++ stl using regex.
Wed, Apr 17, 2:51 PM · Build System, Frameworks
apol added a comment to D20558: Bug fix: find c++ stl using regex.

Why are you using it without Android.cmake?

I am building Krita and to add create-apk target I am using include (ECMAndroidDeployQt.cmake), I can't use Android.cmake because of some variables as it expects to run as a toolchain.

Wed, Apr 17, 2:36 PM · Build System, Frameworks

Mon, Apr 15

sh-zam updated the diff for D20558: Bug fix: find c++ stl using regex.
  • disallow whitespaces in path
Mon, Apr 15, 4:53 PM · Build System, Frameworks
sh-zam added a comment to D20558: Bug fix: find c++ stl using regex.

Why are you using it without Android.cmake?

Mon, Apr 15, 12:28 AM · Build System, Frameworks
sh-zam updated the diff for D20558: Bug fix: find c++ stl using regex.
  • disallow whitespaces in path
Mon, Apr 15, 12:28 AM · Build System, Frameworks
sh-zam closed D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.
Mon, Apr 15, 12:13 AM · Build System, Frameworks
apol added a comment to D20558: Bug fix: find c++ stl using regex.

Can we add a cmake_minimum_required to 3.7? Regex fails to compile on older versions, when run outside of Android.cmake.

Mon, Apr 15, 12:08 AM · Build System, Frameworks

Sun, Apr 14

apol accepted D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.

Looks good to me, can you land it?

Sun, Apr 14, 11:59 PM · Build System, Frameworks
sh-zam added a comment to D20558: Bug fix: find c++ stl using regex.

Can we add a cmake_minimum_required to 3.7? Regex fails to compile on older versions, when run outside of Android.cmake.

Sun, Apr 14, 10:29 PM · Build System, Frameworks
sh-zam updated the diff for D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.

improved name and code flow

Sun, Apr 14, 8:59 PM · Build System, Frameworks
sh-zam requested review of D20558: Bug fix: find c++ stl using regex.
Sun, Apr 14, 8:55 PM · Build System, Frameworks
apol requested changes to D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.
Sun, Apr 14, 5:47 PM · Build System, Frameworks

Sat, Apr 13

lbeltrame added reviewers for D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix: apol, vkrause.
Sat, Apr 13, 10:47 PM · Build System, Frameworks
sh-zam updated the diff for D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.

minor typo fixed

Sat, Apr 13, 5:01 PM · Build System, Frameworks
sh-zam requested review of D20509: Detect duplicate ANDROID_EXTRA_LIBS and minor bug fix.
Sat, Apr 13, 3:02 PM · Build System, Frameworks

Sun, Apr 7

dfaure closed D20349: Unconditionally enable -DQT_STRICT_ITERATORS, not just in debug mode.
Sun, Apr 7, 5:00 PM · Build System, Frameworks
aacid accepted D20349: Unconditionally enable -DQT_STRICT_ITERATORS, not just in debug mode.
Sun, Apr 7, 4:56 PM · Build System, Frameworks
dfaure added a comment to D20349: Unconditionally enable -DQT_STRICT_ITERATORS, not just in debug mode.

All the modules that use this cmake file, already build fine in debug mode with strict iterators.
This enables the same setting in release mode, for code that already builds fine in debug mode. So unless people have bad code in #ifndef NDEBUG, I don't expect this to break anything at all.

Sun, Apr 7, 4:23 PM · Build System, Frameworks
aacid added a comment to D20349: Unconditionally enable -DQT_STRICT_ITERATORS, not just in debug mode.

Are we sure we don't break code by introducing this? Or we're just going to play Whac-A-Mole with compile errors until all it's compiling again?

Sun, Apr 7, 4:04 PM · Build System, Frameworks
dfaure requested review of D20349: Unconditionally enable -DQT_STRICT_ITERATORS, not just in debug mode.
Sun, Apr 7, 1:23 PM · Build System, Frameworks

Tue, Apr 2

bencreasy added a watcher for Build System: bencreasy.
Tue, Apr 2, 1:07 AM

Fri, Mar 29

apol closed D20105: ecm_add_wayland_client_protocol: Improve error messages.
Fri, Mar 29, 1:10 PM · Build System, Frameworks
zzag added a comment to D20105: ecm_add_wayland_client_protocol: Improve error messages.

Hmm it looks like we're using ancient code option.

Fri, Mar 29, 9:14 AM · Build System, Frameworks
zzag accepted D20105: ecm_add_wayland_client_protocol: Improve error messages.
Fri, Mar 29, 9:06 AM · Build System, Frameworks
cmollekopf added a comment to D20059: Take clang-cl into account..

FWIW, I have meanwhile used this patch to build all kube dependencies on linux and osx as well, and it seems like it doesn't break anything.

Fri, Mar 29, 7:23 AM · Build System, Frameworks
apol updated the diff for D20105: ecm_add_wayland_client_protocol: Improve error messages.

for generating server files too

Fri, Mar 29, 3:20 AM · Build System, Frameworks
apol requested review of D20105: ecm_add_wayland_client_protocol: Improve error messages.
Fri, Mar 29, 3:16 AM · Build System, Frameworks

Tue, Mar 26

apol added a comment to D20059: Take clang-cl into account..

LGTM, looks like clang-cl will be missing a few arguments though.

Tue, Mar 26, 2:31 PM · Build System, Frameworks
apol added a reviewer for D20059: Take clang-cl into account.: Windows.
Tue, Mar 26, 2:29 PM · Build System, Frameworks
cmollekopf added a comment to D20059: Take clang-cl into account..

There was already an earlier (abandoned) attempt at this: https://git.reviewboard.kde.org/r/128779

Tue, Mar 26, 9:11 AM · Build System, Frameworks
cmollekopf requested review of D20059: Take clang-cl into account..
Tue, Mar 26, 9:09 AM · Build System, Frameworks

Sun, Mar 24

knauss added a task to D19996: WIP Add a global test for insecure http: URLs used in code or documentation: T10716: Add check for accidental http: usage.
Sun, Mar 24, 8:24 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

fixing a typo.

Sun, Mar 24, 3:11 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

make parallel network requests.

Sun, Mar 24, 2:29 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

complete diff.

Sun, Mar 24, 12:31 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

now with glob matching and another round of renaming blacklist files.

Sun, Mar 24, 12:19 PM · Build System, Frameworks

Sat, Mar 23

knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

remove BLACKLSIT for httpcheck

Sat, Mar 23, 10:17 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

add httpupdate and split out blacklist file

Sat, Mar 23, 10:14 PM · Build System, Frameworks
knauss added a comment to D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

this would be a nice addition to Krazy. on my todo list.

Sat, Mar 23, 9:43 PM · Build System, Frameworks
winterz added a comment to D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

this would be a nice addition to Krazy. on my todo list.

Sat, Mar 23, 3:57 PM · Build System, Frameworks
vkrause added inline comments to D19996: WIP Add a global test for insecure http: URLs used in code or documentation.
Sat, Mar 23, 3:32 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

search for http://\S

Sat, Mar 23, 2:07 PM · Build System, Frameworks
cgiboudeaux added inline comments to D19996: WIP Add a global test for insecure http: URLs used in code or documentation.
Sat, Mar 23, 2:04 PM · Build System, Frameworks
vkrause added inline comments to D19996: WIP Add a global test for insecure http: URLs used in code or documentation.
Sat, Mar 23, 1:58 PM · Build System, Frameworks
knauss updated the diff for D19996: WIP Add a global test for insecure http: URLs used in code or documentation.

add more excludes.

Sat, Mar 23, 1:58 PM · Build System, Frameworks