Using sed to match comma newline endbracket

Question

I would like to write a sed statement that removes the last comma.

DROP TABLE IF EXISTS person;$
CREATE TABLE person ($
  id int(11) NOT NULL,$
  name varchar(500) DEFAULT NULL,$
  gender char(1) DEFAULT NULL,$
  birthdate date DEFAULT NULL,$
  deathdate date DEFAULT NULL,$
  height int(11) DEFAULT NULL,$
) ;$

code snippet is from vim with :set list

The wanted output is

DROP TABLE IF EXISTS person;$
CREATE TABLE person ($
  id int(11) NOT NULL,$
  name varchar(500) DEFAULT NULL,$
  gender char(1) DEFAULT NULL,$
  birthdate date DEFAULT NULL,$
  deathdate date DEFAULT NULL,$
  height int(11) DEFAULT NULL$
) ;$

I tryed to do it like this

sed -e 's@,$)@$)@'

but it does not match

last comma? Can you show expected output. – sat Apr 07 '14 at 15:47 — sat, Apr 07 '14 at 15:47
which comma are you atalking about ? – aelor Apr 07 '14 at 15:49 — aelor, Apr 07 '14 at 15:49
I hope it is clear now. – Kim Ahlstrøm Meyn Mathiassen Apr 07 '14 at 15:49 — Kim Ahlstrøm Meyn Mathiassen, Apr 07 '14 at 15:49

devnull · Accepted Answer · 2014-04-07T16:19:29.417

4

sed process one line of input at a time. You need to append the next line to the current one, perform the substitution and print the pattern space:

sed 'N;s/,\n)/\n)/;P;D' inputfile

For your sample input, this would produce:

DROP TABLE IF EXISTS person;
CREATE TABLE person (
  id int(11) NOT NULL,
  name varchar(500) DEFAULT NULL,
  gender char(1) DEFAULT NULL,
  birthdate date DEFAULT NULL,
  deathdate date DEFAULT NULL,
  height int(11) DEFAULT NULL
) ;

edited Apr 07 '14 at 16:19

answered Apr 07 '14 at 15:55

devnull

118,548
33
236
227

@aelor Could you please elaborate _doesnt work_? Does it produce an error, or unexpected results? – devnull Apr 07 '14 at 16:07
Why split it up into 2 statements? – potong Apr 07 '14 at 16:16
@potong Remove the split. Thanks. – devnull Apr 07 '14 at 16:19
i dont knw why its behaving differently for me i am using ubuntu 12.04 and the above command doesnt remove any commas\ – aelor Apr 07 '14 at 18:39
@aelor I suspect that you're considering the `$` at the end of lines to be a part of the input -- those are not a part of the input. – devnull Apr 08 '14 at 01:31

score 0 · Answer 2 · edited May 23 '17 at 12:13

0

You can act on files in vim with argdo.

This expression matched last comma: (using /)

/,[\s\n]*)\s*;

Also: VIM, Run a command on multiple files

edited May 23 '17 at 12:13

Community

1
1

answered Apr 07 '14 at 15:57

Niloct

9,491
3
44
57

score 0 · Answer 3 · answered Apr 07 '14 at 15:59

0

this may work for you:

awk -v RS="" '{gsub(/,\n\)/,"\n)")}7' file

answered Apr 07 '14 at 15:59

Kent

189,393
32
233
301

Using sed to match comma newline endbracket

3 Answers3