From IRC as documentation of the next steps: TL;DR: rebuild llvm with -g1 but with -DNDEBUG and retry then [09:05] LocutusOfBorg, cpaelzer: what are you trying? we are back with two one gigabyte debug builds on ppc64el and s390x, and of course the failing amd64 build [09:09] doko, -g1 is breaking clamav [09:09] -g works [09:09] I'm doing a -g -NDEBUG to see what happens [09:12] doko: bug 1717574 [09:12] bug 1717574 in clamav (Ubuntu) "new llvm3.9 breaking freshclam bytecode execution" [Medium,Confirmed] https://launchpad.net/bugs/1717574 [09:14] cpaelzer: is this upstreamed to clamav, or are we trying to change things which we don't understand? why is a change for the debug information responsible for that? [09:15] doko: I only rerun the test and found the link of the issue to llvm - there is no clamav change [09:16] doko: so nothing to upstream there that we'd know yet [09:16] doko: it is just that their bytecode worked before a certina llvm upload and fails since then [09:16] cpaelzer: did you try to rebuild clamav with the same flags as llvm? [09:17] cpaelzer: or did you try to rebuild clamav at all? [09:17] we rebuilt clamav with old and new llvm, but not change flags [09:17] none of the rebuilds had any effect [09:17] we also tried a new clamav just to be sure [09:17] but yeah, no changes in flags, except for llvm [09:18] please can you report that upstream? [09:25] cpaelzer: we can't accomodate these builds if they take too much disk space to build, and even if, each of these packages take 1 gb, that makes 4gb for every upload ... [09:28] doko: I did never say "please make them big" I'm only naively debugged a clamav crash and found that it is broken since these llvm uploads [09:28] I beg your pardon if that got to you differently [09:29] I'm reporting to clamav and will link it up [09:30] doko: if you follow the bug I linked maybe you can see what the actual root cause might be the reason for that incompatibility [09:31] cpaelzer: what is the change in compiler and linker flags? [09:32] cpaelzer, https://launchpad.net/~costamagnagianfranco/+archive/ubuntu/costamagnagianfranco-ppa/+packages [09:32] I uploaded a new llvm with -g1 (to make doko and infra happy) and with -DNDEBUG [09:32] what I discovered, is that ppassing RELWITHDEBINFO flags, overrides the cmake default ones [09:32] and that -DNDEBUG is stripped then [09:32] so the change has probably been a side-effect of that -g1, but not directly related [09:34] I'll test that once it is built LocutusOfBorg [09:34] cpaelzer: there's nothing about this -DNDEBUG in the bug report [09:35] and then find out why clamav is relying on that ... [09:35] I discovered it some minutes ago, melding the failed amd64 and the succeeded one [09:36] because in my ppa it was good that build [09:36] you should turn on creation of debug package in your ppa if you want to simulate the builders [09:38] both enabled, build and publish [09:38] thanks for the reminder, I admit I didn't recheck it [09:47] interesting, libclamav is using some NDEBUG stuff [09:48] LocutusOfBorg: doko: I reported to clamav and linked it up in the bug [09:49] LocutusOfBorg: let me know if/once there is a build I shall try for you [09:49] configure: CXXFLAGS from llvm-config: -I/usr/lib/llvm-3.9/include -std=c++0x -gsplit-dwarf -Wl,-fuse-ld=gold -fPIC -fvisibility-inlines-hidden -Wall -W -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wno-missing-field-initializers -pedantic -Wno-long-long -Wno-maybe-uninitialized -Wdelete-non-virtual-dtor -Wno-comment -Werror=date-time -std=c++11 -ffunction-sections -fdata-sections -O2 -g -DNDEBUG [09:49] -fno-exceptions -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS [09:49] interestingly, clamav is getting the build flags from llvm, so a no change rebuild should change them too [09:50] and in fact this seems to happen [09:50] but you rebuilt a new clamav [09:50] also I did with a local build vs the new library and -dev packages [09:52] but clamav not picking up NDEBUG from llvm flags, means that the code ifdefs are not triggered [09:52] this means less asserts [09:53] but the clamav currently in artful uploaded by mdeslaur still built by picking -DNDEBUG up as it was before the llvm changes [09:53] I think I'll directly upload to Ubuntu this one [09:53] https://launchpadlibrarian.net/333379617/buildlog_ubuntu-artful-amd64.clamav_0.99.2+dfsg-6ubuntu2_BUILDING.txt.gz [09:54] LocutusOfBorg: do you mean we have lost the -NDEBUG in llvm and those "extra" asserts might be the reason to break it now? [09:54] somewhat this is a possible reason [09:54] rebuilding makes it assert because the current llvm in artful proposed is built without [09:54] probably rebuilding it with -DNDEBUG makes it pass [09:56] LocutusOfBorg: ok, so since clamav is already built with -DNDEBUG from the old llvm builds that would mean likely no re-upload there. Once your ppa is ready I would install from it and then retry + rebuild+retry - ok? [09:57] ok [09:57] so waiting on https://launchpad.net/~costamagnagianfranco/+archive/ubuntu/costamagnagianfranco-ppa/+build/13399109 now [09:57] I could also upload directly to Ubuntu [09:57] I personally prefer uploading to ubuntu what is known or at least expected to work [09:57] the current proposed one has debug symbols too huge [09:57] ah, ok I see the reason [09:58] even if not working, it is worth an upload [09:58] well, are you sure the ubuntu5 at least builds and is as good/bad than the last one that had small symbols? [09:58] I'll carry all this discussion into the bug for documentation