> > I have no idea where the correct location would be, so anything you > > come with is fine with me. > > Perhaps indeed /usr/include/i2c/linux/i2c? > > And I2C_HEADERS need to point (and default) to /usr/include/i2c, while > > #include <linux/i2c/....> > > picks the header either from a proper Linux kernel tree or from > I2C_HEADERS. Since the includes actually are <linux/...>, not <linux/i2c/...>, that would be /use/include/i2c/linux. It is obviously important to have the same include "tree" (even if it has just a depth of 1) in /usr/include (or wherever it'll end) than we have in the Linux kernel source tree. > If not /usr/include/i2c then perhaps /usr/local/include/i2c to be in > sync with the other /usr/local defaults. i2c and lm_sensors default to /usr/local for everything they install (except for the libsensors configuration file) and I don't think we will change this now. Thus I would go for /usr/local/include/i2c, but of course you could do /usr/include/i2c on your RPM packages if it fits the Red Hat/Fedora policy better. Anyway, before going there, we'd better fix i2c 2.9.0 so that if does install the header files at the old location again, so that people can compile lm_sensors. -- Jean Delvare http://khali.linux-fr.org/
