IRC logs for #baserock for Thursday, 2017-10-26

*** gitlab-br-bot has quit IRC04:27
*** gtristan has quit IRC05:45
*** gtristan has joined #baserock05:54
*** adds68 has joined #baserock06:47
*** jude_ has joined #baserock06:49
*** adds68 has quit IRC06:53
*** paulwaters_ has joined #baserock07:20
*** adds68 has joined #baserock07:48
*** adds68 has quit IRC08:03
*** adds68_ has joined #baserock08:03
*** adds68__ has joined #baserock08:40
*** adds68_ has quit IRC08:42
*** jonathanmaw has joined #baserock08:59
*** ssam2 has joined #baserock09:07
*** ChanServ sets mode: +v ssam209:07
ssam2pipeline for https://gitlab.com/baserock/definitions/merge_requests/59 succeeded!09:19
ssam2could someone give it an approval ?09:23
*** gitlab-br-bot has joined #baserock09:31
*** adds68__ has quit IRC09:32
gitlab-br-botdefinitions: merge request (sam/bootstrap-from-baserock-binaries->master: BuildStream builds should bootstrap from Baserock binaries, not the Freedesktop SDK) #58 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/5810:05
jjardonpipeline for https://gitlab.com/baserock/definitions/merge_requests/64 succeeded! Can I get another review?10:22
gitlab-br-botdefinitions: merge request (jjardon/no_master2->master: Do not use master of our build tools, but a specific version instead) #64 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/6410:42
*** adds68 has joined #baserock11:21
*** adds68 has joined #baserock11:23
*** adds68 has joined #baserock11:23
gitlab-br-botdefinitions: merge request (jjardon/no_master2->master: Do not use master of our build tools, but a specific version instead) #64 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/6411:27
jjardonbenbrown_: ssam2 sorry, can I get a reaproval ^? I had to do a manual rebase11:29
gitlab-br-botdefinitions: merge request (jjardon/no_master2->master: Do not use master of our build tools, but a specific version instead) #64 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/6411:30
jjardonssam2: Do we still have the limitation to be a protected branch to be able to push to the bst cache server?11:32
benbrown_jjardon: AFAIA, yes11:34
ssam2yes11:37
jjardonssam2: now that we have our own docker image; maybe we can put the SSH key there?11:43
ssam2the image would have to be private11:43
benbrown_jjardon: I definitely prefer how it is now to storing it in the docker image11:44
jjardonyeah11:45
*** adds68 has quit IRC12:05
*** adds68 has joined #baserock12:32
*** adds68 has quit IRC12:59
*** adds68 has joined #baserock13:01
gitlab-br-botdefinitions: merge request (jjardon/no_master2->master: Do not use master of our build tools, but a specific version instead) #64 changed state ("merged"): https://gitlab.com/baserock/definitions/merge_requests/6413:34
*** toscalix has joined #baserock14:01
*** jude_ has quit IRC14:02
ssam2the compression of debug symbols has broken debug stripping in baserock14:26
ssam2i guess the simplest thing is to override strip-commands in the baserock project.conf for now14:26
ssam2i wonder why it doesn't work though ... perhaps we build stage2-binutils without libz14:26
ssam2in fact, we definitely do that14:27
gtristanssam2, has it ?14:27
ssam2yeah14:28
gtristanssam2, then I need to fix that, to be honest I assumed it would just work, but maybe it needs more fiddling than just adding the argument to the initial objcopy ?14:28
ssam2https://paste.baserock.org/hupasozoce14:28
ssam2and the result is no stripping is done at all14:28
ssam2and the sysroot ends up at 1.2GB14:28
ssam2maybe there's a way that objcopy could tell us whether it supports compression14:29
ssam2or maybe we could add zlib to stage214:29
gtristanoh weird14:29
gtristanI dont get that error message at all14:29
ssam2me neither14:29
gtristanyou did get that error message, I mean, you received it14:30
ssam2it's in the log14:30
gtristanI did not even receive it14:30
ssam2oh, right14:30
ssam2I thought you meant you don't understand it :-)14:30
gtristanyeah got that :)14:30
gtristanheh14:30
ssam2did you do a baserock bootstrap to test this? I suspect it's due to the way we build binutils14:30
gtristanssam2, So I suspect yeah it might have to do with zlib, but note that I changed the project defaults to split up the macros a bit14:31
ssam2right14:31
gtristanso you can easily disable it for the bootstrap phase until it works, without having to rewrite a whole ton of strip commands14:31
ssam2ohh, nice14:31
gtristanits something like objcopy-something-args14:31
ssam2hmm, we're also using binutils from 201414:34
ssam2still, one thing at a time... don't have time to fix every issue in baserock right now. i'll just disable compression14:35
*** gtristan has quit IRC14:36
*** gtristan has joined #baserock14:39
*** adds68 has quit IRC14:48
*** adds68 has joined #baserock14:48
ssam2the stripping seems to work past gnu-toolchain.bst, so i think this is indeed just that zlib is not in stage214:52
* ssam2 works around14:52
gtristanssam2, ok great to know that it works :)14:52
gtristanssam2, any idea if the compression actually makes smaller debug files ?14:52
ssam2not got an easy way to benchmark that14:53
ssam2well, i guess i could strip something twice14:53
ssam2let me try gcc14:53
ssam2ah, here's the difference14:56
ssam2cc1 debug symbols are 127MB when extracted normally, or 53MB when extracted with --compress-debug-symbols14:57
ssam2so, yes :-)14:57
ssam2on the flip side, objcopy takes 8 seconds vs. 0.3 seconds14:58
ssam2so there's a time cost to the compression14:58
*** adds68 has quit IRC15:00
gtristanssam2, sounds great to me thanks for trying it out !15:14
gtristan:)15:14
gtristanoverall, I think thats a desirable win, I guess some setups will want to disable it15:15
gtristanbut for developers it should save some disk space15:16
ssam2yeah. 8 seconds isn't too bad considering this was cc1, one of the biggest binaries we build15:16
*** adds68 has joined #baserock15:59
*** toscalix has quit IRC16:29
*** adds68 has quit IRC16:30
gitlab-br-botdefinitions: merge request (sam/bootstrap-from-baserock-binaries->master: BuildStream builds should bootstrap from Baserock binaries, not the Freedesktop SDK) #58 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/5816:51
gitlab-br-botdefinitions: merge request (sam/bootstrap-from-baserock-binaries->master: BuildStream builds should bootstrap from Baserock binaries, not the Freedesktop SDK) #58 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/5816:53
ssam2alright! stage3-sysroot/x86_64 branch pushed to https://ostree.baserock.org/releases/16:53
ssam2bootstrap wise... we have liftoff16:54
ssam2assuming https://gitlab.com/baserock/definitions/merge_requests/58 builds OK, which is a big assumption!16:54
gtristanssam2, is that a circular build ?16:54
gtristanssam2, boostrapping off of our bootstrap ?16:54
ssam2there's a manual step, if that's what you mean16:54
gtristanyeah but, committed the result to an ostree, and then rebuilding off of that one ?16:55
ssam2yes16:55
gtristanNice !16:55
ssam2only x86_64 so far but.. baby steps16:55
ssam2the sysroot is about 350MB; not perfect but reasonable16:55
gtristanIt's the most important, from there if we have cross-bootstrap, even if only supported on x86_64, everything is possible :)16:56
ssam2argh, i seem to have deleted the commits that update the README16:57
gitlab-br-botdefinitions: merge request (sam/bootstrap-from-baserock-binaries->master: BuildStream builds should bootstrap from Baserock binaries, not the Freedesktop SDK) #58 changed state ("opened"): https://gitlab.com/baserock/definitions/merge_requests/5816:58
ssam2but, if that builds, then i just need to clean up some stuff on the infrastructure side tomorrow, and it's pretty done17:00
*** jonathanmaw has quit IRC17:00
*** jonathanmaw has joined #baserock17:00
*** ssam2 has quit IRC17:10
*** adds68 has joined #baserock18:15
*** adds68 has quit IRC18:35
gitlab-br-botinfrastructure: issue #1 ("Set up cache for GitLab CI runners") changed state ("opened") undefined18:44
gitlab-br-botinfrastructure: issue #1 ("Set up cache for GitLab CI runners") changed state ("opened") undefined19:24
gitlab-br-botinfrastructure: merge request (pedro/gitlab-bot-fixes-2->master: Route traffic to gitlab irc bot) #16 changed state ("opened"): https://gitlab.com/baserock/infrastructure/merge_requests/1619:24
*** gtristan has quit IRC22:45

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!