Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#251 closed defect (fixed)

Header files not completely installed

Reported by: dmisev Owned by: klipskoch
Priority: major Milestone: 8.4
Component: build system Version: 8.3
Keywords: Cc:
Complexity:

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 dmisev 4 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 4 years ago by abeccati

  • Milestone set to 8.4
  • Owner changed from dmisev to klipskoch
  • Status changed from new to assigned

comment:2 Changed 4 years ago by dmisev

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 Changed 4 years ago by klipskoch

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 follow-up: Changed 4 years ago by 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).

Changed 4 years ago by dmisev

comment:5 in reply to: ↑ 4 Changed 4 years ago by klipskoch

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 Changed 4 years ago by dmisev

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 Changed 4 years ago by klipskoch

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

comment:8 Changed 4 years ago by dmisev

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

comment:9 Changed 4 years ago by klipskoch

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.