git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 0/2] Add cmd_gud and detect libiconv path for Mac OS

From
Bagas Sanjaya <bagasdotme@gmail.com>
Date
Sep 8, 2021, 06:01 UTC
Message-ID
<a1ba4d11-222a-92a1-257d-522dc4e0d8e0@gmail.com>
In-Reply-To
<20210908051340.13332-1-colinpcurtis826@ucla.edu>
On 08/09/21 12.13, Colin Curtis wrote:
Show 13 quoted lines
> From: Colin Curtis <colinpcurtis@gmail.com>
> 
> The gud command opens the Pro Git book webpage in the default
> web browser.  The reason to add this command is due to the
> play on words when saying 'git gud', which sounds like 'get good'.
> Hence this command when invoked will open up the Pro Git
> webpage to allow the user to 'get good' at git.
> 
> We also fix a bug in the Makefile when running on Mac OS, namely
> the libiconv path when using a brew install for the library.  Previously
> the developer would have to manually change the path to the library when
> developing on Mac OS.
> 

Two distinct topics in one patch series. Why don't you create separate patch series for each topic?

For `git gud`, I think there's no point on adding such easter egg, since users can simply access git-scm.com with their browser, without needing specialized command just to invoke it.

-- 
An old man doll... just what I always wanted! - Clara
Previous: Carlo ArenasNext: Ævar Arnfjörð Bjarmason
Message 5 of 6 in “Add cmd_gud and detect libiconv path for Mac OS”
  1. 0/2 Add cmd_gud and detect libiconv path for Mac OSColin Curtis, Sep 8, 2021
  2. 1/2 add cmd_gud to open git-scm.com webpageColin Curtis, Sep 8, 2021
  3. 2/2 add liconv link for makefileColin Curtis, Sep 8, 2021
  4. Carlo ArenasSep 8, 2021
  5. Bagas SanjayaSep 8, 2021
  6. Ævar Arnfjörð BjarmasonSep 8, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.