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

Re: [PATCH v3] git-p4: add "--path-encoding" option

From
Lars Schneider <larsxschneider@gmail.com>
Date
Sep 1, 2015, 15:02 UTC
Message-ID
<3942C6D1-A758-4F50-9865-9D83C5CD9128@gmail.com>
In-Reply-To
<55E5B798.40108@web.de>
On 01 Sep 2015, at 16:35, Torsten Bögershausen <tboegi@web.de> wrote:
Show 21 quoted lines
> On 2015-09-01 14.47, Lars Schneider wrote:
>>>> +test_expect_success 'Create a repo containing iso8859-1 encoded paths' '
>>>>>> +	cd "$cli" &&
>>>>>> +
>>>>>> +	ISO8859="$(printf "$UTF8_ESCAPED" | iconv -f utf-8 -t iso8859-1)" &&
>>>>>> +	>"$ISO8859" &&
>>>>>> +	p4 add "$ISO8859" &&
>>>>>> +	p4 submit -d "test commit"
>>>>>> +'
>>>> Sorry for being persistant,
>>>> but you can't create files with names that  are ISO-8859-1 encoded under Mac OS,
>>>> we end up like this:
>>>> 
>>>> a-%E4_o-%F6_u-%FC.txt
>> You are right. However, my goal is not to create a file with ISO-8859-1 characters in Mac OS. My goal is to create this file in Perforce and this approach seems to work.
>> 
>>>> 
> But this line creates a file, doesn't it ?
>> "$ISO8859" &&
> 
> (I just wonder how this works on you machine )
I tested it on OS X (HPFS) and Linux (ext4). Can you reproduce problems on your machine? If yes, what is your OS/filesystem?
Show 7 quoted lines
> And, may be, we could fill the file with some content, to be double-sure that
> the file name conversion works with Perforce ?
> 
> like
> echo content >"$ISO8859" &&
> 
> and test the content later ?
OK, I will add this.

Thanks, Lars

Previous: Torsten Bögershausen
Message 13 of 13 in “git-p4: add "--path-encoding" option”
  1. git-p4: add "--path-encoding" optionlarsxschneider@gmail.com, Aug 31, 2015
  2. git-p4: add "--path-encoding" optionlarsxschneider@gmail.com, Aug 31, 2015
  3. Junio C HamanoAug 31, 2015
  4. Lars SchneiderSep 1, 2015
  5. Junio C HamanoSep 1, 2015
  6. Junio C HamanoSep 1, 2015
  7. Lars SchneiderSep 1, 2015
  8. Junio C HamanoSep 1, 2015
  9. Torsten BögershausenSep 1, 2015
  10. Fwd: Re: [PATCH v3] git-p4: add "--path-encoding" optionTorsten Bögershausen, Sep 1, 2015
  11. Lars SchneiderSep 1, 2015
  12. Torsten BögershausenSep 1, 2015
  13. Lars SchneiderSep 1, 2015

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.