ESC meeting minutes: 2022-02-17

Miklos Vajna vmiklos at collabora.com
Thu Feb 17 15:37:48 UTC 2022


* Present:
     + Cloph, Heiko, Kendy, Olivier, Caolan, Hossein, Michael W, Michael S, Stephan, Miklos, Xisco, Michael M, Eike, Gabriel

* Completed Action Items:
     + try to revert the wasm-related change that results in .component loading build failures and see if it really helps (Cloph)
       + done, thanks Noel

* Pending Action Items:
     + None

* Release Engineering update (Cloph)
     + 7.3 status: 7.3.1 rc2 in 1 week
       + rc1 was waiting for the AVX usage revert
       + rc1 tag was force-pushed, as the version bump was missing
     + 7.2 status: 7.2.6 rc2 in 2 weeks
     + Remotes: Android, iOS
     + Android viewer

* Documentation (Olivier)
     + New Help:
         + fixes (buovjaga)
         + improvements to come (R. Lima)

     + Helpcontents2
         + Maintenance (Johnny_M, A. Gelmini)
         + Typos after translation round (S. Horacek)
         + Updates (S. Chaiklin, Olivier, A. romedenne, R. Lima, A Barrientos)

     + Guides
         + Update on Base Guide
             + opportunity to review Base (Vasudev, flywire)
             + Coordination by ohallot
             + time based release of the guide.
         + Getting started Guide
             + final revisions (K. Kriek, P. Schofield, J Weber)

      + Google Season of Doc
         + Open for ideas, please speak up !

     + Bugzilla Documentation statistics
         241(241) bugs open
     + Updates:
         BZ changes   1 week   1 month    3 months   12 months
            created     15(2)     51(-11)   125(10)     333(-1)
          commented     34(1)    156(-25)   454(-2)    1652(-17)
           resolved      4(1)     17(-5)     59(4)      201(0)
     + top 10 contributors:
           Olivier Hallot made 63 changes in 1 month, and 579 changes in 1 year
           Nabet, Julien made 23 changes in 1 month, and 62 changes in 1 year
           Heiko Tietze made 22 changes in 1 month, and 95 changes in 1 year
           Rafael Lima made 22 changes in 1 month, and 370 changes in 1 year
           Ilmari Lauhakangas made 19 changes in 1 month, and 120 changes in 1 year
           Kaganski, Mike made 16 changes in 1 month, and 99 changes in 1 year
           Michael Warner made 9 changes in 1 month, and 10 changes in 1 year
           Nucleos made 8 changes in 1 month, and 8 changes in 1 year
           Xisco Fauli made 8 changes in 1 month, and 88 changes in 1 year
           Alain Romedenne made 7 changes in 1 month, and 34 changes in 1 year

* UX Update (Heiko)
     + Bugzilla (topicUI) statistics
         259(259) (topicUI) bugs open, 105(105) (needsUXEval) needs to be evaluated by the UXteam
     + Updates:
         BZ changes   1 week   1 month   3 months   12 months
              added      2(-1)     8(-7)    13(-8)      61(-8)
          commented     55(4)    233(1)    649(-25)   2810(-6)
            removed      3(2)      7(2)     12(2)       35(-1)
           resolved     14(4)     48(4)    105(3)      403(0)
     + top 10 contributors:
           Heiko Tietze made 250 changes in 1 month, and 1938 changes in 1 year
           Seth Chaiklin made 35 changes in 1 month, and 136 changes in 1 year
           Foote, V Stuart made 22 changes in 1 month, and 487 changes in 1 year
           Telesto made 14 changes in 1 month, and 293 changes in 1 year
           Timur made 12 changes in 1 month, and 113 changes in 1 year
           Dieter made 11 changes in 1 month, and 247 changes in 1 year
           Ilmari Lauhakangas made 11 changes in 1 month, and 299 changes in 1 year
           Albrecht Müller made 10 changes in 1 month, and 10 changes in 1 year
           *UNKNOWN* made 10 changes in 1 month, and 10 changes in 1 year
           Kainz, Andreas made 10 changes in 1 month, and 32 changes in 1 year

     + 6 new tickets with needsUXEval Feb/11-17
       + [Bug 147473] Rearrange items the Numbering type list on Customize tab
              in B&N dialog
       + [Bug 147444] Calc - interaction between filters
       + [Bug 147286] ‘Default Paragraph Style’ should be renamed ‘Basic’
       + [Bug 147373] UI: Default file format on export mess (Draw:GIF;
              Calc: WEBP: Writer:EPUB; Impress:PDF)
       + [Bug 147346] simplify gallery usage
       + [Bug 147344] Integrate Auto Text to gallery sidebar

* Crash Testing (Caolan)
     + 13(+0) import failure, 6(+0) export failures
       - a new calc limit related issue, now fixed
     + 4 coverity issues
       - still using the old coverity, will bring to zero again and try new coverity when website returns
     + 9 ossfuzz issues

* Crash Reporting (Xisco)
    + https://crashreport.libreoffice.org/stats/version/7.2.5.2
          + (+2) 1778 1776 1707 1860 1608 1246 691 0
    + https://crashreport.libreoffice.org/stats/version/7.3.0.3
          + (+124) 2384 2260 347 0

    + https://crashreport.libreoffice.org/stats/signature/_GLOBAL__sub_I_protectiondlg.cxx.lto_priv.2643

        - Should be fixed in 7.3.1.1 - Thanks to Lubos


     + https://crashreport.libreoffice.org/stats/signature/rtl::str::assign%3C_rtl_uString%3E(_rtl_uString%20*%20*,_rtl_uString%20*)

     - Happening in LibreOffice 7.2.x only

     - not yet clear what commit fixed it


* Mentoring/EasyHack update (Hossein)
           committer...   1 week      1 month     3 months     12 months
                   open      47(-38)      94(-32)    150(-19)      204(-15)
                reviews     348(-100)   1396(-16)   3498(-10)    12100(228)
                 merged     271(-63)    1348(-3)    3948(-104)   14745(266)
              abandoned      16(2)        62(-3)     197(5)        688(16)
            own commits     203(-55)    1074(-6)    3065(-54)    12588(-78)
         review commits      75(-21)     345(19)     947(-39)     3951(-47)
         contributor...   1 week      1 month     3 months     12 months
                   open      16(-1)       28(-5)      75(-15)      181(-3)
                reviews     700(-250)   3126(-40)   8774(-170)   32692(544)
                 merged      21(-12)     100(19)     267(-22)     1600(20)
              abandoned       4(-11)      26(3)       57(-1)       435(4)
            own commits      31(-15)     130(30)     381(-44)     1745(-21)
         review commits       0(0)         0(0)        0(0)          0(0)
     + easyHack statistics:
        needsDevEval 8(8)   needsUXEval 1(1)   cleanup_comments 289(289)
        total 340(340)   assigned 27(27)   open 284(284)
     + top 10 contributors:
           Johnny_M made 76 patches in 1 month, and 904 patches in 1 year
           Attila Bakos made 5 patches in 1 month, and 30 patches in 1 year
           Po-Yen Huang made 5 patches in 1 month, and 12 patches in 1 year
           Kevin Suo made 4 patches in 1 month, and 35 patches in 1 year
           Vaibhav Malik made 4 patches in 1 month, and 9 patches in 1 year
           Dhiraj Holden made 3 patches in 1 month, and 6 patches in 1 year
           Donkers, Winfried made 2 patches in 1 month, and 4 patches in 1 year
           Nagy Tibor made 2 patches in 1 month, and 34 patches in 1 year
           jwtiyar nariman made 2 patches in 1 month, and 2 patches in 1 year
           Attila Szűcs made 1 patches in 1 month, and 18 patches in 1 year
     + top 10 reviewers:
           Olivier Hallot made 126 review comments in 1 month, and 1306 in 1 year
           Michael Stahl made 124 review comments in 1 month, and 996 in 1 year
           Adolfo Jayme Barrientos made 116 review comments in 1 month, and 914 in 1 year
           Xisco Fauli made 92 review comments in 1 month, and 854 in 1 year
           Thorsten Behrens made 88 review comments in 1 month, and 694 in 1 year
           Kaganski, Mike made 86 review comments in 1 month, and 802 in 1 year
           Nabet, Julien made 84 review comments in 1 month, and 510 in 1 year
           Rathke, Eike made 52 review comments in 1 month, and 308 in 1 year
           Bergmann, Stephan made 50 review comments in 1 month, and 226 in 1 year
           Heiko Tietze made 48 review comments in 1 month, and 272 in 1 year
     + big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
           javier torres
           jwtiyar nariman
           Renwa Hiwa
           Sinduja Y
           Pragat Pandya
           Kunal Pawar
           + 2 of these are language-related (locale data, etc)
           + trying to steer them towards more complex tasks
     + draft blog post: adding new UNO command, WIP

* GSoC (Ilmari)
     + GSoC 2022 application is filed
     + https://developers.google.com/open-source/gsoc/timeline
     + next step: 7th March: accepted orgs are announced
     + just waiting for now (Thorsten)

* Commit Access
   + None

* Developer Certification (Stephan/Kendy/Thorsten/László)
   + resting since: 7 weeks (limit: 20 weeks)

* Jenkins / CI update (Cloph)
     from:Thu Feb 10 14:56:35 2022
     master linux rel  jobs: 121 ok: 121 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
     master linux dbg  jobs: 107 ok:  85 ko:  22 fail ratio: 20.56 % break:   8 broken duration:15.53%
     master mac rel    jobs:  89 ok:  89 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
     master mac dbg    jobs:  93 ok:  93 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
     master win rel    jobs:  86 ok:  86 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
     master win dbg    jobs:  87 ok:  86 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
     master win64 dbg  jobs:  85 ok:  74 ko:  11 fail ratio: 12.94 % break:   1 broken duration: 9.08%
     master gerrit lin jobs: 439 ok: 348 ko:  69 fail ratio: 15.72% time for ok: mean:  19 median:  16
     master gerrit plg jobs: 380 ok: 240 ko: 110 fail ratio: 28.95% time for ok: mean:  42 median:  40
     master gerrit win jobs: 601 ok: 289 ko: 264 fail ratio: 43.93% time for ok: mean:  54 median:  52
     master gerrit mac jobs: 436 ok: 311 ko:  94 fail ratio: 21.56% time for ok: mean:  35 median:  29
     master android    jobs: 370 ok: 314 ko:  28 fail ratio:  7.57% time for ok: mean:  20 median:  14
     master gerrit all jobs: 421 ok: 207 ko: 157 fail ratio: 37.29% time for ok: mean:  95 median:  71
    + https://dev-www.libreoffice.org/tmp/gerrit_220217.html
    + tests that failed more than twice in last seven days
       3 CppunitTest_sw_layoutwriter                gerrit_linux_clang_dbgutil
       5 Killed by the kill-wrapper                gerrit_linux_clang_dbgutil
      35 CppunitTest_sc_macros_test                gerrit_windows (22 on a single bot)

    + was a good week in short
      + investigation with .component loading on Windows was not entirely successful, but improved cygwin load balancing -> 10-15 mins faster (Cloph)
      + with load limiting: https://frupic.frubar.net/shots/42843.png
      + only with parallelism: https://frupic.frubar.net/shots/42848.png

* l10n (Sophie)
   + removal of product name from extended tips -> was removed due to performance
     + affects l10n

* Various baseline bumps (Stephan)
   + Update on bumping macOS build baseline to Xcode 12.5 and macOS 11.0
     + still outstanding, one bot is not yet updated (Cloph)
   + Suggestion to bump Clang loplugin baseline to Clang 12.0.1
     + rolling it out in Jenkins
   + and bump Clang non-loplugin baseline on Linux and Android to Clang 8.0.1
     + turns out Android uses some custom clang, but "clang 8" seemt to work

* Naming for new environment variables (Miklos)
   + related to https://gerrit.libreoffice.org/c/core/+/129992 from Heiko
   + recently introduced SAL_NO_QUERYSAVE was renamed to NO_QUERYSAVE
   + existing prefixes are LO_ or '<named of module>_' (e.g. SAL_)
   + a bit concerned about no prefixes, easy to affect behavior of other software if you put that in some global config (e.g. bashrc)
   + list of env vars at https://wiki.documentfoundation.org/Development/Environment_variables
   + makes sense to have some prefix (Michael W)
   + no problem with having a prefix (Heiko)
     + don't like the mess with LIBO_, LO_, etc.
   + was pointing out just the "SAL_" prefix, not hinting no prefixes (Stephan)
   + fine with any prefix (Stephan)
   + SAL_ prefix usually means the "sal" part of VCL (Michael S)
   + don't like tons of new environment variables anyway (Stephan)
     + too many settings
     + increasing complexity
     + they are hard to document
     + the list of environment variables is not perfect
       + some of them are build-time names
       + some other are missing, when not going via getenv()
AI: add a note to the wiki page to have prefixes (to prevent collision) + think twice before adding a new one (Hossein)
AI: go back to have a prefix for NO_QUERYSAVE (Heiko)

       + https://gerrit.libreoffice.org/c/core/+/130026


* What’s cooking (Miklos)
   + Lubos is working towards non-experimental jumbo sheets in Calc
     + hopes to enable 16k cols as non-experimental on master perhaps next week
     + intention is to have more feedback, early

* QA update (Xisco)

     + UNCONFIRMED: 1688 (+19)
         + enhancements: 265  (-1)
         + needsUXEval: 2 (-7)
         + haveBackTrace: 6 (+1)
         + needsDevAdvice: 25 (+0)
         + documentation:  5 (+4)
         + android:  1 (+0)
         + iOS:  0 (+0)
         + Online:  10 (+0)


     + Most pressing bugs:
      New:

     None

      Old:
         + Slow typing with version 7.1 or above on Windows 11
             + https://bugs.documentfoundation.org/show_bug.cgi?id=145963
             + Lubos?
             + seems to be Windows 11-only

         + macOS: Can't save or open files using Finder dialog on Standard accounts on macOS 10.15 Catalina
            + https://bugs.documentfoundation.org/show_bug.cgi?id=128233
            + started to happen with a new macOS version

         + Text runs of RTL scripts (e.g. Arabic, Hebrew, Persian) from
           imported PDF are reversed, PDFIProcessor::mirrorString not behaving
             + https://bugs.documentfoundation.org/show_bug.cgi?id=104597
             + already listed here for 2 years, and many duplicates
     Fixed:
         + Crash in: _GLOBAL__sub_I_protectiondlg.cxx.lto_priv.2643 ( Calc crashes )
             + https://bugs.documentfoundation.org/show_bug.cgi?id=147135
             + Thanks to Lubos

     + New high severity bugs of the week:
         + Writer stops responding when a group of shapes is anchored to character
             + https://bugs.documentfoundation.org/show_bug.cgi?id=146597
             + Daniel?

* QA stats
     + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
       +154  +26 (-158) overall)
       many thanks to the top bug squashers:
        Heiko Tietze            15
        QA Administrators       14
        Justin L                11
        Luboš Luňák             10
        Xisco Faulí             9
        Buovjaga                8
        Gabor Kelemen (allotropia) 7
        Caolán McNamara         6
        Timur                   6
        Dieter                  5

     + top 10 bugs reporters:
        Telesto                 14
        Mike Kaganski           7
        Alex Thurgood           4
        Colin                   3
        Heiko Tietze            3
        BogdanB                 2
        Dreamquartz             2
        Gabor Kelemen (allotropia) 2
        Jens-D Doll             2
        Luke Kendall            2

     + top 10 bugs fixers:
        Luboš Luňák             7
        Heiko Tietze            5
        Xisco Fauli             5
        Justin Luth             4
        McNamara, Caolán        4
        Németh, László          4
        Andreas Heinisch        3
        Melenchuk, Vasily       2
        Michael Stahl           2
        Nabet, Julien           2

     + top 10 bugs confirmers:
        Raal                    14
        Dieter                  12
        m.a.riosv               12
        Heiko Tietze            7
        Xisco Fauli             6
        Ezinne Nnamani          4
        Telesto                 4
        *UNKNOWN*               3
        Aleksandra              3
        Alex Thurgood           3


* Highest-Priority bugs (aka "MABs"):
    + http://bit.ly/2lJAAJI
      6.0 : 1/53 -     1 %  (+0)
      5.4 : 0/41 -     0 %  (+0)
      5.3 : 0/54 -     0 %  (+0)
      5.2 : 0/43 -     0 %  (+0)
      5.1 : 1/36 -     2 %  (+0)
      5.0 : 0/64 -     0 %  (+0)
      4.4 : 0/76 -     0 %  (+0)
      4.3 : 0/73 -     0 %  (+0)
      4.2 : 2/134 -    1 %  (+0)
      4.1 : 3/85 -     3 %  (+0)
      4.0 : 2/83 -     2 %  (+0)
      old : 2/247 -    0 %  (+0)


* Bisected bugs open: keyword 'bisected'
    + more accurate - down to a single commit.
    + http://bit.ly/2dyIfDy
    +

      done by:
         Raal                   6
         Aron Budea             4
         Timur                  2
         Kelemen, Gabor         2
         Roman Kuznetsov        1
         Ilmari Lauhakangas     1
         Terrence Enger         1
         Xisco Fauli            1


* Bibisected bugs open: keyword 'bibisected'
    + http://bit.ly/2cSCXlS
    +

      done by:
         Raal                   5
         Aron Budea             4
         Timur                  2
         Kelemen, Gabor         2
         Roman Kuznetsov        1
         Ilmari Lauhakangas     1
         Terrence Enger         1
         Xisco Fauli            1


* all bugs tagged with 'regression'
    + 1261(-19) bugs open of 11573(+11) total 11(+0) high prio.

      done by:
         Telesto                4
         Robert Großkopf        2
         Ilmari Lauhakangas     1
         Kelemen, Gabor         1
         m.a.riosv              1
         Németh, László         1
         Raal                   1
         Nabet, Julien          1
         Xisco Fauli            1


* ~Component   count net * high severity regressions
    + http://bit.ly/1HWHb3E
      Writer        - 7(+0)
      LibreOffice   - 2(+0)
      Calc          - 1(+0)
      Draw          - 1(+0)

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


* ~Component   count net * all regressions
    + http://bit.ly/1BUdI8i
      Writer: other            - 435(+1)
      Calc                     - 217(-5)
      Impress                  - 124(-6)
      Writer: docx filter      - 57(-1)
      LibreOffice              - 49(+0)
      Crashes                  - 48(+0)
      Draw                     - 45(+0)
      UI                       - 44(-1)
      Borders                  - 40(+0)
      Writer: perf             - 37(-1)
      Writer: other filter     - 35(+0)
      Base                     - 33(+0)
      Chart                    - 29(+0)
      RTL                      - 26(+0)
      Writer: doc filter       - 22(+0)
      Printing and PDF export  - 21(+0)
      graphics stack           - 20(+0)
      BASIC                    - 18(-1)
      filters and storage      - 16(+0)
      Formula Editor           - 2(+0)
      framework                - 1(+0)
      sdk                      - 1(+0)


More information about the LibreOffice mailing list