ESC meeting minutes: 2024-06-20

  • Present:

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

  • Pending Action Items:

  • Release Engineering update (Cloph)

    • 24.2: 24.2.5 RC1 in 1 week
    • 24.8:
      • beta2 this week, if needed → so far no requests, so will be skipped
      • string / UI freeze in 2 weeks
      • late features?
        • late addition to the notes page in Impress in the works (Thorsten)
    • feedback on Windows 7 vs Python 3.newer (Cloph)
      • rest of the office works nicely, except actual Python functionality
      • fails gracefull, so no need to take any action at this moment
  • Documentation (Olivier)

    • New Help
      • Links to documentation in Help pages (ohallot)
    • Helpcontents2
      • Updates and fixes (ohallot, D. Maddern)
    • Guides
      • Published Impress guide + post
    • Bugzilla Documentation statistics
      252(252) bugs open
    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      created 10(3) 43(-5) 101(-2) 234(1)
      commented 22(10) 91(-9) 237(4) 896(-2)
      resolved 3(1) 23(-3) 63(1) 134(0)
    • top 10 contributors:
      Dione Maddern made 56 changes in 1 month, and 100 changes in 1 year
      Olivier Hallot made 44 changes in 1 month, and 269 changes in 1 year
      Vernon, Stuart Foote made 15 changes in 1 month, and 121 changes in 1 year
      Ilmari Lauhakangas made 12 changes in 1 month, and 81 changes in 1 year
      Heiko Tietze made 11 changes in 1 month, and 83 changes in 1 year
      Stéphane Guillou made 9 changes in 1 month, and 223 changes in 1 year
      Kaganski, Mike made 8 changes in 1 month, and 79 changes in 1 year
      Justin Luth made 6 changes in 1 month, and 8 changes in 1 year
      Timur made 6 changes in 1 month, and 8 changes in 1 year
      Bogdan B made 4 changes in 1 month, and 13 changes in 1 year
  • UX Update (Heiko)

    • Bugzilla (topicUI) statistics
      239(239) (topicUI) bugs open, 38(38) (needsUXEval) needs to be evaluated by the UXteam
    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      added 5(2) 12(1) 16(0) 24(1)
      commented 37(-4) 189(-5) 504(-10) 1756(25)
      removed 2(2) 2(2) 7(2) 12(2)
      resolved 6(3) 28(0) 80(1) 285(-4)
    • top 10 contributors:
      Heiko Tietze made 128 changes in 1 month, and 964 changes in 1 year
      Stéphane Guillou made 104 changes in 1 month, and 659 changes in 1 year
      Eyal Rozenberg made 25 changes in 1 month, and 156 changes in 1 year
      Dieter made 23 changes in 1 month, and 149 changes in 1 year
      Vernon, Stuart Foote made 22 changes in 1 month, and 310 changes in 1 year
      Telesto made 12 changes in 1 month, and 35 changes in 1 year
      Ilmari Lauhakangas made 10 changes in 1 month, and 131 changes in 1 year
      UNKNOWN made 9 changes in 1 month, and 12 changes in 1 year
      Henschel, Regina made 9 changes in 1 month, and 41 changes in 1 year
      Kaganski, Mike made 6 changes in 1 month, and 80 changes in 1 year
      → + [Bug 161641] CTRL+A should select only the closest range of cells filled
      with data
      + https://gerrit.libreoffice.org/c/core/+/169276
      + if it’s too controversial, may need to add a setting to keep the unchanged behavior as well
    • [Bug 157684] Add option to set default Wrap for inserted pictures
      → + [Bug 158520] Writer: Index-tagged headings not considered headings
      cross-reference-wise
    • [Bug 161344] Changing the language for Selected text clutters up language lists
    • [Bug 161574] Name index-related context menu items based on the type of index
      that was right-clicked
      → + [Bug 161555] Missing shape algebra operation: Fragment
    • [Bug 161061] [UI] improve Calc grid lines options wording (was: “Grid lines are
      visible on all cells when option ‘Show on colored cells’ is chosen”)
  • Vertical tabpages state (Samuel/Thorsten)

    • maybe a safe move would be to revert on 24-8 and keep working on master (Xisco)
      • perhaps prepare the revert and discuss on gerrit

=> CC Samuel/Thorsten on the gerrit reverts, push on the branch after a timeout (all)

  • Crash Testing (Caolan)

  • Crash Reporting (Xisco)

    • 24.2.1.2 17451(+523)
    • 24.2.2.2 15373(+693)
    • 24.2.3.2 11334(+1533)
    • 24.2.4.2 1642(+1141)
  • Mentoring (Hossein)

    • Fixed a previous unwanted code behaviour change from mentee’s refactoring
    • .NET progress (thanks to Ritobroto)
      ‘netmaker’, which generates C# stubs from IDL specs
    • Will be merged soon:
      .NET Bindings: disable switch for old windows CLI bindings
      https://gerrit.libreoffice.org/c/core/+/168751
    • CI .NET installation?
      • would prefer first just the native platform, Windows 64bit (Cloph)

      • for macOS, a single Intel bot is left

      • for Windows arm64, we always cross-build (Cloph)

        committer… 1 week 1 month 3 months 12 months
        open 135(36) 249(60) 250(59) 250(59)
        reviews 464(68) 1296(74) 3216(-46) 12594(56)
        merged 363(151) 1075(78) 3279(145) 12698(41)
        abandoned 17(-18) 74(4) 179(1) 691(3)
        own commits 204(56) 722(-10) 2339(65) 9286(8)
        review commits 61(21) 234(9) 716(-5) 3005(10)
        contributor… 1 week 1 month 3 months 12 months
        open 31(-6) 85(-2) 87(0) 87(0)
        reviews 964(220) 2810(160) 7560(292) 29506(56)
        merged 16(7) 63(-3) 290(-5) 1533(-7)
        abandoned 8(-7) 36(3) 108(0) 719(2)
        own commits 24(11) 95(5) 344(2) 1144(10)
        review commits 0(0) 0(0) 0(0) 0(0)

    • easyHack statistics:
      needsDevEval 9(9) needsUXEval 1(1) cleanup_comments 327(327)
      total 412(412) assigned 26(26) open 351(351)
    • top 10 contributors:
      Dione Maddern made 25 patches in 1 month, and 65 patches in 1 year
      Bogdan B made 8 patches in 1 month, and 25 patches in 1 year
      Weblate made 7 patches in 1 month, and 32 patches in 1 year
      Stéphane Guillou made 4 patches in 1 month, and 62 patches in 1 year
      LeSasse made 4 patches in 1 month, and 23 patches in 1 year
      Ritobroto Mukherjee made 3 patches in 1 month, and 18 patches in 1 year
      Hakim Ottey made 3 patches in 1 month, and 4 patches in 1 year
      Srebotnjak, Martin made 2 patches in 1 month, and 17 patches in 1 year
      Adam Seskunas made 2 patches in 1 month, and 14 patches in 1 year
      Darshan Upadhyay made 2 patches in 1 month, and 11 patches in 1 year
    • top 10 reviewers:
      Vajna, Miklos made 140 review comments in 1 month, and 1126 in 1 year
      Caolán McNamara made 128 review comments in 1 month, and 1542 in 1 year
      Nabet, Julien made 82 review comments in 1 month, and 850 in 1 year
      Thorsten Behrens made 78 review comments in 1 month, and 640 in 1 year
      Ilmari Lauhakangas made 76 review comments in 1 month, and 556 in 1 year
      Adolfo Jayme Barrientos made 70 review comments in 1 month, and 580 in 1 year
      Heiko Tietze made 62 review comments in 1 month, and 264 in 1 year
      Xisco Fauli made 62 review comments in 1 month, and 690 in 1 year
      Michael Stahl made 60 review comments in 1 month, and 892 in 1 year
      Hossein made 58 review comments in 1 month, and 310 in 1 year
    • Patches automatically abandoned:
      tdf#149256 Ship LibreOfficeKit headers in packages ( Hossein )
      → Restored
      - https://gerrit.libreoffice.org/#/c/134863
      [WIP] Create new data structure for Primitive2DContainer ( Noel Grandin )
      - https://gerrit.libreoffice.org/#/c/135319
      WIP enable supports32Bitmap everywhere ( Noel Grandin )
      - https://gerrit.libreoffice.org/#/c/155713
      WIP JsonWriterStruct ( Noel Grandin )
      - https://gerrit.libreoffice.org/#/c/164293
      tdf#158803 remove unused imports ( Leo Sa )
      → OK, changed solenv/gdb, which was not intended
      - https://gerrit.libreoffice.org/#/c/165387
      Add NaN payload check at startup ( Xuan Chen )
      → Restored
      - https://gerrit.libreoffice.org/#/c/165391
      tdf#114441 sal_uLong to better integer types ( Aaron Bourdeaux )
      → OK, no reply from the author
      - https://gerrit.libreoffice.org/#/c/165805
      WIP Simplify *SalBitmap::Create methods ( Noel Grandin )
      - https://gerrit.libreoffice.org/#/c/165923
      restore clear of cache on rw-access to ItemSet ( Caolán McNamara )
      - https://gerrit.libreoffice.org/#/c/166066
    • big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
      Richard Trimble
      Yuki Joou
  • GSoC (Ilmari)

    • Midterm evaluations deadline 12 July
  • Commit Access

  • Developer Certification (Stephan/Miklos/László/Gabriel)

    • resting since: 9 weeks (limit: 20 weeks)
  • Jenkins / CI update (Cloph)
    gerrit_android_aarch64 jobs: 188 ok: 182 ko: 4 fail ratio: 2.13% mean_ok: 11 ( 21) median_ok: 8 ( 12)
    gerrit_android_arm jobs: 187 ok: 181 ko: 4 fail ratio: 2.14% mean_ok: 9 ( 18) median_ok: 7 ( 11)
    gerrit_android_x86 jobs: 188 ok: 181 ko: 5 fail ratio: 2.66% mean_ok: 10 ( 20) median_ok: 8 ( 13)
    gerrit_android_x86_64 jobs: 188 ok: 184 ko: 3 fail ratio: 1.60% mean_ok: 10 ( 19) median_ok: 8 ( 12)
    gerrit_linux_clang_dbgutil jobs: 378 ok: 312 ko: 58 fail ratio: 15.34% mean_ok: 46 ( 57) median_ok: 45 ( 52)
    gerrit_linux_gcc_release jobs: 436 ok: 363 ko: 51 fail ratio: 11.70% mean_ok: 19 ( 27) median_ok: 15 ( 20)
    gerrit_mac jobs: 382 ok: 341 ko: 24 fail ratio: 6.28% mean_ok: 51 ( 60) median_ok: 48 ( 52)
    gerrit_windows jobs: 405 ok: 299 ko: 84 fail ratio: 20.74% mean_ok: 68 ( 89) median_ok: 64 ( 73)
    gerrit_master_ml jobs: 458 ok: 263 ko: 148 fail ratio: 32.31% mean_ok: 115 (115) median_ok: 99 ( 99)
    gerrit_master jobs: 48 ok: 25 ko: 19 fail ratio: 39.58% mean_ok: 79 ( 83) median_ok: 60 ( 63)
    gerrit_master_seq jobs: 415 ok: 243 ko: 129 fail ratio: 31.08% mean_ok: 116 (118) median_ok: 101 (102)

    • tests that failed more than twice in last seven days
      3 Killed by the kill-wrapper gerrit_linux_clang_dbgutil
      3 CppunitTest_chart2_xshape gerrit_linux_gcc_release
      3 CppunitTest_sc_subsequent_export_test3 gerrit_windows
      4 CppunitTest_chart2_export2 gerrit_linux_gcc_release
      5 CppunitTest_sw_layoutwriter2 gerrit_windows
      8 aborted by jenkins job timeout gerrit_windows
      15 CppunitTest_sw_a11y gerrit_windows
      • looks good wrt overall performance / failure rates (Cloph)
      • update Jenkins, java 17 is required by clients
        • affects lots/wollmux
        • related: no lode.git mirror on GitHub (Miklos)
          • just mail hostmaster@ if there is a need (Cloph)
  • What’s cooking (Miklos)

  • QA update (Xisco)

    • UNCONFIRMED: 1168 (+1)

      • enhancements: 268 (-4)
      • needsUXEval: 10 (+0)
      • haveBackTrace: 9 (-1)
      • needsDevAdvice: 47 (+1)
      • documentation: 6 (+1)
      • android: 0 (+0)
      • iOS: 0 (+0)
      • Online: 9 (+0)
    • Most pressing bugs:
      New:

    None

  • QA stats (Stéphane)

    • Weekly Bug Summary
      +122 -22 (-135) overall)
      many thanks to the top bug squashers:
      Stéphane Guillou (stragu) 22
      Mike Kaganski 14
      QA Administrators 14
      László Németh 9
      Buovjaga 8
      Jonathan Clark 7
      m_a_riosv 7
      Caolán McNamara 4
      Julien Nabet 4
      Michael Stahl (allotropia) 3

    • top 10 bugs reporters:
      László Németh 5
      Ahmed Hamed 4
      Gabor Kelemen (allotropia) 3
      Regina Henschel 3
      Xisco Faulí 3
      Albert Acer 2
      Ash 2
      Attila Szűcs 2
      Eyal Rozenberg 2
      Joanmarie Diggs 2

    • top 10 bugs fixers:
      Kaganski, Mike 7
      Németh, László 6
      Caolán McNamara 4
      Michael Stahl 4
      Patrick Luby 3
      Balland-Poirier, Laurent 2
      Heiko Tietze 2
      Ilmari Lauhakangas 2
      Rafael Lima 2
      Vajna, Miklos 2

    • top 10 bugs confirmers:
      Stéphane Guillou 15
      Nabet, Julien 8
      Xisco Fauli 5
      m.a.riosv 5
      Ahmed Hamed 4
      Dieter 4
      Heiko Tietze 4
      Ilmari Lauhakangas 4
      Kaganski, Mike 4
      Robert Großkopf 4

  • Bisected bugs open: keyword ‘bisected’

    done by:
       Ilmari Lauhakangas     4
       Stéphane Guillou       2
       Xisco Fauli            2
       Raal                   1
    
  • Bibisected bugs open: keyword ‘bibisected’

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

    • 1228(-6) bugs open of 13431(+13) total 25(+1) high prio.

      done by:
      Stéphane Guillou 4
      Ilmari Lauhakangas 3
      Robert Großkopf 2
      Telesto 2
      Xisco Fauli 2
      m.a.riosv 1

  • ~Component count net * high severity regressions

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

    by OS:
    All - 16(-1)
    Linux - 5(+0)
    Mac OS X - 0(+0)
    Windows - 3(+1)

  • ~Component count net * all regressions

    • Bug List
      Writer: other - 423(+1)
      Calc - 231(+0)
      Impress - 122(-3)
      LibreOffice - 51(-2)
      Draw - 48(-3)
      Writer: docx filter - 44(-1)
      Crashes - 43(+1)
      Base - 41(+2)
      UI - 39(+0)
      Borders - 34(+0)
      Writer: perf - 34(-1)
      Writer: other filter - 32(+0)
      Printing and PDF export - 27(+1)
      Chart - 24(+0)
      RTL - 23(+0)
      filters and storage - 21(+0)
      BASIC - 19(+0)
      Writer: doc filter - 16(+0)
      graphics stack - 13(+0)
      Formula Editor - 8(+1)
      framework - 3(+0)
      Extensions - 2(+0)
      sdk - 2(+0)
      Linguistic - 1(+0)