I agree with ruilong. Make clean should clean files generated only in make process.
Make distclean should clean files generated in make and configure process. xunzhang Send from my iPhone 发自我的 iPhone > 在 2016年7月12日，11:40，Ruilong Huo <[email protected]> 写道： > > I think "make clean" should only clean the files generated during compile > process. > The *.out files are created by feature test code with psql, so I would > prefer keep them. > If you would like to clean them, that should be in feature test framework, > not in makefile. > > Any comments? > > Best regards, > Ruilong Huo > >> On Tue, Jul 12, 2016 at 11:37 AM, Xiang Sheng <[email protected]> wrote: >> >> We indeed should delete these useless files after runing make clean or make >> distclean. >> >>> On Tue, Jul 12, 2016 at 11:35 AM, Ming Li <[email protected]> wrote: >>> >>> LGTM. +1. >>> >>>> On Tue, Jul 12, 2016 at 11:32 AM, Paul Guo <[email protected]> wrote: >>>> >>>> I'd mask the output files after running feature tests in .gitignore and >>>> clean it up after running "make clean" or "make distclean". Anyone has >>> any >>>> suggestions? Thanks. >>>> >>>> diff --git a/src/test/feature/.gitignore b/src/test/feature/.gitignore >>>> index a2e6bd4..c7332b2 100644 >>>> --- a/src/test/feature/.gitignore >>>> +++ b/src/test/feature/.gitignore >>>> @@ -1 +1,2 @@ >>>> doc/ >>>> +**/*.out >>>> >>>> diff --git a/src/test/feature/Makefile b/src/test/feature/Makefile >>>> index adc6acc..e0985d1 100644 >>>> --- a/src/test/feature/Makefile >>>> +++ b/src/test/feature/Makefile >>>> @@ -35,6 +35,7 @@ doc: >>>> doxygen doxygen_template >>>> >>>> clean distclean: sharelibclean >>>> + find . -type f -name "*.out" |xargs rm -f >>>> $(RM) feature-test >>>> $(RM) feature-test.dSYM >> >> >> >> -- >> Best Regards, >> Xiang Sheng >>
