ESC meeting minutes: 2024-10-31

  • Present:

    • Thorsten, Jonathan, Regina, Cloph, Heiko, Michael W., Olivier,
      Hossein, Ilmari, Caolan, Samuel A, Xisco
  • Completed Action Items:

  • Pending Action Items:

    • None
  • Release Engineering update (Cloph)

  • Documentation (Olivier)

    • Helpcontents2

      • Updates and fixes (ohallot)
      • GSOC mentors / @all: Please work on the grammar and clarity
        + affects translation
        + “impersonate” end-users / put yourself into user’s shoes
      • refactor for maintenance (Pierre F)
    • Guides

      • Writer Guide 24.8 released (Jean Weber)
      • Work in progress for Calc Guide 24.8
    • Bugzilla Documentation statistics
      229(229) bugs open

    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      created 1(1) 12(12) 112(112) 257(257)
      commented 3(3) 33(33) 341(341) 993(993)
      resolved 0(0) 5(5) 67(67) 170(170)

    • top 10 contributors:
      Bogdan B made 10 changes in 1 month, and 22 changes in 1 year
      Nabet, Julien made 10 changes in 1 month, and 56 changes in 1 year
      Pierre F made 5 changes in 1 month, and 223 changes in 1 year
      Ilmari Lauhakangas made 4 changes in 1 month, and 120 changes in 1 year
      m.a.riosv made 4 changes in 1 month, and 35 changes in 1 year
      Kaganski, Mike made 3 changes in 1 month, and 75 changes in 1 year
      Olivier Hallot made 3 changes in 1 month, and 336 changes in 1 year
      Dione Maddern made 2 changes in 1 month, and 106 changes in 1 year
      Ticrob made 2 changes in 1 month, and 2 changes in 1 year
      Xisco Fauli made 2 changes in 1 month, and 5 changes in 1 year

  • UX Update (Heiko)

    • Bugzilla (topicUI) statistics
      233(233) (topicUI) bugs open, 40(40) (needsUXEval) needs to be evaluated by the UXteam
    • Updates:
      BZ changes 1 week 1 month 3 months 12 months
      added 12(12) 23(23) 28(28) 35(35)
      commented 108(108) 176(176) 369(369) 1712(1712)
      removed 1(1) 2(2) 3(3) 12(12)
      resolved 9(9) 13(13) 58(58) 258(258)
    • top 10 contributors:
      Heiko Tietze made 85 changes in 1 month, and 916 changes in 1 year
      Vernon, Stuart Foote made 74 changes in 1 month, and 418 changes in 1 year
      Eyal Rozenberg made 73 changes in 1 month, and 206 changes in 1 year
      Ilmari Lauhakangas made 22 changes in 1 month, and 176 changes in 1 year
      Kelemen, Gabor made 20 changes in 1 month, and 42 changes in 1 year
      Dieter made 16 changes in 1 month, and 131 changes in 1 year
      Sarper Akdemir made 12 changes in 1 month, and 21 changes in 1 year
      Henschel, Regina made 8 changes in 1 month, and 54 changes in 1 year
      Seth Chaiklin made 7 changes in 1 month, and 27 changes in 1 year
      Dennis Roczek made 6 changes in 1 month, and 6 changes in 1 year
    • [Bug 163685] Provide ability for user to add - remove formula examples to the
      self-made Elements → Examples panel
    • [Bug 163646] Tooltip for Headings in Navigator should show a correct
      word/characters amount if there is nested outline
    • [Bug 162858] Allow the user to delete his own bibliography references
    • [Bug 163616] Match diacritics
  • Crash Reporting (Caolan)

  • Crash Reporting (from crashreport.libreoffice.org)

    • 24.2.6.2 2952(+0)
    • 24.8.0.3 21059(+0)
    • 24.8.1.2 17394(+0)
    • 24.8.2.1 15065(+0)
    • seems script is off? AI xisco to prod the stats script
  • Mentoring (Hossein)

    • Reviewed submissions from newcomers
    • Added new EasyHack
      tdf#163691 - Use std::copy() instead of memcpy()
      163691 – Use std::copy() instead of memcpy()
    • Drafted a blog post on dev blog
      Notebookbar part 1: custom widgets for the tabbed interface
      Notebookbar part 1: custom widgets for the tabbed interface - LibreOffice Development Blog
    • Takashi kindly provided a fix for Cygwin hang problem on master
      [PATCH] Cygwin: console: Inherit pcon hand over from parent pty
      [PATCH] Cygwin: console: Inherit pcon hand over from parent pty
      newlib-cygwin - newlib and cygwin
      Description of the Cygwin hang problem:
      Cause of Cygwin terminal gets stuck during the build
      • AI: hossein try latest nightly / snapshot with the fix
        • Issue was not reproducible with the latest 3.6 snapshot (Hossein)
        • Probably need to cherry pick the fix to 3.5 (Hossein)
      • 3.6.0 snapshots: Index of /pub/cygwin/x86_64/release/cygwin
        committer… 1 week 1 month 3 months 12 months
        open 76(76) 108(108) 158(158) 162(162)
        reviews 530(530) 1244(1244) 3434(3434) 13388(13388)
        merged 322(322) 1207(1207) 3599(3599) 13890(13890)
        abandoned 10(10) 37(37) 177(177) 765(765)
        own commits 206(206) 826(826) 2592(2592) 9934(9934)
        review commits 104(104) 260(260) 733(733) 3187(3187)
        contributor… 1 week 1 month 3 months 12 months
        open 119(119) 359(359) 399(399) 401(401)
        reviews 1118(1118) 2894(2894) 8164(8164) 30844(30844)
        merged 55(55) 106(106) 321(321) 1331(1331)
        abandoned 3(3) 11(11) 60(60) 554(554)
        own commits 65(65) 124(124) 316(316) 1331(1331)
        review commits 0(0) 0(0) 0(0) 0(0)
      • easyHack statistics:
        needsDevEval 8(8) needsUXEval 1(1) cleanup_comments 332(332)
        total 419(419) assigned 27(27) open 355(355)
      • top 10 contributors:
        Bogdan B made 36 patches in 1 month, and 123 patches in 1 year
        Armin Le Grand (allotropia) made 6 patches in 1 month, and 83 patches in 1 year
        Dr. David Alan Gilbert made 6 patches in 1 month, and 31 patches in 1 year
        Mohit Marathe made 5 patches in 1 month, and 16 patches in 1 year
        Pierre F made 4 patches in 1 month, and 88 patches in 1 year
        Sahil Gautam made 3 patches in 1 month, and 10 patches in 1 year
        Kira Tubo made 3 patches in 1 month, and 29 patches in 1 year
        Ashok made 3 patches in 1 month, and 6 patches in 1 year
        Aditya (Addy) Sahu made 3 patches in 1 month, and 3 patches in 1 year
        Xuan Chen made 2 patches in 1 month, and 6 patches in 1 year
      • top 10 reviewers:
        Xisco Fauli made 130 review comments in 1 month, and 820 in 1 year
        Caolán McNamara made 96 review comments in 1 month, and 1380 in 1 year
        Vajna, Miklos made 92 review comments in 1 month, and 1154 in 1 year
        Thorsten Behrens made 88 review comments in 1 month, and 696 in 1 year
        Nabet, Julien made 84 review comments in 1 month, and 894 in 1 year
        Michael Stahl made 70 review comments in 1 month, and 860 in 1 year
        Adolfo Jayme Barrientos made 58 review comments in 1 month, and 854 in 1 year
        Ilmari Lauhakangas made 58 review comments in 1 month, and 644 in 1 year
        Gabor Kelemen made 56 review comments in 1 month, and 204 in 1 year
        Grandin, Noel made 54 review comments in 1 month, and 684 in 1 year
      • Patches automatically abandoned:
        sw: disable header if empty ( Tomaž Vajngerl )
        - https://gerrit.libreoffice.org/#/c/163884
        tdf#132007 improve password message locked document ( Devansh Varshney )
        - https://gerrit.libreoffice.org/#/c/163924
      • big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
        Samuel Abraham abrahamadekunle50@gmail.com
        Samuel Adesola adesolasamuel2022@outlook.com
        Chizoba ODINAKA chizobajames21@gmail.com
        Abrar Emad eng.abraremad.ae@gmail.com
        muluh godsonmuluh@gmail.com
        Melvin George melvingeorge204@gmail.com
        Ndidi Ogboi ndidiamakaogboi@gmail.com
  • Commit Access

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

  • Jenkins / CI update (Cloph)
    gerrit_android_aarch64 jobs: 156 ok: 149 ko: 5 fail ratio: 3.21% mean_ok: 10 ( 23) median_ok: 6 ( 18)
    gerrit_android_arm jobs: 155 ok: 147 ko: 4 fail ratio: 2.58% mean_ok: 9 ( 23) median_ok: 7 ( 15)
    gerrit_android_x86 jobs: 156 ok: 149 ko: 5 fail ratio: 3.21% mean_ok: 9 ( 22) median_ok: 7 ( 17)
    gerrit_android_x86_64 jobs: 155 ok: 146 ko: 4 fail ratio: 2.58% mean_ok: 9 ( 24) median_ok: 7 ( 18)
    gerrit_linux_clang_dbgutil jobs: 317 ok: 254 ko: 50 fail ratio: 15.77% mean_ok: 45 ( 55) median_ok: 45 ( 51)
    gerrit_linux_gcc_release jobs: 399 ok: 294 ko: 60 fail ratio: 15.04% mean_ok: 24 ( 37) median_ok: 19 ( 30)
    gerrit_mac jobs: 318 ok: 282 ko: 27 fail ratio: 8.49% mean_ok: 54 ( 54) median_ok: 51 ( 51)
    gerrit_windows jobs: 330 ok: 267 ko: 56 fail ratio: 16.97% mean_ok: 38 ( 44) median_ok: 36 ( 37)
    gerrit_master_ml jobs: 428 ok: 226 ko: 147 fail ratio: 34.35% mean_ok: 98 ( 98) median_ok: 84 ( 85)
    gerrit_master jobs: 58 ok: 33 ko: 20 fail ratio: 34.48% mean_ok: 77 ( 77) median_ok: 68 ( 68)
    gerrit_master_seq jobs: 362 ok: 193 ko: 120 fail ratio: 33.15% mean_ok: 101 (101) median_ok: 89 ( 89)

    • tests that failed more than twice in last seven days
      3 aborted by jenkins job timeout gerrit_windows
      3 UITest_sw_ui_frmdlg gerrit_linux_clang_dbgutil
      11 CppunitTest_sw_tiledrendering2 gerrit_linux_gcc_release
    • one mac bot has frequent failure copying jar / triggers permission issue
      java.nio.file.FileSystemException: /Volumes/lode/lode/jenkins/workspace/lo_gerrit/Config/macosx_clang_dbgutil/instdir/LibreOfficeDev.app/Contents/Resources/java/sdbc_hsqldb.jar: Operation not permitted
      → rule to auto-retry
    • CppunitTest_xmlsecurity_signing break on current baseline
      • mismatch between nss 3.101 and crypto-policies package
        → workaround is to set NSS_IGNORE_SYSTEM_POLICY=1
      • perhaps at some stage needs updating the test signatures, to use newer algos?
  • Feature locking (Andreas)

    • https://gerrit.libreoffice.org/c/core/+/174865 “Remove blocking functions feature from core”
    • re-visit from last week - any new views?
    • Heiko: having a ‘freemium’ label in the code - is not ideal. but more a question
      of labelling stuff, rather than technically wrong. And not affecting the desktop
      version anyway
    • Ilmari: but Andreas is not complaining about the label, but about the feature
    • Hossein: perhaps this can be a compile-time option instead?
    • Michael W.: unclear if that resolves it for Andreas
    • Ilmari: does not see the point either
    • Cloph: sees no prob here, either - several other places with ‘locks’ already in
    • AI thorsten conclude this from the side of the ESC, leave comment in the patch
  • What’s cooking (Miklos)

  • QA update (Xisco)

    • UNCONFIRMED: 1375 (+1375)

      • enhancements: 340 (+340)
      • needsUXEval: 20 (+20)
      • haveBackTrace: 11 (+11)
      • needsDevAdvice: 48 (+48)
      • documentation: 3 (+3)
      • android: 0 (+0)
      • iOS: 0 (+0)
      • Online: 9 (+9)
    • Most pressing bugs:
      New:

      • None
        Old:
    • New high severity bugs of the week:

      • None
  • QA stats

    • Weekly Bug Summary
      +133 +0 (-93) overall)
      many thanks to the top bug squashers:
      QA Administrators 10
      Justin L 7
      Buovjaga 6
      Heiko Tietze 6
      m_a_riosv 6
      Michael Weghorn 5
      Noel Grandin 5
      Julien Nabet 4
      László Németh 3
      V Stuart Foote 3

    • top 10 bugs reporters:
      Eyal Rozenberg 27
      peter josvai 3
      Elmar 2
      FredrikL 2
      Landon 2
      Manu 2
      Vaibhavi 2
      Xisco Faulí 2
      neswal.peter 2
      Cor Nouws 1

    • top 10 bugs fixers:
      Grandin, Noel 5
      Jim Raykowski 2
      Jonathan Clark 2
      Justin Luth 2
      Németh, László 2
      Weghorn, Michael 2
      Balazs Varga 1
      Caolán McNamara 1
      Heiko Tietze 1
      Ilmari Lauhakangas 1

    • top 10 bugs confirmers:
      Heiko Tietze 15
      m.a.riosv 10
      Ilmari Lauhakangas 8
      Xisco Fauli 4
      Dieter 3
      Raal 3
      Weghorn, Michael 3
      UNKNOWN 2
      Ady 2
      Nabet, Julien 2

  • Bisected bugs open: keyword ‘bisected’

    done by:
    Raal 3
    Justin Luth 1
    Kelemen, Gabor 1
    Stéphane Guillou 1
    Xisco Fauli 1

  • Bibisected bugs open: keyword ‘bibisected’

    done by:
    Raal 3
    Dennis Roczek 1
    Justin Luth 1
    Kelemen, Gabor 1
    Stéphane Guillou 1
    Xisco Fauli 1

  • all bugs tagged with ‘regression’

    • 1211(+1211) bugs open of 13692(+13692) total 19(+0) high prio.

      done by:
      m.a.riosv 2
      Raal 2
      Kelemen, Gabor 1
      Kaganski, Mike 1
      Németh, László 1
      Xisco Fauli 1

  • ~Component count net * high severity regressions

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

    by OS:
    All - 12(+0)
    Linux - 5(+0)
    Mac OS X - 0(+0)
    Windows - 2(+0)

  • ~Component count net * all regressions

    • Bug List
      Writer: other - 396(+0)
      Calc - 244(+0)
      Impress - 125(+0)
      LibreOffice - 52(+0)
      Draw - 48(+0)
      Writer: docx filter - 43(+0)
      Base - 38(+0)
      Crashes - 38(+0)
      UI - 33(+0)
      Writer: perf - 33(+0)
      Borders - 32(+0)
      Writer: other filter - 31(+0)
      Printing and PDF export - 26(+0)
      Chart - 24(+0)
      RTL - 24(+0)
      BASIC - 22(+0)
      filters and storage - 20(+0)
      Writer: doc filter - 17(+0)
      Formula Editor - 12(+0)
      graphics stack - 12(+0)
      framework - 3(+0)
      sdk - 3(+0)
      Extensions - 2(+0)
      Linguistic - 2(+0)