Re: PATCH: depcomp cpp method output
On Oct 17, 2000, Akim Demaille <[EMAIL PROTECTED]> wrote: > | 2000-06-21 Alex Hornby <[EMAIL PROTECTED]> > | > | * depcomp: Altered sed clause to strip of inclusion depth > | numbers generated by cpp. Also output in gcc style. > Fine with me. Any objection? Nope. -- Alexandre Oliva Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/ Red Hat GCC Developer aoliva@{cygnus.com, redhat.com} CS PhD student at IC-Unicampoliva@{lsd.ic.unicamp.br, gnu.org} Free Software Evangelist*Please* write to mailing lists, not to me
Re: PATCH: depcomp cpp method output
| 2000-06-21 Alex Hornby <[EMAIL PROTECTED]> | | * depcomp: Altered sed clause to strip of inclusion depth | numbers generated by cpp. Also output in gcc style. | | Index: depcomp | === | RCS file: /cvs/automake/automake/depcomp,v | retrieving revision 1.8 | diff -u -r1.8 depcomp | --- depcomp 2000/08/20 15:58:45 1.8 | +++ depcomp 2000/09/05 09:35:43 | @@ -253,7 +253,8 @@ |;; | esac | "$@" -E | | -sed -n '/^# [0-9][0-9]* "\([^"]*\)"/ s::'"$object"'\: \1:p' > "$tmpdepfile" | +sed -n '/^# [0-9][0-9]* "\([^"]*\)".*/ s:: \1 \\:p' | | +sed '$ s: \\$::' > "$tmpdepfile" |) & |proc=$! |"$@" | @@ -261,8 +262,9 @@ |wait "$proc" |if test "$stat" != 0; then exit $stat; fi |rm -f "$depfile" | - cat < "$tmpdepfile" > "$depfile" | - sed < "$tmpdepfile" -e 's/^[^:]*: //' -e 's/$/ :/' >> "$depfile" | + echo "$object : \\" > "$depfile" | + cat < "$tmpdepfile" >> "$depfile" | + sed < "$tmpdepfile" -e '/^$/d' -e 's/^ //' -e 's/ \\$//' -e 's/$/ :/' >> |"$depfile" |rm -f "$tmpdepfile" |;; Fine with me. Any objection? I'll change the last sed into sed < "$tmpdepfile" '/^$/d;s/^ //;s/ \\$//;s/$/ :/' >> "$depfile"
Re: PATCH: depcomp cpp method output
Tom Tromey writes: > > "Alex" == Alex Hornby <[EMAIL PROTECTED]> writes: > > Alex> Here is the first broken out part of my parallel built sources > Alex> patch, which I agreed to split up and resubmit back in the midst > Alex> of time. I hope you still have the time to review these. > > It is time for us to start writing test code for depcomp. > Do you have a test case for this? I'd like to add it to the automake > test suite before checking in this change. > > Tom Sorry, no test case, though I do use it daily. The nice thing about this change is that any such test case could now be done by comparing depmode=cpp against depmode=gcc output. I'm unlikely to have the time to do this at the moment though Alex.
Re: PATCH: depcomp cpp method output
> "Alex" == Alex Hornby <[EMAIL PROTECTED]> writes: Alex> Here is the first broken out part of my parallel built sources Alex> patch, which I agreed to split up and resubmit back in the midst Alex> of time. I hope you still have the time to review these. It is time for us to start writing test code for depcomp. Do you have a test case for this? I'd like to add it to the automake test suite before checking in this change. Tom