[Debian] Re: Solution for #576227

Tue Sep 7 07:27:03 EDT 2010

On 09/07/2010 02:02 PM, Ola Lundqvist wrote:
> Hi Kir
>
> Your suggestion solved that build problem. Thanks a lot for that.
>
> I will try to convince the release team about the changes. If I fail I will
> request help from you. But before that you need to solve the following
> issue for me.
>
> gcc -DSBINDIR=\"/usr/sbin\" -DPKGCONFDIR=\"/etc/vz\"
> -DPKGLIBDIR=\"/usr/lib/vzct
>                          l\" -Wall -g -O2 -Wall -Wpointer-arith
> -Wcast-qual -Winline -Wextra -Wcast-align
>                                         -Wno-unused-parameter
> -D_FILE_OFFSET_BITS=64 -o .libs/vzeventd vzeventd.o  ../s
>
> rc/lib/.libs/libvzctl.so
> vzeventd.o: In function `parse_event':
> /home/ola/build/debian/vzctl/vzctl-3.0.24/src/vzeventd.c:117:
> undefined reference to `set_log_ctid'
> vzeventd.o: In function `read_events':
> /home/ola/build/debian/vzctl/vzctl-3.0.24/src/vzeventd.c:178:
> undefined reference to `set_log_ctid'
> vzeventd.o: In function `daemon_read_events':
> /home/ola/build/debian/vzctl/vzctl-3.0.24/src/vzeventd.c:205:
> undefined reference to `set_log_quiet'
> collect2: ld returned 1 exit status
>
> Do you know why that is? Do this patch depend on some other patch as well?
>    

Yeah, this one:
http://git.openvz.org/?p=vzctl;a=commitdiff;h=f3bf51833e4a1533f9f92132d1872d84e106b3cf

Sorry I haven't mentioned it.

> // Ola
>
> Quoting "Ola Lundqvist"<ola at inguza.com>:
>    
>> Hi Kir
>>
>> autogen.sh do not exist in the released version of the source. It
>> only exists in git. :-)
>>
>> However I found it.
>>
>> I missed to run libtoolize and was missing parameters for automake.
>>
>> However I think I will have some problem to convince the release
>> managers about this. The patch is very invasive, especially when we
>> change the build system...
>>
>> Would it be possible for you to take the .tar.gz from Debian, apply
>> the debian patch, apply the two patches and then run autogen.sh with
>> the versions you have. I have a feeling that this will cause a lot
>> less differences than it do for me with completely different
>> automake, libtool and autoconf versions.
>>
>> The reason why I ask you for this is that I think I will have a
>> problem to convince the release managers about this already without
>> the build problem issues.
>>
>> Best regards,
>>
>> // Ola
>>
>> Quoting "Kir Kolyshkin"<kir at parallels.com>:
>>      
>>> Try running
>>> make distclean; ./autogen.sh
>>>
>>> It should help.
>>>
>>> Ola Lundqvist<ola at inguza.com>  wrote:
>>>
>>>
>>> Hi again Kir
>>>
>>> The problem I end up with after upgrading to a recent version
>>> of autoconf is the following:
>>>
>>> make[3]: Entering directory
>>> `/home/ola/build/debian/vzctl/t/vzctl-3.0.24/src/lib'
>>> /bin/sh ../../libtool --tag=CC   --mode=compile gcc
>>> -DPACKAGE_NAME=\"vzctl\" -DPACKAGE_TARNAME=\"vzctl\"
>>> -DPACKAGE_VERSION=\"3.0.24\" -DPACKAGE_STRING=\"vzctl\ 3.0.24\"
>>> -DPACKAGE_BUGREPORT=\"devel at openvz.org\" -DPACKAGE_URL=\"\"
>>> -DPACKAGE=\"vzctl\" -DVERSION=\"3.0.24\" -DSTDC_HEADERS=1
>>> -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1
>>> -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1
>>> -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1
>>> -DHAVE_DLFCN_H=1 -I. -I../../include   -DSBINDIR=\"/usr/sbin\"
>>> -DPKGCONFDIR=\"/etc/vz\" -DPKGLIBDIR=\"/usr/lib/vzctl\" -Wall -g
>>> -O2 -Wall -Wpointer-arith -Wcast-qual -Winline -Wextra -Wcast-align
>>> -Wno-unused-parameter -D_FILE_OFFSET_BITS=64 -MT cap.lo -MD -MP -MF
>>> .deps/cap.Tpo -c -o cap.lo cap.c
>>> ../../libtool: line 467: CDPATH: command not found
>>> ../../libtool: line 1145: func_opt_split: command not found
>>> libtool: Version mismatch error.  This is libtool 2.2.6b, but the
>>> libtool: definition of this LT_INIT comes from an older release.
>>> libtool: You should recreate aclocal.m4 with macros from libtool 2.2.6b
>>> libtool: and run autoconf again.
>>>
>>> So to me it would help a lot of I could have a patch so the only thing
>>> I need to run to make it work is ./configure and then make.
>>>
>>> Best regards,
>>>
>>> // Ola
>>>
>>> On Tue, Sep 07, 2010 at 08:19:24AM +0200, Ola Lundqvist wrote:
>>>        
>>>> Hi Kir
>>>>
>>>> I can see in bugzilla that this is now considered to be solved with
>>>> the two following patches:
>>>>
>>>> http://git.openvz.org/?p=vzctl;a=commit;h=4cc6257616e1a725650d6b026a42d17cd1c3cce8
>>>> http://git.openvz.org/?p=vzctl;a=commit;h=5b35797753892bfa507b2aa0a1207f0c97bba005
>>>>
>>>> These two are rather invasive patches and requires the kernel
>>>> to have this support.
>>>>
>>>> I have looked through the patch and can see that the mk_reboot_script
>>>> code is now removed in the second patch in script.c.
>>>>
>>>> But do we remove this entry if it already exists and the kernel
>>>> support this?
>>>>
>>>> Also, did you get an answer from the Kernel maintainers about
>>>> whether they are allowing this to be solved or not?
>>>>
>>>> Also, would anything bad happen in case I apply the patch and the
>>>> kernel do not support this new functionality?
>>>>
>>>> In addition it looks like I have a too old version of autoconf.
>>>> Can you please send me a patch with the changes after automake run as
>>>> well so all the .am file changes take effect? That would help me
>>>> quite a lot.
>>>>
>>>> Best regards,
>>>>
>>>> // Ola
>>>>
>>>> --
>>>>          