Why doesn't this simple RegEx work with sed?

最新推荐文章于 2022-04-25 08:45:04 发布

world_hello_100

最新推荐文章于 2022-04-25 08:45:04 发布

阅读量1.1k

点赞数

分类专栏： bash

bash 专栏收录该内容

14 篇文章 0 订阅

订阅专栏

Take the 2-minute tour×

Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.

Why doesn't this simple RegEx work with sed?

up vote 0 down vote favorite

This is a really simple RegEx that isn't working, and I can't figure out why. According tothis, it should work.

I'm on a Mac (OS X 10.8.2).

script.sh

#!/bin/bash
ZIP="software-1.3-licensetypeone.zip"
VERSION=$(sed 's/software-//g;s/-(licensetypeone|licensetypetwo).zip//g' <<< $ZIP)

echo $VERSION

terminal

$ sh script.sh
1.3-licensetypeone.zip

editedDec 19 '13 at 14:53

asked Nov 12 '12 at 12:51

curtisblackwell
386 2 17

add comment

4 Answers

active oldest votes

up vote 3 down vote accepted

Looking at the regex documentation for OS X 10.7.4 (but should apply to OP's 10.8.2), it is mentioned in the last paragraph that

Obsolete (basic) regular expressions differ in several respects. | is an ordinary character and there is no equivalent for its functionality...

... The parentheses for nested subexpressions are \(' and)'...

sed, without any options, uses basic regular expression (BRE).

To use | in OS X or BSD's sed, you need to enable extended regular expression (ERE) via-E option, i.e.

sed -E 's/software-//g;s/-(licensetypeone|licensetypetwo).zip//g'

p/s: \| in BRE is a GNU extension.

Alternative ways to extract version number

chop-chop (parameter expansion)

VERSION=${ZIP#software-}
VERSION=${VERSION%-license*.zip}

sed
```
VERSION=$(sed 's/software-$.*$-license.*/\1/' <<< "$ZIP")
```
You don't necessarily have to match strings word-by-word with shell patterns or regex.

editedNov 14 '12 at 3:35

answered Nov 14 '12 at 3:19

doubleDown
3,001 9 23

add comment

up vote 3 down vote

sed works with simple regular expressions. You have to backslash parentheses and a vertical bar to make it work.

sed 's/software-//g;s/-\(licensetypeone\|licensetypetwo\)\.zip//g'

Note that I backslashed the dot, too. Otherwise, it would have matched any character.

answered Nov 12 '12 at 12:59

choroba
42.5k 3 27 60

Copied and pasted your line in there, but it still didn't work for me.– curtisblackwell Nov 12 '12 at 13:07

@curtisblackwell: Does yoursed support ; to separate commands? In some versions, multiple-e are needed: sed -e 's///' -e 's///'.– choroba Nov 12 '12 at 13:14

yeah, it supports multiple commands separated by;. I'm using that elsewhere.– curtisblackwell Nov 12 '12 at 13:16

@curtisblackwell: Does $ZIP contain whitespace? If it can, use<<< "$ZIP".– choroba Nov 12 '12 at 13:18

no, it doesn't, but i should probably do use quotes in case it gets renamed.– curtisblackwell Nov 12 '12 at 23:57

add comment

up vote 3 down vote

You can do this in the shell, don't need sed, parameter expansion suffices:

shopt -s extglob
ZIP="software-1.3-licensetypeone.zip"
tmp=${ZIP#software-}
VERSION=${tmp%-licensetype@(one|two).zip}

With a recent version of bash (may not ship with OSX) you can use regular expressions

if [[ $ZIP =~ software-([0-9.]+)-licensetype(one|two).zip ]]; then
    VERSION=${BASH_REMATCH[1]}
fi

or, if you just want the 2nd word in a hyphen-separated string

VERSION=$(IFS=-; set -- $ZIP; echo $2)

answered Nov 12 '12 at 18:09

glenn jackman
58.5k 6 31 66

Your last suggestion solves my problem. Thank you! However, I still don't understand why the RegEx doesn't work.– curtisblackwell Nov 13 '12 at 0:00

choroba explained it in his answer. Different programs (perl, sed, bash, etc) implement regular expressions in different ways. With sed, you need to escape the parentheses to give them their special meaning.– glenn jackman Nov 13 '12 at 0:45

That sounds right, but even when I escaped them, it didn't work. Is it possible that my version of sed just doesn't support parentheses and/or pipes in regular expressions?– curtisblackwell Nov 13 '12 at 1:00

highly unlikely. Did you also escape the pipe? Read choroba's answer very carefully. You will also want to checkman sed and (perhaps) man ed to learn about regexes implemented by sed.– glenn jackman Nov 13 '12 at 1:03

I read his answer carefully. Re-read many times, because I thought it was correct. However, even copying and pasting the RegEx didn't work. I'll check man sed again, and look into ed. Thanks.– curtisblackwell Nov 13 '12 at 1:06

add comment

up vote 0 down vote

$ man sed | grep "regexp-extended" -A2
       -r, --regexp-extended

              use extended regular expressions in the script.

answered Nov 12 '12 at 13:03

csgwro
131 1 4

add comment

Your Answer

discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions taggedregex sed or ask your own question.

question feed

Technology			Life / Arts	Culture / Recreation	Science	Other
Stack Overflow Server Fault Super User Web Applications Ask Ubuntu Webmasters Game Development TeX - LaTeX	Programmers Unix & Linux Ask Different (Apple) WordPress Development Geographic Information Systems Electrical Engineering Android Enthusiasts Information Security	Database Administrators Drupal Answers SharePoint User Experience Mathematica more (14)	Photography Science Fiction & Fantasy Seasoned Advice (cooking) Home Improvement Personal Finance & Money more (12)	English Language & Usage Skeptics Mi Yodeya (Judaism) Travel Christianity Arqade (gaming) Bicycles Role-playing Games more (21)	Mathematics Cross Validated (stats) Theoretical Computer Science Physics MathOverflow more (7)	Stack Apps Meta Stack Overflow Area 51 Stack Overflow Careers