-
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
Cyclic dependency checking broken between version 2015-11 to 2014.12 #212
Comments
@gdcarver This is puzzling. I have just done a diff between 2014.11.0 and 2014.12.0. I cannot see any change related to |
@gdcarver I am unable to repeat this problem in several simple test cases. |
@matthewrmshin I was also a bit puzzled as I looked at the changes doc and couldn't see anything that I thought might affect it. The tests are repeatable but I will check again to make sure. I'll work on reducing the code and send a test case in a day or two. |
@matthewrmshin Oddly, more testing show 2014-12 behaving correctly not but 2015 releases. But I am getting inconsistent results which might be related to the number of threads I'm using. |
This is the normal expectation. |
Then I have some odd behaviour that I do not understand at present. Installed multiple versions of FCM, I build in separate directories for each version (i.e. mode=new) and delete previous build dir to be sure. I use the same source dir and 1 thread for each test. In these tests, FCM should not be able to build the final executable but should correctly report a FAIL and a cyclic dependency and not appear to finish successfully. I get:
Another point to note is that if the FAIL is reported, it is not always the same dependency list between versions. So, for some versions, they may or may not correctly report the FAIL. I do not understand why. Maybe there is some caching or a race condition going on somewhere but I don't understand where with only 1 thread e.g. fcm make -j1. Have you ever seen anything like this before? I am working on reducing the code to send you a small test. |
Matt, apologies for not getting back to you about this yet. Creating the test case was not easy and then I had to work on something else. Will get back to this ASAP. |
@gdcarver any further progress in creating the test case? |
This relates to issue 151.
A cyclic dependency exists in some versions of the IFS code (see issue above). Earlier versions of FCM correctly catch this and report it (see report below). However, all versions from 2014.12 to 2015-11 fail to report the cyclic dependency, will appear to work with no fail messages but not produce executable.
I've found that the change happened between the releases 2014.11.0 and 2014.12.0.
Example output from 2014.11.0:
Example output from 2014.12.0:
The text was updated successfully, but these errors were encountered: