-
Notifications
You must be signed in to change notification settings - Fork 17
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
Pretty error (or warning) in not found include folder #17
Comments
I am strongly against making this a warning instead if an error. Because in at least 90% of all cases the include directory is essential for further processing. |
What you are describing is a problem of "missing include", not a problem of "inexisting includefolder". Above error gets thrown when you set a folder that doesn't exist as includefolder (argument -i), it's not even an error in includes themselves, it's a hard crash because armake2 can't canonicalize an inexisting folder path. But error or warning, it should be something, not a panic. On your "fail fast", that's not always favorable, especially when you want to see all errors/warnings to fix them faster without constant rebuilding. Not really applicable here, but generally to armake2 where possible, which is not in a lot of places because you'd just get a false chain. |
No I really was referring to the inexisting include-folder (which would then lead to a missing include). Actually I was saying that if the non-existent folder wasn't treated as an error but a warning instead, armake would very likely throw an error because of a missing include afterwards. And because one tends to look at errors only if the occur, one could oversee the actual root of the problem.
This is certainly correct.
Agreed. But in a chain of interdependent processes it should be applied. Because an error in the beginning of the chain will most likely create "false-errors" further down the chain as it has to deal with invalid states. |
Agreed. Maybe unrelated but something that would help HEMTT a great deal: return code separate into "PBO specific error" (syntax error, missing include, etc.) and "global error" (invalid argument like missing include folder, no space on disk etc). Would allow for better user experience as HEMTT could easily determine if it should continue building further PBOs or stop. |
Right now, armake2 just prints the following when given an in-existing include folder (
./inc
does not exist in this example):First, actual missing include folder name is not printed.
Second, ideally this would just be a warning and folder gets skipped. Maybe in main to not constantly print it, but then programs using armake2 as library would have to implement same checks before passing it to eg.
cmd_build
.The text was updated successfully, but these errors were encountered: