Re: CVS commit: src/usr.bin/sed

2021-03-30 Thread Christos Zoulas
In article ,
John Klos   wrote:
>Log Message:
>Use the same options like m4 (-g turns on GNU, -G turns off GNU) Suggested
>by uwe@
>
>It seems, based on comparing netbsd-9 and -current, that the default was 
>-g (GNU on), and is now -G (GNU off). Should the default be mentioned in 
>the man page?

NetBSD-9 did not support gnu extensions, but it silently ignored
them (e.g. \w became w). NetBSD-current supports them but they are
off by default; it also warns about \ escapes it does not recognize.

All the packages that warn now and did not warn before, were just not
working properly before.

christos



Re: CVS commit: src/usr.bin/sed

2021-03-30 Thread John Klos

Log Message:
Use the same options like m4 (-g turns on GNU, -G turns off GNU) Suggested
by uwe@

It seems, based on comparing netbsd-9 and -current, that the default was 
-g (GNU on), and is now -G (GNU off). Should the default be mentioned in 
the man page?


John


Re: CVS commit: src/usr.bin/sed

2021-03-11 Thread Christos Zoulas
In article ,
Valery Ushakov   wrote:
>On Thu, Mar 11, 2021 at 10:15:05 -0500, Christos Zoulas wrote:
>
>> Module Name: src
>> Committed By:christos
>> Date:Thu Mar 11 15:15:05 UTC 2021
>> 
>> Modified Files:
>>  src/usr.bin/sed: main.c sed.1
>> 
>> Log Message:
>> Add -G to support GNU extensions
>
>This is inverse of m4(1) where -g/--gnu enables GNU extensions and
>-G/--traditional disables them.  Can we be consistent about that?  Do
>we have other programs with gnu-compat mode?

I have made it the same as m4. Looking for more programs with gnu extensions.

christos



Re: CVS commit: src/usr.bin/sed

2021-03-11 Thread Valery Ushakov
On Thu, Mar 11, 2021 at 10:15:05 -0500, Christos Zoulas wrote:

> Module Name:  src
> Committed By: christos
> Date: Thu Mar 11 15:15:05 UTC 2021
> 
> Modified Files:
>   src/usr.bin/sed: main.c sed.1
> 
> Log Message:
> Add -G to support GNU extensions

This is inverse of m4(1) where -g/--gnu enables GNU extensions and
-G/--traditional disables them.  Can we be consistent about that?  Do
we have other programs with gnu-compat mode?

-uwe


Re: CVS commit: src/usr.bin/sed

2015-03-01 Thread Warner Losh

 On Mar 1, 2015, at 8:45 AM, Aleksej Saushev a...@inbox.ru wrote:
 
 chris...@astron.com (Christos Zoulas) writes:
 
 In article 20150228215653.f2f0...@cvs.netbsd.org,
 Aleksej Saushev source-changes-d@NetBSD.org wrote:
 -=-=-=-=-=-
 
 Module Name:src
 Committed By:   asau
 Date:   Sat Feb 28 21:56:53 UTC 2015
 
 Modified Files:
 src/usr.bin/sed: compile.c extern.h main.c process.c
 
 Log Message:
 Improve modularity of sed source:
 - move program source input subroutines into compiler part;
 - move data I/O subroutines into processor part.
 
 It there a good reason for that? We try to keep it synced with FreeBSD..
 
 It makes code a lot cleaner and allows embedding it.
 
 If that's problem, I'll submit changes to FreeBSD.

Sounds like a good idea to me…

Warner


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: CVS commit: src/usr.bin/sed

2015-03-01 Thread Aleksej Saushev
chris...@astron.com (Christos Zoulas) writes:

 In article 20150228215653.f2f0...@cvs.netbsd.org,
 Aleksej Saushev source-changes-d@NetBSD.org wrote:
-=-=-=-=-=-

Module Name:  src
Committed By: asau
Date: Sat Feb 28 21:56:53 UTC 2015

Modified Files:
  src/usr.bin/sed: compile.c extern.h main.c process.c

Log Message:
Improve modularity of sed source:
- move program source input subroutines into compiler part;
- move data I/O subroutines into processor part.

 It there a good reason for that? We try to keep it synced with FreeBSD..

It makes code a lot cleaner and allows embedding it.

If that's problem, I'll submit changes to FreeBSD.


-- 
HE CE3OH...



Re: CVS commit: src/usr.bin/sed

2015-02-28 Thread Christos Zoulas
In article 20150228215653.f2f0...@cvs.netbsd.org,
Aleksej Saushev source-changes-d@NetBSD.org wrote:
-=-=-=-=-=-

Module Name:   src
Committed By:  asau
Date:  Sat Feb 28 21:56:53 UTC 2015

Modified Files:
   src/usr.bin/sed: compile.c extern.h main.c process.c

Log Message:
Improve modularity of sed source:
- move program source input subroutines into compiler part;
- move data I/O subroutines into processor part.

It there a good reason for that? We try to keep it synced with FreeBSD..

christos



Re: CVS commit: src/usr.bin/sed

2014-06-07 Thread Ryo ONODERA
Hi,

This change breaks many pkgsrc packages.
Please take a look at PR bin/48880.

Thank you.

From: Christos Zoulas chris...@netbsd.org, Date: Thu, 5 Jun 2014 20:13:13 
-0400

 Module Name:  src
 Committed By: christos
 Date: Fri Jun  6 00:13:13 UTC 2014
 
 Modified Files:
   src/usr.bin/sed: Makefile POSIX compile.c defs.h extern.h main.c misc.c
   process.c sed.1
 
 Log Message:
 Merge our changes.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.14 -r1.15 src/usr.bin/sed/Makefile
 cvs rdiff -u -r1.4 -r1.5 src/usr.bin/sed/POSIX
 cvs rdiff -u -r1.40 -r1.41 src/usr.bin/sed/compile.c
 cvs rdiff -u -r1.10 -r1.11 src/usr.bin/sed/defs.h
 cvs rdiff -u -r1.11 -r1.12 src/usr.bin/sed/extern.h src/usr.bin/sed/misc.c
 cvs rdiff -u -r1.21 -r1.22 src/usr.bin/sed/main.c
 cvs rdiff -u -r1.39 -r1.40 src/usr.bin/sed/process.c
 cvs rdiff -u -r1.32 -r1.33 src/usr.bin/sed/sed.1
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.
 

--
Ryo ONODERA // ryo...@yk.rim.or.jp
PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3