Minutes of ESC call: 2015-07-30

* Present:
    + Muthu, Thorsten, Eike, Kendy, Miklos, Norbert, Jan-Marek,
      Christian, David T., Sophie, Giuseppe C., Lionel, Robinson, Bjoern
 
* Completed Action Items:
    + create presenter console cherry-pick request (Kendy)
        [ there already was one by Michael, but abandoned, due to the need of more testing ]
    + check EMF+ rendering on Windows with the presenter console bug fixed (Robinson)
        [ let the presenter console bugfix for 5.0.]
    + get a non-TDF branded 5-0 set into git / master (Cloph)
        [ have the new files, but OS X dnd - but in the works
          will push the new 5.0 branding to the libreoffice-5-0 branch ]
    + move existing git images into a private TDF branded pack (Cloph)
    + increase availble disk for crash-test VM136 (Cloph/Caolan)
        [ no need for a change ]
    + file conference papers [!] (All)
        + please submit abstracts:
        + http://conference.libreoffice.org/2015/call-for-papers/
             + Ace of Base (Lionel)
    + poke Jay / Kendy do we need to revert UI / menu changes ? (Sophie)
        [ nothing necessary to revert in 5.0 (Sophie) ]
    + review ongoing pootle maintenance funding arrangements (Floeff)
        [ done (Sophie) ]
 
* Pending Action Items:
    + review Giuseppe's webdav work in gerrit (Kendy/?)
        + https://gerrit.libreoffice.org/#/c/17189/
        [ have a VM for testing against, code looks good in general, need to verify & push (Kendy) ]
    + come up with a fun new name for 'blocker' (Robinson)
        [ still accepting new ideas ]
    + register at the conference: http://conference.libreoffice.org/2015/registration/ (all)
        [ mandatory from security reasons (Sophie) ]
    + re-arrange the help XML for the menu changes (Jay)
        [ Jay was away for vacation, promised to have a look at
          HelpAuthoring.oxt over the weekend (Kendy) ]
    + track & support help XML status (Kendy)
        [ new HelpAuthoring.oxt released, more reports from Regina (Kendy) ]
    + UserAgent - drop bundled-languages (Michael)
    + UserAgent - produce a patch for review (Michael)
    + check 'make check' on a Windows 32bit box instead (Norbert)
        [ still don't have the machine ]
    + promote Development Mentoring Lead Job Offer through all channels: Universities, Companies, 
Clubs, ex-GSoCers etc. (all)
      
https://blog.documentfoundation.org/2015/07/15/tdf-freelance-job-opening-201507-01-development-mentoring-lead/
        [ please spread the word (Bjoern) ]
        [ print some leaflets too & distribute (Bjoern) ]
 
* Release Engineering update (Christian)
    + 4.4.5 - RC2 status
        + released as final earlier today, done
        + will update the online update soon
    + 5.0.0 - status / branch
        + heavy-duty triple-review process for libreoffice-5-0-0 branch
            + 1x review for 5-0 and +2 more for 5-0-0
        + will tag the final later today
        + Major bugs
            + presenter console - not rendering on Windows (Michael)
                + https://bugs.documentfoundation.org/show_bug.cgi?id=91574
                  [ deferred to 5.0.1, invasive fix ]
            + was a problem with dictionaries on OS X (Cloph)
                + fixed now, and Norbert will double-check before building
            + anything else (Robinson)
        + 3 pending patches, please review! (Cloph)
AI:         + will poke people on the IRC (Cloph)
        + Late Feature Status (Michael)
              + LibreOfficeKit / Online tweaks (Kendy)
                  + no changes -> mostly changes going into master anyway.
              + OpenGL / double-buffer RenderContext (Kendy)
                  + OpenGL by default not achievable for 5.0.0
                      + propose re-visit @ ESC for 5.0.1 for some H/W on Windows.
                  + won't affect anything TDF ships for 5.0
                  + ongoing, but nothing for 5.0.0
          + 5.0 splash / startcenter graphics / about dialog (Kendy?)
              + https://bugs.documentfoundation.org/show_bug.cgi?id=90452#c45
                  + did everything get in & done.
              + https://bugs.documentfoundation.org/show_bug.cgi?id=93004
AI:               + Cloph will fix the graphics accordingly to the report (Cloph)
                  + no time for a full round-trip via the designer (Kendy)
    + Android & iOS Remote
 
* UX Update (Kendy)
    + call canceled, everybody on vacation
 
* Documentation (Olivier)
    + new HelpAuthoring-3.1.0.oxt (Kendy)
        + and updated https://wiki.documentfoundation.org/Documentation/Help
        + Regina reported some problems, will have a look (Kendy)
 
* Crashtest update (Caolán)
    + still close to 0 (Miklos)
        + 1 crash in DOCX import
        + 1 crash in ODT export (may be related to the import problem)
 
* GSoC (Cedric)
    + Next important dates:
        + Aug. 21: pencil down & start of evaluations
        + Aug. 28: final evaluations deadline
    + conference travel - poking the students now (Cedric)
        + 3 students will definitely go, have registered
        + let's hope there will be more :-)
 
* Hackfests (Bjoern)
    + Aarhus LibreOffice conference bits ?
        + Hackfest topic: "C++11 in LibreOffice"
        + look into easy, easy C++11 EasyHacks for that
            + like Stephans "init list ctors for UNO-Sequences", or
              using type-alias etc.
            + !!!please file easy hacks if you have ideas!!! (Bjoern)
    + Hamburg, Germany: Oct 23rd -> 25th
        + https://wiki.documentfoundation.org/Hackfest/Hamburg2015
        + would be great to see everyone there.
    + next venues / suggestions
        + Belgium another candidate ...
 
* l10n Update (Sophie)
    + issues with dictionaries, solved now
    + Emojis testing?
        + it's OK (Sophie)
 
* Areas that require unit tests with each bugfix (Markus/???)
    + https://wiki.documentfoundation.org/Development/Unit_Tests
        + updated the page with the areas where tests for fixes are mandatory (Kendy)
        + so far: chart2, writer filters (import + export)
        + do we have more areas where we can have this rule? (Kendy)
            + important to have the maintainer who is checking the code, and the test itself too 
(Kendy)
            + happened in the past that the test did not fail when the fix was reverted (Kendy)
    + would be good to be able to make tests even for bugs that are not fixed yet (Bjoern)
        + happened in the past too, much appreciated, helps fixing (Miklos)
        + do we have a way to mark such a case? (Jan-Marek)
            + not really (Norbert, Miklos)
        + 'make -k stagingcheck' - for tests that do not have a fix yet (Bjoern)
            + isn't it enough to have them in gerrit? (Miklos)
            + can prototype that (Bjoern)
                + that's easy, harder is the infra to show them to the people (Bjoern)
            + might be better to have 'expected failures' tests (Jan-Marek)
                + with easy way to move them to 'expected passes' (Jan-Marek)
        + writing a test needs some knowledge, couldn't we actually mentor the author to do a fix 
too? (Kendy)
            + can be much harder (Norbert, Bjoern)
        + problem is how to promote the existing tests that don't have the fixes yet (Norbert)
            + more concerned to have some xxx tests run, yyy failed, zzz passed (Norbert)
            + have the special rule could work, would like not to see that some tests may fail
              in the normal build, so that it is not perceived as 'normal that some tests fail 
(Kendy)
AI:     + will have a look at the CppUnit to implement 'expected failure' (Jan-Marek)
            + Cpp logs are e.g. in workdir/CppunitTest/sal_rtl_math.test.log
    + people should look if there has been a -1 (Norbert)
        + -1 is not sticky, it is bound to the version of the patch (Norbert)
        + please look if somebody gave -1 before you are reviewing and pushing, to avoid too
          many people using -2 which is sticky (Norbert)
    + if reverting a patch, please don't forget to let the original author know (Kendy)
        + should we do a git hook to mail when a revert happened? (Norbert)
        + mechanic message maybe could make problems - human interaction probably better? (Bjoern)
        + but letting know is important; usually there is a good reason for a revent (Norbert)
            + it is not personal, it just happens (Norbert)
AI:         + can have a look (Norbert)
                + mail to the author + committer + the person who reverted (Bjoern)
        + best to get more reviewers in gerrit, so that reverts are seldom (Bjoern)
    + how to handle when something is pushed to these areas anyway?
        + the same rules as when breaking the build (Miklos)
            + when the author is responsive, and working on the test, let it in (Miklos)
            + when not, revert (Miklos)
            + timing is up to the maintainer (Miklos, Norbert)
            + it is always a trade-off (Bjoern)
                + best to talk to the maintainer, not to overrule him/her (Norbert)
            + important to talk to people, don't push without talking when overruling (Eike)
 
* WebDAV: switch to Serf? (Michael S./Giuseppe C.)
    + how much is serf and neon impl. different feature-wise? (Kendy)
        + from the client perspective yes, but from the development perspective,
          everything needs to be done by the developer (Giuseppe)
    + serf is a pipeline, not a webdav implementation (Giuseppe)
        + one has to implement everything by himself/herself (Giuseppe)
        + more a http + https manipulator (Giuseppe)
    + why then serf was used, and not curl that is already in the codebase? (Michael S.)
        + maybe it would work too (Giuseppe)
    + serf impl. implements only the minimum needed (Giuseppe)
        + neon is more complete (Giuseppe)
    + is there anything that works with --with-webdav=neon, but not with --with-webdav=serf 
(Kendy/Norbert)
        + some unused capabilities are not implemented (Giuseppe)
        + not implemented in serf: locking for resources being copied or moved (Giuseppe)
        + only locking was missing in serf (Giuseppe)
        + but other than that, the serf impl. is complete too (Giuseppe)
    + using serf is more agile, one can achieve more when something is not implemented yet 
(Giuseppe)
 
* Developer Certification (Stephan/Bjoern/Kendy/Thorsten)
    + pending / shelved for now - wait 4 weeks.
 
* Jenkins update (Norbert)
    + windows terrible this week, the rest OK
      from:Thu Jul 23 13:26:58 2015
      master linux rel  jobs:126 ok:124 ko:2  fail ratio:1.58%  break:2 broken duration:0.99%
      master linux dbg  jobs:81  ok:77  ko:3  fail ratio:3.7%   break:3 broken duration:1.35%
      master mac rel    jobs:131 ok:115 ko:16 fail ratio:12.21% break:3 broken duration:4.16%
      master mac dbg    jobs:121 ok:109 ko:12 fail ratio:9.91%  break:3 broken duration:4.21%
      master win rel    jobs:63  ok:39  ko:24 fail ratio:38.09% break:6 broken duration:44.55%
      master win dbg    jobs:64  ok:34  ko:30 fail ratio:46.87% break:5 broken duration:61.54%
      master win64 dbg  jobs:68  ok:24  ko:44 fail ratio:64.7%  break:8 broken duration:71.55%
      lo-4.4 mac        jobs:9   ok:9   ko:0  fail ratio:0%     break:0 broken duration:0%
      gerrit win rel    jobs:134 ok:85  ko:49 fail ratio:36.56%
      gerrit lin rel    jobs:133 ok:123 ko:10 fail ratio:7.51%
      gerrit mac rel    jobs:138 ok:119 ko:19 fail ratio:13.76%
      gerrit all        jobs:133 ok:81  ko:52 fail ratio:39.09%
 
* Easy Hacks:
    + new easy-hacks thanks to:
        + Markus Mohrhard
        + Tomaž Vajngerl
        + http://bit.ly/1KR4HWX
    + create your own by setting whiteboard to eg.
            "easyHack difficultyBeginner skillCpp topicCleanup"
 
* Regression Ideas
    + now going to board; results of scoring by 12x ESC attendees
 
      Name                                  Cost - person weeks  Score.
      A. Cross-platform font/shaping stubs for layout tests   5  8.18
      G. Improved Format Validity Checks                      3  7.44
      B. Automated Help/Documentation screenshot creation     8  7.42
      D. Automated a11y based UI testing                      8  7.08
      C. SSDs for prominent QA developers                Eur 2k  6.36
      H. Always Green Master                                  3  6.00
      E. Checking for DSO dependencies                        1  5.30
      F. Android Unit Testing                                 4  5.00
  
* QA update (Robinson)
    + UNCONFIRMED at 413
    + needAdvice at 11
    + Testing new MAB-replace Bugzilla patches
        + Even without restrictions, Priority:highest remains clean
    + Most Pressing Bugs:
        http://bit.ly/1CIXqPZ
           + Priority highest in the last week ...
               + tdf#92732 - Writer crash after editing with index entry dialog
           + Outstanding bugs
               + tdf#91293 - Links in Impress Are Removed When Exported to PPTX
                 [ a little older ]
 
* QA stats:
  + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
    +131    -76 (+55 overall)
    many thanks to the top bug squashers:
        tommy27                8
        Maxim Monastirsky      8
        Jean-Baptiste Faure    6
        Alex Thurgood          6
        V Stuart Foote         6
        Miklos Vajna           4
        m.a.riosv              4
        Caolán McNamara        3
        Luke                   3
        steve -_-              2
        Lionel Elie Mamane     2
        Eike Rathke            2
        Gordo                  2
        Markus Mohrhard        2
 
* Open 5.0 MAB
  + 1/12 1/12 1/12 1/12 2/12 2/12 3/12 5/11 6/12 5/11 6/11 5/9 6/9 5/8 5/8 4/7
  + https://bugs.documentfoundation.org/showdependencytree.cgi?id=86696&hide_resolved=1
 
* Open 4.4 MAB
  + 10/60 10/60 10/60 10/60 12/60 14/61 16/61 17/61 17/61 16/60 19/60 15/56 16/56
  + https://bugs.documentfoundation.org/showdependencytree.cgi?id=79641&hide_resolved=1
 
* Open 4.3 MAB
  + 61/168 61/168 62/168 62/168 64/169 65/169 72/169 73/170 75/170 78/170 77/169
  + https://bugs.documentfoundation.org/showdependencytree.cgi?id=75025&hide_resolved=1
 
* Bibisected bugs open: whiteboard 'bibisected'
  + 309/956 311/951 305/936 305/931 310/925 310/921 320/915 330/909 329/898 335/895
    + http://tdf.io/bibibugs
 
AI: Overall and component mismatch - missing a component or two ? (Michael)
 
* all bugs tagged with 'regression'
        + 608(+?) bugs open of 4059(+?) total 37 high prio.
 
        * ~Component   count net * high severity regressions
                  Calc - 14 (+0)
                Writer -  9 (+0)
           LibreOffice -  6 (+0)
               Impress -  5 (+0)
          filters and storage -  1 (+0)
                    UI -  1 (+1)
                  Base -  1 (-1)
                + http://bit.ly/1HWHb3E
 
        * ~Component   count net * all regressions
                Writer - 207 (+2)
                  Calc - 115 (+1)
               Impress - 50 (+0)
                    UI - 42 (+4)
               Crashes - 41 (-2)
           LibreOffice - 40 (-2)
               Borders - 24 (+1)
        graphics stack - 24 (-1)
                  Base - 21 (-1)
                  Draw - 20 (+0)
                 Chart - 11 (+0)
   filters and storage - 11 (+1)
Printing and PDF export- 10 (+0)
                 BASIC -  8 (+3)
             framework -  5 (+0)
        Formula Editor -  4 (-1)
            Linguistic -  3 (+1)
            Extensions -  2 (+0)
                + http://bit.ly/1BUdI8i