sed question

[dtanner]

Hi all,
I am attempting to remove a random line in a file. I am thinking that sed should do it something like this:
cat access | sed '/$choice/d' > access.new
Where $choice is a variable filled in by the user, but alas, it does not work.
Thanx in advance!
DT In order to prevent possible injury to yourself, or your computer, and to ensure readability, please do not smoke, eat, drink, spindle, bend, fold, staple, or mutilate this message.

 

[dtanner]

Never mind. I just needed double quotes instead of single quotes.
In order to prevent possible injury to yourself, or your computer, and to ensure readability, please do not smoke, eat, drink, spindle, bend, fold, staple, or mutilate this message.

 

[vgersh99]

Thanks for 'not working'?

Problems:
1. shell quoting
2. UUOC

sed "/${choice}/d" access > access.new vlad
+---------------------------+
|#include<disclaimer.h> |
+---------------------------+

 

[dtanner]

Thanx!! I will change it again.
DT In order to prevent possible injury to yourself, or your computer, and to ensure readability, please do not smoke, eat, drink, spindle, bend, fold, staple, or mutilate this message.

 

[jamisar]

why you cat filename|sed ??
this is correct and really NOT the same: cat filename*|sed
else sed '.....' filename >newfname
vox clamantis in deserto.

 

[vgersh99]

#define UUOC &quot;UselessUseOfCat&quot; vlad
+---------------------------+
|#include<disclaimer.h> |
+---------------------------+