Afternoon. I'm less rubbish than last week (in as much as I'm here), but more rubbish than I should be because I havem't had a chance to do the agenda.
Dave
This is definitely on it though: https://xmpp.org/extensions/inbox/muc-avatars.html
Dave
So this would be the agenda, if I read correctly: https://docs.google.com/spreadsheets/d/1AZ-Sna6OiRG--b-mJMKv3XXfrn3Nehm0kAtlyJvImL0/edit#gid=0&range=B93
Dave
No, some (many) of these have been voted on.
Dave
Right, corrected, though I'll fill the votes in later for the ones that have been done.
SamWhitedcrams before the meeting
Dave
Righty, 'tis time.
Dave
1) Roll Call
Dave
Who be here?
Kev
Here. Entirely unprepared, but here.
Dave
I'm pretty unprepared as well.
SamWhited
I am here, and have read at least one of the protoxeps
SamWhited
But also pretty unprepared
Dave
SamWhited, There's only one protoXEP, plus one PR.
SamWhited
Oh, I thought a second one came in. I have read the protoxep!
Dave
No Daniel or Georg?
Dave
2) Voting On Stuff
Lancehas joined
Dave
a) MUC Avatars https://xmpp.org/extensions/inbox/muc-avatars.html
daniel
Here
daniel
+1
Kev
I'll try to get this one on-list, but realistically I'm flat out until holiday and it might expire.
Zash
I heard Georg is on vacation
Dave
I'm +1 on this, though i would like to understand how it differs from the de-facto that it suggests exists.
SamWhited
I'd be curious to hear what others (especially client authors) think about this one. I tend to think it's unnecessary, because you can just do it all already as far as I can see and having a second XEP that says more or less the same as vcard-temp feels awkward to me.
daniel
Minus presence plus in Disco info
Link Mauve
Dave, I tried to explain that in both the introduction and section 5.2.
Kev
I'm +1, have just scanned it.
daniel
But I'm not entirely sure about the motivation
Link Mauve
SamWhited, I hesitated about making it just informational, still not too sure.
Dave
Link Mauve, Ah, I thought that was just about presence.
Dave
SamWhited, It's mostly usage and semantics rather than protocol, but it seems useful to document.
SamWhited
That might make a bit more sense, having a standards track XEP that rehashes a historical one and adds discovery just feels odd
Link Mauve
Dave, presence vs. disco#info is the only part where my proposal differs from the status quo.
Dave
Link Mauve, Gotcha.
SamWhited
"That" == "Informational"
daniel
But since servers will do that anyway now I don't think that this xep will retroactively fix that
daniel
I mean sorry that clients broke and all
SamWhited
Documenting things sounds find, but it feels poor to have multiple XEPs in various states documenting the same thing. Couldn't we equally just add a note to vcard-temp saying "remember, MUCs are JIDs too"
Link Mauve
At least Prosody will probably skip on the presence way.
daniel
But the demage is done
daniel
But we don't have to discuss this on council
SamWhited
Or a nod to vcard temp in 0045 even
Dave
SamWhited, Are you voting on this or thinking? (Georg is presumed to be on list anyway)
daniel
I'm +1 and then we can discuss on list once the xep is out
SamWhited
I'm thinking out loud, just curious to get others opinions. I guess I'm on list. I'm hesitant to publish this, but it seems fine protocol-wise.
Dave
Cool.
genofirehas left
Dave
b) XEP-0060: Add an example on returning fewer items than requested https://github.com/xsf/xeps/pull/695
daniel
Also really if your client breaks if you get a presence from a bare jid. Wtf. Just fix your client
daniel
The neat thing about doing the presence hash thing is that it worked instantly w/o any changes
Dave
daniel, Feel free to argue this position on list. :-)
Dave
Anyone any votes for this PR?
Dave
I personally think it's straightforward, so +1.
SamWhited
+1 on the PR, though I'll leave an editorial note for the author, but it won't change the substance of it
Kev
I'm not sure why this is a thing. What's the backstory here?
Dave
Link Mauve, ?
Dave
Kev, FWIW, the XEP doesn't say what happens if a clientrequests N most recent when there are only M (M < N) items present. This is the obvious thing to do.
Kev
It being the obvious thing made me wonder why it needed to be a thing.
Link Mauve
Hmm, it was during the XMPP Sprint with MattJ, we IIRC encountered a case where Prosody was sending back an error instead of zero items.
Link Mauve
This makes sure servers behave correctly in this case.
Kev
Fair enough. No objections here.
Dave
Kev, Is that a 0 or a +1?
daniel
+1
SamWhited
Added some minor nits on wording; only the first one really needs to be fixed. The other is just me being picky and I don't think it actually matters, feel free to disregard.
SamWhited
(I'm +1 either way with my council hat on, nits are with my editor hat on)
Kev
0 because I don't have time to do the full review involved in +1ing it. The patch looks fine, but I don't think it's sufficiently diligent to approve just based on the diff - we know how that has worked in the past with complex XEPs like 50.
Dave
OK
Dave
3) AOB
labdsfhas left
Kev
Nowt.
Dave
4) Next Meeting
Dave
I saw you were saying you'll be away for a bit, Kev?
labdsfhas joined
Kev
I'm out of action until October.
Dave
Anyone else going to be MIA?
Kev
(That'll be fully out of action, not on-list, unfortunately)
Dave
I have no idea how you'll survive.
Dave
I'll assume the rest of us will meet same time, same channel.
SamWhited
WFM
Dave
So absent anything else, thanks all, see you next week.
Dave
5) Ite, Meeting Est.
Syndacehas left
Syndacehas joined
Link Mauve
Btw, there are still quite a few outstanding PRs at https://github.com/xsf/xeps/pulls which are tagged as Needs Council, could you add them all to next week’s agenda?
Dave
Link Mauve, I believe that those have had votes and expired. I'll see about tidying these up and ensuring they're documented etc.
Link Mauve
Ta.
guus.der.kinderen
Dave: #579 was voted on?
guus.der.kinderen
The h thingy on SM
guus.der.kinderen
It was in February, but was modified since.
peterhas joined
Kevhas left
Davehas left
Lancehas left
Lancehas joined
Lancehas left
Kevhas left
martinhas left
genofirehas left
martinhas joined
Davehas left
Link Mauvehas left
labdsfhas left
labdsfhas joined
Davehas left
genofirehas left
labdsfhas left
martinhas left
labdsfhas joined
jonaswhas left
labdsfhas left
Kevhas left
Davehas left
Davehas left
vanitasvitaehas left
lnjhas left
lnjhas joined
martinhas left
martinhas left
martinhas left
Davehas left
Davehas left
genofirehas left
jonaswhas left
labdsfhas joined
labdsfhas left
labdsfhas joined
SamWhitedhas left
danielhas left
labdsfhas left
labdsfhas joined
danielhas left
flow
guus.der.kinderen, #579 appears to be conflicting with master (at least gh tells me so)
labdsfhas left
labdsfhas joined
labdsfhas left
labdsfhas joined
flow
guus.der.kinderen, I would suggest squashing into a single commit and rebaseing on the current master