Project

General

Profile

Actions
Copy link

Bug #497

closed

add -P flag to m4(1)

Added by pavalos over 17 years ago. Updated over 17 years ago.

Status:
Closed
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Here's a patch that will add the -P option to our m4. Please let me know
what you think. This is needed to bring in the latest flex sources.

http://www.theshell.com/~pavalos/wip/m4.patch

Thanks,
Peter

Files

Download all files
m4.patch (2.62 KB) m4.patch pavalos, 12/28/2006 07:09 AM
m4-2.patch (2.74 KB) m4-2.patch pavalos, 01/06/2007 07:30 PM
Actions
Copy link
 #1

Updated by dillon over 17 years ago

:Here's a patch that will add the -P option to our m4. Please let me know
:what you think. This is needed to bring in the latest flex sources.
:
:http://www.theshell.com/~pavalos/wip/m4.patch
:
:Thanks,
:Peter

Hmm.  Running the getopt loop twice is kinda a hack.  If you absolutely
    have to do it at least put getopt's control string in one place so it
    isn't duplicated.
-Matt
Actions
Copy link
 #2

Updated by joerg over 17 years ago

...and use optreset please.

Joerg

Actions
Copy link
 #3

Updated by dillon over 17 years ago

:We need to know if -P is set before we handle the -D and -U options.
:
:> ...and use optreset please.
:>=20
:
:Ok, how's this one look:
:
:http://www.theshell.com/~pavalos/wip/m4-2.patch
:
:--Peter

That looks much better.  I looked at some of the other OS dists
    and NetBSD does something similar.
-Matt
Actions
Copy link
 #4

Updated by pavalos over 17 years ago

We need to know if -P is set before we handle the -D and -U options.

Ok, how's this one look:

http://www.theshell.com/~pavalos/wip/m4-2.patch

--Peter

Actions
Copy link

Also available in: Atom PDF