Pre-build steps are not checked for errors

cancel
Showing results for 
Show  only  | Search instead for 
Did you mean: 

Pre-build steps are not checked for errors

653 Views
stefanct
Contributor III

MCUXpresso allows for custom commands to be executed before (and also after) builds via the project settings "C/C++ Build/Settings/Build Steps". However, the exit value of the command is not checked but intentionally(?) ignored, e.g., with simply "false" the beginning of the build log is the following:

12:42:33 **** Incremental Build of configuration Release for project proj_name ****
make -r -j16 all
false
make[1]: [makefile:64: pre-build] Error 1 (ignored)


It then continues normally. I consider this a bug. At least the final/overall exit value of the command(s) should be checked. If a user actually wants to ignore that he/she can easily do so by chaining the commands, e.g., by appending "|| true".

I assume this applies to all architectures but just in case: This is a project for an RT1020.

0 Kudos
Reply
5 Replies

642 Views
AdrianOltean
NXP Employee
NXP Employee

Your observations are completely valid but the behavior is expected according to [1]. Unfortunately, it's not much we can do...

AdrianOltean_0-1718121158379.png

 

Thanks,

Adrian

[1] https://help.eclipse.org/latest/index.jsp?topic=%2Forg.eclipse.cdt.doc.user%2Freference%2Fcdt_u_prop...

0 Kudos
Reply

633 Views
stefanct
Contributor III

Unfortunately, it's not much we can do...

This is not true. You can file a bug upstream (https://bugs.eclipse.org/bugs/enter_bug.cgi), which is is the expected way to escalate such problems in (commercially used) open-source projects.

0 Kudos
Reply

621 Views
AdrianOltean
NXP Employee
NXP Employee
0 Kudos
Reply

121 Views
rshipman
Contributor V

This is a problem for me as well. Pre- (and post-) build script errors should be observed. I imagine this would be a quick and easy fix for the eclipse team. Please would you raise this with them, because a case coming from NXP is more likely to yield results.

The bug report from the link above is dated 2015 and eclipse has migrated to GitHub since then, so it is worth submitting a new request.

Many thanks.

0 Kudos
Reply

86 Views
AdrianOltean
NXP Employee
NXP Employee
I know this is frustrating but there's no special treatment for NXP, or any other individual user or contributor. We had quite a few requests for CDT or other Eclipse projects but no happy ending for all of them. Unfortunately, for this particular request, there's nothing we can easily do on our side to address it - we looked into this.

Regards,
Adrian
0 Kudos
Reply