IDL install makefile leaves dirty src/idl directory

[messer]

src/idl/Makefile

run this for a newly prepared idl file

install: $(EXT) lists
cp ext_$<.h ext_$<.cpp ../cpp/ext/
$(REPO_ADD) ../cpp/ext/ext_$&lt;.h ../cpp/ext/ext_$<.cpp
$(REPO_ADD) ../cpp/ext/profile/extprofile_$&lt;.h
cp $<.inc ../lib/system/
$(REPO_ADD) ../lib/system/$&lt;.inc
touch ../lib/system/builtin_symbols.cpp
cp test_ext_$<.h test_ext_$<.cpp ../test/
$(REPO_ADD) ../test/test_ext_$&lt;.h ../test/test_ext_$<.cpp

Recommend using move commands instead.

Example:

marcel@hiphop:~/hiphop/git/hiphop-php/src$ find . | grep -i marcel
./lib/system/marcel.inc
./idl/ext_marcel.h
./idl/ext_marcel.cpp
./idl/marcel.idl.php
./idl/test_ext_marcel.h
./idl/test_ext_marcel.cpp
./idl/marcel.inc
./test/test_ext_marcel.h
./test/test_ext_marcel.cpp
./cpp/ext/ext_marcel.h
./cpp/ext/ext_marcel.cpp
./cpp/ext/profile/extprofile_marcel.h

[jtarchie]

Sorry, checking something. Please ignore closing of bug.

[sgolemon]

So the short (and admittedly late) answer to this is that the IDL makefile is working as intended. Yes, it means that you have some git cleaning to do after creating a new extension, but the benefit of having ready-to-go skeletons balances that. If I've misunderstood your request, please feel free to reopen with more detail, I promise to get back to you more quickly than you had to wait this last time. :)