sed deleting file

[ddrew78]

Hello Monks, I have a file that looks like this:

(3.3.3.3,4.4.4.4,)

What I need to do is get rid of the comma at the end. I tried using sed, like this:

system "sed 's/,)/)/' pbxiplp > pbxiplp1";

but, instead of giving me

(3.3.3.3,4.4.4.4)

, it just removes everything within the file. Any ideas? I've used 'sed' plenty in the past, but have never run into this before. The file will always only contain one line.

[johngg]

Why use sed at all, you can do it all in Perl?

knoppix@Microknoppix:~$ perl -Mstrict -wE '
> my $pbxiplp = <<EOF;
> (3.3.3.3,4.4.4.4,)
> EOF
> print $pbxiplp;
> 
> my $pbxiplp1;
> open my $inFH, q{<}, \ $pbxiplp or die $!;
> open my $outFH, q{>}, \ $pbxiplp1 or die $!;
> while ( <$inFH> )
> {
>     s{ , (?= \) ) }{};
>     print $outFH $_;
> }
> close $inFH or die $!;
> close $outFH or die $!;
> 
> print $pbxiplp1;'
(3.3.3.3,4.4.4.4,)
(3.3.3.3,4.4.4.4)
knoppix@Microknoppix:~$

I'm using scalar refs. here as I'm doing this on the command line but it would work just as well with real files.

I hope this is helpful.

[Fletch]

Calling out to sed from perl is pretty carrying-coal-to-Newcastle. Just open the old file, open the output, loop over the old lines using the same substitution, and print to the output file.

[Your Mother]

This Perl works from the command line (add the -i if you test it and it seems safe). You can do similar tricks in a script with $^I.

perl -p -e 's/,(?=\)\Z)//' [file list]

[snoopy]

Its always a good idea to do basic status checking of even the most trivial system call. There's always a chance of failure on a live system. For example:
• incorrect path setup
• exhaustion of resources such as physical disk
• shell not available
• permissions or process limits
• missing utilities (although unlikely with sed)
• many others ....

system "sed 's/,)/)/' pbxiplp > pbxiplp1"
    and die "system call failed: $!";

See also system.

[roboticus]

ddrew78:

It looks like you left off the -e switch.

$ cat foo
(3.3.3.3,4.4.4.4,)

$ sed -e 's/,)/)/' foo 
(3.3.3.3,4.4.4.4)

...roboticus

When your only tool is a hammer, all problems look like your thumb.

[ddrew78]

tried it, still doesn't work :-(

[JavaFan]

sed questions aren't Perl question. However, I cannot reproduce your problem:

$ echo '(3.3.3.3,4.4.4.4,)' | sed 's/,)/)/'
(3.3.3.3,4.4.4.4)
$

I get that result both for BSD and GNU sed.

[choroba]

Try to modify it as follows:

system "sed 's/,)/)/' pbxiplp > pbxiplp1" and die $?,$!;

[ddrew78]

This one is going to be the end of me...still removes the content of the file/gives blank output

[choroba]

You should not write to the same file you are reading. You can use sed's -i~ flag to change the file "in place", or save the output under a different name.
Update: sorry, misread the question.

[ddrew78]

I'm not...if you look, the file I'm writing to has a "1" at the end

[cdarke]

This is not sed(1) overwriting the output file, it is the shell. Redirection  > pbxiplp1 is a shell operation - if the file already exists it clobbers it (truncates to zero bytes). This is done by the shell before the sed(1) program is executed. By the time poor old sed gets to read the file the data is long gone.

Use perl for this, or redirect to a different filename then rename it. Misread the question.