Skip to content
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

Install .cmt[i] files. #6691

Closed
vicuna opened this issue Dec 4, 2014 · 7 comments
Closed

Install .cmt[i] files. #6691

vicuna opened this issue Dec 4, 2014 · 7 comments

Comments

@vicuna
Copy link

vicuna commented Dec 4, 2014

Original bug ID: 6691
Reporter: @Drup
Status: closed (set by @damiendoligez on 2015-07-20T14:11:00Z)
Resolution: fixed
Priority: normal
Severity: minor
Target version: 4.02.3+dev
Fixed in version: 4.02.3+dev
Category: configure and build/install
Tags: junior_job
Monitored by: @Drup @hcarty @dbuenzli @yakobowski

Bug description

It would be nice to install .cmt[i] files for the stdlib and compiler-libs by default (they seem to be generated anyway). It would help the various tools to browse the documentation.

File attachments

@vicuna
Copy link
Author

vicuna commented Dec 5, 2014

Comment author: @gasche

I think that's a good idea.

@vicuna
Copy link
Author

vicuna commented Jan 13, 2015

Comment author: @damiendoligez

+1

@vicuna
Copy link
Author

vicuna commented Jan 15, 2015

Comment author: dsheets

Patch attached. See also #136

@vicuna
Copy link
Author

vicuna commented Jan 16, 2015

Comment author: dsheets

New patch, install_cmti_with_nt.patch, with Makefile.nt edited as well uploaded as suggested by doligez on GitHub.

I can't test the Makefile.nt edition but I'm reasonably certain it does the same thing after careful review.

@vicuna
Copy link
Author

vicuna commented Jan 24, 2015

Comment author: @gasche

Merged in trunk, thanks!

@vicuna
Copy link
Author

vicuna commented Jul 17, 2015

Comment author: @damiendoligez

Reopening to include in 4.02.3.

@vicuna
Copy link
Author

vicuna commented Jul 20, 2015

Comment author: @damiendoligez

merged in 4.02 (rev 16223)

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

No branches or pull requests

1 participant