|
Parrot 6.8.0 "Little Lorikeet" | parrot.org/ | Log: irclog.perlgeek.de/parrot | #parrotsketch meeting Tuesday 19:30 UTC Set by moderator on 17 September 2014. |
|||
|
01:49
FROGGS__ joined
05:47
rurban joined
|
|||
| rurban | working on github.com/parrot/parrot/pull/944 => gerdr/socket-readline-gh944 | 06:41 | |
| dalek | rrot/gerdr/socket-readline-gh944: c01a0a5 | (Gerhard R)++ | src/io/utilities.c: Make readline() always return complete lines instead of partial ones, even if we have to block |
07:00 | |
| rrot/gerdr/socket-readline-gh944: e084eba | (Gerhard R)++ | src/io/utilities.c: adjust FIXME comment |
|||
| rrot/gerdr/socket-readline-gh944: 243bad2 | (Gerhard R)++ | src/io/utilities.c: Next shot at fixing io_readline_encoded_string(). All regression should be gone and HTTP::Easy still works. I also identified a potential problem already present in the original code. |
|||
| rrot/gerdr/socket-readline-gh944: 7fac981 | (Gerhard R)++ | / (7 files): Purge set_eof from IO subsystem. Being at EOF is something the underlying system handle keeps track of, and not something we should be able to affect from userspace. |
|||
| rrot/gerdr/socket-readline-gh944: ab3776e | (Gerhard R)++ | / (3 files): Replace raw_reads from IO_BUFFER with PIO_BF_UNDERFLOW flag |
|||
| rrot/gerdr/socket-readline-gh944: 08b1e99 | (Gerhard R)++ | src/pmc/ (4 files): Implement get_bool(), eof(), is_open() in Handle PMC, not its subclasses |
|||
| rrot/gerdr/socket-readline-gh944: 60cf550 | (Gerhard R)++ | src/io/filehandle.c: Set EOF in io_filehandle_read_b() |
|||
| rrot/gerdr/socket-readline-gh944: aa8122d | (Gerhard R)++ | t/pmc/filehandle.t: Change test to match new get_bool() semantics |
|||
| rrot/gerdr/socket-readline-gh944: bf0701a | (Gerhard R)++ | src/io/api.c: Fix bug in Parrot_io_readline_s() io_verify_has_read_buffer() expects a flags argument, not the buffer size |
|||
| rrot/gerdr/socket-readline-gh944: 265980f | (Gerhard R)++ | src/io/socket.c: Consider buffered sockets at EOF if the last read failed |
|||
| rrot/gerdr/socket-readline-gh944: 65674cd | (Gerhard R)++ | src/io/ (2 files): Check for is_eof() in io_readline_encoded_string() instead of the prior hack |
|||
| rrot/gerdr/socket-readline-gh944: d74b834 | rurban++ | / (11 files): Merge branch 'fix-socket-readline' of github.com/gerdr/parrot into gerdr/socket-readline-gh944 Conflicts: \tsrc/io/filehandle.c \tsrc/io/pipe.c \tsrc/io/socket.c \tsrc/io/stringhandle.c \tsrc/io/utilities.c \tsrc/pmc/filehandle.pmc \tsrc/pmc/socket.pmc \tsrc/pmc/stringhandle.pmc |
|||
| rrot/gerdr/socket-readline-gh944: 58a7c68 | rurban++ | src/ (8 files): More fixups for the merge of gerdr/fix-socket-readline All tests pass, but ./parrot_nci_thunk_gen hangs after having written the buffer. |
|||
| rrot/smoke-me/pio_nr_open-3-gh1093: ed18fad | rurban++ | / (2 files): [io] optimize PIO_NR_OPEN from 256 to 3 [GH #1093] PIO_NR_OPEN is now the number of statically allocated interpreter std handles for stdin, stdout and stderr only. It is only used safely in one place, accessing only 0-2. It has nothing to do with the total number of open filehandles, which are allocated dynamically as PMC. Leads to a small memory optimization, 253 less words for each interpreter. |
07:33 | ||
|
07:34
rurban joined
08:13
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1097 (smoke-me/pio_nr_open-3-gh1093 - ed18fad : Reini Urban): The build passed. | 08:13 | |
| [travis-ci] Change view : github.com/parrot/parrot/commit/ed18fad22bc6 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37087889 | |||
|
08:13
travis-ci left
|
|||
| dalek | rrot/smoke-me/pio_nr_open-3-gh1093: 82a5747 | rurban++ | / (3 files): [io] optimize PIO_NR_OPEN from 256 to 3 [GH #1093] PIO_NR_OPEN is now the number of statically allocated interpreter std handles for stdin, stdout and stderr only. It is only used safely in one place, accessing only 0-2. It has nothing to do with the total number of open filehandles, which are allocated dynamically as PMC. Leads to a small memory optimization, 253 less words for each interpreter. |
08:15 | |
|
08:29
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1098 (smoke-me/pio_nr_open-3-gh1093 - 82a5747 : Reini Urban): The build passed. | 08:29 | |
| [travis-ci] Change view : github.com/parrot/parrot/commit/82a5747c38fd | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37089056 | |||
|
08:29
travis-ci left
|
|||
| rurban | darwin has the same encoding problems as the bsd's. fixed right now | 09:17 | |
| dalek | rrot/smoke-me/darwin-encodings: fa21c66 | rurban++ | / (4 files): Add darwin platform encodings, esp. UTF8 Copied from freebsd, fixes t/spec/S05-mass/charsets.rakudo.parrot ./perl6-p -e'my $latin-chars = [~] chr(0)..chr(0xFF); my $test = "abcdefghijklmnopqrstuvwxyzµßàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿ"; say $latin-chars.comb(/<lower>/).join, $test' See [GH #1092] |
09:32 | |
| rrot/smoke-me/darwin-encodings: 53e5544 | rurban++ | / (3 files): [cage] fix copyright formats in 2 new files, tune changelog |
|||
|
09:43
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1099 (smoke-me/darwin-encodings - 53e5544 : Reini Urban): The build passed. | 09:43 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/f...e5544c68ac | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37091874 | |||
|
09:43
travis-ci left
|
|||
| dalek | rrot: fa21c66 | rurban++ | / (4 files): Add darwin platform encodings, esp. UTF8 Copied from freebsd, fixes t/spec/S05-mass/charsets.rakudo.parrot ./perl6-p -e'my $latin-chars = [~] chr(0)..chr(0xFF); my $test = "abcdefghijklmnopqrstuvwxyzµßàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿ"; say $latin-chars.comb(/<lower>/).join, $test' See [GH #1092] |
09:48 | |
| rrot: 53e5544 | rurban++ | / (3 files): [cage] fix copyright formats in 2 new files, tune changelog |
|||
| rrot: 188fda6 | rurban++ | / (2 files): [test] smoke: simplify smolder resend usage export the 3 Parrot::Harness::Smoke subs, as they are needed to resend smolder reports. no use base. Improves GH #1078. perl -Ilib -MParrot::Harness::Smoke \\ -e"send_archive_to_smolder(collect_test_environment_data())" |
09:52 | ||
|
10:01
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1100 (master - 53e5544 : Reini Urban): The build passed. | 10:01 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/d...e5544c68ac | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37092559 | |||
|
10:01
travis-ci left
10:10
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1101 (master - 188fda6 : Reini Urban): The build passed. | 10:10 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/5...8fda672f60 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37092787 | |||
|
10:10
travis-ci left
|
|||
| dalek | rrot: 4176098 | rurban++ | / (3 files): [ops] enable deprecated trap int3 on x86_64 also |
10:35 | |
|
10:48
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1102 (master - 4176098 : Reini Urban): The build passed. | 10:48 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/1...7609861246 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37095053 | |||
|
10:48
travis-ci left
11:51
kid51 joined
|
|||
| dalek | rrot: 24f2988 | rurban++ | t/run/ (2 files): [test] fix mingw test issues with \\r\\n see perl514.cpanel.net/build/builders/p...logs/stdio |
12:07 | |
| rrot: af84d15 | rurban++ | t/ (3 files): [test] fixup mingw fixes |
|||
| rrot: 1ce5a06 | rurban++ | / (5 files): [cage] add const to env api STRING args and fix 2 uninitialized vars |
|||
| rrot: f38a9bc | rurban++ | / (10 files): [cage] fix instances of -Wshadow fix all instances of warning: declaration shadows a local variable tested with latest clang++ |
|||
| rrot: be2ef5e | rurban++ | / (2 files): [frontend]: activate -?/--help for pbc_merge also some cage: comment removed variable, fix -Wshadow |
|||
|
12:18
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1103 (master - be2ef5e : Reini Urban): The build passed. | 12:18 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/4...2ef5e79385 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37099133 | |||
|
12:18
travis-ci left
|
|||
| kid51 | Have I lost my commit bit, perhaps due to inactivity? | 12:35 | |
| I just tried this: | |||
| $ git push origin newlines2-gh1087:smoke-me/newlines2-gh1087 | |||
| fatal: remote error: | |||
| You can't push to git://github.com/parrot/parrot.git | |||
| Use github.com/parrot/parrot.git | |||
| FROGGS__ | kid51: that sounds more like something is blocking port 22 or what git:// uses | 12:38 | |
| kid51 | I made commits in March of this year, so my commit bit should still be valid. | 12:39 | |
| rurban | you need git@github.com:parrot/parrot.git for push | 12:40 | |
| git:// is only for fetch | |||
| but did you change anything in this branch? it already exists, better rename it | 12:41 | ||
| dalek | rrot/smoke-me/newlines2-gh1087: 9268c44 | jkeenan++ | src/platform/freebsd/encoding.c: Properly format copyright notice. Keep t/codingstd/copyright.t happy by adding a comma. |
12:42 | |
| kid51 | Thanks, that worked. | ||
| rurban | I already did that | ||
| kid51 | It passed in master but failed in branch. I was smoke-testing branch as that was specifically requested. | 12:43 | |
| rurban | I see, I did it only for the darwin branch | ||
| I'll rebase it away. thanks | |||
| I was only looking for the 0xa0 error :) | |||
| kid51 | I was also getting errors in t/codingstd/c_code_coda.t in branch | ||
| Now: a more serious issue. | 12:44 | ||
| I try to build and test Parrot 1x/month on all machines I have available. | |||
| rurban | yes? | 12:45 | |
| kid51 | Including my Mac OS X 10.4.11 darwin/ppc with which I started out in the project in 2006 | ||
| But adding src/platform/darwin/encoding.c has caused that build to fail many tests | |||
| rurban | my darwin/ppc machine is on the ship also. that's interesting. maybe we need a different c api for ppc. better define it out there | 12:46 | |
| kid51 | smolder.parrot.org/app/projects/rep...ails/45024 | ||
| I can post the output of 'make' somewhere, if you wish. | |||
| In the past, when we've encountered problems like this, we've worked around by testing for the version of Darwin and the platform. | 12:47 | ||
| Tomorrow I will try to test HEAD of master on a more modern Mac (x86_64) | 12:48 | ||
| rurban | I'll disable the code for ppc there, and fix it later. | 12:51 | |
| I tested it on a 10.9 darwin | |||
|
12:53
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1104 (smoke-me/newlines2-gh1087 - 9268c44 : James E Keenan): The build passed. | 12:53 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/d...68c449387a | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37100631 | |||
|
12:53
travis-ci left
|
|||
| rurban | thanks, I'll push now the rebased version, and the hopeful ppc fix | 13:00 | |
| kid51 | Recorded utf-8 problem at github.com/parrot/parrot/issues/1094 | 13:02 | |
| rurban | I see; ppc prints: Unknown 'codeset utf-8' defaulting to ASCII | ||
| kid51 | Exactly | 13:03 | |
| 'make' completes despite these warnings, but tests fail massively | |||
| rurban | I'll check for this string also now | 13:04 | |
| dalek | rrot/smoke-me/darwin-encoding-gh1094: c37f6c4 | rurban++ | src/platform/darwin/encoding.c: Adjust darwin/encoding to accept utf-8 for older ppc on newer systems uppercase UTF-8 is returned by setlocale |
13:09 | |
|
13:24
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1105 (smoke-me/darwin-encoding-gh1094 - c37f6c4 : Reini Urban): The build passed. | 13:24 | |
| [travis-ci] Change view : github.com/parrot/parrot/commit/c37f6c45a457 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37101921 | |||
|
13:24
travis-ci left
|
|||
| rurban | also fixed the code_coda issue | 13:54 | |
| dalek | rrot/smoke-me/newlines2-gh1087: 391b79b | (Tobias Leich)++ | src/string/encoding/ (2 files): RT #122341 treat all of 0x0a-0x0d,\\x85,\\x2028,\\x2029 as newlines The script tools/dev/gen_charset_tables.pl was not used because it removes character properties of chars in the range of 0x81 to 0xFF. Additionally, u_iscclass now checks for characters with enum_cclass_newline property, which it did not do at all before. |
14:22 | |
| rrot/smoke-me/newlines2-gh1087: 1463167 | rurban++ | / (5 files): [tools] typetables: fix gen_charset_tables.pl and regenerate Removes defunct and since 2010 unused Parrot_ascii_typetable. Adds \\v to CCLASS_NEWLINE manually (confirmed), \\x85\\xa0 confirmed to be now in the whitespace cclass, but several old systems fail with the \\xa0 (non-breaking whitespace) test for whitespace. Several chars 160..191 are not in the [[:punct:]] class anymore. Added bootstrap-tables make target, update the tables automatically. Improved src/string/encoding/tables.c pod. Closes PR #1087 |
|||
|
14:35
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1106 (smoke-me/newlines2-gh1087 - 1463167 : Reini Urban): The build passed. | 14:35 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/3...6316774d57 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37105951 | |||
|
14:35
travis-ci left
|
|||
| dalek | rrot: c37f6c4 | rurban++ | src/platform/darwin/encoding.c: Adjust darwin/encoding to accept utf-8 for older ppc on newer systems uppercase UTF-8 is returned by setlocale |
16:04 | |
| rurban | ok, done for today. still waiting for someone to break smoke-me/newlines3-gh1087 with 0xa0 not as whitespace | 16:08 | |
|
16:18
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1107 (master - c37f6c4 : Reini Urban): The build passed. | 16:18 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/b...7f6c45a457 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37111853 | |||
|
16:18
travis-ci left
16:23
dalek joined
16:27
Coke joined
16:28
Util joined
|
|||
| dalek | rrot: 06f6c46 | rurban++ | / (3 files): [docs] add 3 last missing manpages |
18:16 | |
| rrot: c001c9f | rurban++ | ChangeLog: ChangeLog: ws only |
18:24 | ||
|
18:28
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1108 (master - 06f6c46 : Reini Urban): The build passed. | 18:28 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/c...f6c4611a9c | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37119847 | |||
|
18:28
travis-ci left
18:40
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1109 (master - c001c9f : Reini Urban): The build passed. | 18:40 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/0...01c9f43548 | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37120574 | |||
|
18:40
travis-ci left
|
|||
| dalek | rrot: 0318f96 | rurban++ | / (3 files): [io] optimize PIO_NR_OPEN from 256 to 3 [GH #1093] PIO_NR_OPEN is now the number of statically allocated interpreter std handles for stdin, stdout and stderr only. It is only used safely in one place, accessing only 0-2. It has nothing to do with the total number of open filehandles, which are allocated dynamically as PMC. Leads to a small memory optimization, 253 less words for each interpreter. |
18:52 | |
|
19:07
travis-ci joined
|
|||
| travis-ci | [travis-ci] parrot/parrot#1110 (master - 0318f96 : Reini Urban): The build passed. | 19:07 | |
| [travis-ci] Change view : github.com/parrot/parrot/compare/c...18f962288f | |||
| [travis-ci] Build details : travis-ci.org/parrot/parrot/builds/37122171 | |||
|
19:07
travis-ci left
19:22
stryx` joined
|
|||