On 2007-04-10 13:45:07 +0200, Arjen Laarhoven wrote: > On Tue, Apr 10, 2007 at 11:48:01AM +0200, Karl Hasselström wrote: > > > On 2007-04-08 22:09:39 +0200, Arjen Laarhoven wrote: > > > > > ppc/{sha1ppc.S => sha1ppc.in.S} | 110 +++++++++++++++++++------------------- > > > > Wouldn't it be prettier if this filename was .S.in instead of > > .in.S? Additional file suffixes are usually added at the end (e.g. > > .tar.gz), and it makes more sense too. > > Using the .S suffix makes gcc automatically do the right thing. > .S.in requires an extra '-x assembler-with-cpp' option to gcc. Of > course, it's trivial fix. I just read the Makefile changes again, a bit slower this time, and noticed that you _first_ feed the .in.S file to gcc, and _then_ to the perl script, instead of the other way around like I was expecting. With that arrangement, your naming makes sense, since it reflects which file format is contained in which. Sorry for the noise. -- Karl Hasselström, kha@xxxxxxxxxxx www.treskal.com/kalle - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html