<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi,<br>
    I think Kohei has covered most of the questions below but..<br>
    On 17/08/11 19:49, Lionel Elie Mamane wrote:
    <blockquote cite="mid:20110817184943.GB32660@capsaicin.mamane.lu"
      type="cite">
      <pre wrap="">Hi,

</pre>
    </blockquote>
    [...]<br>
    <blockquote cite="mid:20110817184943.GB32660@capsaicin.mamane.lu"
      type="cite">
      <pre wrap="">
So I don't know when to close a bug. When it is fixed in master?</pre>
    </blockquote>
    yes, and normally the committer does that, is it a rule? I don't
    know, it's just what I and at least I know others do. Normally I put
    the commit id(s) and branches in the comment too. This time you
    closed the bug before I got around to doing that and that kinda
    distracted me from doing what I normally do<br>
    <blockquote cite="mid:20110817184943.GB32660@capsaicin.mamane.lu"
      type="cite">
      <pre wrap=""> When
it is fixed in all branches where we want it fixed?

For example, bug #40079; it is fixed in master, so I closed it. But
now it shows up as not blocking "LibreOffice 3.4 most annoying bugs"
anymore (it is striked out), </pre>
    </blockquote>
    when we want a bug to be fixed in the 3.4 release then we need to
    get the patch into the general dev branch associated with 3.4 ( e.g.
    the libreoffice-3-4 branch )&nbsp; and also the release branch e.g.&nbsp;
    libreoffice-3-4-3. There are rules though about committing to these
    branches. Currently you need the approval of 1 reviewer to commit to
    the libreoffice-3-4 branch and the approval of 3 reviewers to commit
    to the release branch. I have already pushed the patch to 3.4 and<i>
    </i>I already asked for a review so that this bug can go into
    libreoffice-3-4-3. It is a good idea to monitor the mailing list and
    the information that the patch was pushed to master, 3.4 and request
    for approval to commit to libreoffice-3-4-3 is all under that same
    thread you started about the patch for that issue.<br>
    <br>
    Noel<br>
    <br>
  </body>
</html>