IRC logs for #buildstream for Tuesday, 2018-06-19

*** tristan has joined #buildstream00:35
*** Prince781 has quit IRC01:03
*** toscalix has joined #buildstream01:15
*** Prince781 has joined #buildstream02:02
*** Prince781 has quit IRC02:03
*** Prince781 has joined #buildstream02:05
*** Prince781 has quit IRC02:25
*** Prince781 has joined #buildstream02:26
*** Prince781_ has joined #buildstream04:36
*** Prince781 has quit IRC04:38
*** Prince781_ is now known as Prince78104:38
gitlab-br-botbuildstream: merge request (remove_shebangs->master: Remove shebangs from python files) #506 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50604:41
*** Prince781 has quit IRC04:42
*** Prince781 has joined #buildstream04:50
*** Prince781 has quit IRC04:52
*** Prince781 has joined #buildstream04:56
*** ernestask has joined #buildstream05:13
gitlab-br-botbuildstream: issue #424 ("Python files have shebangs unnecessarily") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/42405:16
gitlab-br-botbuildstream: merge request (remove_shebangs->master: Remove shebangs from python files) #506 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/50605:16
*** tristan has quit IRC05:16
*** Prince781 has quit IRC06:18
*** toscalix has quit IRC07:17
*** Phil has joined #buildstream07:18
*** coldtom has joined #buildstream07:27
NexusTwo of my branches are hanging on "tests/examples/flatpak-autotools.py::test_autotools_build" Does anyone else experience this?08:14
PhilNope08:15
Nexuswell darn08:16
*** bethw has joined #buildstream08:17
*** toscalix has joined #buildstream08:34
*** jonathanmaw has joined #buildstream08:38
gitlab-br-botbuildstream: issue #430 ("BuildStream.doap is incorrectly included in MANIFEST.in") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/43008:40
*** mohan43u has quit IRC08:49
*** dominic has joined #buildstream08:52
*** dominic has joined #buildstream08:53
laurencevalentind, thanks for the update re Add 'workspacedir' configuration MR -  https://gitlab.com/BuildStream/buildstream/merge_requests/444#note_8246916408:58
laurencehave you pushed for a review from someone in this channel?08:58
valentindNo. Sorry. That was a long time ago.08:59
laurencevalentind, nw! we're all busy :)09:00
laurenceI'm just trying to tidy up the MR backlog before things get waaay outdated09:00
laurencei'm doing some housekeeping :)09:00
gitlab-br-botbuildstream: merge request (valentindavid/331_include->master: WIP: Add support for include in project.conf) #471 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47109:01
gitlab-br-botbuildstream: merge request (dp0/pylint_restrict->master: WIP: Restrict version of pylint) #503 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50309:03
gitlab-br-botbuildstream: merge request (dp0/pylint_restrict->master: Restrict version of pylint) #503 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50309:05
gitlab-br-botbuildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47409:30
*** mohan43u has joined #buildstream09:54
*** toscalix has quit IRC10:00
*** aday has quit IRC10:04
*** mohan43u has quit IRC10:10
Nexusi'm getting this error and i'm having a hard time debugging it, does anyone have any ideas?10:11
Nexushttps://hastebin.com/yohimecuni.md10:11
jmacNexus: Is this a new test or an unusual branch? That version of test_autotools_build isn't from current master10:15
Nexusit's on my master ._.10:17
jmacIt is on master, you're right, in tests/examples/flatpak-autotools10:21
jmacBut I can't see where it gets hello.bst from10:21
*** aday has joined #buildstream10:49
Nexus:(11:00
gitlab-br-botbuildstream: issue #428 ("Unhandled exception when trying to push to the bst CAS") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/42811:21
jjardonHi, is there plans for a 1.1.4 release soonish?11:41
*** Phil has quit IRC12:34
*** Phil has joined #buildstream12:34
*** jonathanmaw has quit IRC12:40
gitlab-br-botbuildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47412:40
*** jonathanmaw has joined #buildstream12:55
*** ernestask has quit IRC13:15
jmacStill having problems getting BuildGrid and BuildStream to run on the same system. I think there's some incompatible state being written by their setup scripts.13:19
*** dominic has quit IRC13:19
*** dominic has joined #buildstream13:19
*** xjuan has joined #buildstream13:23
*** aday has quit IRC13:42
*** aday has joined #buildstream13:43
*** tristan has joined #buildstream13:55
gitlab-br-botbuildstream: merge request (gokcen/source_transform->master: WIP: Source transform plugin) #505 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50514:27
coldtomhi, i'm new to buildstream, can someone point me to a relatively easy issue to sort out?14:48
*** Phil has quit IRC14:49
tristancoldtom, hmmm, I'll take a look - just running out for breakfast and will be back14:49
tristancoldtom, recommend you request developer access in the gitlab UI14:50
tristanthat lets you use our CI setup when you push branches, and makes it easier for your to follow upstream if you can work in the same repo14:51
coldtomwill do :)14:52
*** tristan has quit IRC15:12
gitlab-br-botbuildstream: merge request (gokcen/source_transform->master: WIP: Source transform plugin) #505 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50515:15
*** Prince781 has joined #buildstream15:21
*** Phil has joined #buildstream15:58
laurencecoldtom, jennis and I spoke f2f earlier and he highlighted this issue as a decent place to start - https://gitlab.com/BuildStream/buildstream/issues/30116:17
laurenceor at least, a relatively simple one for a newcomer to pick up16:17
laurencetristan may or may not agree16:17
coldtomi feel like that should be within my ability to fix16:24
*** tristan has joined #buildstream16:26
Nexustristan: would you be able to look at my plan for https://gitlab.com/BuildStream/buildstream/issues/413 please?16:26
*** coldtom has quit IRC16:31
jennisWe receive a stack trace if we try to checkout an element that hasn't yet been built: https://paste.baserock.org/misemiwiva, looks like we could improve this? Thoughts?16:33
tristanjennis, file an issue I guess ?16:42
tristanNexus, added comments to 41316:42
Nexusty :)16:43
tristanNexus, I'm keeping my eye on https://gitlab.com/BuildStream/buildstream/merge_requests/482, as I understand it unlocks the rest16:46
Nexustristan: i'm getting an error on it that i can't fix16:47
tristanNexus, I've been having a hard time keeping track though, I think you are resolving comments in the MR without pushing the new branches which resolve them16:47
Nexusonce that tests passes, it's ready to be reviewed/merged16:47
tristanfor instance: https://gitlab.com/BuildStream/buildstream/merge_requests/482/diffs#note_8140126416:47
Nexusyeah i resolved it with the intention of pushing the changes, but can't get past the test fails16:48
gitlab-br-botbuildstream: merge request (430-buildstream-doap-is-incorrectly-included-in-manifest-in->master: WIP: Resolve "BuildStream.doap is incorrectly included in MANIFEST.in") #508 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50816:48
gitlab-br-botbuildstream: merge request (430-buildstream-doap-is-incorrectly-included-in-manifest-in->master: WIP: Resolve "BuildStream.doap is incorrectly included in MANIFEST.in") #508 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50816:48
tristanNexus, neither does the version of the branch on gitlab :-S16:48
gitlab-br-botbuildstream: merge request (reduce_history_in_cache->master: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/48216:48
Nexusthat has my latest version now16:50
gitlab-br-botbuildstream: merge request (430-buildstream-doap-is-incorrectly-included-in-manifest-in->master: WIP: Resolve "BuildStream.doap is incorrectly included in MANIFEST.in") #508 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50816:50
gitlab-br-botbuildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47416:51
gitlab-br-botbuildstream: merge request (caching_build_trees->master: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47416:52
gitlab-br-botbuildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/47416:52
gitlab-br-botbuildstream: merge request (430-buildstream-doap-is-incorrectly-included-in-manifest-in->master: WIP: Resolve "BuildStream.doap is incorrectly included in MANIFEST.in") #508 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/50816:52
tristanNexus, and did we figure out why we are cloning all the history from ${ref}...HEAD into our checkouts ? or is that supposed to be fixed here ?16:52
tristanor is it impossible ?16:52
Nexusthat's been changed in this latest version16:54
Nexusoriginally it was an intermediate step, i was working under a misunderstanding of how git dealt with cloning, so i thought i was dropping the points past REF, but i wasn't16:54
Nexusnow it should16:55
Nexustristan: any thoughts on this ?https://hastebin.com/yohimecuni.md as this is currently blocking me from passing my tests and i've been unable to debug it16:56
tristanNexus, where is it documented that "ext::git -c blackMagic.voodoo=true %s..." does anything ?16:59
Nexusyeahhhhh i should probably put a comment on that17:00
skullmanhttps://git-scm.com/docs/git-config#git-config-uploadpackallowAnySHA1InWant17:01
tristanNexus, the documentation seems to say that the remote needs to be configured with those enabled17:01
tristanNexus, since we maintain the remote (i.e. in this case the remote is our local repo in the source cache), then can we just configure the repo that way ?17:02
skullmanext:: makes it run the git command instead of the usual remote command for file:/// URIs, the -c forces it for the period of the invocation.17:02
tristanI may not be reading deeply enough, this git fetch pack page for instance references it: https://git-scm.com/docs/git-fetch-pack17:02
*** bethw has quit IRC17:04
gitlab-br-botbuildstream: merge request (reduce_history_in_cache->master: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/48217:05
skullmantristan: setting it in repo config works. Might need some work to migrate cached git repositories if doing it that way. I suggested the voodoo to Nexus to keep everything isolated to that call site.17:08
tristanok but we're still relying on the HEAD state, and using the "number of commits between <ref> and HEAD" as a meaningful number17:09
tristanAs I mentioned in another comment, it's racy17:10
tristanNexus, skullman - the problem is what if another process is doing `bst track` and we have a different HEAD state before and after ... around line 192 ?17:11
*** Phil has quit IRC17:11
skullmansounds like we need to resolve HEAD first thing and then use the sha1 of that for the rest17:12
tristanskullman, I'm not sure why we care what HEAD is in the first place17:13
skullmanI'd not read up on that bit, and right now my brain is dribbling out of my ears, so I'm afraid I can't help with that.17:14
*** jonathanmaw has quit IRC17:15
Nexustristan: how else would we know how deep to go?17:22
tristanNexus, the question is not "how else would we know how deep past HEAD to ask for at fetch time", it is rather "there must be another way to fetch precisely what we need, without fussing about what HEAD happens to be"17:26
tristanNexus, as long as we are A.) Calculating how many commits in one step, and B.) Cloning based on that "how many commits" number... and that (A) and (B) are not an atomic operation; then this approach is dangerous17:27
*** Prince781 has quit IRC17:56
*** aday has quit IRC18:01
*** aday has joined #buildstream18:02
tristanNexus, I've been chatting for a while in #git18:11
tristanhttps://bpaste.net/show/56d0ff8982e718:11
tristanNexus, I think we should investigate usage of `git clone --reference` or `git clone --shared`18:13
*** xjuan has quit IRC18:14
*** Prince781 has joined #buildstream18:15
*** Prince781 has quit IRC18:16
*** Prince781 has joined #buildstream18:18
*** dominic has quit IRC18:19
tristanNexus, capturing more useful information there...18:22
*** tristan has quit IRC18:29
*** tristan has joined #buildstream18:32
*** cs_shadow has quit IRC18:33
gitlab-br-botbuildstream: issue #244 ("Need a method of mirroring all sources so they are always available") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/24419:05
*** toscalix has joined #buildstream19:10
*** xjuan has joined #buildstream19:20
* tristan hadn't forgotten coldtom but wasn't quick enough19:34
gitlab-br-botbuildstream: issue #301 ("Fix circular imports") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/30119:38
*** Prince781 has quit IRC19:43
*** Prince781 has joined #buildstream19:45
*** Prince781 has quit IRC20:04
*** tristan has quit IRC20:49
*** xjuan has quit IRC21:01
*** toscalix has quit IRC21:23
*** aday has quit IRC23:07

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