<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Hi Ben,</p>
    <p>I've just noticed that when you commit a diff from phab (as
      below), you are assigned both as committer and *author* (see also
      here
<a class="moz-txt-link-freetext" href="https://git.haskell.org/ghc.git/commitdiff/a1c0b70638949a73bbd404c11797f2edf28f5965">https://git.haskell.org/ghc.git/commitdiff/a1c0b70638949a73bbd404c11797f2edf28f5965</a>).</p>
    <p>Reviewers and subscribers to the phab diff are indicated in the
      commit message but not the author (here Ingo Blechschmidt
      "iblech"). I'm worried that if the Phabricator instance goes down
      we won't be able to retrieve commit authors. And also that they
      are not credited appropriately (cf git shortlog -sne).</p>
    <p>(By the way, I've also noticed that .mailmap contents isn't up to
      date: `git shortlog -se | cut -f2 | cut -d'<' -f1 | uniq -d`
      isn't empty. Maybe you could add a check in a script somewhere to
      ensure that it stays empty when you push a commit?).</p>
    <p>Regards,<br>
      Sylvain<br>
    </p>
    <div class="moz-forward-container"><br>
      <br>
      -------- Forwarded Message --------
      <table class="moz-email-headers-table" cellspacing="0"
        cellpadding="0" border="0">
        <tbody>
          <tr>
            <th valign="BASELINE" nowrap="nowrap" align="RIGHT">Subject:
            </th>
            <td>[commit: ghc] master: Use https links in user-facing
              startup and error messages (a1c0b70)</td>
          </tr>
          <tr>
            <th valign="BASELINE" nowrap="nowrap" align="RIGHT">Date: </th>
            <td>Sat, 15 Dec 2018 00:49:47 +0000 (UTC)</td>
          </tr>
          <tr>
            <th valign="BASELINE" nowrap="nowrap" align="RIGHT">From: </th>
            <td><a class="moz-txt-link-abbreviated" href="mailto:git@git.haskell.org">git@git.haskell.org</a></td>
          </tr>
          <tr>
            <th valign="BASELINE" nowrap="nowrap" align="RIGHT">Reply-To:
            </th>
            <td><a class="moz-txt-link-abbreviated" href="mailto:ghc-devs@haskell.org">ghc-devs@haskell.org</a></td>
          </tr>
          <tr>
            <th valign="BASELINE" nowrap="nowrap" align="RIGHT">To: </th>
            <td><a class="moz-txt-link-abbreviated" href="mailto:ghc-commits@haskell.org">ghc-commits@haskell.org</a></td>
          </tr>
        </tbody>
      </table>
      <br>
      <br>
      Repository : <a class="moz-txt-link-freetext" href="ssh://git@git.haskell.org/ghc">ssh://git@git.haskell.org/ghc</a><br>
      <br>
      On branch : master<br>
      Link :
<a class="moz-txt-link-freetext" href="http://ghc.haskell.org/trac/ghc/changeset/a1c0b70638949a73bbd404c11797f2edf28f5965/ghc">http://ghc.haskell.org/trac/ghc/changeset/a1c0b70638949a73bbd404c11797f2edf28f5965/ghc</a><br>
      <br>
      <blockquote type="cite">---------------------------------------------------------------<br>
      </blockquote>
      <br>
      commit a1c0b70638949a73bbd404c11797f2edf28f5965<br>
      Author: Ben Gamari <a class="moz-txt-link-rfc2396E" href="mailto:ben@smart-cactus.org"><ben@smart-cactus.org></a><br>
      Date: Fri Dec 14 11:10:56 2018 -0500<br>
      <br>
      Use https links in user-facing startup and error messages<br>
      I consider myself lucky that in my circle of friends, `http` urls
      (as<br>
      opposed to `https` urls) are frowned upon in that we generally<br>
      apologize in the rase cases that we share an `http` url.<br>
      This pull request changes `http` links into their `https`
      analogues in<br>
      the following places:<br>
      * In the GHCI startup message (and parts of the User's Guide,
      where<br>
      there are verbatim transcripts of GHCi sessions).<br>
      * In a couple of error messages, asking the user to report a bug.<br>
      (I also took the liberty to change a single space before the
      reportabug<br>
      url into two spaces, harmonizing this occurence with the others.)<br>
      I'm not trying to start a war. I just had a moment to spare and
      felt<br>
      like preparing this diff. Merge or don't merge as you wish!<br>
      Reviewers: bgamari, erikd, simonmar<br>
      Subscribers: goldfire, rwbarton, carter<br>
      Differential Revision: <a class="moz-txt-link-freetext" href="https://phabricator.haskell.org/D5450">https://phabricator.haskell.org/D5450</a><br>
      <br>
      <br>
      <blockquote type="cite">---------------------------------------------------------------<br>
      </blockquote>
      <br>
      a1c0b70638949a73bbd404c11797f2edf28f5965<br>
      compiler/typecheck/TcTyClsDecls.hs | 2 +-<br>
      compiler/utils/Panic.hs | 2 +-<br>
      docs/users_guide/ghci.rst | 4 ++--<br>
      ghc/GHCi/UI.hs | 2 +-<br>
      rts/RtsMessages.c | 2 +-<br>
      5 files changed, 6 insertions(+), 6 deletions(-)<br>
      <br>
      diff --git a/compiler/typecheck/TcTyClsDecls.hs
      b/compiler/typecheck/TcTyClsDecls.hs<br>
      index cc9779a..71899a1 100644<br>
      --- a/compiler/typecheck/TcTyClsDecls.hs<br>
      +++ b/compiler/typecheck/TcTyClsDecls.hs<br>
      @@ -3609,7 +3609,7 @@ checkValidRoles tc<br>
      report_error doc<br>
      = addErrTc $ vcat [text "Internal error in role inference:",<br>
      doc,<br>
      - text "Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/reportabug">http://www.haskell.org/ghc/reportabug</a>"]<br>
      + text "Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/reportabug">https://www.haskell.org/ghc/reportabug</a>"]<br>
      {-<br>
************************************************************************<br>
      diff --git a/compiler/utils/Panic.hs b/compiler/utils/Panic.hs<br>
      index 03f095b..4f0f3b1 100644<br>
      --- a/compiler/utils/Panic.hs<br>
      +++ b/compiler/utils/Panic.hs<br>
      @@ -168,7 +168,7 @@ showGhcException exception<br>
      showString "panic! (the 'impossible' happened)\n"<br>
      . showString (" (GHC version " ++ cProjectVersion ++ " for " ++
      TargetPlatform_NAME ++ "):\n\t")<br>
      . s . showString "\n\n"<br>
      - . showString "Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/reportabug\n">http://www.haskell.org/ghc/reportabug\n</a>"<br>
      + . showString "Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/reportabug\n">https://www.haskell.org/ghc/reportabug\n</a>"<br>
      throwGhcException :: GhcException -> a<br>
      diff --git a/docs/users_guide/ghci.rst b/docs/users_guide/ghci.rst<br>
      index 49a96ca..f468e80 100644<br>
      --- a/docs/users_guide/ghci.rst<br>
      +++ b/docs/users_guide/ghci.rst<br>
      @@ -37,7 +37,7 @@ command ``ghci``:<br>
      .. code-block:: none<br>
      $ ghci<br>
      - GHCi, version 8.y.z: <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/">http://www.haskell.org/ghc/</a> :? for help<br>
      + GHCi, version 8.y.z: <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/">https://www.haskell.org/ghc/</a> :? for help<br>
      Prelude><br>
      There may be a short pause while GHCi loads the prelude and
      standard<br>
      @@ -2052,7 +2052,7 @@ by using the :ghc-flag:`-package ⟨pkg⟩`
      flag:<br>
      .. code-block:: none<br>
      $ ghci -package readline<br>
      - GHCi, version 8.y.z: <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/">http://www.haskell.org/ghc/</a> :? for help<br>
      + GHCi, version 8.y.z: <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/">https://www.haskell.org/ghc/</a> :? for help<br>
      Loading package base ... linking ... done.<br>
      Loading package readline-1.0 ... linking ... done.<br>
      Prelude><br>
      diff --git a/ghc/GHCi/UI.hs b/ghc/GHCi/UI.hs<br>
      index ae8ba02..13275f8 100644<br>
      --- a/ghc/GHCi/UI.hs<br>
      +++ b/ghc/GHCi/UI.hs<br>
      @@ -162,7 +162,7 @@ defaultGhciSettings =<br>
      ghciWelcomeMsg :: String<br>
      ghciWelcomeMsg = "GHCi, version " ++ cProjectVersion ++<br>
      - ": <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/">http://www.haskell.org/ghc/</a> :? for help"<br>
      + ": <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/">https://www.haskell.org/ghc/</a> :? for help"<br>
      ghciCommands :: [Command]<br>
      ghciCommands = map mkCmd [<br>
      diff --git a/rts/RtsMessages.c b/rts/RtsMessages.c<br>
      index 053805e..a90962e 100644<br>
      --- a/rts/RtsMessages.c<br>
      +++ b/rts/RtsMessages.c<br>
      @@ -172,7 +172,7 @@ rtsFatalInternalErrorFn(const char *s, va_list
      ap)<br>
      #endif<br>
      fprintf(stderr, "\n");<br>
      fprintf(stderr, " (GHC version %s for %s)\n", ProjectVersion,
      xstr(HostPlatform_TYPE));<br>
      - fprintf(stderr, " Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="http://www.haskell.org/ghc/reportabug\n">http://www.haskell.org/ghc/reportabug\n</a>");<br>
      + fprintf(stderr, " Please report this as a GHC bug:
      <a class="moz-txt-link-freetext" href="https://www.haskell.org/ghc/reportabug\n">https://www.haskell.org/ghc/reportabug\n</a>");<br>
      fflush(stderr);<br>
      }<br>
      #if defined(mingw32_HOST_OS)<br>
      <br>
      _______________________________________________<br>
      ghc-commits mailing list<br>
      <a class="moz-txt-link-abbreviated" href="mailto:ghc-commits@haskell.org">ghc-commits@haskell.org</a><br>
      <a class="moz-txt-link-freetext" href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-commits">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-commits</a><br>
    </div>
  </body>
</html>