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

Fixed target clean #6

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

Fixed target clean #6

wants to merge 2 commits into from

Conversation

jorbakk
Copy link
Contributor

@jorbakk jorbakk commented Oct 29, 2024

Also remove main.exe when building for windows (native or cross-compile).

@tekknolagi
Copy link
Owner

Use $(TARGET), no? Or is there a reason you remove both separately?

@jorbakk
Copy link
Contributor Author

jorbakk commented Oct 29, 2024

The reason is just that the makefile is simpler and that you don't have to care about which target you build in order to clean up (less error prone) ... it just cleans up. However, if you don't like it, I can make a target-specific clean up.

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