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

Makefile improvements #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

iam28th
Copy link

@iam28th iam28th commented Mar 23, 2024

Greetings!
There are some minor additions I'd like to make to the building process; I know that CoLoRd is currently not under active development, but hopefully it will still be of some help (I briefly checked, and it looks like similar changes -- if you like them:) -- can be implemented in other repositories).

PR consists of 2 commits:

  1. The first one cleans up some missed files.
(a spoiler with the picture to see what files these are; .gitignore was temporarily deleted to make them visible)

image

  1. The second is more of a convenience to simplify development workflow. It allows make to automatically detect when a .cpp files includes a recently modified header, and rebuild the corresponding object.
    Previously changes in .h files were not picked up, which could lead to unexpected stuff when objects contained outdated logic; and the only solution to such problem would be to make clean and recompile everything anew every time.
    Now such situations will not happen when editing files in src/colord.

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.

None yet

1 participant