{"action":"created","issue":{"url":"https://api.github.com/repos/janestreet/jbuilder/issues/237","repository_url":"https://api.github.com/repos/janestreet/jbuilder","labels_url":"https://api.github.com/repos/janestreet/jbuilder/issues/237/labels{/name}","comments_url":"https://api.github.com/repos/janestreet/jbuilder/issues/237/comments","events_url":"https://api.github.com/repos/janestreet/jbuilder/issues/237/events","html_url":"https://github.com/janestreet/jbuilder/issues/237","id":253025757,"number":237,"title":"JBuilder sometimes does not recompile executables","user":{"login":"smondet","id":617111,"avatar_url":"https://avatars3.githubusercontent.com/u/617111?v=4","gravatar_id":"","url":"https://api.github.com/users/smondet","html_url":"https://github.com/smondet","followers_url":"https://api.github.com/users/smondet/followers","following_url":"https://api.github.com/users/smondet/following{/other_user}","gists_url":"https://api.github.com/users/smondet/gists{/gist_id}","starred_url":"https://api.github.com/users/smondet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smondet/subscriptions","organizations_url":"https://api.github.com/users/smondet/orgs","repos_url":"https://api.github.com/users/smondet/repos","events_url":"https://api.github.com/users/smondet/events{/privacy}","received_events_url":"https://api.github.com/users/smondet/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2017-08-25T21:47:27Z","updated_at":"2017-08-29T17:55:31Z","closed_at":null,"body":"\r\nI have a very simple one-ocaml-file → one-executable project\r\n\r\n```scheme\r\n(jbuild_version 1)\r\n\r\n(executable\r\n ((name vidimetro)\r\n  (libraries (lablgl.glut nonstd sosa cmdliner misuja))))\r\n\r\n(install ((section bin) (files ((vidimetro.exe as vidimetro)))))\r\n```\r\n\r\nI've noticed that some small but functional changes to `vidimetro.ml` do not trigger relinking of the executable:\r\n\r\n```\r\n $ jbuilder --version\r\n1.0+beta12\r\n $ jbuilder build --dev @install\r\n    ocamldep vidimetro.depends.ocamldep-output\r\n      ocamlc vidimetro.{cmi,cmo,cmt}\r\n    ocamlopt vidimetro.{cmx,o}\r\n```\r\n\r\nit does not rebuild `vidimetro.exe`.\r\n\r\nIf I `clean` I get the right `.exe`:\r\n\r\n```\r\n $ jbuilder clean\r\n $ jbuilder build --dev @install\r\n    ocamldep vidimetro.depends.ocamldep-output\r\n      ocamlc vidimetro.{cmi,cmo,cmt}\r\n    ocamlopt vidimetro.{cmx,o}\r\n    ocamlopt vidimetro.exe\r\n```\r\n\r\n(I doubt that it is a file time-stamps problem, I'm on a standard `ext4` file-system)\r\n\r\n(the code is in this branch: https://gitlab.com/smondet/vidimetro/commits/packaging-improvements)\r\n\r\nPS: maybe (?) we had similar issues in solvuu-build, need to depend on `.cmx` **and** `.o` files\r\nhttps://github.com/solvuu/solvuu-build/issues/65\r\n"},"comment":{"url":"https://api.github.com/repos/janestreet/jbuilder/issues/comments/325743765","html_url":"https://github.com/janestreet/jbuilder/issues/237#issuecomment-325743765","issue_url":"https://api.github.com/repos/janestreet/jbuilder/issues/237","id":325743765,"user":{"login":"copy","id":1816456,"avatar_url":"https://avatars3.githubusercontent.com/u/1816456?v=4","gravatar_id":"","url":"https://api.github.com/users/copy","html_url":"https://github.com/copy","followers_url":"https://api.github.com/users/copy/followers","following_url":"https://api.github.com/users/copy/following{/other_user}","gists_url":"https://api.github.com/users/copy/gists{/gist_id}","starred_url":"https://api.github.com/users/copy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/copy/subscriptions","organizations_url":"https://api.github.com/users/copy/orgs","repos_url":"https://api.github.com/users/copy/repos","events_url":"https://api.github.com/users/copy/events{/privacy}","received_events_url":"https://api.github.com/users/copy/received_events","type":"User","site_admin":false},"created_at":"2017-08-29T17:55:31Z","updated_at":"2017-08-29T17:55:31Z","body":"In case someone runs into the same issue as I did: `ppx_expect` failures of the kind `File … changed, you need rebuild inline_test_runner to be able to run expect tests` are also fixed by this."}}
{"actor":{"display_login":"copy"}}