{"action":"created","issue":{"url":"https://api.github.com/repos/hercules-390/hyperion/issues/165","repository_url":"https://api.github.com/repos/hercules-390/hyperion","labels_url":"https://api.github.com/repos/hercules-390/hyperion/issues/165/labels{/name}","comments_url":"https://api.github.com/repos/hercules-390/hyperion/issues/165/comments","events_url":"https://api.github.com/repos/hercules-390/hyperion/issues/165/events","html_url":"https://github.com/hercules-390/hyperion/issues/165","id":185699921,"number":165,"title":"Runtest.cmd (Windows) includes ostailor null command and runtest (Linux) does not.","user":{"login":"srorso","id":17149494,"avatar_url":"https://avatars.githubusercontent.com/u/17149494?v=3","gravatar_id":"","url":"https://api.github.com/users/srorso","html_url":"https://github.com/srorso","followers_url":"https://api.github.com/users/srorso/followers","following_url":"https://api.github.com/users/srorso/following{/other_user}","gists_url":"https://api.github.com/users/srorso/gists{/gist_id}","starred_url":"https://api.github.com/users/srorso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srorso/subscriptions","organizations_url":"https://api.github.com/users/srorso/orgs","repos_url":"https://api.github.com/users/srorso/repos","events_url":"https://api.github.com/users/srorso/events{/privacy}","received_events_url":"https://api.github.com/users/srorso/received_events","type":"User","site_admin":false},"labels":[{"id":69570415,"url":"https://api.github.com/repos/hercules-390/hyperion/labels/BUG","name":"BUG","color":"FF3300","default":false},{"id":193435220,"url":"https://api.github.com/repos/hercules-390/hyperion/labels/Fix%20being%20tested...","name":"Fix being tested...","color":"22DA5C","default":false},{"id":123868782,"url":"https://api.github.com/repos/hercules-390/hyperion/labels/In%20progress...","name":"In progress...","color":"FF9933","default":false}],"state":"open","locked":false,"assignee":{"login":"Fish-Git","id":1486786,"avatar_url":"https://avatars.githubusercontent.com/u/1486786?v=3","gravatar_id":"","url":"https://api.github.com/users/Fish-Git","html_url":"https://github.com/Fish-Git","followers_url":"https://api.github.com/users/Fish-Git/followers","following_url":"https://api.github.com/users/Fish-Git/following{/other_user}","gists_url":"https://api.github.com/users/Fish-Git/gists{/gist_id}","starred_url":"https://api.github.com/users/Fish-Git/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fish-Git/subscriptions","organizations_url":"https://api.github.com/users/Fish-Git/orgs","repos_url":"https://api.github.com/users/Fish-Git/repos","events_url":"https://api.github.com/users/Fish-Git/events{/privacy}","received_events_url":"https://api.github.com/users/Fish-Git/received_events","type":"User","site_admin":false},"assignees":[{"login":"Fish-Git","id":1486786,"avatar_url":"https://avatars.githubusercontent.com/u/1486786?v=3","gravatar_id":"","url":"https://api.github.com/users/Fish-Git","html_url":"https://github.com/Fish-Git","followers_url":"https://api.github.com/users/Fish-Git/followers","following_url":"https://api.github.com/users/Fish-Git/following{/other_user}","gists_url":"https://api.github.com/users/Fish-Git/gists{/gist_id}","starred_url":"https://api.github.com/users/Fish-Git/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fish-Git/subscriptions","organizations_url":"https://api.github.com/users/Fish-Git/orgs","repos_url":"https://api.github.com/users/Fish-Git/repos","events_url":"https://api.github.com/users/Fish-Git/events{/privacy}","received_events_url":"https://api.github.com/users/Fish-Git/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2016-10-27T15:21:12Z","updated_at":"2016-11-18T03:43:23Z","closed_at":null,"body":"The Windows version of runtest includes an `ostailor null` command between each test case, and the linux shell script does not.  When a test case uses `ostailor quiet`, subsequent test cases that depend on program check detection via `*Program` report failed compares under *nix.  The same test cases pass on Windows.  \n\nIn the current test case stream, six cases fail for this reason when the BFP test cases are included.  The test cases are:\n\n```\npfpo-esa     \nssk370         (2 of 2 cases)\nsske           (1 of 2 cases in the file)\nsske370        (1 of 12 cases in the  file)\nsske390        (1 of 12 cases in the file)\n```\n\nBFP test cases include `ostailor quiet` because they intentionally create program checks to verify floating point results when traps are enabled.  There is no value to including the program check event in the alltests.out log.  \n\nWhile this issue was discovered on the Hyperion that included Softfloat mods, it was reproduced on an unmodified Hyperion by excluding ostailor null from runtest.cmd and including ostailor quiet in a test case near the beginning.  \n\nI propose that the runtest shell script be modified to include `ostailor` null between each test case.  Likewise for `numcpu 1` and `mainsize 2m`.\n\nAlternatively, we could remove the `ostailor null` _etc._ between test cases from runtest.cmd and create the expectation that test case developers include ostailor null at the end of the test case should they alter the ostailor setting in the case.  But my preference is the addition of ostailor null to the shell script.  \n\nAnd that's the core of the issue.  How should these two scripts be made consistent with respect to their processing of test cases?\n"},"comment":{"url":"https://api.github.com/repos/hercules-390/hyperion/issues/comments/261441671","html_url":"https://github.com/hercules-390/hyperion/issues/165#issuecomment-261441671","issue_url":"https://api.github.com/repos/hercules-390/hyperion/issues/165","id":261441671,"user":{"login":"Fish-Git","id":1486786,"avatar_url":"https://avatars.githubusercontent.com/u/1486786?v=3","gravatar_id":"","url":"https://api.github.com/users/Fish-Git","html_url":"https://github.com/Fish-Git","followers_url":"https://api.github.com/users/Fish-Git/followers","following_url":"https://api.github.com/users/Fish-Git/following{/other_user}","gists_url":"https://api.github.com/users/Fish-Git/gists{/gist_id}","starred_url":"https://api.github.com/users/Fish-Git/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fish-Git/subscriptions","organizations_url":"https://api.github.com/users/Fish-Git/orgs","repos_url":"https://api.github.com/users/Fish-Git/repos","events_url":"https://api.github.com/users/Fish-Git/events{/privacy}","received_events_url":"https://api.github.com/users/Fish-Git/received_events","type":"User","site_admin":false},"created_at":"2016-11-18T03:43:23Z","updated_at":"2016-11-18T03:43:23Z","body":"> I'll see if I can tweak the Linux runtest script to cat each .tst test file individually so we can insert the needed statements just like the Windows version is already doing.\r\n\r\nDone and tested. Works fine. Will commit and close this issue."}}
{"actor":{"display_login":"Fish-Git"}}