2015-06-01 UTC
KevinMarks_, fuzzyhorns, KevinMarks, ivc\zz and netweb joined the channel
KevinMarks, KevinMarks2, kez, Left_Turn, Erkan_Yilmaz, glennjones, KartikPrabhu, eschnou, krijnhoetmerbot, frontwards, adactio, Mark87_, benborges and netweb joined the channel
# 12:49 glennjones tantek: KevinMarks: Adding a couple comments to rel-urls brainstroming wiki page, about inconsistent structure and a new property for rel=tag which is the last path segment
eschnou and TallTed joined the channel
fuzzyhorns, fuzzyhor_, glennjones and benborges joined the channel
# 16:50 tantek glennjones: taking a look at the rel-urls stuff now
# 16:59 tantek !tell glennjones no need to return last path segment of the URL, because the URL is already there - and that's just a library/framework utility function to get the last path segment of a URL
# 16:59 Loqi Ok, I'll tell them that when I see them next
# 17:01 tantek and thanks for the editorial fix to the parsing spec!
gRegorLove joined the channel
# 17:13 Loqi glennjones: tantek left you a message 13 minutes ago: no need to return last path segment of the URL, because the URL is already there - and that's just a library/framework utility function to get the last path segment of a URL
# 17:16 tantek glennjones: I'm having trouble seeing the differences in those two long pages
# 17:18 tantek oh I see it now - the mf2py parser forgot to put the "alternate" into the rel-urls collection
# 17:19 tantek yes - 5 looks correct - looks like a bug in the mf2py parser implementation of rel-urls
# 17:19 tantek might want to ping KevinMarks since that's his code I'm pretty sure
# 17:21 glennjones Yes, I was just wonder if mf2py was correct, but the way I read the logic on the parsing page it should include the "alternate"
KevinMarks_ and KartikPrabhu joined the channel
eschnou, KartikPrabhu, KevinMarks___ and KevinMarks_ joined the channel
# 19:35 KevinMarks I left alternates out of rel-urls because they were left out of rels
# 19:35 KevinMarks my assumption was that you'd look up the rel you were interested in, then get more info about each URL represented
# 19:36 KevinMarks I can put the redundant alternate in rel-urls, but then we should put it in rels as well
KartikPrabhu joined the channel
glennjones joined the channel
# 20:18 gRegorLove Is the main reason to not break backwards compatibility? The rel-alternates hash has array keys already, but the other rels don't
# 20:22 KevinMarks I found that looking up by rel and walking the list of urls, then getting more info if needed worked well
# 20:34 gRegorLove And having rels: coworker: url: would break consumers that aren't expecting the 'url' key, but just the actual url
glennjones_ joined the channel
# 20:42 KevinMarks I did consider adding another magic thing like alternates for xfn, but then I'd need another one for enclosures and so on
# 20:45 gRegorLove So rels should still be the first place to look and if you need more info, rel-urls.
# 20:45 gRegorLove With the exception of rel-alternates, which would have the extra info in both spots
fuzzyhorns joined the channel
# 21:10 KevinMarks yeah, which is why i think alternates shouldn't be in both, or if it is should eb consistent
tantek and voxpelli joined the channel
# 21:21 tantek alternates was separate only because it needed more information
# 21:21 tantek that is - for any alternate to be useful you need more information, so it didn't make sense in the simple "rels" collection
# 21:22 tantek the goal was to keep the rels collection simple
# 21:22 tantek whereas with rel-urls, the goals was to include all the information necessary for all kinds of use-cases of various rels
# 21:22 tantek thus it makes sense to include alternate in there along with all other rels
# 21:22 KevinMarks right, but as yu have written it there is no 'alternate' in rels but there is in rel-urls
# 21:23 tantek 'alternate' doesn't make sense in rels because all use-cases for it require more information
# 21:23 tantek whereas 'alternate' *does* make sense in rel-urls because all the information for them is provided there
# 21:23 Loqi KevinMarks meant to say: but given we have alternates, why put it in rel-urls too?
# 21:24 tantek I'd be ok with dropping 'alternates' collection as I don't know of any current consuming code
# 21:25 tantek KevinMarks: because the method of look-up in rel-urls makes more sense (is easier) than the way the current alternates collection works
# 21:25 tantek I would expect anyone actually looking for alternates to now use the rel-urls map
# 21:25 KevinMarks except that I see rels as provdidng the keys and rel-urls the details
# 21:26 tantek right, so if we drop 'alternates' then we must add them, even as just urls, to rels
# 21:26 tantek point being, for now, it makes sense to have the alternates in rel-urls
# 21:26 tantek it's separate issue to consider dropping alternates and along with that change, adding alternate urls to rels
# 21:27 tantek that would make sense if we dropped 'alternates'
# 21:28 tantek because anyone actually looking for a specific alternate (which all use-cases do) would use the alternates collection currently
# 21:28 KevinMarks I'm saying it doesn't make sense to have a url in rel-urls that isn't in rels
# 21:28 tantek but that's more work for everyone, so it doesn't make sense
# 21:29 tantek it's easier to just put all the rel urls into rel-urls, than have to explain why they are not all in there
# 21:29 tantek !tell aaronpk does your indie reader use mf2 parsing to look for alternates?
# 21:29 Loqi Ok, I'll tell them that when I see them next
# 21:30 KevinMarks it doesn't make sense to put the alternate urls in rel-urls without putting them un rels
twisted` joined the channel
# 21:31 tantek of course it does. rel-urls is new, and thus it makes sense to get it right.
# 21:31 tantek it's less code to put all the rel urls into rel-urls
# 21:31 tantek to put rel alternate urls into rel-urls, regardless of any other conditions
# 21:32 KevinMarks I've written the code, you haven't. its the same amount, it just depends where the else goes
# 21:32 tantek well I wrote the pseudo-code in the spec for it ;)
# 21:37 tantek what do other folks think? shall we include alternates in the "rels" collection and drop "alternates"?
# 21:38 KevinMarks it is actually simpler to make sense of as the alternates case is truly separate
# 21:38 tantek KevinMarks: I think you screwed up a list nesting level :P
# 21:39 tantek but as glennjones was pointing out, it makes the parsed output more confusing
# 21:39 tantek which is why I'm now advocating dropping "alternates" as the path forward
andicascadesf joined the channel
# 21:41 tantek I'd rather go for simplifying KevinMarks - based on glennjones feedback
# 21:46 tantek alright, I'm going to revert those change then - as I'd rather have the intended rel-urls functionality be in the spec
# 21:47 Loqi tantek meant to say: alright, I'm going to revert those changes then - as I'd rather have the intended rel-urls functionality be in the spec
KartikPrabhu joined the channel
# 22:07 tantek KevinMarks: no no - you're familiar with this - just check the fragmention I pasted ^^
# 22:09 tantek for the rels stuff I just added warnings where the algorithm will change
# 22:14 KevinMarks you'd want the rels for teh exmaple to be "rels": { "author": [ "http://example.com/a", "http://example.com/b" ], "in-reply-to": [ "http://example.com/1", "http://example.com/2" ],"alternate":["http://example.com/fr"] }
,
# 22:17 KevinMarks that nested h-object issue looks good, and the implied name rule means that the value will match the name rule fro whitespace, whatever the resolution, which is good as that was a point of contention
KartikPrabhu joined the channel
# 22:44 tantek nope - adding the whole alternates removal thing as an issue
# 22:44 KevinMarks as written the pesudocode is inconsistent between rels and rel-urls
# 22:45 tantek no - as written rel-urls is complete, and adding to rels is a separate issue
# 22:46 tantek will add a note about the alternates in the list
# 22:47 tantek nope, double-checked and existing warnings about pending changes cover the case of including "alternate" in rels
# 22:48 tantek because it's not special-cased anymore, it gets added along with everything else
# 22:51 KevinMarks to use alternate you need the keys I just added to the example
# 22:54 tantek you do not need that KevinMarks - you can simply iterate through rel-urls
# 22:54 Loqi aaronpk: tantek left you a message 1 hour, 24 minutes ago: does your indie reader use mf2 parsing to look for alternates?
# 22:54 tantek KevinMarks: but you're right that the example had that omission - will add that in a warning note
# 22:55 tantek kevinmarks please stop - trying to get broader discussion before actually making the changes to the spec
# 22:55 tantek that's the point of putting in the warnings first
# 22:56 KevinMarks it makes no sense to iterate the rel-urls looking for the alternates
# 22:59 KevinMarks (I wasn't tryign to have an edit war, but to have something to point to
# 22:59 tantek since it's a *change* to how things are spec and implemented
# 23:00 tantek it actually *does* work as spec'd. and yes there can be improvements. hence another issue.
# 23:01 tantek thanks for pointing out that the example did not document the change that would occur
# 23:01 tantek I believe I just recaptured that in that edit
# 23:02 KevinMarks have rels and rel-urls be parallel is key. Your spec text puts them in an inconsistent stare
# 23:02 Loqi KevinMarks meant to say: have rels and rel-urls be parallel is key. Your spec text puts them in an inconsistent state
# 23:02 tantek having rels expanded is better, but it's not "key"
# 23:02 KevinMarks which requires the workaround fo walking the rel-urls again checking their rels
# 23:05 tantek KevinMarks: not as documented in the brainstorm which we discussed and agreed, which includes alternates in rel-urls
# 23:06 tantek so that's what's spec'd since that's what we agreed on: microformats2-parsing-brainstorming#more_information_for_rel-based_formats
# 23:07 KevinMarks hm, my "This generalises to other rel's too, such as rel-feed and rel-alternate that have type, lang etc attributes." was ambiguous there
# 23:08 tantek KevinMarks_: not ambiguous at all - by explicitly listing rel-alternate in that sentence the intent of inclusion in rel-urls was very clear
# 23:08 KevinMarks but I in no way thought that the rels and rel-urls would not match
# 23:09 tantek I was doing quite literal brainstorm application
# 23:09 tantek since we discussed and agreed on what was documente
# 23:10 KevinMarks you're saying that your theoretical implementation of an ambiguous brainstorm trumps my actual implmentation of it, that is lined to from it?
# 23:23 tantek this is why implementations != specs, but rather implementations help double-check specs
fuzzyhorns joined the channel
# 23:55 tantek also why I specifically asked for it to be written up on the brainstorming page, rather than just "code is the spec" in an implementation