LADSoft / OrangeC

OrangeC Compiler And Tool Chain
http://ladsoft.tripod.com/orange_c_compiler.html
Other
291 stars 39 forks source link

What do you think about adding something like `osed`? #934

Closed ghost closed 1 year ago

ghost commented 1 year ago

You already have ogrep 😄

chuggafan commented 1 year ago

While grep is fundamentally a simple program to make in terms of "idea", "ed" (and therefore, sed) are much more complex editor programs, I don't support the idea of adding a fully featured sed.

GitMensch commented 1 year ago

Me neither, but I wouldn't complain if @iahunq provides a PR to add it.

ghost commented 1 year ago

GNU sed has a standalone build for Windows if anyone interested:

https://github.com/mbuilov/sed-windows

LADSoft commented 1 year ago

i haven't forgotten this... i may add gnu sed at some point but not sure yet how to make a .configure script for it so I can get all the sources I need. Tried using autoconf but it generates errors, and then the script it generates doesn't work...

I also think it may not quite be standalone; it seems to use various unix packages...

ghost commented 1 year ago

i haven't forgotten this... i may add gnu sed at some point but not sure yet how to make a .configure script for it so I can get all the sources I need. Tried using autoconf but it generates errors, and then the script it generates doesn't work...

I also think it may not quite be standalone; it seems to use various unix packages...

Why? Why do you need to add gnu sed to Orange C? There is already a standalone version of gnu sed that available for everyone!

https://github.com/mbuilov/sed-windows

Note: ogrep is your own implementation of grep then osed is your own implementation of sed, this is what I mean when I suggested you to add osed; and other people were right; there is no need for a homegrown implementation of sed; sed is more complex than grep and people can always use gnu sed if they have a need for sed.