[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Pattern Specific Variable Limitation
From: |
Paul D. Smith |
Subject: |
Re: Pattern Specific Variable Limitation |
Date: |
Sun, 18 Mar 2001 22:35:13 -0500 |
%% "Matthew Von-Maszewski" <address@hidden> writes:
mv> I found a small, but in my case defeating, limitation with the
mv> Pattern Specific Variable code. The code finds the first match
mv> for a pattern, not necessarily the most specific match.
I'll have to think about this change. In all the other pattern rules in
make, the first appropriate match always wins; nowhere else in make
does it do any kind of "best fit" match such as you are proposing here.
mv> Please let me know if you believe this to be a properly formed
mv> patch.
Actually, not :). It _would_ help if you could provide a proper patch.
First, the order of arguments to diff is "diff <old> <new>"; you have
used "diff rule.c rul.org" which I interpret to be "diff <new> <old>",
so your diff is backwards (showing lines removed that were really added
and added that were really removed).
Also, any diff created for the purpose of patch should use either the -c
(context) or -u (unified) diff format; these forms include 3 lines of
unchanged text around the actual changes which allows patch to be much
more accurate about finding the right chunk of code to apply the changes
to. I have a preference for -u, others prefer -c; it doesn't really
matter.
Thanks.
--
-------------------------------------------------------------------------------
Paul D. Smith <address@hidden> Find some GNU make tips at:
http://www.gnu.org http://www.paulandlesley.org/gmake/
"Please remain calm...I may be mad, but I am a professional." --Mad Scientist