ESC meeting minutes: 2024-05-02

  • Present:

    • Cloph, Hossein, Olivier, Caolan, Heiko, Ilmari, Jonathan, Michael W, Eike, Regina, Stephan, Xisco, Thorsten, Stephane, Michael S, Miklos
  • Completed Action Items:

  • Pending Action Items:

  • Release Engineering update (Cloph)

    • 7.6: 7.6.7 RC2 this week
      • last release in this line? (Xisco)
      • last planned, yes (Cloph)
    • 24.2: 24.2.4 RC1 in 2 weeks
    • 24.8: first alpha in 1 week
  • Documentation (Olivier)

    • Sometimes missing features to be documented
      • good if Olivier is aware
    • Highlight of the cell background during editing e.g.
      • so translators would discover these features as well.
    • Helpcontent2
      • Updates and fixes (ohallot, stragu)
      • Page for SEQUENCE Calc function (ohallot)
    • Guides
      • Stalled.
    • Bugzilla Documentation statistics
      264(264) bugs open
    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      created 10(-9) 38(2) 101(3) 223(1)
      commented 18(0) 63(3) 251(13) 850(-2)
      resolved 6(-3) 19(3) 49(3) 119(2)
    • top 10 contributors:
      Olivier Hallot made 33 changes in 1 month, and 256 changes in 1 year
      Stéphane Guillou made 29 changes in 1 month, and 255 changes in 1 year
      Vernon, Stuart Foote made 22 changes in 1 month, and 105 changes in 1 year
      Dione Maddern made 10 changes in 1 month, and 31 changes in 1 year
      MeBugzilla made 7 changes in 1 month, and 7 changes in 1 year
      Nabet, Julien made 7 changes in 1 month, and 53 changes in 1 year
      nobu made 7 changes in 1 month, and 16 changes in 1 year
      Ilmari Lauhakangas made 6 changes in 1 month, and 71 changes in 1 year
      Cor Nouws made 5 changes in 1 month, and 8 changes in 1 year
      Kaganski, Mike made 4 changes in 1 month, and 68 changes in 1 year
  • UX Update (Heiko)

    • Bugzilla (topicUI) statistics
      242(242) (topicUI) bugs open, 35(35) (needsUXEval) needs to be evaluated by the UXteam
    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      added 5(2) 8(-3) 15(-3) 24(-2)
      commented 38(-11) 173(-18) 559(6) 1842(-60)
      removed 3(3) 5(3) 5(3) 14(3)
      resolved 7(-2) 30(-1) 87(-1) 315(0)
    • top 10 contributors:
      Heiko Tietze made 107 changes in 1 month, and 1078 changes in 1 year
      Vernon, Stuart Foote made 105 changes in 1 month, and 306 changes in 1 year
      Stéphane Guillou made 57 changes in 1 month, and 643 changes in 1 year
      Ilmari Lauhakangas made 19 changes in 1 month, and 142 changes in 1 year
      Dieter made 15 changes in 1 month, and 136 changes in 1 year
      Eyal Rozenberg made 9 changes in 1 month, and 133 changes in 1 year
      Kaganski, Mike made 7 changes in 1 month, and 96 changes in 1 year
      Petros Christopoulos made 7 changes in 1 month, and 7 changes in 1 year
      m.a.riosv made 6 changes in 1 month, and 49 changes in 1 year
      UNKNOWN made 5 changes in 1 month, and 5 changes in 1 year
    • [Bug 160657] Improve how Writer Navigator Headings are displayed when
      alphabetically sorted
    • [Bug 160694] Add .uno:VRuler button to NB Tabbed/Tabbed Compact cutomize
      View list
    • [Bug 160873] Usability: “OLE Object” probably not familiar language
      (Menu: Insert/OLE Object)
    • [Bug 160219] Add confirmation dialog to “Delete all comments”
      (.uno:DeleteAllNotes) in Calc
      → + [Bug 160465] Ctrl+Shift+F6 is marked reserved, but doesn’t do anything
      • ReservedKeys in svapp.cxx not working for triple-state
        → + [Bug 95274] Wrong editing languages offered
      • gaelic or welsh not correctly reported per libexttextcat, unclear
        and input is highly welcome
  • Crash Testing (Caolan)

    • 63(+39) import failure, 59(+40) export failures
      • down from a higher, unreported numbers
      • it’s an improvement, in fact.
    • 21 coverity issues
    • 5 ossfuzz issues
  • Crash Reporting (Xisco)

    • 7.6.5.2 9807(+812)
    • 7.6.6.3 2944(+1016)
    • 24.2.1.2 12822(+961)
    • 24.2.2.2 6745(+2138)
    • let’s wait until next week to see the results from the new release
  • Mentoring (Hossein)

    • Reviewd submissions from newcomers
    • Added a new draft blog post for fixing crashes:
      Crash fixes, part 4: Assertion failure
      (Previous parts were around segfaults, aborts, and how to fix and test them)
      crash fixes, part 4: Assertion failure - LibreOffice Development Blog
      committer… 1 week 1 month 3 months 12 months
      open 104(22) 169(38) 201(42) 201(42)
      reviews 356(26) 1152(-40) 3648(-116) 12418(-90)
      merged 288(75) 981(47) 3241(-10) 12462(-29)
      abandoned 10(-12) 57(3) 174(1) 657(-9)
      own commits 194(22) 694(22) 2169(2) 9190(-9)
      review commits 47(-16) 224(-35) 856(-15) 3056(-11)
      contributor… 1 week 1 month 3 months 12 months
      open 35(-6) 71(-19) 95(-5) 95(-5)
      reviews 792(86) 2546(0) 7698(32) 29426(-22)
      merged 17(-10) 114(-27) 378(24) 1693(17)
      abandoned 13(2) 54(3) 298(1) 707(15)
      own commits 22(-8) 106(-43) 436(-3) 1127(6)
      review commits 0(0) 0(0) 0(0) 0(0)
      • easyHack statistics:
        needsDevEval 8(8) needsUXEval 1(1) cleanup_comments 331(331)
        total 411(411) assigned 28(28) open 349(349)
      • top 10 contributors:
        Armin Le Grand (allotropia) made 8 patches in 1 month, and 89 patches in 1 year
        Ritobroto Mukherjee made 7 patches in 1 month, and 14 patches in 1 year
        Rafał Dobrakowski made 5 patches in 1 month, and 16 patches in 1 year
        made 5 patches in 1 month, and 5 patches in 1 year
        Juan José González made 4 patches in 1 month, and 11 patches in 1 year
        Kira Tubo made 4 patches in 1 month, and 14 patches in 1 year
        Dione Maddern made 4 patches in 1 month, and 34 patches in 1 year
        colton Garrett made 4 patches in 1 month, and 4 patches in 1 year
        Stéphane Guillou made 3 patches in 1 month, and 7 patches in 1 year
        Stéphane Guillou made 3 patches in 1 month, and 60 patches in 1 year
      • top 10 reviewers:
        Adolfo Jayme Barrientos made 108 review comments in 1 month, and 496 in 1 year
        Caolán McNamara made 94 review comments in 1 month, and 1418 in 1 year
        Ilmari Lauhakangas made 92 review comments in 1 month, and 526 in 1 year
        Hossein made 76 review comments in 1 month, and 326 in 1 year
        Thorsten Behrens made 74 review comments in 1 month, and 640 in 1 year
        Vajna, Miklos made 60 review comments in 1 month, and 1086 in 1 year
        Michael Stahl made 60 review comments in 1 month, and 918 in 1 year
        Olivier Hallot made 60 review comments in 1 month, and 352 in 1 year
        Xisco Fauli made 56 review comments in 1 month, and 712 in 1 year
        Grandin, Noel made 52 review comments in 1 month, and 688 in 1 year
      • Patches automatically abandoned:
        tdf#157029 changed Korean Font priority for LibreOffice UI ( DaeHyun Sung )
        - https://gerrit.libreoffice.org/#/c/157494
        tdf#42982: Logs for RuntimeErrors ( Achintya Sharma )
        - https://gerrit.libreoffice.org/#/c/158688
        tdf#42982: improve UNO API error reporting ( Achintya Sharma )
        - https://gerrit.libreoffice.org/#/c/158689
        tdf#126573 Add SORT function to Libreoffice Calc ( Sahil Gautam )
        - https://gerrit.libreoffice.org/#/c/161613
        tdf#155856 Added cmake to complextoolbarcontrols ( Konstantin Limarev )
        - https://gerrit.libreoffice.org/#/c/162396
        tdf#155856 Added cmake to Draw example ( Konstantin Limarev )
        - https://gerrit.libreoffice.org/#/c/162670
        tdf#130857 refactor vclbuilder ( Omkar Acharekar )
        - https://gerrit.libreoffice.org/#/c/163103
      • big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
        Jakub Kościelak
  • GSoC (Ilmari)

    • 8 projects accepted: Google Summer of Code
    • bonus project from Linux Foundation: Google Summer of Code
      • good if UX keeps an eye on this (Ilmari)
      • the idea is that the UI doesn’t change (Michael W)
    • next deadline: end of bonding period: 27th of May
  • Commit Access

  • LibreOffice Technology hackfest at Budapest (Miklos)

    • 4th & 5th June (Tue-Wed)
    • more details soon
  • Developer Certification (Stephan/Miklos/László/Gabriel)

    • resting since: 3 weeks (limit: 20 weeks)
  • Jenkins / CI update (Cloph)

    • stats are missing, problem with Jenkins

    • apart from that, (relatively) new problem with the macOS builders

      • some older builders start to segfault on the code (compiler crashes)
    • Noel had a stab at fixing those clang crashes, is this still outstanding? (Caolan)

      • not sure (Cloph)
      • seems to be limited to the macOS 12.5 / xcode 14.2 (Cloph)
      • newer macs are delivered with new macOS / newer xcode
  • What’s cooking (Miklos)

    • A11y update (Michael W)
      • gtk4: improve a11y role mapping upstream and in LO
      • gtk3: working on issues with broken a11y hierarchy and wrong positions being
        reported (tdf#159369, tdf#160806, tdf#160837)
    • RTL/CTL/CJK update (Jonathan)
      • tdf#61444 Text layout broken across formatting changes
      • tdf#124116 Hebrew & Arabic punctuation shifted across formatting changes
        • In-progress
      • Do you need commit access or you expect reviews for your changes? (Hossein)
        • Wouldn’t mind a review, good to have a 2nd set of eyes
        • appreciate if others also take a look.
    • build with wsl-as-helper but from within git-bash (Cloph)
      • all changes are submitted to master
      • works with srcdir != builddir
      • not sure if to keep doxygen in wsl or not
        • initially was running out of memory with it, but have new ideas
      • considering it as finished for now
        • installer uses strawberry perl, not git-bash-perl for now.
      • thanks a lot, this is greatly appreciated, helped the Windows arm work (Thorsten)
        • seeing some hangs, though.
        • when building with help, saw those (Cloph)
        • saw it without help, but only on arm (Stephan)
        • would be interesting to see if the hanging process is also suspended (Cloph)
          • if so, that would be the one Cloph is also seeing with xsltproc/xargs
    • RFC: makefile cleanup/simplification (Cloph)
      • example: https://gerrit.libreoffice.org/c/core/+/167005
      • if no concerns, would continue with this, turning these calls into just variablse
      • idea: give it a week for people to look at it, and decide
      • anything that makes gbuild less indirect, love it (Thorsten, Stephan)
      • what is the counter-argument? (Caolan)
        • makes the history harder to read (Cloph)
        • suggest to not wait for a week (Caolan)
    • Progress on Dev Guide API doc links (Ilmari)
      • changing to the use of wiki templates, that point to the correct API doc pages
      • even did some mass-replacements
      • perhaps even script some of this
      • 1400 items left, but improving
  • QA update (Xisco)

    • UNCONFIRMED: 1171 (+1)

      • enhancements: 274 (-4)
      • needsUXEval: 5 (-7)
      • haveBackTrace: 10 (+0)
      • needsDevAdvice: 47 (+0)
      • documentation: 4 (+0)
      • android: 0 (+0)
      • iOS: 0 (+0)
      • Online: 9 (+0)
    • Most pressing bugs:
      New:

    None

  • QA stats (Stéphane)

    • this week’s focus: crashes

      • reviewing untouched reports, often able to duplicate or mark as fixed
    • Weekly Bug Summary
      +76 +25 (-84) overall)
      many thanks to the top bug squashers:
      QA Administrators 14
      Stéphane Guillou (stragu) 10
      Heiko Tietze 9
      Buovjaga 4
      V Stuart Foote 4
      László Németh 3
      Michael Stahl (allotropia) 3
      Noel Grandin 3
      Patrick Luby (volunteer) 3
      m_a_riosv 3

    • top 10 bugs reporters:
      Cor Nouws 3
      Stéphane Guillou (stragu) 3
      peter josvai 3
      Eyal Rozenberg 2
      Heiko Tietze 2
      László Németh 2
      Roland Baudin 2
      Tuomas Hietala 2
      Xisco Faulí 2
      nobu 2

    • top 10 bugs fixers:
      Heiko Tietze 4
      Stéphane Guillou 4
      Grandin, Noel 3
      Michael Stahl 3
      Németh, László 3
      Patrick Luby 3
      Kaganski, Mike 2
      Kelemen, Gabor 2
      Andreas Heinisch 1
      CP 1

    • top 10 bugs confirmers:
      Stéphane Guillou 16
      m.a.riosv 8
      Ilmari Lauhakangas 6
      Dieter 5
      Vernon, Stuart Foote 5
      Heiko Tietze 4
      Weghorn, Michael 4
      Ady 2
      Nabet, Julien 2
      Németh, László 2

  • Bisected bugs open: keyword ‘bisected’

    done by:
       Raal                   3
       Stéphane Guillou       1
       Xisco Fauli            1
    
  • Bibisected bugs open: keyword ‘bibisected’

    done by:
       Raal                   3
       Stéphane Guillou       1
       Xisco Fauli            1
    
  • all bugs tagged with ‘regression’

    • 1234(-5) bugs open of 13325(+9) total 22(-3) high prio.

      done by:
      m.a.riosv 2
      Stéphane Guillou 1
      Vernon, Stuart Foote 1
      Xisco Fauli 1

  • ~Component count net * high severity regressions

    • Bug List
      Writer - 15(-1)
      Calc - 2(-2)
      Impress - 2(+0)
      LibreOffice - 1(+0)
      UI - 1(+0)

    by OS:
    All - 17(-3)
    Linux - 3(+0)
    Mac OS X - 0(+0)
    Windows - 2(+0)

  • ~Component count net * all regressions

    • Bug List
      Writer: other - 434(-3)
      Calc - 225(-2)
      Impress - 127(-1)
      Draw - 53(+0)
      LibreOffice - 50(-2)
      Writer: docx filter - 45(+0)
      Base - 40(+0)
      Crashes - 40(-3)
      UI - 37(+0)
      Borders - 35(+0)
      Writer: perf - 35(+0)
      Writer: other filter - 30(+0)
      Chart - 24(+0)
      Printing and PDF export - 24(+0)
      RTL - 23(+0)
      filters and storage - 22(+1)
      BASIC - 20(+0)
      Writer: doc filter - 16(+0)
      graphics stack - 12(+0)
      Formula Editor - 7(+0)
      framework - 3(+0)
      Extensions - 2(+0)
      sdk - 2(+0)
      Linguistic - 1(+0)