search

google-code-export / tovid

Automatically exported from code.google.com/p/tovid
1 stars 0 forks source link

sed(1) does not understand curly braces #154

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The current code:

CRLF="sed {s/\r/\n/g}"  # Turn carriage-returns into newlines

may be working with GNU-sed, but it is rejected by normal sed:

FreeBSD-8:
sed: 1: "{s/n/r/g}": bad flag in substitute command: '}'

Solaris-10:
sed: command garbled: {s/\n/\r/g}

It is, probably, better to use tr(1) instead of sed(1) anyway as per the 
attached patch (untested).

Original issue reported on code.google.com by virtuale...@gmail.com on 18 Nov 2011 at 10:08

Attachments:

GoogleCodeExporter commented 9 years ago 
You are using a version of tovid that is many years old (5+ years ?).  The code 
that you are supplying a patch for has been removed a long time ago.  I realize 
that for various reasons freebsd uses older versions of programs, so I guess 
this patch will have to be applied only to the (patched) freebsd release.

Original comment by grepper@gmail.com on 26 Nov 2011 at 5:29

GoogleCodeExporter commented 9 years ago 
I guess, I wasted several hours by working on the latest release available from 
SourceFourge `tovid' project... Please, do post a note there (both on web-pages 
and in the "download" directory) advising people to seek latest files 
elsewhere...

Better yet -- why not upload your new versions to both Google AND SourceForge?

Original comment by virtuale...@gmail.com on 2 Dec 2011 at 12:41

GoogleCodeExporter commented 9 years ago 
Unfortunately I have never had admin privileges on the sourceforge site, so I 
can't change that. And the original owner has retired. Sorry about that.

Original comment by grepper@gmail.com on 21 May 2014 at 1:17