XSF Editor Team - 2019-10-15

  1. debacle has left

  2. ralphm has left

  3. ralphm has joined

  4. Tobi has joined

  5. lnj has joined

  6. lnj has left

  7. lnj has joined

  8. debacle has joined

  9. debacle has left

  10. pep.

    hmm, how do I push on georg's branch btw, on github

  11. pep.

    Is that possible?

  12. pep.

    jonas’, ^

  13. pep.

    How do you usually do? Do you just prepare the merge on your repo and only then push to master?

  14. pep.

    note for later: I'll want to add support to send-updates.py to use my local msmtp setup

  15. pep.

    heh the XEP- headers are a nice touch

  16. jonas’

    pep., I think there are instructions for that on the PR page

  17. jonas’

    pep., and yes, normally I prepare just loclly

  18. pep.

    Right, it doesn't talk about what I want to do, it's another workflow, ok

  19. pep.

    unrelated, pilotfiber.dl.sourceforge.net has an expired cert since Sep 22 apparently.. it's being pulled in xep-docker-base

  20. pep.


  21. jonas’

    I have a fix for that in the pipeline

  22. pep.


  23. jonas’

    for now, just use the latest build off docker hub

  24. pep.

    https://github.com/xsf/xeps/pull/843 should I also put that in the batch or wait a bit?

  25. stpeter has joined

  26. pep.

    https://github.com/Ppjet6/xeps/tree/merge-xep-423%2C402%2C405 jonas’

  27. pep.


  28. pep.

    I've been rebasing stuff. Maybe I should force the author not to be me to have that a bit cleaner

  29. pep.

    Or I can just merge

  30. jonas’

    don’t rebase other peoples commits

  31. jonas’

    that’ll confuse github

  32. pep.

    yeah I can see that..

  33. jonas’

    $ cat ~/.local/bin/git-make-pr-branch #!/bin/bash set -euo pipefail branch_name="$1" remote="$2" pr="$3" commit_id="$(git ls-remote "$remote" "refs/pull/$pr/head" | cut -f1)" git fetch "$remote" "$commit_id" git checkout -b "$branch_name" "$commit_id"

  34. jonas’

    that script will make a new branch with the HEAD of the PR

  35. jonas’

    you can add commits there, and then do a merge to master

  36. pep.

    I know, I'm already doing that

  37. pep.

    Ok, so merge

  38. jonas’

    yes, merge

  39. jonas’

    XEP-0423 lacks a revision block in the tree you linked me

  40. pep.

    wait I did add one though

  41. pep.


  42. jonas’

    you can add https://github.com/xsf/xeps/pull/843 to the merge

  43. jonas’

    pep., that’s not showing up in https://github.com/xsf/xeps/compare/master...Ppjet6:merge-xep-423,402,405?expand=1 for some reason

  44. pep.

    it is

  45. jonas’


  46. pep.

    Only under my name

  47. jonas’

    I’m too stupid to read diffs

  48. pep.


  49. jonas’

    did you check that the CI passes on this?

  50. pep.

    I have an awfully slow connection atm, docker barfs on me at each attempt

  51. pep.

    I get disconnected on a regular basis

  52. jonas’

    pep., the CI is run by github, too

  53. jonas’

    just make a PR

  54. pep.

    Right. let me do merges then, and I'll submit a PR

  55. jonas’


  56. jonas’

    other than that, the formal stuff looks ok

  57. lnj has left

  58. pep.

    Sent, https://github.com/xsf/xeps/pull/844

  59. pep.

    I'm moving to c3d2, see you online in a bit :)

  60. jonas’

    oh, that’s close

  61. pep.

    Yes it's not far from you :p

  62. pep.

    I'm here until thursday morning, then moving to munich

  63. pep.

    daniel should be there tonight, and Astro, don't know if you know him

  64. jonas’

    yeah, met astro at some meetpu

  65. pep.


  66. stpeter has left

  67. stpeter has joined

  68. stpeter has left

  69. pep.

    Ok PR passed. I'll merge and spam standards

  70. jonas’

    sounds good

  71. pep.

    smtplib.SMTPSenderRefused: (452, b'4.3.1 Insufficient system storage', 'pep@bouah.net')

  72. pep.

    oops, thanks smtplib for reminding me

  73. pep.

    hmm, that didn't catch 335?

  74. jonas’

    because 335 doesn’t have a revision block, and the transition isn’t handled specially

  75. jonas’

    please use the dry run mode to double-check output before sending emails

  76. jonas’

    it’s fine to not-announce the transition tho

  77. jonas’

    (we never did that before, either)

  78. debacle has joined

  79. pep.


  80. pep.

    Yeah I did check emails before. The one for 423 is a bit mangled but ~

  81. pep.

    I mean the revision block

  82. ralphm has left

  83. ralphm has joined

  84. jonas’


  85. pep.

    I'm also waiting for dwd to send an email or sth to standards for the agenda for tomorrow, to add 292 which is stuck in LC

  86. pep.

    But I guess it's probably not happening at this point (the email)

  87. jonas’

    yeah, dwd sent a mail to council@

  88. jonas’

    he won’t be able to write an agenda before tomorrow morning

  89. jonas’

    he won’t be able to write an agenda before tomorrow noon

  90. jonas’

    you can send agenda suggestions to dave@cridland.net

  91. pep.

    is council@ private?

  92. pep.

    Or am I just not subscribed to it

  93. pep.

    ok it's not private

  94. pep.

    I have a reliable connection now! I can check your docker PR

  95. jonas’

    pep., https://github.com/xsf/xep-docker-base/pull/4

  96. pep.

    heh, ok

  97. pep.

    I'm curious how much work this is going to be to update to buster tbh..

  98. jonas’

    I have another PR ;)

  99. jonas’

    oh I don’t

  100. jonas’


  101. pep.


  102. ralphm has left

  103. ralphm has joined

  104. stpeter has joined

  105. stpeter has left

  106. stpeter has joined

  107. stpeter has left

  108. stpeter has joined

  109. pep.

    jonas’, https://github.com/xsf/xeps/pull/810 that's ready to merge right?

  110. pep.

    I can take it as well if you want

  111. jonas’

    doesn’t kev have to ack that first?

  112. pep.

    Ah indeed

  113. pep.

    https://github.com/xsf/xeps/pull/819 this is editor land though right?

  114. pep.

    Ah I tagged it already

  115. pep.

    "update copyright year"

  116. flow

    pep., btw https://github.com/xsf/xeps/pull/825 is "ready to merge", I didn't put up the label because I'm OP

  117. stpeter has left

  118. flow

    in case you want to have something to test your workflow with

  119. pep.

    I was looking for an ACK from Lance but yeah you have that in there

  120. pep.

    jonas’, I'm wondering if we shouldn't stop assigning editors(?) I personally don't mind if you merge stuff I have reviewed, and I guess in time it should be reciprocal :p

  121. pep.

    Maybe we want a "please don't touch this, I'll do it" signal

  122. pep.

    And that should be assigning?

  123. pep.

    And that could be assigning?

  124. Tobi has left

  125. lnj has joined

  126. lnj has left

  127. stpeter has joined

  128. stpeter has left

  129. debacle has left