Perl 6 - the future is here, just unevenly distributed

IRC log for #pdl, 2015-03-03

| Channels | #pdl index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:06 mohawk soon as this improvement to alienhdf4 is confirmed as passing, i'm adding badges to the gh repo
00:06 mohawk since we now have sexy PASSING code
00:13 opkick [pdl] wchristian force-pushed alienhdf4 from 6cbdba7 to be1f880: http://git.io/x2TT
00:13 opkick pdl/alienhdf4 be1f880 Ed J: Use Alien::HDF4 in PDL::IO::HDF build
00:25 opkick [pdl] mohawk2 comment on issue #11: Am I right that this is only possible with updating the README.md, and it will show there? http://git.io/x2lt
00:25 opkick [pdl] coveralls comment on issue #51:
01:02 travis-ci joined #pdl
01:02 travis-ci PDLPorters/pdl#38 (config-devel-checklib - 3d5ebf0 : Zakariyya Mughal): The build has errored.
01:02 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/8cc6a55b9388...3d5ebf0b6eaf
01:02 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52823192
01:02 travis-ci left #pdl
01:05 mohawk sivoais, the error above is because the Makefile.PL was "2.9e+4 secs in the future"
01:05 mohawk which is a bit crazy
01:05 mohawk i think it might be related to how both the push and the pr travis's were given #38
01:06 mohawk anyway, the other #38 is running now so we'll see
01:06 mohawk sivoais, also, we should switch the travis-ci bot to not join the channel to reduce the noise slightly
01:25 opkick [pdl] mohawk2 force-pushed hainest-assgn_badflag from e5b7a5d to 824bc6e: http://git.io/xgxg
01:25 opkick pdl/hainest-assgn_badflag 9727ab7 Ed J: Rewrite t/ops.t with Test::More
01:25 opkick pdl/hainest-assgn_badflag 824bc6e Tim Haines: Add propagation of badflag with .= (Ops::assgn) for sf.net bug 3543056.
01:30 opkick [pdl] coveralls comment on issue #22:
02:02 travis-ci joined #pdl
02:02 travis-ci PDLPorters/pdl#42 (alienhdf4 - be1f880 : Ed J): The build has errored.
02:02 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/6cbdba70eb01...be1f88069cff
02:02 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52830508
02:02 travis-ci left #pdl
02:07 opkick [pdl] mohawk2 force-pushed alienhdf4 from be1f880 to 6b68ca5: http://git.io/x2TT
02:07 opkick pdl/alienhdf4 6b68ca5 Ed J: Use Alien::HDF4 in PDL::IO::HDF build
02:11 mohawk sivoais, do we really need coverage tests of BADVAL=0?
02:16 sivoais mohawk: I was actually thinking we don't. It generates 2 coveralls comments and doesn't seem to add much
02:16 sivoais mohawk: ok, I'll look into that Travis-CI IRC setting. Probably a way to do notice
02:18 mohawk hold on, i don't like notice
02:18 mohawk message is fine
02:18 mohawk it's the skip_join setting we need
02:18 mohawk also, we are having a race condition problem with both branch and pr
02:18 mohawk i think what i'll do is close the PRs for now, we can reopen them when they pass ;-)
02:19 opkick [pdl] mohawk2 comment on issue #52: Temporary close because Travis fouls up with race condition when both branch and PR. http://git.io/x2Fh
02:20 opkick [pdl] mohawk2 closed pull request #51: Add propagation of badflag with .= (Ops::assgn) for sf.net bug 3543056. (master...hainest-assgn_badflag) http://git.io/xgux
02:20 opkick [pdl] mohawk2 comment on issue #51: Temporary close because Travis fouls up with race condition when both branch and PR. http://git.io/x2bm
02:21 opkick [pdl] mohawk2 closed pull request #21: Use Alien::HDF4 in PDL::IO::HDF build (master...alienhdf4) http://git.io/xCv0
02:22 sivoais that's weird
02:22 mohawk what?
02:23 sivoais that it would cause a race-condition
02:23 mohawk and yet
02:23 mohawk it's certainly doing 2 sets of builds, off the same commit SHA in 2 contexts
02:23 mohawk that's a mistake in my mind
02:24 mohawk may i suggest you make a branch and try the skip_join and 1 coverage only thing?
02:24 mohawk i'm looking at a USENAN=0 problem on lldfp1
02:24 sivoais yeah, I just popped my stack to that. I'll do that now
02:25 mohawk nice
02:25 mohawk good news is, your checklib one works except for this doubling-up thing (so i cancelled one job, it's actually green)
02:26 mohawk even better, revealed that alien::hdf4 doesn't seem to install, so waiting for more details on that
02:26 mohawk AND found lldfp1 problem with USENAN=0
02:26 mohawk this CI business is fantastic
02:27 mohawk can't wait till SF implement it on their site and we can ditch nasty old github
02:29 opkick [pdl] zmughal pushed 2 new commits to master: http://git.io/x2AL
02:29 opkick pdl/master 460a039 Zakariyya Mughal: Travis-CI: IRC notifications: skip_join to reduce noise
02:29 opkick pdl/master 37ec8ba Zakariyya Mughal: Travis-CI: remove COVERAGE=1 + WITH_BADVAL=0 build...
02:29 sivoais hah
02:30 sivoais I thought SF had a build farm or am I misremembering?
02:30 sivoais lame, it shutdown around 2007
02:33 travis-ci joined #pdl
02:33 travis-ci PDLPorters/pdl#44 (hainest-assgn_badflag - 824bc6e : Tim Haines): The build passed.
02:33 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/e5b7a5d71f0b...824bc6e358f0
02:33 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52836611
02:33 travis-ci left #pdl
02:33 sivoais \o/
02:36 travis-ci joined #pdl
02:36 travis-ci PDLPorters/pdl#46 (alienhdf4 - 6b68ca5 : Ed J): The build has errored.
02:36 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/be1f88069cff...6b68ca57236d
02:36 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52839765
02:36 travis-ci left #pdl
02:37 sivoais /o\
02:45 opkick [pdl] hainest comment on issue #2: Terribly sorry. I had completely forgotten about this. Thanks for chasing it down! http://git.io/x2hV
02:47 opkick [pdl] mohawk2 comment on issue #2: Nothing to apologise for! Are you comfortable these days with rebasing? Are there other contributions you'd like to make to PDL? http://git.io/x2jg
02:48 sivoais maybe this might help: <https://github.com/travis-ci/travis-ci/issues/1316> "I have a repo where I create branches and for each branch create a PR that would merge those branches into the master branch."
02:48 sivoais "I only want Travis to build the PRs, not the branches. I do however want Travis to always build the master branch."
02:49 mohawk take a look at this one: https://travis-ci.org/PDLPorters/pdl/jobs/52839770
02:50 sivoais I restarted that job
02:50 mohawk sivoais, yes, that issue looks like the thing
02:50 sivoais implement?
02:51 mohawk i'd say "on a branch first"
02:51 mohawk but this is kind of an exception
02:51 mohawk if it's .t.yml changes only
02:51 sivoais yeah
02:53 mohawk great
02:53 travis-ci joined #pdl
02:53 travis-ci PDLPorters/pdl#46 (alienhdf4 - 6b68ca5 : Ed J): The build has errored.
02:53 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/be1f88069cff...6b68ca57236d
02:53 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52839765
02:53 travis-ci left #pdl
02:53 mohawk i don't think it'll fully restart
02:54 mohawk i'm quite happy with travis building PRs, so long as they aren't branches on our own repo
02:54 mohawk that MUST be a thing it can spot?
02:55 mohawk or rather, i just don't want it double-building
02:55 sivoais ^_^ you would think
02:55 mohawk if it's a a branch of own repo, AND it's been PRed
02:55 sivoais check the SHA!!
02:55 mohawk it doesn't ;-)
02:55 mohawk ...evidently
02:56 sivoais this is actually a good practice though. If we're going to only run them for PR'd branches, the PRs can act as code-review
02:56 sivoais on that branch
02:56 mohawk i disagree
02:56 sivoais no merging until the branch is done
02:56 sivoais no?
02:56 mohawk assuming i understand you right
02:56 sivoais let me show you an example
02:56 mohawk i want travis running on non-PRed branches
02:57 mohawk because that's how i can know it's ok
02:57 mohawk show away :-)
02:59 sivoais there was a specific example I was looking for, but I can't find it now
02:59 sivoais but here is how this project works: <https://github.com/shogun-toolbox/shogun/pull/2325>
02:59 sivoais see how many comments that PR is getting?
03:00 sivoais so for every branch, even the ones on PDLPorters/pdl, we create a PR where we can work through the changes
03:01 sivoais and that PR acts as the point of code review
03:01 travis-ci PDLPorters/pdl#48 (master - 37ec8ba : Zakariyya Mughal): The build passed.
03:01 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/aea62544f94c...37ec8ba947cb
03:01 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52841226
03:01 sivoais we can even put a DO NOT MERGE sign at the top to indicate it is not done yet
03:03 opkick [pdl] zmughal pushed 1 new commit to master: http://git.io/xafr
03:03 opkick pdl/master aedd1ea Zakariyya Mughal: Travis-CI: only build branches if a PR has been made for them...
03:03 sivoais TODO: look at ViennaCL for GPU matrices
03:08 mohawk i like the idea of code review
03:08 mohawk naturally
03:08 mohawk could you make a trivial branch and PR it?
03:08 sivoais test-cleanup!
03:08 mohawk go on then ;-)
03:09 sivoais it shall be the poster child of code review ;-)
03:14 mohawk can you push and PR that now
03:14 mohawk i am especially keen to see if it double-travises
03:16 mohawk no pressure :-)
03:18 sivoais darn these context switches. Someone asked me something in the middle of navigating to the directory
03:18 sivoais thanks for the reminder ;-)
03:19 mohawk grin
03:21 opkick [pdl] zmughal created test-cleanup (+19 new commits): http://git.io/xaTE
03:21 opkick pdl/test-cleanup 2c24694 Zakariyya Mughal: convert aaa_load.t to Test::More
03:21 opkick pdl/test-cleanup 0afa73c Zakariyya Mughal: consolidate test helper function of form `ok( $no, $result )`...
03:21 opkick pdl/test-cleanup a2dc9fd Zakariyya Mughal: done_testing not needed
03:21 mohawk sivoais++
03:23 opkick [pdl] wchristian deleted test-cleanup at 16da656: http://git.io/xaTb
03:23 sivoais ha, one sec
03:23 opkick [pdl] zmughal created test-cleanup (+19 new commits): http://git.io/xaTE
03:23 opkick pdl/test-cleanup 2c24694 Zakariyya Mughal: convert aaa_load.t to Test::More
03:23 opkick pdl/test-cleanup 0afa73c Zakariyya Mughal: consolidate test helper function of form `ok( $no, $result )`...
03:23 opkick pdl/test-cleanup a2dc9fd Zakariyya Mughal: done_testing not needed
03:23 mohawk you pushed only to gh, didn't you
03:23 mohawk always push sf first, then gh
03:24 sivoais yeah, thing is, I didn't fix my SF remote on my other copy of the repo where I was doing the test-cleanup branch
03:24 sivoais now, that should *not* build in Travis
03:24 * sivoais checks
03:24 sivoais and then I'll create a PR
03:25 sivoais cool, it is not building
03:25 sivoais and now I made a PR
03:25 opkick [pdl] zmughal opened pull request #53: Test cleanup (master...test-cleanup) http://git.io/xak0
03:26 sivoais and it is building only once!
03:27 mohawk ok
03:28 mohawk so the workflow is to immediately open a PR with any branch
03:28 mohawk could you make a little mode (or even just rebase and force-push, it'll make new SHA)
03:28 mohawk s/mode/mod
03:28 mohawk cancel that build
03:28 mohawk force-push, see if it only makes one new build?
03:30 travis-ci PDLPorters/pdl#49 (master - aedd1ea : Zakariyya Mughal): The build passed.
03:30 travis-ci Change view : https://github.com/PDLPorters/pdl/compare/37ec8ba947cb...aedd1ea8f9cf
03:30 travis-ci Build details : http://travis-ci.org/PDLPorters/pdl/builds/52843850
03:31 mohawk w00t
03:32 mohawk sivoais, once you're happy a force-pushed PR-ed branch only gets one build, could you reopen the PRs?
03:33 sivoais sure, um...let's take a look
03:34 mohawk Can't load '/home/user/pdl-code/blib/arch/auto/PDL/Image2D/Image2D.so' for module PDL::Image2D: /home/user/pdl-code/blib/arch/auto/PDL/Image2D/Image2D.so: undefined symbol: quick_select_
03:34 mohawk what the actual
03:34 mohawk (that's not truncated, i think)
03:34 mohawk ah, i know exactly what that is
03:35 mohawk quick_select_$TBSULNQFD - that's now supposed to be quick_select_$TBSULNQFDA
03:35 mohawk carry one
03:35 mohawk carry on, as you were
03:35 * sivoais carries the one
03:37 mohawk actually, i'm not sure that's right
03:37 mohawk oh god, i'm going to have to go down the PDL::PP rabbithole
03:37 * mohawk weeps
03:38 sivoais mohawk: why do I need to force push? Can't I just push?
03:38 mohawk sure, that'll do
03:38 mohawk just wanted a quicker turnaround
03:39 mohawk dammit scotty, i need more speed!
03:39 opkick [pdl] zmughal pushed 1 new commit to test-cleanup: http://git.io/xaLd
03:39 opkick pdl/test-cleanup 437682f Zakariyya Mughal: remove pok() function
03:39 sivoais she canna go any faster, cap'n!
03:41 mohawk i don't want excuses, scotty!
03:41 sivoais \o/
03:41 sivoais new build in the USSR...
03:42 sivoais err, I mean ... PR
03:42 mohawk nice work
03:42 mohawk so, reopen the PRs?
03:42 sivoais will do!
03:43 mohawk sivoais++
03:43 opkick [pdl] zmughal reopened pull request #52: longlong-double-fix resurrection part 1 (master...lldfp1) http://git.io/xgaV
03:43 opkick [pdl] zmughal reopened pull request #22: remove Devel::CheckLib from inc/ (master...config-devel-checklib) http://git.io/xRNn
03:44 mohawk nice work!
03:45 mohawk i'll cancel the build on lldfp1, because we know it's broken
03:46 mohawk uh oh, build fail: https://travis-ci.org/PDLPorters/pdl/jobs/52844964
03:46 mohawk and yes, it's what you expected:
03:46 mohawk t/clump.t failed
03:47 mohawk actually, that's not a fail as such because it's on blead so allowed
03:52 opkick [pdl] coveralls comment on issue #53:
03:58 * sivoais wonders what the histogram of log sizes for this channel are now ;-)
03:58 mohawk like a teenager just before waking
03:59 mohawk like a tall tree in the netherlands
03:59 Mithaldu joined #pdl
04:00 mohawk Mithaldu, welcome back dude
04:00 * sivoais didn't know Mithaldu left...
04:00 sivoais I turned off my join/parts
04:02 sivoais Mithaldu: <https://github.com/PDLPorters/pdl/issues>
04:02 mohawk i wasn't aware either
04:02 sivoais even better <https://github.com/PDLPorters/pdl/pulls>
04:03 sivoais jberger: ^
04:04 jberger ye gods the backlof
04:04 jberger backlog
04:04 jberger what did I miss?
04:04 sivoais :-D
04:04 mohawk everything
04:05 mohawk seems Mithaldu only signed out 40 mins ago
04:06 jberger sorry, there has been all kinds of conference planning ... mind-drain
04:08 mohawk fair enough ;-)
04:08 mohawk sivoais, easy ways for a quick coverage win: top "missed lines" is lib/PDL/Transform/Proj4.pm with 2267
04:11 mohawk so knock up a quicky test for that, and everyone wins
04:11 mohawk ...that covers all the lines, naturally ;-)
04:11 mohawk jberger, to summarise - we got travis working well, then found it was double building, then we fixed that and are now fixing problems it's shown us
04:12 mohawk jberger, new workflow - branches don't get travised, only PRs get it
04:13 jberger ah, ok
04:14 jberger I think I have seen that problem elsewhere
04:14 mohawk so lots of chat for coordination, that's the outcome
04:14 jberger cool
04:14 jberger well done!
04:14 mohawk conf = yapc?
04:14 jberger mojoconf
04:14 mohawk it was all thanks to sivoais
04:14 jberger sivoais++ # as always
04:15 sivoais oh, and we're using GH milestones to triage the SF issues
04:15 jberger http://mojoconf.com/
04:15 opkick [pdl] coveralls comment on issue #53:
04:15 mohawk yeah, we've made the substantive migration to gh
04:15 sivoais because doing it by hand was making me weep
04:15 sivoais especially since nothing was ever getting closed
04:16 sivoais even after merges
04:17 mohawk yeah
04:18 mohawk sivoais, https://github.com/PDLPorters/pdl/issues/11
04:18 sivoais mohawk: want me to handle that?
04:19 mohawk yes please
04:19 sivoais will do, just fixing something else non-PDL
04:19 sivoais I'll assign myself
04:19 mohawk may i suggest an email to the list mentioning this: https://github.com/PDLPorters/pdl/issues?q=is%3Aopen+is%3Aissue+milestone%3A%22PDL+v2.008%22
04:19 mohawk grin, cool
04:20 sivoais because... you know... I made the issue in the first place :-D
04:20 * mohawk continues his git rebase -x 'perl Makefile.PL && make core' -i master
04:20 mohawk grind, grind
04:22 jberger :o
04:25 mohawk how's this for some git wizardry
04:25 mohawk so that rebase -x is going along lldfp1
04:26 mohawk but it's with USENAN=1
04:26 mohawk so i inserted a commit at the start setting USENAN=1
04:26 mohawk when it's all done, which it just has
04:26 mohawk i'll remove that USENAN=1 commit, then force-push
04:30 mohawk i suppose another way, especially if some commits changed perldl.conf, would be to have each -x start with "perl -pi -e s/USENAN/blah/" and end with " && git checkout perldl.conf"
04:34 * jberger 's mind splatters on floor
04:35 opkick [pdl] wchristian force-pushed lldfp1 from fbbfbe8 to da84695: http://git.io/x2va
04:35 opkick pdl/lldfp1 5760843 Craig DeForest: remove references to progenitors and mutators.  Update core version number.
04:35 opkick pdl/lldfp1 0e634d2 Craig DeForest: some typos ("propogate"->"propagate" throughout; some typos in CallExt documentation)
04:35 opkick pdl/lldfp1 19cfe47 Craig DeForest: more cleanup of pdl.h.PL. Also, cut out foomethod() nonsense since it is not...
04:35 mohawk jberger, too much git magic? ;-)
04:36 jberger the horror, the horror
04:36 * jberger totally switched motifs there
04:36 mohawk sivoais, why are there 2 builds showing now on lldfp1?
04:36 mohawk https://github.com/PDLPorters/pdl/pull/52
04:38 sivoais mohawk: did you rebase on master?
04:39 mohawk yes
04:39 sivoais :-/
04:39 * sivoais looks at .travis.yml
04:39 mohawk what difference would rebasing on master make, anyway?
04:39 mohawk shit, wait
04:39 mohawk i think i erred
04:39 sivoais it would bring in the "branches: only: master"
04:40 sivoais :-)
04:40 mohawk i didn't get latest master
04:40 mohawk my bad
04:40 sivoais :-P
04:40 mohawk i'm not used to other people making changes ;-)
04:42 mohawk want me to rebase / repush the others?
04:42 opkick [pdl] mohawk2 force-pushed lldfp1 from da84695 to ac82533: http://git.io/x2va
04:42 opkick pdl/lldfp1 50ebf4d Craig DeForest: remove references to progenitors and mutators.  Update core version number.
04:42 opkick pdl/lldfp1 57e8644 Craig DeForest: some typos ("propogate"->"propagate" throughout; some typos in CallExt documentation)
04:42 opkick pdl/lldfp1 d8f2f92 Craig DeForest: more cleanup of pdl.h.PL. Also, cut out foomethod() nonsense since it is not...
04:42 mohawk or did you already?
04:43 sivoais no, not yet. I'm doing something else now
04:44 mohawk ok
04:44 mohawk i'll leave it then
04:44 sivoais I'll do the readme.md in #11, but that's all I can do tonight ;-)
04:44 mohawk i believe latest lldfp1 should pass though
04:44 mohawk man, that's a lot
04:44 mohawk you da man
04:44 opkick [pdl] coveralls comment on issue #51:
04:45 mohawk we might want to tweak the coveralls message, looks like it's putting a blank line at start and irc isn't showing substance, or something
04:46 * sivoais looks
04:52 sivoais mohawk: it isn't configurable *sniff*
05:11 opkick [pdl] coveralls comment on issue #22:
05:42 opkick [pdl] coveralls comment on issue #52:
05:44 mohawk doh
05:47 opkick [pdl] zmughal force-pushed config-devel-checklib from 3d5ebf0 to f3a4ce3: http://git.io/xgjP
05:47 opkick pdl/config-devel-checklib f3a4ce3 Zakariyya Mughal: remove Devel::CheckLib from inc/...
05:49 opkick [pdl] zmughal force-pushed alienhdf4 from 6b68ca5 to 1ee4bab: http://git.io/x2TT
05:49 opkick pdl/alienhdf4 1ee4bab Ed J: Use Alien::HDF4 in PDL::IO::HDF build
05:50 opkick [pdl] zmughal force-pushed hainest-assgn_badflag from 824bc6e to 0a29cb6: http://git.io/xgxg
05:50 opkick pdl/hainest-assgn_badflag 79bc72e Ed J: Rewrite t/ops.t with Test::More
05:50 opkick pdl/hainest-assgn_badflag 0a29cb6 Tim Haines: Add propagation of badflag with .= (Ops::assgn) for sf.net bug 3543056.
05:55 sivoais mohawk: README.md in MANIFEST.SKIP ?
05:56 sivoais so should .travis.yml
06:01 opkick [pdl] zmughal created github-sprucing (+2 new commits): http://git.io/xaam
06:01 opkick pdl/github-sprucing 005e983 Zakariyya Mughal: add .travis.yml to MANIFEST.SKIP
06:01 opkick pdl/github-sprucing 12882dd Zakariyya Mughal: add README.md to spruce up the GitHub repository
06:02 mohawk sivoais, good call
06:04 mohawk the README.md looks sexy
06:04 mohawk now all it needs is travis + coverage badges
06:04 mohawk damn fine work
06:05 mohawk (we should probably make coveralls not comment if coverage stays same)
06:06 opkick [pdl] zmughal opened pull request #54: GitHub sprucing (master...github-sprucing) http://git.io/xaa9
06:07 sivoais I think that can be done
06:11 opkick [pdl] zmughal pushed 1 new commit to github-sprucing: http://git.io/xawW
06:11 opkick pdl/github-sprucing cf38d2c Zakariyya Mughal: add build and coverage badges to README.md...
06:12 opkick [pdl] zmughal comment on issue #54: Updated to add badges.... http://git.io/xawX
06:13 mohawk NICE
06:14 mohawk i think that version badge needs a webhook setting?
06:14 mohawk https://github.com/PDLPorters/pdl/blob/github-sprucing/README.md
06:14 mohawk that looks so...
06:14 mohawk 2015
06:14 sivoais :-D
06:15 sivoais we have only begun!
06:15 mohawk ha ha
06:15 mohawk but srsly, did i remember right the version badge thing needs a webhook?
06:15 mohawk i don't know how to check if we've set that up
06:16 sivoais nope, that just uses MetaCPAN (through fury.io)
06:16 mohawk great
06:16 sivoais <https://badge.fury.io/pl/PDL.svg>
06:17 opkick [pdl] coveralls comment on issue #22:
06:21 * sivoais clocks out
06:21 * sivoais will send the e-mail about the milestone to pdl-devel in the morning
06:23 sivoais I still didn't get to that vsearch doc patch :-/
06:24 sivoais I'll assign it to myself <https://github.com/PDLPorters/pdl/issues/43>
06:35 mohawk that'll do it
06:38 sivoais will you look at that <https://github.com/PDLPorters/pdl/pulse>!
06:38 sivoais for 1 week: "Excluding merges, 9 authors have pushed 31 commits to master and 104 commits to all branches. On master, 44 files have changed and there have been 6,986 additions and 7,109 deletions."
06:38 mohawk yes there have
06:42 opkick [pdl] coveralls comment on issue #51:
07:12 opkick [pdl] coveralls comment on issue #54:
13:31 Mithaldu mohawk, sivoais: that was only my bouncer getting disconnected and reconnecting
13:34 Mithaldu mohawk: do you think you'll be able to restrain yourself from mocking people today?
15:16 rindolf joined #pdl
16:06 sivoais jberger: if you need help on the perl4science website conversion, let me know. We can split the work if there's anything manual involved.
16:08 jberger man, there is so much going on for me atm, I would love to convert to statocles, but its on the back burner
16:34 opkick [pdl] zmughal opened issue #55: Make sure all code has use strict and use warnings http://git.io/xKGC
16:35 opkick [pdl] zmughal opened issue #56: Make sure that $a and $b are not used http://git.io/xKGD
16:46 opkick [pdl] zmughal comment on issue #29: Using my sleuthing skills, I do think it does [ [ pdl-Bugs-3543056 ] propagate badflag with .=  ](http://mailman.jach.hawaii.edu/pipermail/pdl-porters/2012-July/004896.html)  links to <https://sourceforge.net/tracker/?func=detail&amp;atid=100612&amp;aid=3543056&amp;group_id=612> (same bug number) which redirects to <https://sourceforge.net/p/pdl/bugs/308/>. http://git.io/xK4K
16:48 mohawk Mithaldu, it's so hard to make predictions, especially about the future
16:48 mohawk i can't promise
16:48 mohawk and i can't promise to try
16:48 Mithaldu not funny
16:48 mohawk but i'll try to try
16:48 Mithaldu i'll make it clear
16:48 Mithaldu you're mocking chm
16:48 Mithaldu you're mocking everyone else working on this project
16:49 mohawk in fact i am not
16:49 Mithaldu and when i try to point out to you how astonishingly dumb that is, you mock me
16:49 mohawk i regret that you think i am
16:49 Mithaldu 15-03-02@21:39:50 (mohawk) in fact, better than museum
16:49 Mithaldu 15-03-02@21:39:53 (mohawk) mausoleum
16:49 Mithaldu ^- mocking
16:49 Mithaldu don't even try to rationalize what you were thinking, that is how it comes across to anyone
16:50 mohawk that was making a fairly reasonable, albeit obviously exaggerated, description, of the bug list
16:50 Mithaldu and if you think that's productive, then feel free to go on, but don't be surprised if chm says "fuck you"
16:50 mohawk during that time, chm has, of his own volition, advocated for then executed, adding me and sivoais as devs
16:50 mohawk this is obvious to everyone
16:51 mohawk if he were to say that, then so be it
16:51 jberger ok ok ok, there has been plenty of acidic conversation for everyone
16:51 jberger we all want to see pdl progress
16:51 jberger lets get it there
16:51 jberger I've been frustrated too
16:52 mohawk jberger, well said
16:52 jberger I admit, and I apologize
16:52 sivoais hmm, I looking at that badflag propagation...
16:52 mohawk me too
16:53 sivoais the old mailing list threads seem to have some interesting conversation. I need to see if the discussed code is in the tests
16:53 mohawk sivoais, good research
16:53 sivoais like <http://mailman.jach.hawaii.edu/pipermail//pdl-porters/2012-July/004892.html>
17:08 sivoais mohawk: I'm going to add a test to the assgn branch
17:09 mohawk sivoais, on the branch?
17:09 sivoais yeah, on the branch
17:09 mohawk be my guest
17:09 sivoais just another test of bad value propagation
17:10 Mithaldu left #pdl
17:11 sivoais hmm, but the thing is... you have *set* badflag to 1 explicitly
17:13 sivoais mohawk: idea for later... based on the branch name, we can kick off smaller versions of the test suite
17:14 sivoais git checkout -b my-thing-wip my-thing;
17:14 sivoais and all -wip branches only test 5.20 with badflag=1
17:15 sivoais and then we just fast forward my-thing-wip onto my-thing when we want the full build
17:15 sivoais that'll give a ~8 min turnaround
17:16 mohawk i say that's a waste of effort
17:17 mohawk if you want to run one set of things, do it locally
17:17 mohawk that's what i did with lldfp1
17:17 sivoais right
17:17 mohawk please let's not get all "i got me a hammer, where the nails at"
17:18 sivoais ok, that Developer Testing mode would be really nice though ;-)
17:18 mohawk "make coretest"
17:18 sivoais the one where it doesn't update the mtime if it doesn't have to
17:18 mohawk if you don't change the build system, the make works well
17:18 mohawk have you tried it?
17:19 mohawk and it's right to do a full rebuild if you do change the build system, because then all bets are off
17:19 mohawk i still don't know what "codegen" you think is happening unnecessarily?
17:20 sivoais yeah, I'm probably doing something wrong :-P
17:21 mohawk srsly, if you pastebin output showing such, i'll look at it
17:21 mohawk eumm is approaching a release so i want to be sure it's perfect
17:24 sivoais mohawk: I don't think the badflag prop will be fixed until the tests don't have to set ->badflag(1)
17:24 sivoais I'm going to turn these into TODO tests
17:27 mohawk don't do that
17:27 mohawk or rather, do, but also adjust the test itself so it's correct
17:27 mohawk and please edit the actual commit itself
17:29 sivoais mohawk: but the thing is, the test is not correct. It does not test the bug because it sets the badflag manually when it should be automatic
17:30 sivoais so I removed the lines that set the badflag manually
17:30 sivoais is that what you mean?
17:32 sivoais and you want me to do a soft reset and amend the commit?
17:35 mohawk soft reset, or git rebase and "edit" the commit
17:35 mohawk either
17:35 mohawk make the commit be correct, rather than preserving tim's code verbatim :-)
17:35 sivoais right, sounds good, it'll land in a few
17:36 sivoais also, add more comments!
17:36 sivoais because nobody wants to read tests without knowing why they are there! :-)
17:36 mohawk you could have stopped that before "without"
17:39 opkick [pdl] zmughal force-pushed hainest-assgn_badflag from 0a29cb6 to 6f8d393: http://git.io/xgxg
17:39 opkick pdl/hainest-assgn_badflag 6f8d393 Tim Haines: Add propagation of badflag with .= (Ops::assgn) for sf.net bug 3543056....
17:40 sivoais hahah, true
17:41 mohawk sivoais++
17:41 sivoais but we're in the wrong field if we want to avoid reading code
17:42 sivoais *sigh* I see more test-cleanup in my future...
17:43 sivoais I added a low-hanging-fruit label <https://github.com/PDLPorters/pdl/issues?q=is%3Aopen+label%3Alow-hanging-fruit>
17:43 * sivoais needs some coffee
17:53 opkick [pdl] zmughal comment on issue #51: LIke I was saying in IRC <http://irclog.perlgeek.de/pdl/2015-03-03#i_10215431>, this won't fix the code until the `badflag` does not need to be set. Turning into TODO tests for now. http://git.io/x6Jt
18:49 opkick [pdl] coveralls comment on issue #51:
19:16 opkick [pdl] zmughal comment on issue #6: @wchristian, could we also do the same for... http://git.io/x67I
19:18 sivoais mohawk: there are other PDL repos that could also be mirrored to GitHub
19:18 sivoais I listed them in my comment above
19:18 opkick [pdl] zmughal reopened issue #6: This repo very out of date http://git.io/xChu
19:18 sivoais reopening as a reminder
19:19 sivoais actually, that's silly. I'll make another issue
19:20 opkick [pdl] zmughal opened issue #57: Mirror other PDL repos http://git.io/x6ds
19:20 opkick [pdl] zmughal closed issue #6: This repo very out of date http://git.io/xChu
19:27 opkick [Alien-HDF4] zmughal created travis-ci (+1 new commit): http://git.io/x6Nd
19:27 opkick Alien-HDF4/travis-ci dd04f16 Zakariyya Mughal: add .travis.yml
19:34 opkick [Alien-HDF4] zmughal force-pushed travis-ci from dd04f16 to 312498a: http://git.io/x6hR
19:34 opkick Alien-HDF4/travis-ci 312498a Zakariyya Mughal: add .travis.yml
19:35 opkick [Alien-HDF4] zmughal force-pushed travis-ci from 312498a to f56d4fb: http://git.io/x6hR
19:35 opkick Alien-HDF4/travis-ci f56d4fb Zakariyya Mughal: add .travis.yml
19:46 opkick [Alien-HDF4] zmughal force-pushed travis-ci from f56d4fb to b67edeb: http://git.io/x6hR
19:46 opkick Alien-HDF4/travis-ci b67edeb Zakariyya Mughal: add .travis.yml
19:54 travis-ci PDLPorters/Alien-HDF4#5 (travis-ci - b67edeb : Zakariyya Mughal): The build passed.
19:54 travis-ci Change view : https://github.com/PDLPorters/Alien-HDF4/compare/f56d4fbd47ea...b67edebff7c7
19:54 travis-ci Build details : http://travis-ci.org/PDLPorters/Alien-HDF4/builds/52944720
19:55 opkick [Alien-HDF4] zmughal opened pull request #3: add .travis.yml (master...travis-ci) http://git.io/xitf
20:01 opkick [Alien-HDF4] zmughal force-pushed travis-ci from b67edeb to eba7c56: http://git.io/x6hR
20:01 opkick Alien-HDF4/travis-ci eba7c56 Zakariyya Mughal: add .travis.yml
20:28 drrho joined #pdl
23:00 sivoais mohawk: an example of that build thing would be the difference between editing the docs versus the PP code
23:11 opkick [pdl] zmughal created doc/vsearch-example (+1 new commit): http://git.io/xPHm
23:11 opkick pdl/doc/vsearch-example 444ce66 Zakariyya Mughal: add vsearch example...
23:12 opkick [pdl] zmughal opened pull request #58: add vsearch example (master...doc/vsearch-example) http://git.io/xPHW
23:17 opkick [pdl] zmughal closed issue #39: SF#347 t/pdl_from_string.t has a failure if BADVAL_NAN=1 http://git.io/xgWr
23:20 opkick [pdl] zmughal comment on issue #49: @mohawk2, you made the change that fixes this bug that @kmx brought up on SF. You built on Windows 64-bit when you merged #18, correct? Close? http://git.io/xP5V
23:40 opkick [pdl] zmughal pushed 1 new commit to doc/vsearch-example: http://git.io/xPhl
23:40 opkick pdl/doc/vsearch-example b635451 Zakariyya Mughal: add links to other vsearch_* functions

| Channels | #pdl index | Today | | Search | Google Search | Plain-Text | summary