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
missing dependencies at byte-code link with mlpack #5752
Comments
Comment author: @bobzhang this patch also worked for my case |
Comment author: @lefessan I just add the patch that was proposed on the caml-list, I don't know if it correctly solves the problem (I don't use ocamlbuild...). Index: ocamlbuild/ocaml_compiler.ml--- ocamlbuild/ocaml_compiler.ml (revision 10605)
|
Comment author: @bobzhang which revision? |
Comment author: meyer It's been not committed, as I'm thinking of better way to do this. |
Comment author: @bobzhang meyer, do you have a solution now? |
Comment author: nlucaroni My group ran into this issue as well. Another work around is to mention the missing module in the mli file that defines the mlpack. We narrowed down the root module that was causing the issue in the mlpack through the depends files. Then added an unused function in the mli file containing a type from the missing module. |
Comment author: Matthieu Lemerre I also just ran into the issue. |
Comment author: meyer hongboz: Looking into this at the moment |
Comment author: meyer Eventually I applied the community patch in commit 13154 on trunk - because I didn't find better way. Could anybody confirm the problem is fixed? |
Comment author: nlucaroni Meyer, I'll try the patch at the end of the week. Thanks, I can understand why this solution may not be optimal --As I recall from looking at the patch, it applies to both native and byte code dependency resolution, even though the issue is only in bytecode. I had a compilation issue with .d.byte as well, but true:debug in the _tags file with byte compilation worked perfectly fine, so I'll follow up on that as well and open a new issue if it warrants it. Thanks, |
Comment author: nlucaroni I'm sorry Meyer, the patch may have fixed the issue in the case provided in the forum, but it doesn't fix the issue I've experienced. Maybe the expression of two different issues. I compiled OCaml with svn commit revision 13167. I get the same error about a missing dependency. I first compiled the .native (worked), then the .byte (failed), then .d.byte (failed, but differently). Let me know if there is anything I can do to help out further. I'll work on compiling the example on groups, and if that works, trying to define a small example from my source code. |
Comment author: nlucaroni Yes, the example at #4592 (with an additional _tags file that was missing), fails even after this patch. |
Comment author: meyer Nlucaroni, sorry to keep you waiting, ocamlbuild didn't have a proper testing infrastructure, now it has. Will have a second tour with this bug over this weekend. Thanks for the report. |
Comment author: meyer nlucaroni: Could you submit a testcase or an example that exposes the problem? Preferably in the form of patch for testsuite, please look at ocamlbuild/testsuite/level0.ml for an example. Thanks. |
Comment author: nlucaroni I mentioned the files linked in the duplicate report [0] also fails for me. Otherwise, it's the same thing as mentioned by Ashish [1], and Grégoire Sutre [2]. I only have a large example, and busy getting a release out shortly. 0 : #4592 1 : http://caml.inria.fr/pub/ml-archives/caml-list/2008/06/784c154d0f7b53995c0167ac63fc5bb9.en.html |
Comment author: meyer 0/ Is a user problem, please see the comments Commit 13272 implements testing of both cases and the other one from mailing list. Nlucaroni, I'd appreciate if you could double check your use case, and see if the trunk version still fails, if not, please update us. I can't reproduce the bug in any of the use cases you have pointed out, so if it still fails, then when you have a little bit of time, please try to reduce it to a small test case (preferably as a patch to level0.ml) Thanks. |
Comment author: nlucaroni Thanks meyer. I double checked the comment in [0] and you are right, the issues are different. So I modified it with something along the issue we are having and I was able to recreate the issue, see attachment to this issue. It looks like ocaml 4.01.0+dev10-2012-10-16 builds the test case I uploaded properly. This must be the patch you submitted above? but not my larger project. I'll try to minimize it some other time. |
Comment author: @damiendoligez I confirm the test case "ex.zip" builds correctly on 4.02.1, so we're now waiting for a new repro case. |
Comment author: @xavierleroy In the absence of new information, can we consider this issue fixed in 4.02.1? |
Comment author: @damiendoligez ocamlbuild is now a separate project that lives on GitHub. |
Original bug ID: 5752
Reporter: @bobzhang
Status: resolved (set by @damiendoligez on 2017-02-27T12:47:35Z)
Resolution: suspended
Priority: normal
Severity: minor
Target version: later
Category: -for ocamlbuild use https://github.com/ocaml/ocamlbuild/issues
Duplicate of: #4592
Related to: #6326
Monitored by: @hcarty
Bug description
This problem was reported twice in caml list. I came across such a problem once again.
https://groups.google.com/forum/?fromgroups=#!searchin/fa.caml/byte$20code$20mlpack/fa.caml/c5Q8q6ZdvDw/nIFGpdVuAaUJ
Thanks
File attachments
The text was updated successfully, but these errors were encountered: