Opened 17 years ago
Last modified 16 years ago
#1243 accepted defect
Problem with 'sed' again in parsing SVN version info
| Reported by: | Michael Wood-Vasey | Owned by: | Paul Price |
|---|---|---|---|
| Priority: | lowest | Milestone: | |
| Component: | psconfig | Version: | current |
| Severity: | minor | Keywords: | |
| Cc: | jester@… |
Description
/usr/bin/sed -e "s|@PSLIB_VERSION@|\"/usr/bin/svnversion ../..\"|" -e "s|@PSLIB_BRANCH@|\"/usr/bin/svn info ../.. | /usr/bin/sed -n -e '/URL:/ h' -e '/Repository Root:/ { x; H; x; s|Repository Root: \(.*\)\nURL: \1\(.*\)|\2| ; s|^/|| ; s|/[a-zA-Z]*/src.*|| ; p }'\"|" -e "s|@PSLIB_SOURCE@|\"/usr/bin/svn info | /usr/bin/sed -n -e 's/Repository UUID: // p'\"|" psVersionDefinitions.h.in > psVersionDefinitions.h
sed: 1: "/Repository Root:/ { x; ...": extra characters at the end of p command
svn: Write error: Broken pipe
on Mac OS X 10.5.6, Intel
Change History (6)
comment:1 by , 17 years ago
comment:2 by , 17 years ago
It's a BSD sed vs. GNU sed issue. If I compiled GNU sed then it works fine with that.
comment:3 by , 17 years ago
| Priority: | normal → lowest |
|---|---|
| Severity: | blocker → minor |
| Status: | new → accepted |
The workaround (get GNU sed from fink) indicates that this ticket is no longer a "blocker". It's not likely to be fixed any time soon on our side.
comment:4 by , 17 years ago
| Cc: | added |
|---|
Could we add the specific version of GNU sed that works to extlibs?
comment:5 by , 16 years ago
we should replace the long and un-readable sed line with a small perl script that lives in the psconfig directory.
the perl script could also handle the question of svn export reported in bug #1250
comment:6 by , 16 years ago
| Component: | build → psconfig |
|---|

Please report what version of sed you're using. It seems that the Mac sed is not compatible with the GNU sed.