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

Re: [PATCH] sha1_name: allow to add @{...} alias via config

From
Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date
Sep 8, 2010, 10:12 UTC
Message-ID
<AANLkTimR35gKwXg9Ve5y2SqaxyxcM410h3p=eM9cHWY2@mail.gmail.com>
In-Reply-To
<AANLkTin83BQ0YhWHuDqkY4HYWP9+bUso1BDNbFtRskDp@mail.gmail.com>
2010/9/8 Santi Béjar <santi@agolina.net>:
Show 8 quoted lines
> 2010/9/8 Santi Béjar <santi@agolina.net>:
>> 2010/9/8 Nguyễn Thái Ngọc Duy <pclouds@gmail.com>:
>>> This allows users to add new @{..} alias via ref-at.* config
>>> variables. The rewrite rule is printf-alike.
>>
>> Another issue is that it can shadow builtin @{}s, like @{upstream}.
>
> In fact this patch breaks @{upstream}.
My fault. There should not be an equal in
if (substitute_ref_at(&new_str, str) >= 0)
-- 
Duy
Previous: Santi BéjarNext: Nguyen Thai Ngoc Duy
Message 4 of 5 in “sha1_name: allow to add @{...} alias via config”
  1. sha1_name: allow to add @{...} alias via configNguyễn Thái Ngọc Duy, Sep 8, 2010
  2. Santi BéjarSep 8, 2010
  3. Santi BéjarSep 8, 2010
  4. Nguyen Thai Ngoc DuySep 8, 2010
  5. Nguyen Thai Ngoc DuySep 8, 2010

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.