Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 79231 - openssl ebuild stops with sed error: "sed: -e expression #1, char 114: unknown option to `s'"
Summary: openssl ebuild stops with sed error: "sed: -e expression #1, char 114: unknow...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Daniel Ahlberg (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-23 09:43 UTC by Markus Wernig
Modified: 2005-02-06 00:39 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch for checking forward slashes in CFLAGS in openssl-0.9.7e.ebuild (openssl.patch,316 bytes, patch)
2005-01-23 09:44 UTC, Markus Wernig
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Wernig 2005-01-23 09:43:02 UTC
I have added "-I/usr/local/include -L/usr/local/lib" to CFLAGS in make.conf.
There is a line (currently line no. 68) in the openssl-0.9.7e-ebuild that runs sed on the contents of the $CFLAGS variable. sed gives the above error if CFLAGS contains a forward slash ("/"). This is actually what I would expect it to do.

As a workaround I added the following line before the sed call on line no. 68:

NEW_CFLAGS="$( echo $NEW_CFLAGS | sed -e s/'\/'/'\\\/'/g  )"
(See attached patch)

I don't know if this is specific to openssl, all other ebuilds seem to run correctly

Reproducible: Always
Steps to Reproduce:
1. Add -I/usr/local/include -L/usr/local/lib" to CFLAGS in make.conf
2. run "emerge openssl"
3.

Actual Results:  
emerge stops with error "sed: -e expression #1, char 114: unknown option to `s'"

Expected Results:  
clean install

I've created a patch for the ebuild that I'll attach.
Comment 1 Markus Wernig 2005-01-23 09:44:20 UTC
Created attachment 49303 [details, diff]
Patch for checking forward slashes in CFLAGS in openssl-0.9.7e.ebuild
Comment 2 SpanKY gentoo-dev 2005-02-06 00:39:24 UTC
changed the delimiter to ':' instead in 0.9.7e