Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix 'sed' error in options.sh. v5.0.201 v6.0.103 #3891

Merged
merged 3 commits into from
Nov 23, 2023

Conversation

duiniuluantanqin
Copy link
Member

@duiniuluantanqin duiniuluantanqin commented Nov 22, 2023

The - character, when placed in the middle of a regular expression, is interpreted as a range. It must be placed at the beginning or end to be interpreted as a literal character.


TRANS_BY_GPT4


Co-authored-by: john hondaxiao@tencent.com

@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Nov 22, 2023
@xiaozhihong
Copy link
Collaborator

Before

image

After

image

It works well

@xiaozhihong
Copy link
Collaborator

xiaozhihong commented Nov 22, 2023

The '-' symbol must be placed at the beginning or end. If it's in the middle, it will be interpreted as a range. For example, '0-9' is interpreted as '0123456789'.

TRANS_BY_GPT4

@xiaozhihong xiaozhihong changed the title fix 'sed' error in options.sh fix 'sed' error in options.sh. v5.0.201 v6.0.103 Nov 22, 2023
@xiaozhihong xiaozhihong added the RefinedByAI Refined by AI/GPT. label Nov 22, 2023
@xiaozhihong xiaozhihong merged commit 1b34fc4 into ossrs:develop Nov 23, 2023
17 checks passed
xiaozhihong added a commit that referenced this pull request Nov 23, 2023
The `-` character, when placed in the middle of a regular expression, is
interpreted as a range. It must be placed at the beginning or end to be
interpreted as a literal character.

---------

`TRANS_BY_GPT4`

---------

Co-authored-by: john <hondaxiao@tencent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RefinedByAI Refined by AI/GPT. TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants