Re: PATCH: depcomp cpp method output

[Akim Demaille]

| 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

[Alexandre Oliva]

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

[Tom Tromey]

 "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