IPP Software Navigation Tools IPP Links Communication Pan-STARRS Links

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 Paul Price, 17 years ago

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

comment:2 by Michael Wood-Vasey, 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 Paul Price, 17 years ago

Priority: normallowest
Severity: blockerminor
Status: newaccepted

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 Sebastian Jester, 17 years ago

Cc: jester@… added

Could we add the specific version of GNU sed that works to extlibs?

comment:5 by eugene, 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 eugene, 16 years ago

Component: buildpsconfig
Note: See TracTickets for help on using tickets.