Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

replaced very complex Makefile with a much simpler one #16

Merged
merged 1 commit into from
Jan 7, 2016
Merged

replaced very complex Makefile with a much simpler one #16

merged 1 commit into from
Jan 7, 2016

Conversation

mgkuhn
Copy link
Contributor

@mgkuhn mgkuhn commented Jan 7, 2016

I found the previous Makefile both insanely complex (over 100 kB!), unmaintainable, and it also didn't work properly (as was evident from the need to have additional makepdf and makeps wrappers to build the glossary).

I have, therefore, deleted the existing Makefile, and replaced it with a better, very simple Makefile, one that any student of my CST Part IB "Unix Tools" course should be able to fully understand in less than 5 minutes. I believe my new Makefile is a much better starting point.

I also moved the CUni.* files into the same subdirectory as the other logos, both for consistency, and such that "make clean" can "rm *.eps".

also moved CUni.* to subdirectory, such that "make clean" can "rm *.eps"
@@ -172,8 +142,7 @@ The Computer Laboratory guidelines for technical reports:

## _Q3_: Can I use my own Makefile?

By all means. We are currently using the very nice (and smart) `Makefile` built
specifically for LaTeX:
By all means. Previously we used the horrendously complex `Makefile` at
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha 😄

Yes, it was amazingly complex.

I totally agree, your Makefile is much more readable and maintainable.

@urbas
Copy link
Member

urbas commented Jan 7, 2016

This is a much needed cleanup.

The world is now a noticeably better place without that big Makefile.

Thank you very much @mgkuhn!

urbas added a commit that referenced this pull request Jan 7, 2016
replaced very complex Makefile with a much simpler one
@urbas urbas merged commit 6aa2efe into cambridge:master Jan 7, 2016
@mgkuhn mgkuhn deleted the makefile branch April 28, 2017 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants