*** tristan has joined #buildstream | 00:35 | |
*** Prince781 has quit IRC | 01:03 | |
*** toscalix has joined #buildstream | 01:15 | |
*** Prince781 has joined #buildstream | 02:02 | |
*** Prince781 has quit IRC | 02:03 | |
*** Prince781 has joined #buildstream | 02:05 | |
*** Prince781 has quit IRC | 02:25 | |
*** Prince781 has joined #buildstream | 02:26 | |
*** Prince781_ has joined #buildstream | 04:36 | |
*** Prince781 has quit IRC | 04:38 | |
*** Prince781_ is now known as Prince781 | 04:38 | |
gitlab-br-bot | buildstream: merge request (remove_shebangs->master: Remove shebangs from python files) #506 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/506 | 04:41 |
---|---|---|
*** Prince781 has quit IRC | 04:42 | |
*** Prince781 has joined #buildstream | 04:50 | |
*** Prince781 has quit IRC | 04:52 | |
*** Prince781 has joined #buildstream | 04:56 | |
*** ernestask has joined #buildstream | 05:13 | |
gitlab-br-bot | buildstream: issue #424 ("Python files have shebangs unnecessarily") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/424 | 05:16 |
gitlab-br-bot | buildstream: merge request (remove_shebangs->master: Remove shebangs from python files) #506 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/506 | 05:16 |
*** tristan has quit IRC | 05:16 | |
*** Prince781 has quit IRC | 06:18 | |
*** toscalix has quit IRC | 07:17 | |
*** Phil has joined #buildstream | 07:18 | |
*** coldtom has joined #buildstream | 07:27 | |
Nexus | Two of my branches are hanging on "tests/examples/flatpak-autotools.py::test_autotools_build" Does anyone else experience this? | 08:14 |
Phil | Nope | 08:15 |
Nexus | well darn | 08:16 |
*** bethw has joined #buildstream | 08:17 | |
*** toscalix has joined #buildstream | 08:34 | |
*** jonathanmaw has joined #buildstream | 08:38 | |
gitlab-br-bot | buildstream: issue #430 ("BuildStream.doap is incorrectly included in MANIFEST.in") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/430 | 08:40 |
*** mohan43u has quit IRC | 08:49 | |
*** dominic has joined #buildstream | 08:52 | |
*** dominic has joined #buildstream | 08:53 | |
laurence | valentind, thanks for the update re Add 'workspacedir' configuration MR - https://gitlab.com/BuildStream/buildstream/merge_requests/444#note_82469164 | 08:58 |
laurence | have you pushed for a review from someone in this channel? | 08:58 |
valentind | No. Sorry. That was a long time ago. | 08:59 |
laurence | valentind, nw! we're all busy :) | 09:00 |
laurence | I'm just trying to tidy up the MR backlog before things get waaay outdated | 09:00 |
laurence | i'm doing some housekeeping :) | 09:00 |
gitlab-br-bot | buildstream: 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/471 | 09:01 |
gitlab-br-bot | buildstream: merge request (dp0/pylint_restrict->master: WIP: Restrict version of pylint) #503 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/503 | 09:03 |
gitlab-br-bot | buildstream: merge request (dp0/pylint_restrict->master: Restrict version of pylint) #503 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/503 | 09:05 |
gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 09:30 |
*** mohan43u has joined #buildstream | 09:54 | |
*** toscalix has quit IRC | 10:00 | |
*** aday has quit IRC | 10:04 | |
*** mohan43u has quit IRC | 10:10 | |
Nexus | i'm getting this error and i'm having a hard time debugging it, does anyone have any ideas? | 10:11 |
Nexus | https://hastebin.com/yohimecuni.md | 10:11 |
jmac | Nexus: Is this a new test or an unusual branch? That version of test_autotools_build isn't from current master | 10:15 |
Nexus | it's on my master ._. | 10:17 |
jmac | It is on master, you're right, in tests/examples/flatpak-autotools | 10:21 |
jmac | But I can't see where it gets hello.bst from | 10:21 |
*** aday has joined #buildstream | 10:49 | |
Nexus | :( | 11:00 |
gitlab-br-bot | buildstream: issue #428 ("Unhandled exception when trying to push to the bst CAS") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/428 | 11:21 |
jjardon | Hi, is there plans for a 1.1.4 release soonish? | 11:41 |
*** Phil has quit IRC | 12:34 | |
*** Phil has joined #buildstream | 12:34 | |
*** jonathanmaw has quit IRC | 12:40 | |
gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 12:40 |
*** jonathanmaw has joined #buildstream | 12:55 | |
*** ernestask has quit IRC | 13:15 | |
jmac | Still 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 IRC | 13:19 | |
*** dominic has joined #buildstream | 13:19 | |
*** xjuan has joined #buildstream | 13:23 | |
*** aday has quit IRC | 13:42 | |
*** aday has joined #buildstream | 13:43 | |
*** tristan has joined #buildstream | 13:55 | |
gitlab-br-bot | buildstream: merge request (gokcen/source_transform->master: WIP: Source transform plugin) #505 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/505 | 14:27 |
coldtom | hi, i'm new to buildstream, can someone point me to a relatively easy issue to sort out? | 14:48 |
*** Phil has quit IRC | 14:49 | |
tristan | coldtom, hmmm, I'll take a look - just running out for breakfast and will be back | 14:49 |
tristan | coldtom, recommend you request developer access in the gitlab UI | 14:50 |
tristan | that 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 repo | 14:51 |
coldtom | will do :) | 14:52 |
*** tristan has quit IRC | 15:12 | |
gitlab-br-bot | buildstream: merge request (gokcen/source_transform->master: WIP: Source transform plugin) #505 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/505 | 15:15 |
*** Prince781 has joined #buildstream | 15:21 | |
*** Phil has joined #buildstream | 15:58 | |
laurence | coldtom, jennis and I spoke f2f earlier and he highlighted this issue as a decent place to start - https://gitlab.com/BuildStream/buildstream/issues/301 | 16:17 |
laurence | or at least, a relatively simple one for a newcomer to pick up | 16:17 |
laurence | tristan may or may not agree | 16:17 |
coldtom | i feel like that should be within my ability to fix | 16:24 |
*** tristan has joined #buildstream | 16:26 | |
Nexus | tristan: would you be able to look at my plan for https://gitlab.com/BuildStream/buildstream/issues/413 please? | 16:26 |
*** coldtom has quit IRC | 16:31 | |
jennis | We 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 |
tristan | jennis, file an issue I guess ? | 16:42 |
tristan | Nexus, added comments to 413 | 16:42 |
Nexus | ty :) | 16:43 |
tristan | Nexus, I'm keeping my eye on https://gitlab.com/BuildStream/buildstream/merge_requests/482, as I understand it unlocks the rest | 16:46 |
Nexus | tristan: i'm getting an error on it that i can't fix | 16:47 |
tristan | Nexus, 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 them | 16:47 |
Nexus | once that tests passes, it's ready to be reviewed/merged | 16:47 |
tristan | for instance: https://gitlab.com/BuildStream/buildstream/merge_requests/482/diffs#note_81401264 | 16:47 |
Nexus | yeah i resolved it with the intention of pushing the changes, but can't get past the test fails | 16:48 |
gitlab-br-bot | buildstream: 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/508 | 16:48 |
gitlab-br-bot | buildstream: 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/508 | 16:48 |
tristan | Nexus, neither does the version of the branch on gitlab :-S | 16:48 |
gitlab-br-bot | buildstream: merge request (reduce_history_in_cache->master: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/482 | 16:48 |
Nexus | that has my latest version now | 16:50 |
gitlab-br-bot | buildstream: 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/508 | 16:50 |
gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 16:51 |
gitlab-br-bot | buildstream: merge request (caching_build_trees->master: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 16:52 |
gitlab-br-bot | buildstream: merge request (caching_build_trees->master: WIP: Caching buildtrees) #474 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/474 | 16:52 |
gitlab-br-bot | buildstream: 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/508 | 16:52 |
tristan | Nexus, 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 |
tristan | or is it impossible ? | 16:52 |
Nexus | that's been changed in this latest version | 16:54 |
Nexus | originally 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't | 16:54 |
Nexus | now it should | 16:55 |
Nexus | tristan: 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 it | 16:56 |
tristan | Nexus, where is it documented that "ext::git -c blackMagic.voodoo=true %s..." does anything ? | 16:59 |
Nexus | yeahhhhh i should probably put a comment on that | 17:00 |
skullman | https://git-scm.com/docs/git-config#git-config-uploadpackallowAnySHA1InWant | 17:01 |
tristan | Nexus, the documentation seems to say that the remote needs to be configured with those enabled | 17:01 |
tristan | Nexus, 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 |
skullman | ext:: 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 |
tristan | I may not be reading deeply enough, this git fetch pack page for instance references it: https://git-scm.com/docs/git-fetch-pack | 17:02 |
*** bethw has quit IRC | 17:04 | |
gitlab-br-bot | buildstream: merge request (reduce_history_in_cache->master: Reduce history in cache) #482 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/482 | 17:05 |
skullman | tristan: 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 |
tristan | ok but we're still relying on the HEAD state, and using the "number of commits between <ref> and HEAD" as a meaningful number | 17:09 |
tristan | As I mentioned in another comment, it's racy | 17:10 |
tristan | Nexus, 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 IRC | 17:11 | |
skullman | sounds like we need to resolve HEAD first thing and then use the sha1 of that for the rest | 17:12 |
tristan | skullman, I'm not sure why we care what HEAD is in the first place | 17:13 |
skullman | I'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 IRC | 17:15 | |
Nexus | tristan: how else would we know how deep to go? | 17:22 |
tristan | Nexus, 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 |
tristan | Nexus, 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 dangerous | 17:27 |
*** Prince781 has quit IRC | 17:56 | |
*** aday has quit IRC | 18:01 | |
*** aday has joined #buildstream | 18:02 | |
tristan | Nexus, I've been chatting for a while in #git | 18:11 |
tristan | https://bpaste.net/show/56d0ff8982e7 | 18:11 |
tristan | Nexus, I think we should investigate usage of `git clone --reference` or `git clone --shared` | 18:13 |
*** xjuan has quit IRC | 18:14 | |
*** Prince781 has joined #buildstream | 18:15 | |
*** Prince781 has quit IRC | 18:16 | |
*** Prince781 has joined #buildstream | 18:18 | |
*** dominic has quit IRC | 18:19 | |
tristan | Nexus, capturing more useful information there... | 18:22 |
*** tristan has quit IRC | 18:29 | |
*** tristan has joined #buildstream | 18:32 | |
*** cs_shadow has quit IRC | 18:33 | |
gitlab-br-bot | buildstream: issue #244 ("Need a method of mirroring all sources so they are always available") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/244 | 19:05 |
*** toscalix has joined #buildstream | 19:10 | |
*** xjuan has joined #buildstream | 19:20 | |
* tristan hadn't forgotten coldtom but wasn't quick enough | 19:34 | |
gitlab-br-bot | buildstream: issue #301 ("Fix circular imports") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/301 | 19:38 |
*** Prince781 has quit IRC | 19:43 | |
*** Prince781 has joined #buildstream | 19:45 | |
*** Prince781 has quit IRC | 20:04 | |
*** tristan has quit IRC | 20:49 | |
*** xjuan has quit IRC | 21:01 | |
*** toscalix has quit IRC | 21:23 | |
*** aday has quit IRC | 23:07 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!