| *** zoli___ has joined #baserock | 00:06 | |
| *** zoli__ has quit IRC | 00:06 | |
| *** zoli___ has quit IRC | 04:26 | |
| *** petefoth has quit IRC | 05:26 | |
| *** zoli__ has joined #baserock | 06:24 | |
| *** petefoth has joined #baserock | 06:36 | |
| *** mike has joined #baserock | 06:58 | |
| *** mike has joined #baserock | 06:58 | |
| *** mike is now known as Guest24079 | 06:59 | |
| *** petefoth has quit IRC | 07:11 | |
| *** petefoth has joined #baserock | 07:12 | |
| *** zoli__ has quit IRC | 07:39 | |
| *** zoli__ has joined #baserock | 07:41 | |
| *** fay_ has joined #baserock | 07:42 | |
| *** zoli__ has quit IRC | 07:44 | |
| *** jonathanmaw has joined #baserock | 07:51 | |
| *** mdunford has joined #baserock | 07:58 | |
| *** bashrc has joined #baserock | 08:05 | |
| paulsherwood | is there a general email address for the infra team? | 08:22 |
|---|---|---|
| rjek | admin at baserock.org IIRC | 08:22 |
| * rjek confirms that that is the case. | 08:22 | |
| paulsherwood | tvm | 08:24 |
| *** SotK_ is now known as SotK | 08:31 | |
| *** gary_perkins has joined #baserock | 08:31 | |
| *** rdale has joined #baserock | 08:47 | |
| *** Guest24079 has quit IRC | 08:50 | |
| *** Guest24079 has joined #baserock | 09:05 | |
| *** ssam2 has joined #baserock | 09:06 | |
| *** ChanServ sets mode: +v ssam2 | 09:06 | |
| *** edcragg has joined #baserock | 09:39 | |
| nowster | ssam2: using a prefix with spaces in it will break lots of naïve scripts... | 09:42 |
| *** zoli__ has joined #baserock | 09:42 | |
| pedroalvarez | nowster: yes | 09:42 |
| ssam2 | nowster: i don't actually care about this (hence I gave a +1) but others do | 09:44 |
| ssam2 | I kind of think that if you use a space in your prefix then you deserve to suffer a little ;) | 09:45 |
| ssam2 | nowster: would you mind if i fixed up the quoting of $PREFIX in your patch, then merged it? | 09:52 |
| nowster | No problem! | 09:53 |
| ssam2 | ok, will do | 09:53 |
| nowster | ssam2: Looking at gerrit 996. Is there anything else that's blocking its inclusion? | 09:54 |
| nowster | (and its friends) | 09:54 |
| ssam2 | not that I know of | 09:54 |
| nowster | Just missing another +1? | 09:55 |
| ssam2 | yeah | 09:55 |
| nowster | Shall I trawl the office looking for victims? :) | 09:55 |
| ratmice___ | not to overreact or anything, but the 'deserve to suffer' reminds me of typhoid mary for some reason | 09:55 |
| ratmice___ | e.g. just because it works you should really avoid spreading the disease :) | 09:56 |
| ssam2 | nowster: i'm happy to merge as is, if i have time to double check them again. but "encouraging" reviewers is always welcome! | 09:56 |
| *** petefoth_ has joined #baserock | 10:18 | |
| *** radiofree has joined #baserock | 10:21 | |
| *** edcragg_ has joined #baserock | 10:21 | |
| *** petefoth has quit IRC | 10:21 | |
| *** radiofree_ has quit IRC | 10:21 | |
| *** Guest24079 has quit IRC | 10:21 | |
| *** petefoth_ is now known as petefoth | 10:22 | |
| *** edcragg has quit IRC | 10:22 | |
| *** Guest24079 has joined #baserock | 10:22 | |
| pedroalvarez | heh, a normal user can read all the /baserock metadata except strata metadata :) | 10:44 |
| ssam2 | because of permissions? heh | 10:46 |
| pedroalvarez | yes | 10:46 |
| pedroalvarez | 644 and 600 | 10:46 |
| pedroalvarez | anyway, I wanted to look into chunks metadata, so everything OK here :) | 10:47 |
| ssam2 | nowster: I didn't merge change 997 because it turns out it's mirroring a repo that is just a tarball import of the upstream tarball | 10:55 |
| nowster | ssam2: I see. | 10:55 |
| ssam2 | i think it's better to do our own tarball import, although there's no real harm in mirroring GStreamer's one | 10:55 |
| ssam2 | calling the repo libmad-tarball instead of libmad would be nice too | 10:55 |
| nowster | ok | 10:55 |
| nowster | ssam2: I've found ftp://ftp.mars.org/pub/mpeg/madplay-0.15.2b.tar.gz | 11:01 |
| ssam2 | that's the one I found, I think that's "upstream2 | 11:01 |
| ssam2 | " | 11:01 |
| nowster | sorry... you're right about libmad's latest | 11:01 |
| ssam2 | the patches in the gstreamer repo look useful, for what it's worth | 11:02 |
| nowster | let's see what the differences are | 11:02 |
| nowster | ugh! Nasty nasty repo. | 11:02 |
| nowster | warning: remote HEAD refers to nonexistent ref, unable to checkout. | 11:02 |
| nowster | no branches | 11:03 |
| nowster | only tags | 11:03 |
| nowster | It looks to me like libmad is not currently under active development (last activity 2006). | 11:06 |
| nowster | ...and the patches from gstreamer look useful for any ARM builds. | 11:06 |
| ssam2 | yes, that's what I thought | 11:07 |
| *** mdunford has quit IRC | 11:07 | |
| ssam2 | but as you say, the GStreamer repo is a little weird, and we can do our own tarball imports | 11:08 |
| ssam2 | do you know how to write an appropriate .lorry file? I can do it if not | 11:09 |
| ssam2 | thinking a bit more, it'd be hard for you to push those patches to a delta/libmad repo on git.baserock.org without push access to git.baserock.org | 11:10 |
| ssam2 | do you have access? I can add you if not | 11:10 |
| ssam2 | alternately, we could merge the current lorry but call the repo libmad-gstreamer-sdk or some such thing | 11:11 |
| nowster | I don't have access. | 11:13 |
| ssam2 | ok. which do you prefer out of using the gstreamer fork, or doing our own tarball import? if you're going to be doing more work on the Baserock reference systems, it'd probably be useful if you had push access to git.baserock.org anyway. i think you've been around here long enough we can trust you :) | 11:15 |
| nowster | ssam2: :) | 11:15 |
| nowster | If we do the tarball import and then layer patches on top, will that affect the lorry mechanism? | 11:15 |
| ssam2 | i'm not quite sure what you mean... it'll mean we need to write a different .lorry file | 11:16 |
| ssam2 | easy to do though, e.g. http://git.baserock.org/cgi-bin/cgit.cgi/baserock/local-config/lorries.git/tree/open-source-lorries/autoconf-tarball.lorry | 11:16 |
| ssam2 | just give the URL to the tarball | 11:16 |
| nowster | If we go with the Underbit tarball, but apply the gstreamer ARM patches on top, how will that work? | 11:18 |
| ssam2 | once Lorry has created a delta/libmad repo, you can clone it, apply the patches in a branch based off the libmad-0.15.1b branch, then push that as 'baserock/0.15.1b-gstreamer-sdk-fork' or some such thing | 11:19 |
| ssam2 | so we fork it but pretend that we haven't because it's more effort to 'officially' have forked it ;) | 11:19 |
| ssam2 | and i doubt anyone from the Baserock project is going to do major changes to it | 11:19 |
| nowster | :) | 11:20 |
| ssam2 | it's more effort up front, "cleaner" in a theoretical sense, but i'm happy for us to take the short cut of just mirroring the gstreamer sdk's fork | 11:20 |
| nowster | right... Let's go with that plan. I'll make a libmad-tarball.lorry if you give me push access to the Baserock repos. | 11:21 |
| ssam2 | ok, will do | 11:21 |
| ssam2 | please mail me your SSH public key | 11:21 |
| nowster | wilco | 11:22 |
| *** mdunford has joined #baserock | 11:22 | |
| ssam2 | does anyone have thoughts on how we should store default split rules and default build commands in definitions.git ? | 11:23 |
| ssam2 | my first idea is to have a file named DEFAULTS at the top level which is YAML | 11:24 |
| ssam2 | and has a 'build-systems' list, containing 'autotools', 'cpan', etc. and the configure-commands, build-commands, and so on for each one | 11:24 |
| ssam2 | and also a 'split-rules' entry, containing 'chunk' and 'stratum', and the regex patterns for each split rule | 11:25 |
| ssam2 | i wonder if it makes sense to put 'repo-alias' in there as well... which raises the question of how we inject 'trove-host' into the URLs (e.g. upstream: prefix expands to git://{TROVE_HOST}/delta) | 11:26 |
| ssam2 | i'll come up with a prototype soon, i just wondered if anyone has thought of this already | 11:26 |
| pedroalvarez | ssam2: my idea was to do something like: | 11:26 |
| pedroalvarez | build-system: strata/defaults/autotools.morph | 11:26 |
| pedroalvarez | in the chunk morphologies. | 11:26 |
| pedroalvarez | but I didn't like the idea of having paths there | 11:27 |
| ssam2 | right... one aspect that confused me is how to avoid hardcoding special paths | 11:27 |
| ssam2 | I don't really want there to be a magic 'defaults/' directory with magic files inside | 11:27 |
| pedroalvarez | but then you are going to hardcode the special path to DEFAULTS in the build tool | 11:28 |
| ssam2 | indeed, but we already do that with VERSION ;) | 11:28 |
| pedroalvarez | fair | 11:28 |
| ssam2 | specifying the 'defaults' path in each .morph file is a good alternative, though | 11:28 |
| ssam2 | maybe i'll try both | 11:28 |
| tiagogomes | I would rather hardcode the build tool to read from DEFAULTS | 11:30 |
| pedroalvarez | regarding repo-alias I don't see any benefit on adding a DEFAULT | 11:30 |
| pedroalvarez | default* | 11:30 |
| ssam2 | yeah, maybe so. I guess the real problem is that it's a pain to change it in morph.conf | 11:30 |
| ssam2 | but that could be fixed in Morph | 11:31 |
| *** radiofree has quit IRC | 11:31 | |
| ssam2 | and in YBD the repo-alias can already be specified by a file committed in definitions.git | 11:31 |
| pedroalvarez | hm.. but I want to be able to build definitions.git using my local trove without adding a patch | 11:31 |
| tiagogomes | If you write `build-system: strata/defaults/autotools.morph` that's not defaults anymore, as you are referencing a specific configuration | 11:31 |
| *** radiofree has joined #baserock | 11:32 | |
| ssam2 | tiagogomes: it gives a lot more flexibility. which could be good or bad :) | 11:32 |
| ssam2 | i see your point that it's not really a 'default' in the same sense | 11:33 |
| ssam2 | could be 'build-system: base/autotools.morph' or something | 11:33 |
| tiagogomes | I would call it complexity instead :) If the defaults don't work, for some project, you can always change them in the chunk morph | 11:34 |
| tiagogomes | and if you added `build-system: path-to-build-system`, you would also need to add `split-rules: path-to-split-rules` | 11:36 |
| ssam2 | we might be able to make the split rules build-system specific | 11:36 |
| ssam2 | but haven't actually tried that | 11:36 |
| ssam2 | nowster: you should be in 'baserock-writers' group on git.baserock.org now, try 'ssh git@git.baserock.org whoami' to test | 11:38 |
| tiagogomes | a compromise, could be putting `build-system: path-to-defaults` in the system morphologies | 11:38 |
| ssam2 | 'defaults: path-to-defaults' would make more sense then! | 11:38 |
| tiagogomes | betters, `defaults: path-to-defaults` in the system morphologies | 11:38 |
| ssam2 | snap | 11:38 |
| tiagogomes | meh, you typed faster :) | 11:39 |
| ssam2 | I think I prefer the DEFAULTS approach, but i haven't tried to write a DEFAULTS file yet, so maybe i'll change my mind :) | 11:39 |
| nowster | ssam2: baserock-writers: Those with write access to Baserock repositories | 11:42 |
| nowster | ssam2: ta | 11:42 |
| nowster | ssam2: and ta | 11:51 |
| * paulsherwood would prefer one defaults declaration, rather than having to declare in each morph file, fwiw | 11:58 | |
| paulsherwood | currently ybd has https://github.com/devcurmudgeon/ybd/blob/master/ybd.conf, overridable by having a ybd.conf in the definitions directory - could defaults be done in a similar way? | 11:59 |
| *** Guest24079 has quit IRC | 12:03 | |
| ssam2 | paulsherwood: that's what I'm proposing with DEFAULTS, really | 12:07 |
| *** zoli__ has quit IRC | 12:08 | |
| ssam2 | if a build tool wants to provide a way for users to override them outside of definitions.git, I think that's up to the build tool authors, rather than the definitions format itself | 12:08 |
| paulsherwood | :) | 12:08 |
| *** zoli__ has joined #baserock | 12:09 | |
| *** zoli___ has joined #baserock | 12:10 | |
| *** zoli__ has quit IRC | 12:14 | |
| *** Guest24079 has joined #baserock | 12:16 | |
| nowster | ssam2: does http://git.baserock.org/cgi-bin/cgit.cgi/delta/libmad-tarball.git look sane to you? | 12:18 |
| ssam2 | looks great, thanks for doing that | 12:32 |
| nowster | now to fix the definitions so that they point at git.baserock, rather than file:///... | 12:35 |
| ssam2 | first pass at a DEFAULTS file: http://paste.baserock.org/opevimacoz | 12:38 |
| persia | ssam2: Am I correct in assuming that chunk definitons must declare one of those build systems, or include a custom build-system? | 12:41 |
| nowster | I *think* that there's some sort of autodetection. | 12:42 |
| persia | I worry a bit if that is true and DEFAULTS is as written. | 12:44 |
| ssam2 | persia: definitions version 6 removes autodetection, so every chunk must specify a build-system | 12:51 |
| ssam2 | so, from v6, every chunk must declare one of those (although it can be declared in the stratum, to avoid needing lots of extra chunk .morph files) | 12:51 |
| persia | Ah, good. We'll have to see how well this works in practice, but I don7t see any obvious issues. | 12:52 |
| *** tiagogomes has quit IRC | 13:04 | |
| *** 7GHAAS8M4 has joined #baserock | 13:15 | |
| *** zoli___ has quit IRC | 13:17 | |
| *** zoli__ has joined #baserock | 13:18 | |
| *** zoli___ has joined #baserock | 13:19 | |
| *** zoli__ has quit IRC | 13:22 | |
| *** zoli___ has quit IRC | 13:27 | |
| *** zoli__ has joined #baserock | 13:27 | |
| *** 7GHAAS8M4 is now known as tiagogomes_ | 13:48 | |
| *** rdale has quit IRC | 13:53 | |
| *** rdale has joined #baserock | 13:53 | |
| *** _longines has quit IRC | 13:54 | |
| *** _longines has joined #baserock | 13:55 | |
| *** inara` has quit IRC | 15:29 | |
| *** Guest24079 has quit IRC | 15:31 | |
| *** ssam2 has quit IRC | 15:34 | |
| *** inara has joined #baserock | 15:37 | |
| *** Guest24079 has joined #baserock | 15:43 | |
| *** ssam2 has joined #baserock | 15:47 | |
| *** ChanServ sets mode: +v ssam2 | 15:47 | |
| * pedroalvarez sends https://gerrit.baserock.org/1005 after seeing Mason failing | 15:59 | |
| *** jonathanmaw has quit IRC | 16:02 | |
| ssam2 | good catch, thanks | 16:03 |
| *** robtaylor has quit IRC | 16:18 | |
| *** Guest24079 has quit IRC | 16:31 | |
| *** bashrc has quit IRC | 16:50 | |
| *** ssam2 has quit IRC | 17:00 | |
| pedroalvarez | if we ever build something in Go in baserock, we might have some problems | 17:01 |
| pedroalvarez | I might be wrong, but this Go component I'm looking at has some imports with URL's to github | 17:01 |
| jmacs | Yes, I think that's a common complaint about Go | 17:02 |
| pedroalvarez | git grep github | wc -l | 17:02 |
| pedroalvarez | 349 | 17:02 |
| *** mdunford has quit IRC | 17:09 | |
| *** mdunford has joined #baserock | 17:37 | |
| *** edcragg_ has quit IRC | 17:39 | |
| *** mdunford has quit IRC | 18:16 | |
| *** brlogger has joined #baserock | 18:35 | |
| *** edcragg has joined #baserock | 21:42 | |
| *** edcragg has quit IRC | 23:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!