| *** zoli_ has joined #baserock | 00:01 | |
| *** tiagogomes_ has quit IRC | 00:15 | |
| *** edcragg has quit IRC | 00:21 | |
| *** tiagogomes_ has joined #baserock | 00:32 | |
| *** locallycompact has quit IRC | 00:34 | |
| *** gtristan has joined #baserock | 02:12 | |
| *** CTtpollard has quit IRC | 08:10 | |
| *** CTtpollard has joined #baserock | 08:23 | |
| *** bruce_ has joined #baserock | 08:26 | |
| *** gtristan has quit IRC | 08:34 | |
| paulsherwood | jjardon: http://paste.baserock.org/ohaciduvey | 08:47 |
|---|---|---|
| paulsherwood | actually, this exposes an interesting problem in the concourse pipeline we are testing | 08:54 |
| paulsherwood | the definitions changes have made the pipeline itself obsolete :) | 08:55 |
| pedroalvarez | mason-x86-64.baserock.org was complaining too | 09:01 |
| paulsherwood | is it just that lorry hadn't done that repo yet? | 09:02 |
| paulsherwood | it seems to be there now | 09:02 |
| jjardon | Oh, submodules; patch coming | 09:03 |
| *** bashrc has joined #baserock | 09:06 | |
| *** edcragg has joined #baserock | 09:24 | |
| *** gtristan has joined #baserock | 09:25 | |
| paulsherwood | jjardon: it seems to be working now | 09:27 |
| paulsherwood | but yes, there are submodules to fix | 09:27 |
| *** jonathanmaw has joined #baserock | 09:30 | |
| *** franred has joined #baserock | 09:36 | |
| *** rdale has joined #baserock | 09:37 | |
| *** toscalix has joined #baserock | 09:43 | |
| pedroalvarez | wow, that DEFAULT's change will rebuild the world :) | 09:51 |
| pedroalvarez | I wonder if it has been tested though | 09:51 |
| tiagogomes_ | sounds like another bug, unless the split rules changed | 09:56 |
| *** ssam2 has joined #baserock | 10:09 | |
| *** ChanServ sets mode: +v ssam2 | 10:09 | |
| *** rdale has quit IRC | 10:15 | |
| *** rdale has joined #baserock | 10:16 | |
| *** edcragg has quit IRC | 10:19 | |
| jjardon | pedroalvarez: I was about to ask that: it didnt rebuild the world here, so not sure that was a bug or not | 10:19 |
| pedroalvarez | oh! | 10:19 |
| pedroalvarez | but it should almost rebuild everything, right? | 10:20 |
| franred | IMO, it should rebuild all the packages which use autotools | 10:20 |
| *** locallycompact has joined #baserock | 10:21 | |
| *** CTtpollard has quit IRC | 10:24 | |
| *** CTtpollard has joined #baserock | 10:28 | |
| *** jonathanmaw has quit IRC | 10:30 | |
| *** paulw has joined #baserock | 10:30 | |
| *** jonathanmaw has joined #baserock | 10:32 | |
| *** edcragg has joined #baserock | 10:36 | |
| pedroalvarez | franred: yup, and since zlib has that build system, and is in build-essentials.. | 10:46 |
| pedroalvarez | rebuild the world it is | 10:46 |
| paulsherwood | is it merged? | 10:46 |
| jjardon | How I tell morph/ybd to ignore previous caches and do a full build? | 10:50 |
| paulsherwood | jjardon: mv artifacts dir? | 10:51 |
| ssam2 | if you change a build-system in DEFAULTS then it should automatically rebuild anything that uses that build-system | 10:56 |
| pedroalvarez | jjardon: does your question mean that either morph/ybd are not rebuilding anything with your patch in? | 10:56 |
| ssam2 | because the build commands are part of the cache key | 10:56 |
| ssam2 | so if the build commands change, the cache key changes | 10:57 |
| * ssam2 congratulates jjardon for misspelling the name of the 'spell-checking' stratum in https://gerrit.baserock.org/#/c/1825/1 | 11:00 | |
| jjardon | :) | 11:00 |
| Zara | =D | 11:00 |
| jjardon | pedroalvarez: no, they are not rebuilding the stuff | 11:00 |
| paulsherwood | bug, then | 11:01 |
| paulsherwood | where is your branch, jjardon? | 11:01 |
| pedroalvarez | `git fetch https://gerrit.baserock.org/baserock/baserock/definitions refs/changes/18/1818/1 && git checkout FETCH_HEAD` | 11:01 |
| pedroalvarez | there are checkout, cherry-pick, etc buttons in gerrit | 11:01 |
| jjardon | paulsherwood: very useful to see what the commands are actually doing, btw | 11:02 |
| paulsherwood | :) | 11:03 |
| pedroalvarez | I can say that Morph doesn't have that bug | 11:04 |
| pedroalvarez | and that the patch doesn't work | 11:04 |
| pedroalvarez | http://paste.baserock.org/cecujivugu | 11:04 |
| ssam2 | hmm.. good catch | 11:05 |
| ssam2 | however I think the problem is that ZLib isn't autotools | 11:05 |
| ssam2 | so we should probably stop pretending that it is | 11:05 |
| * richard_maw pointed this out in his review of the patch | 11:05 | |
| *** tiagogomes_ has quit IRC | 11:06 | |
| *** jonathanmaw has quit IRC | 11:09 | |
| pedroalvarez | you did :) | 11:09 |
| jjardon | ok, lets fix the incorrect chunks then! | 11:09 |
| paulsherwood | and ybd :/ | 11:09 |
| jjardon | pedroalvarez: sorry, I though I tried this with morph but seems I only used ybd | 11:10 |
| pedroalvarez | no worries, I just wanted to double check | 11:11 |
| *** jonathanmaw has joined #baserock | 11:12 | |
| *** CTtpollard has quit IRC | 11:18 | |
| *** cosm has joined #baserock | 11:20 | |
| *** tiagogomes_ has joined #baserock | 11:21 | |
| locallycompact | I'm looking at morph and I can't figure out how it handles nested submodules. update_submodules(app, repo_dir) doesn't return anything or call itself on its submodules. | 11:21 |
| *** CTtpollard has joined #baserock | 11:21 | |
| ssam2 | maybe it doesn't | 11:23 |
| richard_maw | locallycompact: the Submodules.load() function does the depth walking | 11:24 |
| ssam2 | hmm | 11:24 |
| ssam2 | oh yeah | 11:24 |
| locallycompact | does it? how? | 11:25 |
| richard_maw | ah, I misparsed Submodule as Submodules, hmm | 11:26 |
| richard_maw | ah, you need morphlib.builder.extract_sources | 11:27 |
| *** cosm has quit IRC | 11:33 | |
| ssam2 | as usual Morph has two implementations of the same thing :-) | 11:37 |
| *** paulw has quit IRC | 11:38 | |
| locallycompact | oh | 11:43 |
| locallycompact | update_submodules is only called from plugins | 11:43 |
| locallycompact | what do those do? | 11:43 |
| pedroalvarez | "get_repo_plugin.py": you give it the name of a chunk and a path, and it will clone the repo in that path | 11:44 |
| locallycompact | extract_sources doesn't seem to make any calls to RepoAliasResolver though. | 11:47 |
| *** cosm has joined #baserock | 11:47 | |
| richard_maw | locallycompact: it gets the repo object from the LocalRepoCache.get_repo() method | 11:52 |
| richard_maw | which uses the resolver to fill in the url | 11:52 |
| locallycompact | ok | 11:54 |
| ssam2 | my favourite thing about the get_repo_plugin is that there is an expand_repo_plugin that does the same thing :-) | 11:56 |
| ssam2 | which is largely my fault | 11:56 |
| tiagogomes_ | Both need to die | 12:00 |
| jjardon | ok to merge the zlib fix? https://gerrit.baserock.org/#/c/1830/ | 12:02 |
| * richard_maw reviews | 12:03 | |
| locallycompact | I'm unsure how to make this change without breaking something | 12:05 |
| locallycompact | Does app carry a copy of definitions I can query? | 12:07 |
| richard_maw | it didn't the last time I looked | 12:08 |
| richard_maw | (partially because we don't have a neat data structure describing definitions) | 12:08 |
| richard_maw | (also because eww, singletons) | 12:08 |
| franred | jjardon, looks ok to me, I haven't tested, though | 12:13 |
| ssam2 | locallycompact: there are two code paths that are affected by this I think | 12:16 |
| *** cosm has quit IRC | 12:16 | |
| ssam2 | locallycompact: one codepath is fairly easy, it uses the DefinitionsRepo class | 12:16 |
| ssam2 | actually I'm not sure that has anything to do with submodules anyway | 12:17 |
| ssam2 | the other code path goes through the SourceResolver module | 12:18 |
| ssam2 | which creates a bunch of Source objects | 12:18 |
| ssam2 | which can carry any data, e.g. what submodules were specified in the stratum .morph file | 12:18 |
| *** cosm has joined #baserock | 12:28 | |
| *** edcragg has quit IRC | 12:38 | |
| *** edcragg has joined #baserock | 12:40 | |
| *** cosm has quit IRC | 13:07 | |
| jjardon | Hey ssam2 , seems the '>-' trick doesn't work as expected; I still have to use '\' | 13:25 |
| *** gtristan has quit IRC | 13:28 | |
| richard_maw | jjardon: what's your code? | 13:30 |
| jjardon | richard_maw: https://gerrit.baserock.org/#/c/1817/3/DEFAULTS | 13:31 |
| richard_maw | jjardon: weird, you need to align your --prefix and --sysconfdir to the same column as the ./configure | 13:36 |
| jjardon | richard_maw: uh, is this the intended behaviour in the yalm spec? or is it a bug in the parser? | 13:37 |
| richard_maw | don't know | 13:37 |
| richard_maw | jjardon: added a comment of the other style | 13:39 |
| jjardon | richard_maw: ah, great, thanks | 13:41 |
| *** ctbruce has joined #baserock | 13:51 | |
| *** bruce_ has quit IRC | 13:51 | |
| jjardon | richard_maw: btw, if you have a moment, I think this is what you wanted to happen? https://gerrit.baserock.org/#/q/status:open+project:baserock/baserock/definitions+branch:master+topic:python3-distutils | 14:04 |
| *** ctbruce has quit IRC | 14:06 | |
| ssam2 | jjardon: that's annoying! | 14:09 |
| ssam2 | jjardon: seems the normal mode works though, good | 14:09 |
| jjardon | mmm, I have a problem here; I need to add libva-intel-driver to be able to use the libva in intel platforms: Id put this in the bsp stratum, but this will make that stratum depend on libdrm, wayland and x11. O ther solution is to put it in multimedia-gstreamer; but this will break the build in ARM; any other suggestion? | 14:10 |
| jjardon | I guess I can always create a completely different stratum for this ... | 14:10 |
| paulsherwood | new stratum, arch-specific | 14:11 |
| pedroalvarez | +1 to new stratum, arch-specific | 14:14 |
| jjardon | what do you mean by arch-specific? | 14:17 |
| paulsherwood | only to be used in systems where arch: x86_64, so by convention include x86_64 in the name | 14:18 |
| pedroalvarez | exactly, that would be enougg | 14:19 |
| pedroalvarez | -g+h | 14:19 |
| jjardon | ah, yeah sure; Im suing x86 only actually | 14:19 |
| * paulsherwood is surprised that bsp*.morph don't actually specify arch: | 14:19 | |
| pedroalvarez | almost all of them | 14:20 |
| *** ctbruce has joined #baserock | 14:20 | |
| pedroalvarez | well, in the name, if that's what you mean | 14:21 |
| paulsherwood | i mean surprising they don't have an arch: field, as systems do | 14:21 |
| *** toscalix has quit IRC | 14:27 | |
| *** toscalix has joined #baserock | 14:29 | |
| jjardon | vulkan is here: https://www.khronos.org/vulkan/ | 14:52 |
| jjardon | paulsherwood: pedroalvarez I think I found a better approach in the end: https://gerrit.baserock.org/#/q/topic:hardware_codecs | 15:02 |
| *** bashrc has quit IRC | 15:03 | |
| pedroalvarez | that's what paulsherwood suggested I believe | 15:04 |
| *** bashrc has joined #baserock | 15:05 | |
| jjardon | pedroalvarez: I've moved all the libva stuff there | 15:05 |
| pedroalvarez | jjardon: the approach works for me | 15:10 |
| *** cosm has joined #baserock | 15:17 | |
| *** gtristan has joined #baserock | 16:18 | |
| *** cosm has quit IRC | 16:30 | |
| paulsherwood | http://paste.baserock.org/ofegoleyer | 16:30 |
| paulsherwood | node is broken in master for ybd? | 16:30 |
| pedroalvarez | "configure: error: no such option: --sysconfdir" | 16:31 |
| pedroalvarez | jjardon: ^ | 16:31 |
| pedroalvarez | s/ for ybd?// | 16:32 |
| paulsherwood | same in morph? | 16:32 |
| jjardon | paulsherwood: node doesn't use autotools, the node morph file needs to be fixed | 16:35 |
| jjardon | paulsherwood: can you try the patch from: git fetch https://gerrit.baserock.org/baserock/baserock/definitions refs/changes/35/1835/1 && git checkout FETCH_HEAD | 16:42 |
| *** cosm has joined #baserock | 16:44 | |
| * pedroalvarez +1s https://gerrit.baserock.org/#/c/1835 | 16:46 | |
| * richard_maw wonders if we need an autotools-like build-system which just takes --prefix | 16:47 | |
| *** ctbruce has quit IRC | 16:54 | |
| paulsherwood | jjardon: +1 | 16:55 |
| paulsherwood | works, merged | 16:55 |
| pedroalvarez | thanks | 16:56 |
| ssam2 | richard_maw: we could maybe consider 'autotools-like' things to use the 'build-api' (https://github.com/cgwalters/build-api) | 17:04 |
| ssam2 | interestingly the build API says "The configure script MUST either ignore unknown options that start with --enable- and --disable-, or accept an argument --help which prints valid options." | 17:05 |
| ssam2 | which means that a configure script that blows up on unknown arguments is still valid according to the build-api. not sure if that's useful or not | 17:06 |
| * richard_maw isn't sure we can guarantee that --libdir is accepted, so wouldn't use build-api for node or zlib | 17:07 | |
| jjardon | any reason why gdp is not using stock weston? | 17:14 |
| jjardon | pedroalvarez: maybe you know? | 17:14 |
| paulsherwood | radiofree probably knows more? | 17:14 |
| richard_maw | wouldn't surprise me to discover that it's locked to an old version because they haven't updated the IVI-shell plugins | 17:14 |
| paulsherwood | heh | 17:15 |
| jjardon | at least agl uses stock weston; maybe our definitions are old, as the gdp specific patches has been already upstreamed? | 17:15 |
| ratmice | no mouse around, so hopefully these urls work, but they are the origin of things like libdir... | 17:15 |
| ratmice | https://www.gnu.org/prep/standards/html_node/Directory-Variables.html#Directory-Variables | 17:16 |
| ratmice | https://www.gnu.org/prep/standards/html_node/Configuration.html#Configuration | 17:17 |
| pedroalvarez | jjardon: gdp needs to update some components before they can move to weston 1.9.0 without patches | 17:17 |
| ratmice | so, to be compatible with gnu coding standards configure *should* accept libdir | 17:17 |
| pedroalvarez | jjardon: I've heard that will happen soon | 17:17 |
| pedroalvarez | I tried to implement the changes needed but failed | 17:18 |
| jjardon | pedroalvarez: ah rigth, thanks for the heads up | 17:18 |
| pedroalvarez | I have already tested genivi baseline with weston 1.9.0 and works nice | 17:19 |
| pedroalvarez | but didn't want to break gdp | 17:19 |
| pedroalvarez | so I'll upstream the changes once it's possible | 17:19 |
| jjardon | pedroalvarez: what weston version is being used in gdp? | 17:20 |
| ratmice | so, its weird that build-api and gnu coding standards differ here between should and must... | 17:21 |
| jjardon | uh, 1.6+patches; agl is using 1.8 | 17:24 |
| paulsherwood | jjardon: plase go ahead and update gdp if you can | 17:25 |
| jjardon | I guess the layer_manager is the component that is holding the upgrade? | 17:26 |
| pedroalvarez | jjardon: afaict, gdp-hmi is not compatible with the required version of wayland-ivi-extension | 17:28 |
| pedroalvarez | jjardon: I tried this changes, and managed to compile it http://git.baserock.org/cgi-bin/cgit.cgi/delta/genivi/genivi-demo-platform-hmi.git/log/?h=baserock/pedroalvarez/update-new-wayland-ivi-exts | 17:29 |
| pedroalvarez | but everything in the GUI was wrong when running it | 17:30 |
| jjardon | mmm, that repo looks quite dead; last commit in master almost a year ago | 17:33 |
| * locallycompact identifies a grand total of one jillion thing still pulling submodules from github | 17:33 | |
| jjardon | pedroalvarez: what is this component suppose to do? | 17:33 |
| richard_maw | jjardon: I think that's the example GUI that jonathanmaw was showing up | 17:33 |
| *** ssam2 has quit IRC | 17:41 | |
| *** ssam2 has joined #baserock | 17:42 | |
| *** ChanServ sets mode: +v ssam2 | 17:42 | |
| pedroalvarez | yup, that's the shiny launcher of GDP :P | 17:46 |
| pedroalvarez | TIL: hmi stands for human machine interface | 17:54 |
| richard_maw | because automotive appears to want its own acronyms for everything | 17:56 |
| jmacs | "HMI" is broader than automotive | 17:57 |
| *** jonathanmaw has quit IRC | 17:59 | |
| *** bashrc has quit IRC | 18:05 | |
| *** edcragg has quit IRC | 18:20 | |
| gtristan | fwiw... the bug reported here: http://wiki.baserock.org/guides/arm-be-7/ | 18:21 |
| gtristan | is actually a bug discussed here: https://gmplib.org/list-archives/gmp-bugs/2015-December/003850.html | 18:21 |
| gtristan | and can be worked around by using the gmp library on the gcc infrastructure site (instead of bleeding edge 6.1) | 18:22 |
| *** rdale has quit IRC | 18:23 | |
| *** tiagogomes_ has quit IRC | 18:30 | |
| jjardon | Hi, is it ok to introduce a generic weston system with qt (and qtwayland)? https://gerrit.baserock.org/#/q/status:open+project:baserock/baserock/definitions+branch:master+topic:weston-qt5-system | 18:37 |
| *** toscalix has quit IRC | 18:44 | |
| *** ssam2 has quit IRC | 18:56 | |
| pedroalvarez | gtristan: good to know! Id like to remove our workaround | 19:10 |
| *** bruce_ has joined #baserock | 19:59 | |
| *** toscalix has joined #baserock | 20:02 | |
| *** toscalix has quit IRC | 20:06 | |
| *** bruce_ has quit IRC | 21:28 | |
| *** edcragg has joined #baserock | 23:07 | |
| *** trn has quit IRC | 23:37 | |
| *** trn has joined #baserock | 23:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!