Fix part of #22 #30

Merged
merged 1 commit into from Apr 8, 2013

Conversation

Projects
None yet
3 participants
@vbmithr
Contributor

vbmithr commented Mar 21, 2013

Global command line flag for generating annot files

@mor1

This comment has been minimized.

Show comment Hide comment
@mor1

mor1 Mar 24, 2013

just added a patch to do this myself before noticing this :)

however, i blocked on getting the .annot files into the right place for emacs/typerex to pick them up -- they get generated in the build directory rather than the source directory it seems. any ideas? i've tried constructing and spawning an appropriate cp command line, but it doesn't seem to work (or possibly run) -- i suspect i don't understand ext/filepath.ml and absolute/relative paths well enough...

mor1 commented Mar 24, 2013

just added a patch to do this myself before noticing this :)

however, i blocked on getting the .annot files into the right place for emacs/typerex to pick them up -- they get generated in the build directory rather than the source directory it seems. any ideas? i've tried constructing and spawning an appropriate cp command line, but it doesn't seem to work (or possibly run) -- i suspect i don't understand ext/filepath.ml and absolute/relative paths well enough...

@vincenthz

This comment has been minimized.

Show comment Hide comment
@vincenthz

vincenthz Mar 25, 2013

Member

@mor1: is that a problem that they are not in the source directory ? i.e. can't the search path be specified ? I'm not up-to-date on what's the other system do, as i don't use annot.

Member

vincenthz commented Mar 25, 2013

@mor1: is that a problem that they are not in the source directory ? i.e. can't the search path be specified ? I'm not up-to-date on what's the other system do, as i don't use annot.

@mor1

This comment has been minimized.

Show comment Hide comment
@mor1

mor1 Mar 25, 2013

short answer: don't know. best to talk to thomas about that i think -- current beta of typrerex appears to allow a ROOT/.typerex file that can specify where to find .cmt files; but i couldn't get .cmt files it would accept to be generated, whereas with the current stable release, putting .annot file into the source directory definitely does work... (possibly this is a cul de sac and i should just manage manually until the next release of typerex...)

mor1 commented Mar 25, 2013

short answer: don't know. best to talk to thomas about that i think -- current beta of typrerex appears to allow a ROOT/.typerex file that can specify where to find .cmt files; but i couldn't get .cmt files it would accept to be generated, whereas with the current stable release, putting .annot file into the source directory definitely does work... (possibly this is a cul de sac and i should just manage manually until the next release of typerex...)

@vbmithr

This comment has been minimized.

Show comment Hide comment
@vbmithr

vbmithr Mar 25, 2013

Contributor

I looked at ocamlspot, it does not work well either when .annot/.cmt are not in the same directory as the sources. There is an ad-hoc hack for ocamlbuild as well as a config file to specify a directory to search, but I did not manage to get that working for now

Contributor

vbmithr commented Mar 25, 2013

I looked at ocamlspot, it does not work well either when .annot/.cmt are not in the same directory as the sources. There is an ad-hoc hack for ocamlbuild as well as a config file to specify a directory to search, but I did not manage to get that working for now

@vincenthz

This comment has been minimized.

Show comment Hide comment
@vincenthz

vincenthz Mar 28, 2013

Member

is there a way to execute a command in ocamlspot that would tell ocamlspot where to look for the annot files ?

for example we could have: obuild get annot:src/a.ml => /path/to/a.annot

Member

vincenthz commented Mar 28, 2013

is there a way to execute a command in ocamlspot that would tell ocamlspot where to look for the annot files ?

for example we could have: obuild get annot:src/a.ml => /path/to/a.annot

@vbmithr

This comment has been minimized.

Show comment Hide comment
@vbmithr

vbmithr Mar 28, 2013

Contributor

Yeah there should be, it is just not working well or not easy to make it work. But there is nothing obuild can do about it for sure :)

Contributor

vbmithr commented Mar 28, 2013

Yeah there should be, it is just not working well or not easy to make it work. But there is nothing obuild can do about it for sure :)

@vbmithr

This comment has been minimized.

Show comment Hide comment
@vbmithr

vbmithr Apr 4, 2013

Contributor

Please look if it is correct now.

Contributor

vbmithr commented Apr 4, 2013

Please look if it is correct now.

vincenthz added a commit that referenced this pull request Apr 8, 2013

@vincenthz vincenthz merged commit 11116ca into ocaml-obuild:master Apr 8, 2013

@vincenthz

This comment has been minimized.

Show comment Hide comment
@vincenthz

vincenthz Apr 8, 2013

Member

Thanks @vbmithr. For the location, we probably need to sort it out, but i guess it could be a different issue.

Member

vincenthz commented Apr 8, 2013

Thanks @vbmithr. For the location, we probably need to sort it out, but i guess it could be a different issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment