Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#251 closed defect (fixed)

Header files not completely installed

Reported by: Dimitar Misev Owned by: Kinga Lipskoch
Priority: major Milestone: 8.4
Component: build system Version: 8.3
Keywords: Cc:
Complexity: Medium

Description

make install doesn't seem to completely install all needed rasdaman headers:

$ ls $RMANHOME/include
basictypes.hh  bool.h  globals.hh  rasdaman.hh  stdexcept.h

Attachments (1)

includes.txt (2.7 KB ) - added by Dimitar Misev 11 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by abeccati, 11 years ago

Milestone: 8.4
Owner: changed from Dimitar Misev to Kinga Lipskoch
Status: newassigned

comment:2 by Dimitar Misev, 11 years ago

It seems fine now, only config.h does not get installed:

$ ls include/
basictypes.hh  bool.h  clientcomm  commline  conversion  debug  globals.hh  rasdaman.hh  raslib  rasodmg  stdexcept.h

comment:3 by Kinga Lipskoch, 11 years ago

I just installed rasdaman (until step make install) and I got same results you reported before, i.e., the 5 header files are there, but the directories are not.

comment:4 by Dimitar Misev, 11 years ago

Strange, maybe it depends on the OS. I'm on Debian now, but I think I had that error on Ubuntu or similar. Anyway, at least we know what's missing, and it's reproducible.

Attached is the full list of files that should be installed, just in case (plus config.h should be at top-level).

by Dimitar Misev, 11 years ago

Attachment: includes.txt added

in reply to:  4 comment:5 by Kinga Lipskoch, 11 years ago

Replying to dmisev:

Strange, maybe it depends on the OS. I'm on Debian now, but I think I had that error on Ubuntu or similar. Anyway, at least we know what's missing, and it's reproducible.

Attached is the full list of files that should be installed, just in case (plus config.h should be at top-level).

Ok, so maybe it's the OS. I am using Ubuntu now.
Thanks for the list. This means that the directories and their files are missing.

comment:6 by Dimitar Misev, 11 years ago

Yes, most likely something to be fixed in the source:Makefile.am
If you check it out you'll see

nobase_include_HEADERS=raslib/*.h raslib/*.hh raslib/*.icc raslib/*.cc \
    rasodmg/*.hh rasodmg/*.cc rasodmg/*.icc conversion/*.h conversion/*.hh \
    clientcomm/*.h clientcomm/*.hh commline/*.hh commline/*.cc debug/*.hh

but as it seems it doesn't work on Ubuntu.

This may be helpful: http://www.gnu.org/software/automake/manual/html_node/Headers.html

comment:7 by Kinga Lipskoch, 11 years ago

I take back my previous comments.
All header files except config.h were copied (on both Debian and Ubuntu).

comment:8 by Dimitar Misev, 11 years ago

Ok, then just config.h remains to be added to the Makefile.am

comment:9 by Kinga Lipskoch, 11 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.