On 04/11/10 07:10 AM, [email protected] wrote: > Hi, > > The RDSv3 project will provide a new system header file, > /usr/include/sys/rds.h, as a RDMA interface described in rds(3SOCKET). > > Since this header file is created under usr/include/sys/ib/client/rdsv3 > during the compile time, we need to copy/move this file to usr/include/sys > when a RDSv3 IPS package is created.
Why would you not simply create the file in the correct place to start with? The only reason to put it in sys/ib/client/rdsv3 instead of sys should be if code in ON does "#include <sys/ib/client/rdsv3>," or uses a -I argument, and that sounds wrong. ON code that relies on delivered header files should use them as delivered. ? > For this purpose, as an experiment, I added the following line (the line > starting with "+") to system-header.mf, then the file could be copied > under usr/include/sys in the installation of the IPS package (system/header). > > (system-header.mf) > > file path=usr/include/sys/ib/clients/rdsv3/rds.h > + file usr/include/sys/ib/clients/rdsv3/rds.h path=usr/include/sys/rds.h > > (after the installation) > > # find /usr/include -name rds.h > /usr/include/sys/ib/clients/rdsv3/rds.h > /usr/include/sys/rds.h <--- > > Now I'm wondering if this way is acceptable; we might have another way > such as making a symbolic link to usr/include/sys/ib/clients/rdsv3/rds.h, > but so far there is no symbolic files under /usr/include/sys, so I'm > wondering if copying the file is fine, but then we have a duplicate file > (/usr/include/sys/ib/clients/rdsv3/rds.h and /usr/include/sys/rds.h are > the same) there. No, do not do it this way. This is a capability of pkgsend that we do not use in the ON build. You should create the proto area the same way that you intend to install the system. > Also the line "file path=usr/include/sys/ib/clients/rdsv3/rds.h" seems > yet needed because the following error is put during the nightly without it. > > ==== cpio archives build errors (DEBUG) ==== > > Failed to create generic usr archive: 519230 blocks > cpiotranslate: usr/include/sys/ib/clients/rdsv3/rds.h: no packaging info > > Please let me know your opinions/suggestions on this issue. > > Thanks, > > -Eiji > _______________________________________________ > on-ips-dev mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/on-ips-dev _______________________________________________ on-ips-dev mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/on-ips-dev
