#dev 2017-02-06

2017-02-06 UTC
sebsel, miklb_, KevinMarks, tantek and [dgold] joined the channel
#
loqi.me
created /Pingback_spam (+31) "prompted by tantek and dfn added by tantek"
(view diff)
KevinMarks_ joined the channel
#
tantek.com
edited /DDOS (+944) "Pingback DDOS example on Krebs noted in Ars Technica article"
(view diff)
#
loqi.me
created /Pingback_DDOS (+31) "prompted by tantek and dfn added by tantek"
(view diff)
mblaney and KartikPrabhu joined the channel
#
ben_thatmustbeme
finally, after several weeks or working on it, i'm starting to get something useful for my laravel rewrite of my site
#
ben_thatmustbeme
should be much saner to maintain
#
ben_thatmustbeme
and hopefully i can bundle some things up as libs that others can use, like webmention with vouch
#
ben_thatmustbeme
its a little sad to be starting over from scratch, doing it all while maintaining all my permalinks has been the biggest concern thus far
#
ben_thatmustbeme
has taken most of my time
#
aaronpk
This sounds familiar
#
aaronpk
I also rewrote my site in Laravel :-)
#
ben_thatmustbeme
your code isn't public though is it?
#
ben_thatmustbeme
well, at least you can compare
KevinMarks and KevinMarks_ joined the channel
tantek joined the channel
#
GWG
I agree with the conclusions to a point. But it is also an issue.
#
GWG
The plugin could degrade gracefully.
KevinMarks joined the channel
#
aaronpk
I'm pro not having the plugin register functions in the global namespace. That's how PHP gor itself a bad name long ago.
#
martymcguire[m]
Wow that thread ended up being a lot of discussion for a one line array syntax sugar change to keep backwards compatibility.
#
aaronpk
Supporting 5.2 is a different story
#
GWG
aaronpk: What is the middle solution?
#
tantek.com
edited /Medium (+621) "/* Criticism */ Lightweight Marketing Articles, from HN"
(view diff)
#
aaronpk
You could bundle the mf2 parser with the webmention plugin by doing a sort of manual release of it
#
GWG
aaronpk: You mean a backcompat fork?
#
aaronpk
Kind of.. it wouldn't take much to make a process for backporting the parser code for 5.2
#
aaronpk
luckily the plugin is really just a bunch of functions that happen to be in the \mf2 namespace
#
GWG
aaronpk: That I know nothing about.
#
GWG
How to create such a process, I mean.
#
aaronpk
Would it help if I made a script in the mf2 parser repo that would export a 5.2 compatible version of the code?
#
GWG
aaronpk: Yes. I would certainly use it in my plugin, and I'd imagine pfefferle would do the same for Semantic Linkbacks.
#
GWG
But it would only solve one problem.
#
GWG
The rest still have to be figured out
#
GWG
Still...one problem less
#
kartikprabhu.com
edited /Medium (+204) "/* Other */ add archive snapshot of article"
(view diff)
#
aaronpk
now…where to find a place to run php 5.2
#
KartikPrabhu
does php not have something like a virtual environment?
#
aaronpk
i know on ubuntu i can install multiple versions now
#
aaronpk
haven’t tried on osx
#
aaronpk
also 5.2 is officially unsupported. looks like it’s not available via homebrew
#
tantek
presumably it is supported by various hosting companies, otherwise WordPress wouldn't stick to it
#
GWG
Annoying, isn't it?
KevinMarks_ joined the channel
#
aaronpk
dreamhost is being proactive and appropriately dropping support for old versions
tantek joined the channel
#
aaronpk
dreamhost only goes back to 5.5 right now
#
aaronpk
i can’t even figure out how to get something to install 5.2
#
aaronpk
can’t even figure out if travis-ci.org supports it
#
GWG
aaronpk: And this is, as I said, the tip of the WP iceberg. This is just making sure the library meets minimum standards.
#
GWG
aaronpk: I'm almost certain it does
#
aaronpk
i’d like to be able to test it. know where i can find a host that supports 5.2?
#
aaronpk
travis-ci does support 5.2! so at least i can “run” it there
#
aaronpk
oh wait. noit doesn't
tantek joined the channel
#
KartikPrabhu
that seems useful!
#
aaronpk
installs virtualbox
#
GWG
I installed https://github.com/wimg/PHPCompatibility to try to flag issues
tantek joined the channel
#
aaronpk
finally has a ubuntu VM
tantek and jaduncan[m] joined the channel
#
aaronpk
and bah phpenv doesn’t work
KevinMarks joined the channel
#
aaronpk
will try again tomrorow
#
david.shanske.com
edited /Wordpress_Webmention_Plugin (+525) "/* Issues and Brainstorming */"
(view diff)
#
GWG
What properties do you store from a webmention?
#
GWG
I'm brainstorming
#
aaronpk
i made progress. i have to wade through these docs now https://github.com/php-build/php-build
#
GWG
aaronpk: I feel like I sent you down a rabbit hole
#
david.shanske.com
edited /WordPress/Data (+620) "/* Metadata */"
(view diff)
KevinMarks, tantek and KevinMarks_ joined the channel
#
jonnybarnes
so why was my issue mentioned? GWG
#
GWG
Jonnybarnes, WordPress has a PHP5.2 compatibility we were discussing
#
jonnybarnes
which is just silly, when was the last time 5.2 got a security update?
#
GWG
You can't abandon the WordPress user base just because of that though. The better question is what features of the newer PHP version do you need?
#
jonnybarnes
if i remember correctly 5.3 had a few things that were going to be in PHP6, like autoloading and stuff
#
GWG
Are we using them?
#
GWG
We briefly lost 5.3 support over short array notation
#
GWG
A code shortcut is not a reason to bump a version either
#
GWG
Changing minimum support requirements only makes sense when you need a feature only provided by a newer version
cweiske and pfefferle joined the channel
#
pfefferle
good morning
#
GWG
Good morning, pfefferle. We were talking about Semantic Linkbacks in your absence.
#
pfefferle
GWG what have I missed?
#
pfefferle
GWG the mf2 discussion?
#
GWG
Yes, among others
#
GWG
It started elsewhere. I was thinking about your comments
#
GWG
You said that Semantic Linkbacks was a bit hacky. I was trying to figure out how to make it less so
[cdevroe], singpolyma and tantek joined the channel
#
aaronpk
i now have php 5.2 installed
#
aaronpk
trying to figure out how to install corresponding old version of phpunit to run the tests
#
aaronpk
oh gosh this is not going to go well
#
aaronpk
"Composer requires PHP 5.3.2+ to run"
#
tantek
oh boy
#
aaronpk
looool no anonymous functions in 5.2
#
tantek
is pretty sure cassis still runs in 5.2 :P
#
tantek
can't be sure because his local php is 5.5.x and webhost has been auto-upgrading it for a while
#
aaronpk
i have a VM with 5.2 on it if you’d like me to test it
#
tantek
sure!
#
aaronpk
step 1 is it parses fine!
#
aaronpk
actually the js one doesn’t. the compacted .php version does
#
tantek
weird!
#
tantek
where's the parse error?
#
aaronpk
there’s a duplicate function!
#
aaronpk
date_get_rfc3339 is defined tweice
#
tantek
that might be worth an emergency fix
#
aaronpk
if i take out the one at the top of the file then it parses fine
#
aaronpk
hm, running auto_link outputs a bunch of the JS comments
#
aaronpk
using my php-only version works fine tho
#
tantek
aaronpk - yeah you have to use output buffering
#
aaronpk
around every function call?
#
tantek
oh perhaps
#
aaronpk
yikes. no wonder i didn’t want to use cassis until i made the php-only version
#
tantek
I thought it was only around the include
#
tantek
I need to make that a formal requirement
#
aaronpk
the php-only one is what’s included from composer, and it’s part of the repository which also includes the post-process script to create it, so might not be that important to worry about
#
aaronpk
oh boy okay this isn’t going to be as easy as i’d hoped
#
aaronpk
oh wait… progress!
#
aaronpk
hm this might be a dealbreaker… the $node parameter of DOMDocument::saveHTML was added in 5.3 http://php.net/manual/en/domdocument.savehtml.php
#
aaronpk
other than that, it looks like it’s parsing my home page okay! (the HTML values are missing because of that difference)
#
tantek
aaronpk, still looking into "date_get_rfc3339 is defined twice" - it should only be defined once for PHP then once for JS
#
tantek
yeah exactly - the PHP parser should not be seeing that second one!
#
tantek
It's in PHP comments!
#
aaronpk
then why is there a if(js()) check in the second one?
#
tantek
oh ooops
#
tantek
weird, maybe I switched that?
#
tantek
ok you're right, that version is inconsistent in that way from what I'm running on my site! Argh, must have missed fixing this
KartikPrabhu and KevinMarks joined the channel
#
tantek
finds all kinds of new functions he's been testing in production that need to be committed
#
tantek
starts doing verification diffs
#
tantek
checks his local server before blanking his homepage and all permalinks
#
tantek
aaronpk++ again for helping me set that up
#
Loqi
aaronpk has 33 karma in this channel (1205 overall)
#
tantek
and now for the git diffs
KevinMarks_ joined the channel
#
tantek
uses IRC to keep track of his multi-tasking state
#
tantek
discovers pi to 3 digits in sexagesimal epoch days and an ordinal date!
#
tantek
sexagesimal epoch days: 100 === 1979-314!
#
tantek
aaronpk, great catch of that duplicate function definition
#
tantek
definitely worth fixing before a "release"
#
tantek
cassis.js updated on master - aaronpk - can you try again? (php 5.2)
KevinMarks joined the channel
#
aaronpk
auto_link works now!
#
tantek
it didn't before?
#
tantek
(or rather, was that what you were using for the parse test?)
#
aaronpk
it still outputs when the function is run, but the php-only version solves that
#
tantek
yeah - I noted that in the header comments :)
#
tantek
(I mean, from before even, not a new comment)
#
aaronpk
is that what "and use thereof” means?
#
tantek
I can see how that was too subtle
#
tantek
rewording
#
tantek
aaronpk better? if you see this or "/// var" in the browser, you need to wrap your PHP include of cassis.js and use of functions therein with calls to ob_start and ob_end_clean, e.g.:
#
aaronpk
yeah. and maybe show two examples. one of the include and another of just the function call
#
tantek
I this was good enough (instead of mentioning a specific function) // your code that calls CASSIS functions goes here
KevinMarks joined the channel
#
tantek
well, fixed one bug for today, even if it was a PHP 5.2-only parse bug.
#
tantek
though I did ship (open source) a couple new functions in CASSIS that previously I'd only been testing privately
#
aaronpk
that wasn’t just a 5.2 problem actually! I tried it on 5.6 and it was the same thing
#
tantek
oh cool! thanks!
#
tantek
weird that I wasn't seeing it
#
pfefferle
aaronpk why are you trying to downgrade mf2 to php5.2
#
aaronpk
so that it can be included in wordpress
#
loqi.me
created /exercise_watch (+201) "prompted by tantek and dfn added by tantek"
(view diff)
#
loqi.me
edited /exercise_watch (+30) "/* See Also */ new section"
(view diff)
#
pfefferle
aaronpk I am not sure if this would be the main problem...
#
aaronpk
how so?
#
tantek
what is the main problem?
#
Loqi
It looks like we don't have a page for "main problem" yet. Would you like to create it?_S
#
pfefferle
aaronpk I think we should start the discussion to include the webmention plugin and then iterate... or is there a discussion that this is the last step to integrate all of the plugins?
#
aaronpk
well my understanding was no plugin would be able to include the microformats parser unless it was compatible with 5.2, so i figured i would see what it would take to make it compatible
#
pfefferle
aaronpk the core is PHP 5.2.4+ but they recomment PHP 7
#
pfefferle
aaronpk I think we should start to integrate on plugin after another and if the mf2 lib is a problem, they would tell us...
#
pfefferle
aaronpk it is no problem to release the plugin in the official repo, so I see no need atm.
KevinMarks joined the channel
#
pfefferle
aaronpk I have not seen a official discussion about integrating the plugin into core, so I would wait until we get such a feedback...
#
aaronpk
okay, well i did all the “easy” stuff to make it work with 5.2, but hit a snag with domdocument and I don’t have a quick fix for that
#
aaronpk
so I won’t bother continuing until I know if it’s going to be worth the effort
#
pfefferle
aaronpk that is what I would recommend :)
#
pfefferle
aaronpk It would have been awesome if this was the least thing to do to integrate it into core :) :(
pfefferle, pfefferle_, KevinMarks and KevinMarks_ joined the channel
#
tantek
aaronpk - perhaps capture the snag with domdocument as an issue with how far you got?
#
aaronpk
yeah i can do that
#
tantek
cool, that way you've externalized that internal state and can clear your local cache ;)
#
aaronpk
i think i might commit the other changes to the main branch too, since they’re forwards-compatible and certainly don’t make the code any worse
#
tantek
definitely!
#
tantek
feel free to create an issue for "PHP 5.2.4 backcompat" to link them to
#
tantek
that way the longer term intent is captured
#
tantek
tries to externalize this kind of state as much as he can
KevinMarks joined the channel
#
vanderven.se martijn
edited /YouTube (+1263) "Document atom feeds and be specific about the removal of watch history from the API."
(view diff)
KartikPrabhu joined the channel
#
loqi.me
edited /Accelerated_Mobile_Pages (+68) "KevinMarks added "https://amphtml.wordpress.com/2017/02/06/whats-in-an-amp-url/amp/" to "See Also""
(view diff)
pfefferle and tantek joined the channel
#
sknebel
what is RethinkDB?
#
Loqi
It looks like we don't have a page for "RethinkDB" yet. Would you like to create it?_T
KevinMarks, KevinMarks_, KartikPrabhu, tantek, miklb, petermolnar, prtksxna and mindB joined the channel
#
GWG
aaronpk: I told you it was only one problem.
KevinMarks joined the channel