<div dir="ltr">And Yann's reply for the list archives.<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Yann E. MORIN</b> <span dir="ltr"><<a href="mailto:yann.morin.1998@free.fr">yann.morin.1998@free.fr</a>></span><br>
Date: Tue, Sep 2, 2014 at 4:11 PM<br>Subject: Re: [PATCH v6 1/1] New package: openvmtools<br>To: Károly Kasza <<a href="mailto:kaszak@gmail.com">kaszak@gmail.com</a>><br><br><br>Karoly, All,<br>
<br>
On 2014-09-02 15:59 +0200, Károly Kasza spake thusly:<br>
[--SNIP--]<br>
<div class="">> > > +OPENVMTOOLS_AUTORECONF = YES<br>
> > Please add a comment saying why we need to autoreconf.<br>
> ><br>
> OK. (It's because the package is distributed without a configure script -<br>
> <a href="http://sourceforge.net/p/open-vm-tools/mailman/message/32550385/" target="_blank">http://sourceforge.net/p/open-vm-tools/mailman/message/32550385/</a> )<br>
<br>
</div>OK, add this above the "AUTORECONF = YES" line.<br>
<div><div class="h5"><br>
> > > +ifeq ($(BR2_PACKAGE_OPENVMTOOLS_PROCPS),y)<br>
> > > +OPENVMTOOLS_CONF_ENV += CUSTOM_PROCPS_NAME=procps<br>
> > LDFLAGS="-L$(TARGET_DIR)/usr/lib"<br>
> ><br>
> > Why do you point to TARGET_DIR?<br>
> ><br>
> > Usually, we consider the development files are installed in STAGING_DIR.<br>
> > So, if you need to point to a library path, you should point to<br>
> > STAGING_DIR here.<br>
> ><br>
> > But STAGING_DIR/usr/lib is already searched for by our toolchain, so it<br>
> > should not be necessary to add it.<br>
> ><br>
> > If you really need that, explain why.<br>
> ><br>
><br>
> If I do not add -L$(TARGET_DIR)/usr/lib I simply get an error while<br>
> configuring:<br>
><br>
> checking for getstat in -lprocps... no<br>
> checking for getstat in -lproc-3.2.8... no<br>
> checking for getstat in -lproc-3.2.7... no<br>
> configure: error: libproc not found. Please configure without procps (using<br>
> --without-procps) or install procps - <a href="http://procps.sourceforge.net" target="_blank">http://procps.sourceforge.net</a><br>
> make: ***<br>
> [/home/scythe/buildroot/output/build/openvmtools-9.4.6-1770165/.stamp_configured]<br>
> Error 1<br>
><br>
> I don't know why, but I simply can not find any procps related libraries in<br>
> the staging directory:<br>
><br>
> scythe@buildroot-vm:~/buildroot/output$ find . | grep libproc<br>
> ./build/procps-ng-3.3.9/proc/.libs/libprocps.lai<br>
> ./build/procps-ng-3.3.9/proc/.libs/libprocps.so.3<br>
> ./build/procps-ng-3.3.9/proc/.libs/libprocps.a<br>
> ./build/procps-ng-3.3.9/proc/.libs/libprocps.so.3.0.0<br>
> ./build/procps-ng-3.3.9/proc/.libs/libprocps.so<br>
> ./build/procps-ng-3.3.9/proc/.libs/<a href="http://libprocps.la" target="_blank">libprocps.la</a><br>
> ./build/procps-ng-3.3.9/proc/libprocps.sym<br>
> ./build/procps-ng-3.3.9/proc/libprocps.pc<br>
> ./build/procps-ng-3.3.9/proc/<a href="http://libprocps.pc.in" target="_blank">libprocps.pc.in</a><br>
> ./build/procps-ng-3.3.9/proc/<a href="http://libprocps.la" target="_blank">libprocps.la</a><br>
> ./target/usr/lib/pkgconfig/libprocps.pc<br>
> ./target/usr/lib/libprocps.so.3<br>
> ./target/usr/lib/libprocps.a<br>
> ./target/usr/lib/libprocps.so.3.0.0<br>
> ./target/usr/lib/libprocps.so<br>
> ./target/usr/lib/<a href="http://libprocps.la" target="_blank">libprocps.la</a><br>
> scythe@buildroot-vm:~/buildroot/output$ cd staging<br>
> scythe@buildroot-vm:~/buildroot/output/staging$ find . | grep libproc<br>
><br>
> > +ifeq ($(BR2_PACKAGE_OPENVMTOOLS_DNET),y)<br>
> > > +OPENVMTOOLS_CONF_ENV +=<br>
> > CUSTOM_DNET_CPPFLAGS="-I$(STAGING_DIR)/usr/include"<br>
> ><br>
> > Ditto, STAGING_DIR/usr/include is already searched for by our toolchain,<br>
> > so it should not be needed.<br>
> ><br>
> > If you really need that, explain why.<br>
> ><br>
><br>
> If I omit the CUSTOM_DNET_CPPFLAGS variable, I get an error while<br>
> configuring:<br>
><br>
> checking for dumbnet-config... no<br>
> checking for dnet-config... no<br>
> configure: error: dnet-config was not found on your PATH. Please configure<br>
> without dnet (using --without-dnet) or install dnet -<br>
> <a href="http://libdnet.sourceforge.net" target="_blank">http://libdnet.sourceforge.net</a><br>
> make: ***<br>
> [/home/scythe/buildroot/output/build/openvmtools-9.4.6-1770165/.stamp_configured]<br>
> Error 1<br>
><br>
> I think this is because the configure script is looking for a binary called<br>
> "dnet-config", and it needs the environment variable if the binary is not<br>
> found in the $PATH. So I guess this can't be circumvented.<br>
<br>
</div></div>Ah, then there are a few possibilities, of which the best would be to<br>
change <a href="http://configure.ac" target="_blank">configure.ac</a> to call $DNET_CONFIG is it is set, instead of just<br>
dnet-config, and then set DNT_CONFIG=$(STAGING_DIR)/usr/bin/dnet-config<br>
<div class=""><br>
> I would send a new version, do you have any idea why procps-ng doesn't copy<br>
> it's libraries into the $(STAGING_DIR)?<br>
<br>
</div>Yep, that's because <a href="http://propcs-ng.mk" target="_blank">propcs-ng.mk</a> does not have INSTALL_STAGING = YES.<br>
Provide a (separate) patch that adds this to procps-ng.<br>
<br>
I'll do a further review of your comments later (~16:00 UTC), for now<br>
I've got to go...<br>
<div class="HOEnZb"><div class="h5"><br>
Regards,<br>
Yann E. MORIN.<br>
<br>
--<br>
.-----------------.--------------------.------------------.--------------------.<br>
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |<br>
| <a href="tel:%2B33%20662%20376%20056" value="+33662376056">+33 662 376 056</a> | Software  Designer | \ / CAMPAIGN     |  ___               |<br>
| <a href="tel:%2B33%20223%20225%20172" value="+33223225172">+33 223 225 172</a> `------------.-------:  X  AGAINST      |  \e/  There is no  |<br>
| <a href="http://ymorin.is-a-geek.org/" target="_blank">http://ymorin.is-a-geek.org/</a> | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |<br>
'------------------------------^-------^------------------^--------------------'<br>
</div></div></div><br></div>