IUCompilerCourse / Essentials-of-Compilation

A book about compiling Racket and Python to x86-64 assembly
1.28k stars 140 forks source link

Suggestion for readable `make clean` command #91

Closed stefanos82 closed 2 years ago

stefanos82 commented 2 years ago

Currently we have the following command

clean:
    $(LATEXMK) -C book
    rm -f book.log book.aux book.bbl book.lof book.out book.toc book.blg book.pdf book.ilg book.ind book.lot book.run.xml book.bcf book.fls book.fdb_latexmk authors.idx authors.ilg authors.ind subject.idx subject.ilg subject.ind

which is massive.

Is it OK with you if we could reduce it to

clean:
    $(LATEXMK) -C book
    $(RM) \
        *.idx *.ilg *.ind *.aux *.bbl *.bcf \
        *.blg *.fdb_latexmk *.fls *.lof *.log \
        *.lot *.out *.run.xml *.toc book.pdf

which looks cleaner and much more tight?

jsiek commented 2 years ago

Done.

BTW, for things like this, if you do a pull request, I'll be able to process it sooner and faster.

stefanos82 commented 2 years ago

Traumatic experience from recent past forced me to reduce my PR activities as I was considered an annoying person for caring too much and being overzealous with knowledge.

I hope you understand my reasoning.