| *** xjuan has quit IRC | 00:11 | |
| *** bochecha has quit IRC | 00:25 | |
| *** albfan[m] has quit IRC | 01:05 | |
| *** segfault3[m] has quit IRC | 01:06 | |
| *** abderrahim[m] has quit IRC | 01:06 | |
| *** inigomartinez has quit IRC | 01:06 | |
| *** tlater[m] has quit IRC | 01:06 | |
| *** asingh_[m] has quit IRC | 01:06 | |
| *** persia has quit IRC | 01:07 | |
| *** persia has joined #buildstream | 01:08 | |
| *** albfan[m] has joined #buildstream | 01:12 | |
| *** segfault3[m] has joined #buildstream | 01:21 | |
| *** kailueke[m] has quit IRC | 01:27 | |
| *** abderrahim[m] has joined #buildstream | 01:27 | |
| *** inigomartinez has joined #buildstream | 01:29 | |
| *** tlater[m] has joined #buildstream | 01:30 | |
| *** asingh_[m] has joined #buildstream | 01:37 | |
| *** tlater[m] has quit IRC | 01:38 | |
| *** jjardon[m] has quit IRC | 01:38 | |
| *** inigomartinez has quit IRC | 01:38 | |
| *** ssssam[m] has quit IRC | 01:38 | |
| *** Demos[m] has quit IRC | 01:38 | |
| *** waltervargas[m] has quit IRC | 01:38 | |
| *** rafaelff[m] has quit IRC | 01:38 | |
| *** alatiera has quit IRC | 01:38 | |
| *** cgmcintyre[m] has quit IRC | 01:38 | |
| *** krichter[m] has quit IRC | 01:38 | |
| *** dineshdb[m] has quit IRC | 01:38 | |
| *** connorshea[m] has quit IRC | 01:38 | |
| *** awacheux[m] has quit IRC | 01:38 | |
| *** doras[m] has quit IRC | 01:38 | |
| *** catonano has joined #buildstream | 01:58 | |
| *** kailueke[m] has joined #buildstream | 02:06 | |
| *** jjardon[m] has joined #buildstream | 02:10 | |
| *** dineshdb[m] has joined #buildstream | 02:37 | |
| *** ssssam[m] has joined #buildstream | 02:39 | |
| *** Demos[m] has joined #buildstream | 02:44 | |
| *** tlater[m] has joined #buildstream | 02:56 | |
| *** waltervargas[m] has joined #buildstream | 03:05 | |
| *** inigomartinez has joined #buildstream | 03:17 | |
| *** rafaelff[m] has joined #buildstream | 03:18 | |
| *** catonano has quit IRC | 03:19 | |
| *** alatiera has joined #buildstream | 03:25 | |
| *** cgmcintyre[m] has joined #buildstream | 03:26 | |
| *** krichter[m] has joined #buildstream | 03:36 | |
| *** connorshea[m] has joined #buildstream | 03:44 | |
| *** awacheux[m] has joined #buildstream | 03:49 | |
| *** doras[m] has joined #buildstream | 04:15 | |
| *** alatiera_ has joined #buildstream | 05:20 | |
| *** tristan has joined #buildstream | 07:41 | |
| *** toscalix has joined #buildstream | 07:55 | |
| *** tristan has quit IRC | 08:17 | |
| *** rdale has joined #buildstream | 08:19 | |
| *** alatiera_ has quit IRC | 08:27 | |
| *** jonathanmaw has joined #buildstream | 09:01 | |
| gitlab-br-bot | buildstream: merge request (willsalmon/outOfSourecBuild->master: Out of source builds) #776 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/776 | 09:09 |
|---|---|---|
| *** ssssam[m] has quit IRC | 09:10 | |
| *** dineshdb[m] has quit IRC | 09:14 | |
| *** albfan[m] has quit IRC | 09:14 | |
| *** oknf[m] has quit IRC | 09:16 | |
| *** theawless[m] has quit IRC | 09:16 | |
| *** alatiera has quit IRC | 09:16 | |
| *** rafaelff[m] has quit IRC | 09:17 | |
| flatmush | has anyone built boost using buildstream yet? | 09:17 |
| *** cgmcintyre[m] has quit IRC | 09:17 | |
| *** asingh_[m] has quit IRC | 09:17 | |
| *** jjardon[m] has quit IRC | 09:18 | |
| *** mattiasb has quit IRC | 09:18 | |
| *** awacheux[m] has quit IRC | 09:18 | |
| *** m_22[m] has quit IRC | 09:18 | |
| *** abderrahim[m] has quit IRC | 09:18 | |
| tiagogomes | flatmush is in baserock definitions… | 09:18 |
| flatmush | tiagogomes: yeah, just found it | 09:18 |
| flatmush | thanks | 09:18 |
| *** segfault3[m] has quit IRC | 09:19 | |
| tiagogomes | A lot of links here are broken https://gitlab.com/BuildStream/buildstream/blob/master/CONTRIBUTING.rst | 09:34 |
| *** ssssam[m] has joined #buildstream | 09:35 | |
| toscalix | tiagogomes: is it possible to add the same test here that we have for the website broken links? | 09:36 |
| tiagogomes | Not really, as this links actually work, even though they don't point to the intended section | 09:39 |
| toscalix | I see | 09:41 |
| *** dineshdb[m] has joined #buildstream | 09:43 | |
| *** albfan[m] has joined #buildstream | 09:47 | |
| *** cgmcintyre[m] has joined #buildstream | 10:01 | |
| gitlab-br-bot | buildstream: merge request (jonathan/debug-remote-failed-builds->master: WIP: Jonathan/debug remote failed builds) #869 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/869 | 10:05 |
| *** mattiasb has joined #buildstream | 10:06 | |
| *** lachlan has quit IRC | 10:07 | |
| *** awacheux[m] has joined #buildstream | 10:10 | |
| *** m_22[m] has joined #buildstream | 10:12 | |
| gitlab-br-bot | buildstream: merge request (tpollard/494->master: WIP: Don't pull artifact buildtrees by default) #786 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/786 | 10:17 |
| *** abderrahim[m] has joined #buildstream | 10:17 | |
| *** theawless[m] has joined #buildstream | 10:19 | |
| *** alatiera has joined #buildstream | 10:20 | |
| *** segfault3[m] has joined #buildstream | 10:23 | |
| *** lachlan has joined #buildstream | 10:23 | |
| gitlab-br-bot | buildstream: merge request (chandan/fix-checkout-none-1.2->bst-1.2: Ensure `--deps=none` option works for `bst checkout`) #828 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/828 | 10:29 |
| *** jjardon[m] has joined #buildstream | 10:33 | |
| *** rafaelff[m] has joined #buildstream | 10:40 | |
| *** lachlan has quit IRC | 10:42 | |
| *** asingh_[m] has joined #buildstream | 10:42 | |
| gitlab-br-bot | buildstream: issue #704 ("Add a way to format a patch from a workspace") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/704 | 10:45 |
| *** oknf[m] has joined #buildstream | 10:48 | |
| *** abderrahim has quit IRC | 10:52 | |
| *** abderrahim has joined #buildstream | 10:53 | |
| *** jonathanmaw_ has joined #buildstream | 10:54 | |
| *** jonathanmaw has quit IRC | 10:54 | |
| *** lachlan has joined #buildstream | 10:56 | |
| *** lachlan has quit IRC | 11:01 | |
| *** lachlan has joined #buildstream | 11:10 | |
| *** lachlan has quit IRC | 11:13 | |
| *** lachlan has joined #buildstream | 11:19 | |
| *** lachlan has quit IRC | 11:27 | |
| *** lachlan has joined #buildstream | 11:31 | |
| *** bochecha has joined #buildstream | 11:43 | |
| gitlab-br-bot | buildstream: merge request (tpollard/494->master: WIP: Don't pull artifact buildtrees by default) #786 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/786 | 11:48 |
| gitlab-br-bot | buildstream: merge request (tpollard/494->master: Don't pull artifact buildtrees by default) #786 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/786 | 11:48 |
| *** lachlan has quit IRC | 11:51 | |
| *** tristan has joined #buildstream | 12:01 | |
| *** ChanServ sets mode: +o tristan | 12:01 | |
| gitlab-br-bot | buildstream: merge request (chandan/fix-checkout-none-1.2->bst-1.2: Ensure `--deps=none` option works for `bst checkout`) #828 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/828 | 12:06 |
| *** jonathanmaw_ has quit IRC | 12:46 | |
| *** jonathanmaw has joined #buildstream | 12:46 | |
| *** lachlan has joined #buildstream | 12:59 | |
| *** finn has joined #buildstream | 13:36 | |
| tiagogomes | yocto ALTERNATIVE_TARGET is handy. | 13:42 |
| *** lachlan has quit IRC | 13:42 | |
| *** lachlan has joined #buildstream | 13:48 | |
| *** bochecha has quit IRC | 13:54 | |
| *** lachlan has quit IRC | 13:56 | |
| jmac | Just spoken to someone from Bazel about retying jobs in remote execution... it's pretty complicated! | 14:04 |
| jmac | I'm going to have a read through Bazel's Java and see if we can do the same thing | 14:05 |
| *** lachlan has joined #buildstream | 14:06 | |
| tristan | Interesting | 14:11 |
| tristan | jmac, I presume that so long as output has been produced, the architecture does not really allow for considering that output invalid | 14:12 |
| jmac | Apparently it's also very much necessary to do retries if using RBE | 14:12 |
| jmac | No, I tihnk if an output's been produced then we consider it done | 14:12 |
| tristan | for instance, if we fail to obtain source code due to a network timeout (mostly what BuildStream uses "retries" for), then we did not produce output, and it should be alright | 14:12 |
| tristan | What is RBE ? | 14:13 |
| skullman | is that Google's build service thing? | 14:14 |
| jmac | Yes | 14:16 |
| gitlab-br-bot | buildstream: merge request (jmac/vdir_import_test->master: WIP: Import test for virtual directories) #815 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/815 | 14:34 |
| gitlab-br-bot | buildstream: merge request (jonathan/debug-remote-failed-builds->master: WIP: Jonathan/debug remote failed builds) #869 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/869 | 14:43 |
| *** adds68 has quit IRC | 14:58 | |
| *** adds68 has joined #buildstream | 14:59 | |
| *** tristan has quit IRC | 15:00 | |
| *** lachlan has quit IRC | 15:02 | |
| *** jennis has quit IRC | 15:04 | |
| *** jennis has joined #buildstream | 15:05 | |
| *** alatiera has quit IRC | 15:06 | |
| *** ssssam[m] has quit IRC | 15:07 | |
| *** jennis has quit IRC | 15:10 | |
| *** jennis has joined #buildstream | 15:11 | |
| *** adds68 has quit IRC | 15:13 | |
| *** benbrown has quit IRC | 15:13 | |
| *** Nexus has quit IRC | 15:14 | |
| *** Nexus has joined #buildstream | 15:14 | |
| *** adds68 has joined #buildstream | 15:14 | |
| *** lachlan has joined #buildstream | 15:15 | |
| *** ssssam[m] has joined #buildstream | 15:15 | |
| *** adds68 has quit IRC | 15:19 | |
| *** laurence has quit IRC | 15:20 | |
| *** coldtom has quit IRC | 15:20 | |
| *** adds68 has joined #buildstream | 15:20 | |
| *** alatiera has joined #buildstream | 15:20 | |
| *** coldtom has joined #buildstream | 15:20 | |
| *** jennis has quit IRC | 15:20 | |
| *** benbrown has joined #buildstream | 15:21 | |
| *** jennis has joined #buildstream | 15:21 | |
| *** tristan has joined #buildstream | 15:22 | |
| *** laurence has joined #buildstream | 15:23 | |
| *** jennis has quit IRC | 15:24 | |
| *** jennis has joined #buildstream | 15:25 | |
| *** adds68 has quit IRC | 15:26 | |
| *** benbrown has quit IRC | 15:26 | |
| *** coldtom has quit IRC | 15:26 | |
| *** adds68 has joined #buildstream | 15:27 | |
| *** coldtom has joined #buildstream | 15:27 | |
| *** benbrown has joined #buildstream | 15:29 | |
| *** adds68 has quit IRC | 15:30 | |
| *** coldtom has quit IRC | 15:31 | |
| *** jennis has quit IRC | 15:31 | |
| *** jennis has joined #buildstream | 15:32 | |
| *** coldtom has joined #buildstream | 15:32 | |
| *** adds68 has joined #buildstream | 15:32 | |
| *** coldtom has quit IRC | 15:35 | |
| *** adds68 has quit IRC | 15:35 | |
| *** lachlan has quit IRC | 15:35 | |
| *** lachlan has joined #buildstream | 15:37 | |
| *** coldtom has joined #buildstream | 15:37 | |
| *** jennis has quit IRC | 15:38 | |
| *** adds68 has joined #buildstream | 15:38 | |
| *** coldtom has quit IRC | 15:41 | |
| *** adds68 has quit IRC | 15:41 | |
| *** adds68 has joined #buildstream | 15:41 | |
| *** coldtom has joined #buildstream | 15:41 | |
| *** jennis has joined #buildstream | 15:41 | |
| *** finn has quit IRC | 15:48 | |
| jjardon | Hi, is there any command to get the dependency graph of a given element? | 15:48 |
| skullman | there was an option to show an element and what it depends on to `bst show`, but I can't remember if it did so in a way you could call a dependency graph | 15:50 |
| gitlab-br-bot | buildstream: merge request (tristan/refactor-types-api->master: Refactor types api) #870 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/870 | 15:51 |
| tpollard | bst show should show the dependencies of an element (--deps to control the type of deps) | 15:51 |
| *** ChanServ sets mode: +o tristan | 15:52 | |
| tristan | There is no way to show the *graph*, though | 15:52 |
| tristan | I think this was asked for at some point in the past, rings a bell | 15:52 |
| tristan | probably also by jjardon :) | 15:52 |
| jjardon | skullman: tpollard thanks! | 15:53 |
| jjardon | tristan: :) | 15:53 |
| tpollard | yes it does ring a bell | 15:53 |
| jjardon | mmm, It woud be very handy if bst somehow show the dependency graph when there is a circular one | 15:53 |
| jjardon | https://www.irccloud.com/pastebin/rLWKKIzu/ | 15:54 |
| tristan | whoa, weird bug | 15:54 |
| jjardon | As currently you have to manually know where the circular dependency is | 15:54 |
| tpollard | some graphviz dots would be nice | 15:54 |
| tristan | (A) Open https://gitlab.com/BuildStream/buildstream/merge_requests/870 (B) Right click the pipeline and open in a new tab (C) Go to that tab and click the first job (D) Now click the "Back" button on the browser... Result: Shows a page of JSON structured data instead of the main pipeline page | 15:55 |
| tristan | jjardon, Don't we error out for a circular dependency ? | 15:55 |
| tristan | jjardon, Or, we don't show a very useful error when that happens ? | 15:55 |
| jjardon | tristan: the second: the error message is not useful to know how to break the circular dependency | 15:56 |
| tristan | ahh yes I see | 15:56 |
| tristan | jjardon, I seem to recall having something different there at some point, but it was very tricky to get correctly | 15:57 |
| tristan | jjardon, Do we have an issue open for that ? | 15:57 |
| tristan | Would certainly be a useful thing to do | 15:57 |
| *** lachlan has quit IRC | 16:01 | |
| *** lachlan has joined #buildstream | 16:01 | |
| jjardon | tristan: not sure, I will search and if not I will open one, if that is ok | 16:01 |
| lachlan | tristan: We have had an update to !836 and I have also moved it back to WIP until we have resolved the issues with YAML cache. | 16:01 |
| tristan | jjardon, Appreciate it, tag it "Frontend" (whether it exists or not) - I think we agreed to rename that label "User Experience" | 16:02 |
| jjardon | tristan: sure | 16:03 |
| *** finn has joined #buildstream | 16:03 | |
| finn | Are the outages affecting this file? | 16:03 |
| finn | https://gnome7.codethink.co.uk/tarballs/integration-tests-base.v1.x86_64.tar.xz | 16:03 |
| finn | I can't fetch it via the autotools helloworld example. Which is having an affect on our hackathon today | 16:04 |
| tristan | lachlan, I have a few of them on my plate, and trying to avoid daily lookings at it otherwise there isn't any hope that I get the architecture diagrams done in time (already starting to look grim, hence the midnight hackery) | 16:04 |
| tiagogomes | If project A uses ${some-variable} when building some element, and; project B uses a junction to import that element but refines ${some-variable}; will the element by built with the redefined ${some-variable} | 16:04 |
| tristan | lachlan, I presume that so long as it's WIP, I can let it sleep right ? | 16:04 |
| Kinnison | finn: I believe there are outages at teledata (where that server is hosted) | 16:04 |
| tristan | tiagogomes, No | 16:04 |
| finn | Kinnison, couldn't have happened at a worse time as we're getting people to use bst >_< | 16:05 |
| jmac | skullman: You mentioned some problems with gnome7.codethink.co.uk, I think | 16:05 |
| lachlan | tristan: We are pushing to get at least the YAML Cache functionality merged by the end of this sprint, but I (for one) will defer to any higher priorities. | 16:05 |
| Kinnison | tristan: I fear that despite the WIPness, it'd be beneficial for you to have a look at 836 whenever you can | 16:05 |
| tristan | tiagogomes, For that, the sub-project needs to offer API, as far as I recall - you need the subproject to export a project option, which can be propagated/selected in the junction.bst definition | 16:05 |
| * Kinnison would really like it if we could get this YAML cache stuff bottomed by the end of the gathering | 16:05 | |
| Kinnison | It has dragged on | 16:05 |
| tristan | Kinnison, Would be nice if it passes it's CI | 16:06 |
| skullman | jmac: yep, same problem. | 16:06 |
| Kinnison | tristan: Aye, I know, I'm hoping Lachlan is working on that | 16:06 |
| tristan | Why exactly is it WIP if you are asking for a review ? | 16:06 |
| * tristan doesnt understand this really | 16:06 | |
| Kinnison | tristan: I think this is the interpretation of WIP of "It's not ready for merge, but we'd like input from you" | 16:06 |
| tristan | Kinnison, That is exactly the opposite of WIP | 16:07 |
| lachlan | We have some changes that will require a refactoring and I think it is better that we have a settled solution before this is submitted for review again | 16:07 |
| lachlan | Review with a view to merge that is | 16:07 |
| gitlab-br-bot | buildstream: issue #705 ("Provide some way of visualising the dependency graph of a given element") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/705 | 16:07 |
| Kinnison | lachlan: are you saying you *don't* want tristan to look at it? | 16:07 |
| * Kinnison wishes there was an "RFC" marker for MRs | 16:08 | |
| tristan | Ok ok... sec... Its *always* ok to ask a peer for review on a WIP merge request, including myself | 16:08 |
| tristan | With that out of the way; WIP means upstream will not pay attention | 16:08 |
| Kinnison | Okay | 16:08 |
| lachlan | Kinnison: We have a solution, but it would be better that it is agreed before I refactor and submit for merge. | 16:09 |
| tpollard | finn: any chance you could find a cache with it in? | 16:09 |
| * Kinnison is therefore asking if tristan will please continue to pay attention to 836 :-) | 16:09 | |
| Kinnison | lachlan: and I'm asking that you continue to work closely with tristan to ensure that your approach is agreed | 16:09 |
| Kinnison | :-) | 16:09 |
| tristan | Yes I know - and I'm pulling my hair trying to stay focused on the one thing I have to get done... the pipeline is not passing, and the patch series ends in a SQUASHME | 16:09 |
| tristan | which is, weird | 16:10 |
| lachlan | Kinnison: The changes are pushed - the SQUASHME will need refactoring as long as it makes sense | 16:10 |
| Kinnison | Okay, so, if I'm reading the situation clearly... The series has a patch at the end which is meant to address some/all of tristan's comments. Lachlan doesn't want to spend the time to squash that into the requisite other patches unless tristan agrees that the result of the squashme patch does indeed meet his requirements. Is that fair? | 16:11 |
| jjardon | tristan: coldtom was faster than me :) https://gitlab.com/BuildStream/buildstream/issues/705 | 16:13 |
| tristan | lachlan, https://gitlab.com/BuildStream/buildstream/merge_requests/836#note_106463082 This comment is still unadressed | 16:13 |
| tristan | Simple comment | 16:13 |
| tristan | Looks like whenever we pass multiple elements on the command line, we load/save the yaml cache for every toplevel target element | 16:14 |
| jjardon | tristan: Do you want me to open another specifically for the circular dependency case? | 16:14 |
| tristan | My last batch of comments for 836 were that the YamlCache API (A) exposed private details needlessly and (B) externalized more work to callers than was needed | 16:15 |
| coldtom | jjardon: i could edit my description to add that as a use case if you prefer | 16:15 |
| tristan | jjardon, Those are necessarily separate issues | 16:16 |
| tristan | coldtom, jjardon: I don't see how it could at all be possible to visually display a graph that we failed to load. | 16:16 |
| jjardon | tristan: agreed, let me open another | 16:18 |
| lachlan | tristan: I have forwarded your comments on to jonathanmaw | 16:18 |
| tristan | lachlan, Kinnison: looking only at the squashme commit: https://gitlab.com/BuildStream/buildstream/merge_requests/836/diffs?commit_id=dcfa4a7ddb78d424798f71c348f440217b7f61b6, it appears to address the last batch of comments yes | 16:19 |
| tristan | regarding public/outward facing surfaces of that API | 16:19 |
| Kinnison | tristan: thanks for doing that | 16:19 |
| Kinnison | So it sounds like lachlan needs to actually do the rebasing for those changes, and then look to address the remaining issues | 16:19 |
| Kinnison | Is that fair? | 16:20 |
| lachlan | jonathanmaw is looking at the loop change, it looks like the tests do not need changing | 16:20 |
| gitlab-br-bot | buildstream: issue #706 ("Information when a circular dependency is detected is not good enough") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/706 | 16:22 |
| * Kinnison will leave lachlan and jonathanmaw to it then :-) | 16:22 | |
| jjardon | tristan: done: https://gitlab.com/BuildStream/buildstream/issues/706 | 16:23 |
| gitlab-br-bot | buildstream: merge request (tristan/refactor-types-api->master: Refactor types api) #870 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/870 | 16:23 |
| tristan | Kinnison, lachlan resolved a bunch of the discussions, it appears only the loop and empty test bodies remain; and I didn't verify if the empty test bodies are still there in the branch | 16:25 |
| Kinnison | tristan: progress! Thanks for your efforts, I understand how busy you are | 16:26 |
| gitlab-br-bot | buildstream: merge request (lachlan/pickle-yaml-test-list-composite->master: WIP - Add YAM cache testing to yaml tests and further YAML cache refinements) #836 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/836 | 16:27 |
| lachlan | The empty tests should have already been removed, jonathanmaw has made further changes | 16:27 |
| jonathanmaw | I removed the empty test that I couldn't remember the purpose of | 16:28 |
| Kinnison | finn: Could you try again to fetch that tarball? | 16:28 |
| jonathanmaw | and added a comment for what the other test was meant to test | 16:28 |
| tpollard | finn: jjardon has put it back online | 16:28 |
| jonathanmaw | lachlan: can you look at that test (tests/frontend/yaml.py::test_yamlcache_changed_file) and see if it's still necessary to write a test for that behaviour? | 16:28 |
| jonathanmaw | s/yaml.py/yamlcache.py/ | 16:29 |
| lachlan | jonathanmaw : Will have a look in a bit | 16:29 |
| jonathanmaw | ta | 16:29 |
| tiagogomes | tristan how do I propagate that? options doesn't seem to be allowed in the junction definition | 16:34 |
| *** waltervargas[m] has quit IRC | 16:35 | |
| *** waltervargas[m] has joined #buildstream | 16:36 | |
| tiagogomes | As far as I can see, there's no way of extending / modifying a bst file imported through a junction. This is quite a restriction | 16:36 |
| finn | Thanks, works now :) | 16:37 |
| tristan | tiagogomes, that is a misconception, you do not modify/extend bst files imported through a junction | 16:39 |
| tristan | tiagogomes, You (A) consume the artifacts generated by the junction elements (the elements are in your pipeline), and you (B) configure the project options of the junction when declaring the junction | 16:39 |
| tristan | tiagogomes, I think you are not finding (B), lemme see how it's done... | 16:40 |
| tristan | tiagogomes, "options" dictionary in the junction configuration: https://docs.buildstream.build/elements/junction.html | 16:41 |
| tiagogomes | I understand that's by design, but this makes buildstream language much less expressive than yocto bbapend files | 16:42 |
| tristan | tiagogomes, you can use variables in the parent project to express the values you want to assign to the subproject (this allows some measure of flexibility in the parent, and chaining of commonly known project options when exporting an option to a variable) | 16:42 |
| tristan | Oh certainly it does | 16:42 |
| tristan | tiagogomes, That said, you can also *include* a file from a junction | 16:42 |
| tiagogomes | aah, options is part of the config section | 16:43 |
| tristan | So, for instance you can have a file in a sub-project define the base environment, like project-environment.inc, and then you can "(@): junction.bst:project-environment.inc" to include it somewhere; and the *including* dictionary takes precedence on the cross-junction included dictionary | 16:45 |
| tiagogomes | Right, the problem with options that is that the upstream project needs to define an option for everything a derived project wants to tweak | 16:47 |
| tristan | That is very, very much intentional with junctions yes | 16:47 |
| tristan | It is however, possible to *patch* or fork a junctioned project | 16:48 |
| tristan | That becomes a bit different than the expected usage of junctions, where the artifacts would be the same | 16:48 |
| tristan | tiagogomes, I.e. a junction element is like anything else, it can be a git repo or tarball with the BuildStream project, and can be followed by a `patch` source redefining something | 16:49 |
| *** rdale has quit IRC | 16:58 | |
| *** bochecha has joined #buildstream | 16:59 | |
| gitlab-br-bot | buildstream: merge request (lachlan/pickle-yaml-test-list-composite->master: WIP - Add YAM cache testing to yaml tests and further YAML cache refinements) #836 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/836 | 17:02 |
| *** toscalix has quit IRC | 17:02 | |
| gitlab-br-bot | buildstream: merge request (lachlan/pickle-yaml-test-list-composite->master: WIP - Add YAM cache testing to yaml tests and further YAML cache refinements) #836 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/836 | 17:13 |
| lachlan | tristan: Further updates to both address your issues and correct a problem with tmpdir resolution in the load_yaml_file function (courtesy of jonathanmaw). | 17:15 |
| *** finn has quit IRC | 17:19 | |
| *** tristan has quit IRC | 17:33 | |
| *** tristan has joined #buildstream | 17:35 | |
| gitlab-br-bot | buildstream: issue #708 ("BuildStream can crash if remote-execution endpoint is not working as expected") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/708 | 17:43 |
| lachlan | tristan: We do have one last YAML cache test that jonathanmaw has asked me to look at | 17:53 |
| *** jonathanmaw has quit IRC | 18:01 | |
| gitlab-br-bot | buildstream: merge request (juerg/remote-config->master: _artifactcache: Fix crash in push_directory() without artifact server) #871 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/871 | 18:02 |
| *** lachlan has quit IRC | 18:05 | |
| *** xjuan has joined #buildstream | 18:42 | |
| *** xjuan has quit IRC | 19:19 | |
| *** jjardon has quit IRC | 19:26 | |
| *** jjardon has joined #buildstream | 19:26 | |
| *** ChanServ sets mode: +o jjardon | 19:27 | |
| *** xjuan has joined #buildstream | 20:15 | |
| gitlab-br-bot | buildstream: issue #709 ("Per-architecture CAS servers") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/709 | 20:33 |
| jjardon | have anyone seen this before? unhandled exception when pushing to CAS: https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/107204101 | 20:37 |
| *** ChanServ sets mode: +o tristan | 21:16 | |
| tristan | jjardon, eek I sure haven't | 21:16 |
| tristan | jjardon, please file an issue, a "Name resolution failure" looks like it should be handled as a temporary error, causing a retry | 21:17 |
| jjardon | tristan: sure, I will | 21:21 |
| gitlab-br-bot | buildstream: issue #710 ("Error when pushing to the CAS server") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/710 | 21:25 |
| *** bochecha has quit IRC | 21:28 | |
| *** xjuan has quit IRC | 22:58 | |
| *** mohan43u has quit IRC | 23:14 | |
| *** xjuan has joined #buildstream | 23:14 | |
| *** mohan43u has joined #buildstream | 23:17 | |
| *** tristan has quit IRC | 23:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!