2016-12-10 15:44:29 <andypost> clandmeter, how can I check build for aarch64 for https://github.com/alpinelinux/aports/pull/595
2016-12-10 15:44:44 <andypost> looks php5 nuild affected the same issue
2016-12-10 15:44:50 <andypost> *build
2016-12-10 17:40:27 <fcolista> hi guys
2016-12-10 17:40:28 <fcolista> re:
2016-12-10 17:40:29 <fcolista> https://bugs.alpinelinux.org/issues/6430
2016-12-10 17:40:50 <fcolista> would be fine if i remove the -doc subpackage in perl-unicode-collate-doc?
2016-12-10 17:40:55 <fcolista> perl-doc already include it
2016-12-10 17:42:42 <fcolista> Umh
2016-12-10 17:42:49 <fcolista> a diff shows that they are different
2016-12-10 18:15:00 <jirutka> fcolista: I’d be imho better to just remove conflicting files
2016-12-10 18:15:28 <jirutka> fcolista: why does it install man pages for some test components anyway?
2016-12-10 18:16:50 <fcolista> jirutka, why you say "for some test components"? What you mean? It's a subpackage
2016-12-10 18:17:27 <jirutka> usr/share/man/man3/Test::Builder.3pm.gz
2016-12-10 18:18:01 <fcolista> that's done by perl
2016-12-10 18:18:07 <fcolista> not perl-unicode-collate
2016-12-10 18:18:23 <jirutka> aha
2016-12-10 18:18:56 <jirutka> i don’t understand what’s going on here
2016-12-10 18:19:13 <jirutka> does perl already bundle unicode-collate, or why it contains man pages for it?!
2016-12-10 18:19:45 <fcolista> That's the same i was wondering..that perl-unicode-collate is now part of perl-core
2016-12-10 18:20:07 <fcolista> But i didn't find a reference to it
2016-12-10 18:20:27 <fcolista> the only one is this:
2016-12-10 18:20:28 <fcolista> http://perldoc.perl.org/Unicode/Collate.html
2016-12-10 18:20:51 <fcolista> where the top of the menu mention Unicode::Collate as part oft the core
2016-12-10 18:20:58 <fcolista> nut i'm not 100% sure
2016-12-10 18:22:39 <jirutka> well, i think that when you remove -doc subpkg from some perl-* packages, no one will complain ¯\_(ツ)_/¯
2016-12-10 18:24:17 <fcolista> just the builder complains
2016-12-10 18:24:48 <fcolista> and also i don't know if the man pages installed by perl-doc are correct
2016-12-10 18:24:54 <fcolista> for perl-unicode-collate
2016-12-10 18:25:32 <fcolista> how to manage conflicts?
2016-12-10 18:26:05 <jirutka> why the builder complains? ofc, when you remove -doc from subpkgs, you must also delete /usr/share/man in package()
2016-12-10 18:27:46 <jirutka> fcolista: using https://wiki.alpinelinux.org/wiki/APKBUILD_Reference#replaces, but it’s not a correct solution in this case IMO
2016-12-10 18:28:05 <jirutka> one of these man pages is probably wrong
2016-12-10 18:29:03 <fcolista> yeah..and i think is perl-doc
2016-12-10 18:42:36 <kazblox> does anyone know if alpine has autoconf-archive
2016-12-10 19:06:47 <andypost> jirutka, is there a way to test build under aarch64?
2016-12-10 19:52:11 <jirutka> andypost: if you don’t have aarch64 HW, then unfortunately not (yet)
2016-12-10 19:54:31 <andypost> jirutka, sad cos if PR595 works I'd like to make MR to upstream
2016-12-10 20:07:09 <jirutka> andypost: well, i can try to merge it and we will see
2016-12-10 20:09:52 <andypost> jirutka, would be great because php5 also have no aarch64 build for the same reason
2016-12-10 20:11:32 <jirutka> andypost: please add some explanation to https://github.com/alpinelinux/aports/pull/595/files#diff-e713f68eaa54e3f26802a50e0dc11841
2016-12-10 20:11:49 <jirutka> andypost: i.e. what it’s trying to fix and maybe even that link to SO
2016-12-10 20:12:40 <andypost> sure
2016-12-10 20:16:58 <andypost> jirutka, updated summary
2016-12-10 20:19:40 <jirutka> andypost: I meant add it directly to the patch file, not PR
2016-12-10 20:20:07 <jirutka> andypost: sumary in PR is _not_ stored in git repository
2016-12-10 20:22:53 <andypost> jirutka, what kind of comment makes sense? the same code just above for arm https://github.com/php/php-src/blob/master/Zend/zend_multiply.h#L71
2016-12-10 20:26:24 <jirutka> andypost: it can be the same text from your summary, just copied to the patch file, like in https://github.com/alpinelinux/aports/blob/master/testing/rust/llvm-with-ffi.patch
2016-12-10 20:26:42 <kazblox> <kazblox>does anyone know if alpine has autoconf-archive | asking this because punes can't build without it
2016-12-10 20:26:59 <jirutka> andypost: so when someone look at it later, (s)he can understand why is that patch here
2016-12-10 20:27:06 <andypost> jirutka, got thanx!
2016-12-10 20:27:22 <jirutka> andypost: it’s also useful to add snippet of the build error that you’re trying to fix
2016-12-10 20:27:49 <jirutka> kazblox: I don’t see such package here http://pkgs.alpinelinux.org/packages?name=autoconf-archive&branch=&repo=&arch=&maintainer=
2016-12-10 20:28:05 <jirutka> kazblox: but I have no idea what autoconf-archive is
2016-12-10 20:28:20 <kazblox> https://www.gnu.org/software/autoconf-archive/
2016-12-10 20:28:43 <jirutka> kazblox: I can google it… but I don’t know if it’s some binary or just m4 macros or what
2016-12-10 20:28:48 <kazblox> it's m4 macros
2016-12-10 20:29:03 <kazblox> m4/autoconf macros*
2016-12-10 20:41:01 <andypost> jirutka, I used original patch
2016-12-10 20:41:23 <andypost> jirutka, maybe add link to PR where the details are?
2016-12-10 20:43:31 <jirutka> andypost: I see now, this is definitely better; I’d also add link to https://bugs.php.net/bug.php?id=70015 into the patch
2016-12-10 20:43:54 <andypost> 
ACTION doing

2016-12-10 20:45:28 <andypost> done
2016-12-10 20:58:08 <jirutka> here we go
2016-12-10 21:10:39 <jirutka> andypost: it works! \o/
2016-12-10 21:10:47 <andypost> yay!
2016-12-10 21:11:12 <andypost> now it needs the same pacth for php5 and only readline left
2016-12-10 21:15:15 <andypost> jirutka, does it makes sense to add this patch https://github.com/alpinelinux/aports/pull/592 or better separate PR?
2016-12-10 21:18:34 <jirutka> andypost: do I understand it correctly that the patch fixes bug introduced in security release?
2016-12-10 21:20:01 <andypost> jirutka, not, pr592 is just upgrade but php5 has the same issue with build under aarch64
2016-12-10 21:20:52 <jirutka> but php5 is currently built for aarch64
2016-12-10 21:20:57 <andypost> jirutka, there's no php5 for aarch64 in 3.4 and 3.5 https://pkgs.alpinelinux.org/packages?name=php%3F
2016-12-10 21:21:18 <jirutka> https://pkgs.alpinelinux.org/package/edge/main/aarch64/php5
2016-12-10 21:21:48 <andypost> oh, misread
2016-12-10 21:21:58 <andypost> only 3.4 missing it
2016-12-10 21:24:24 <andypost> jirutka, then better separate pr after security fixed version bump
2016-12-10 21:25:34 <jirutka> i still don’t understand how we managed to build php5 on aarch64 if it contains the same bug as php7
2016-12-10 21:29:51 <jirutka> i should go sleep, don’t know why, but i’m very tired today
2016-12-10 21:33:11 <andypost> jirutka, looks compiler was updated last month
2016-12-12 08:20:57 <ScrumpyJack> morning - happy monday!
2016-12-12 10:23:37 <ScrumpyJack> stuggling to install alpine-linux (in a xen hvm). anyone seen this? st.ilet.to/sd.png
2016-12-12 10:38:34 <ScrumpyJack> just noticed this st.ilet.to/zb.png
2016-12-12 10:39:51 <^7heo> ScrumpyJack: maybe it's related to any of the things I try to fix in mkinitfs
2016-12-12 11:35:22 <fcolista> clandmeter, ping
2016-12-12 13:35:01 <jirutka> Docker does not properly distinguish between numeric UIDs and string usernames.
2016-12-12 13:35:01 <jirutka> Local attackers could possibly escalate their privileges.
2016-12-12 13:35:02 <jirutka> LOL
2016-12-12 14:44:25 <clandmeter> fcolista: pong
2016-12-12 14:44:34 <fcolista> hey clandmeter
2016-12-12 14:44:35 <fcolista> hi
2016-12-12 14:44:58 <clandmeter> howdy
2016-12-12 14:45:15 <fcolista> Not bad :) So...terminator as you (probably) know doesn't work
2016-12-12 14:45:32 <fcolista> in orer to make it work we need keybinder3
2016-12-12 14:45:39 <fcolista> https://github.com/engla/keybinder/releases
2016-12-12 14:45:49 <fcolista> we have keybinder in main
2016-12-12 14:45:59 <fcolista> but it's compiled against gtk3
2016-12-12 14:46:22 <fcolista> *gtk2. While keybinder3 it's for gtk3
2016-12-12 14:46:45 <fcolista> I've it compiled in testing locally
2016-12-12 14:46:50 <fcolista> and terminator works
2016-12-12 14:47:06 <fcolista> http://tpaste.us/AyMx
2016-12-12 14:47:09 <fcolista> the question is
2016-12-12 14:47:18 <fcolista> how should we call it? It's keybinder3 ok?
2016-12-12 14:47:41 <fcolista> i'm following what we did with vte
2016-12-12 14:47:46 <fcolista> we have vte3
2016-12-12 14:48:04 <fcolista> do you think this naming is correct?
2016-12-12 14:49:32 <clandmeter> i just got back from poland :)
2016-12-12 14:49:53 <fcolista> welcome back!
2016-12-12 14:50:00 <fcolista> Did you enjoy?
2016-12-12 14:50:15 <clandmeter> kind of
2016-12-12 14:50:16 <fcolista> I've bisited Warsaw a couple of yrs ago, but i didn't like it too much
2016-12-12 14:50:26 <clandmeter> wroclaw
2016-12-12 14:50:40 <clandmeter> its nice
2016-12-12 14:50:53 <clandmeter> very busy in the center this time of year.
2016-12-12 14:52:50 <clandmeter> fcolista: im looking at fedora
2016-12-12 14:53:06 <clandmeter> they have reguar keybinder and keybinder3
2016-12-12 14:53:32 <fcolista> yes, they are different. Also other distro does like this
2016-12-12 14:53:43 <clandmeter> ok
2016-12-12 14:53:50 <clandmeter> why?
2016-12-12 14:53:53 <clandmeter> what is the difference?
2016-12-12 14:54:06 <clandmeter> i see one has python bindings, and other doesnt.
2016-12-12 14:54:17 <fcolista> one is compiled against gtk2, the other one against gtk3
2016-12-12 14:54:25 <fcolista> s/compiled/supports
2016-12-12 14:56:26 <clandmeter> right
2016-12-12 14:57:15 <clandmeter> i was confused with python vs gtk
2016-12-12 14:57:20 <fcolista> np
2016-12-12 14:57:28 <clandmeter> just follow what fedora does
2016-12-12 14:57:53 <clandmeter> its also what ncopa suggested to me few times.
2016-12-12 14:58:00 <clandmeter> btw, is he back already?
2016-12-12 14:58:26 <fcolista> no, afaik
2016-12-12 15:01:31 <clandmeter> fcolista: so you will introduce new vte and keybinder?
2016-12-12 15:01:44 <fcolista> no, vte3 already exists
2016-12-12 15:01:51 <fcolista> i've just took it as example
2016-12-12 15:02:11 <clandmeter> i thought vte was also an issue
2016-12-12 15:02:15 <fcolista> to say: we already have vte and vte3. Can we have keybinder and keybinder3 ?
2016-12-12 15:02:26 <fcolista> no, it's not an issue at all
2016-12-12 15:05:12 <clandmeter> well, i have no idea how this all works together (im seeing 3 vte version in fedora, api related)
2016-12-12 15:05:41 <clandmeter> in doubth following the naming fedora
2016-12-12 15:05:56 <clandmeter> or else ask ncopa, he should be back soon.
2016-12-12 15:06:25 <clandmeter> ok i got to go again. ttyt
2016-12-12 15:06:31 <fcolista> i need to push keybinder3 in community, though
2016-12-12 15:06:36 <fcolista> byt clandmeter
2016-12-12 15:06:36 <fcolista> thx
2016-12-12 15:54:14 <fcolista> wb ncopa