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

Makefiles missing header file dependencies #48

Closed
wjhun opened this issue Aug 24, 2018 · 7 comments
Closed

Makefiles missing header file dependencies #48

wjhun opened this issue Aug 24, 2018 · 7 comments
Assignees
Labels

Comments

@wjhun
Copy link
Contributor

wjhun commented Aug 24, 2018

I ran into a few out-of-sync builds from objects not being updated after header file changes.

Of possible interest: http://make.mad-scientist.net/papers/advanced-auto-dependency-generation/

@talshadow
Copy link
Contributor

I propose resolve this issue with cmake.
Also i can provide all headers dependencies by build targets

@wjhun
Copy link
Contributor Author

wjhun commented Sep 3, 2018

Hi Andrii. cmake sounds interesting, but could we perhaps solve this in a simpler manner before moving to a whole new build system?

The simplest solution I've seen is at this link, under the headline "TL;DR: The GCC Solution":

http://make.mad-scientist.net/papers/advanced-auto-dependency-generation/#tldr

@talshadow
Copy link
Contributor

I do not insist on using cmake. Just with this build system is easier to work as for me.

@wjhun
Copy link
Contributor Author

wjhun commented Sep 3, 2018

It's something to consider later as an alternative to automake / autoconf, especially to support different build environments. For the time being, I think just getting the existing build working seamlessly is a good goal. Your work in removing "-include"s should help with this as well.

Did you find that link above helpful? It's not that pretty, but it should work.

@talshadow
Copy link
Contributor

Your link looks as solution.

About automake / autoconf... By my point of view this is an old rubbish which has more problems than pluses and is very hard in support . More progressive for now is CMake. As alternative looks good qbs.

@convolvatron
Copy link
Contributor

another vote against autotools.

@mkhon
Copy link
Contributor

mkhon commented Mar 29, 2019

Fixed with #662

@mkhon mkhon closed this as completed Mar 29, 2019
@mkhon mkhon self-assigned this Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants