[devel] Fwd: Re: as-needed sabotage

[devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 863 bytes --]



----------  Пересланное сообщение  ----------

Subject: Re: [devel] as-needed sabotage
Date: 13 мая 2006 12:24
From: Igor Zubkov <icesik@mail.ru>
To: devel@altlinux.ru

В сообщении от 13 мая 2006 00:22 Alexey Tourbin написал(a):
> > bbappconf - bbappconf makes it possible to set some options for the
> > windows blackbox opens * Mon May 01 2006 Igor Zubkov <icesik@altlinux>
> > 0.0.2-alt2
> > - fix rebuild with new ld (by -Wl,--no-as-needed)

Я потратил три часа на попытку починить и забил. У меня есть занятия
повеселее. Внутри там такая каша, что там надо просто переписывать все
Makefile.am и configure.ac.

Если у кого-то есть желание помочь -- welcome.

Причём, все bb* тулзы такие. Достаточно одно рецепта, все остальные я починю
сам.

--
I.F.K. - La Vita Fragile

-------------------------------------------------------

-- 
I.F.K. - La Vita Fragile

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey I. Froloff]

[-- Attachment #1.1: Type: text/plain, Size: 316 bytes --]

* Igor Zubkov <icesik@> [060513 14:13]:
> Я потратил три часа на попытку починить и забил. У меня есть занятия
> повеселее. Внутри там такая каша, что там надо просто переписывать все
> Makefile.am и configure.ac.
Ровно пять минут.  В спеке вызвать %__autoreconf перед
%configure.

-- 
Regards,
Sir Raorn.

[-- Attachment #1.2: bbappconf-0.0.2-alt-link.patch --]
[-- Type: text/plain, Size: 614 bytes --]

--- bbappconf-0.0.2-orig/configure.in	2002-10-15 23:06:27 +0400
+++ bbappconf-0.0.2/configure.in	2006-05-13 14:53:16 +0400
@@ -71,13 +71,13 @@
 
 CFLAGS="$CFLAGS $X_CFLAGS"
 CXXFLAGS="$CXXFLAGS $X_CFLAGS"
-LDFLAGS="$LDFLAGS $X_LIBS $X_PRE_LIBS"
+LIBS="$LIBS $X_LIBS $X_PRE_LIBS"
 
 dnl Checks for X libraries.
-AC_CHECK_LIB(X11, XOpenDisplay, LDFLAGS="$LDFLAGS -lX11",
+AC_CHECK_LIB(X11, XOpenDisplay, LIBS="$LIBS -lX11",
              AC_MSG_ERROR(XOpenDisplay not found in -lX11))
 
-LDFLAGS="$LDFLAGS $X_EXTRA_LIBS"
+LIBS="$LIBS $X_EXTRA_LIBS"
 
 dnl Checks for header files.
 AC_HEADER_STDC

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 449 bytes --]

* Alexey I. Froloff <raorn@> [060513 14:57]:
> > Я потратил три часа на попытку починить и забил. У меня есть занятия
> > повеселее. Внутри там такая каша, что там надо просто переписывать все
> > Makefile.am и configure.ac.
> Ровно пять минут.  В спеке вызвать %__autoreconf перед
> %configure.
Или даже так (untested):

%__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*LIBS.*),LIBS=\1$LIBS\2,' configure.*
%__autoconf

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1.1: Type: text/plain, Size: 4652 bytes --]

В сообщении от 13 мая 2006 13:56 Alexey I. Froloff написал(a):
> * Igor Zubkov <icesik@> [060513 14:13]:
> > Я потратил три часа на попытку починить и забил. У меня есть занятия
> > повеселее. Внутри там такая каша, что там надо просто переписывать все
> > Makefile.am и configure.ac.
>
> Ровно пять минут.  В спеке вызвать %__autoreconf перед
> %configure.

Да ну уж прям... А хоть попробовать сделать то, что посоветовал?

[JT]
Видимо, я сегодня точно напьюсь водки.
Ну как не пить при такой жизни?
[/JT]

Ладно, для тех к мне не верит...

На примере bbdate:
c++ -DPACKAGE=\"bbdate\" -DVERSION=\"0.2.4\" -DSTDC_HEADERS=1 -DHAVE_MALLOC_H=1 -DTIME_WITH_SYS_TIME=1 -DHAVE_SELECT=1 -DHAVE_STRSTR=1  -I. -I.    -DDEFAULT_CONF=\"/usr/share/bbtools/bbdate.conf\"            -DDEFAULT_NOBB=\"/usr/share/bbtools/bbdate.nobb\"  -pipe -Wall -O2 -march=i586 -mtune=pentium4  -c 
Basewindow.cc
c++  -pipe -Wall -O2 -march=i586 -mtune=pentium4  -s   -lSM -lICE -lX11  -o 
bbdate  bbdate.o Image.o LinkedList.o resource.o Baseresource.o Basewindow.o  
bbdate.o: In function `ToolWindow::~ToolWindow()':
bbdate.cc:(.text+0x22): undefined reference to `XUnmapWindow'
bbdate.cc:(.text+0x5a): undefined reference to `XDestroyWindow'
[заскипано]
Basewindow.o: In function `Basewindow::Basewindow(int, char**)':
Basewindow.cc:(.text+0x707): undefined reference to `XOpenDisplay'
collect2: ld returned 1 exit status
make[1]: *** [bbdate] Error 1
make[1]: Leaving directory `/home/icesik/RPM/BUILD/bbdate-0.2.4'
make: *** [all-recursive] Error 1
make: Leaving directory `/home/icesik/RPM/BUILD/bbdate-0.2.4'
ошибка: Неверный код возврата из /home/icesik/tmp/rpm-tmp.41096 (%build)


Ошибки сборки пакетов:
    Неверный код возврата из /home/icesik/tmp/rpm-tmp.41096 (%build)

Добавление autoreconf -fisv:
[icesik@iceberg bbdate-0.2.4]$ autoreconf -fisv
autoreconf-default: Entering directory `.'
autoreconf-default: configure.in: not using Gettext
autoreconf-default: running: aclocal --force 
configure.in:15: warning: underquoted definition of AC_SET_DEBUG
  run info '(automake)Extending aclocal'
  or see http://sources.redhat.com/automake/automake.html#Extending-aclocal
configure.in:22: warning: underquoted definition of AC_SET_NODEBUG
/usr/share/aclocal/libart.m4:11: warning: underquoted definition of 
AM_PATH_LIBART
/usr/share/aclocal/libIDL.m4:6: warning: underquoted definition of 
AM_PATH_LIBIDL
/usr/share/aclocal/ORBit.m4:4: warning: underquoted definition of 
AM_PATH_ORBIT
autoreconf-default: configure.in: tracing
autoreconf-default: configure.in: not using Libtool
autoreconf-default: running: /usr/bin/autoconf-2.5 --force
autoreconf-default: configure.in: not using Autoheader
autoreconf-default: running: automake --add-missing --force-missing
Makefile.am: installing `./depcomp'
Makefile.am:1: `CPPFLAGS' is a user variable, you should not override it;
Makefile.am:1: use `AM_CPPFLAGS' instead.
autoreconf-default: Leaving directory `.'
[icesik@iceberg bbdate-0.2.4]$ ./configure 
checking for a BSD-compatible install... /usr/bin/ginstall -c
[заскипано]
configure: creating ./config.status
config.status: creating version.h
config.status: creating Makefile
config.status: creating data/Makefile
config.status: executing depfiles commands
[icesik@iceberg bbdate-0.2.4]$ make
[заскипано]
g++  -O2 -Wall   -s   -lSM -lICE -lX11  -o bbdate  bbdate.o Image.o 
LinkedList.o resource.o Baseresource.o Basewindow.o  
bbdate.o: In function `ToolWindow::~ToolWindow()':
bbdate.cc:(.text+0x1c): undefined reference to `XUnmapWindow'
bbdate.cc:(.text+0x54): undefined reference to `XDestroyWindow'
[заскипано]
Basewindow.cc:(.text+0x6c5): undefined reference to `XOpenDisplay'
collect2: ld returned 1 exit status
make[1]: *** [bbdate] Ошибка 1
make[1]: Leaving directory `/home/icesik/RPM/SOURCES/bbdate-0.2.4'
make: *** [all-recursive] Ошибка 1
[icesik@iceberg bbdate-0.2.4]$ 


Проблема обитает где-то Makefile.am:
CPPFLAGS =      @CPPFLAGS@ \
                -DDEFAULT_CONF=\"$(datadir)/bbtools/bbdate.conf\" \
                -DDEFAULT_NOBB=\"$(datadir)/bbtools/bbdate.nobb\"


SUBDIRS = data
bin_PROGRAMS = bbdate
bbdate_SOURCES = bbdate.cc Image.cc LinkedList.cc \
                 bbdate.hh Image.hh LinkedList.hh \
                 resource.cc resource.hh \
                 Baseresource.cc Baseresource.hh \
                 Basewindow.cc Basewindow.hh \
                 blackboxstyle.hh
EXTRA_DIST = BUGS TODO
bbdate_LDADD = @X_LIBS@

Что здесь править и как я не воспринимаю...

configure.in я пожалуй тоже приложу...


-- 
ТТ34 - Джек

[-- Attachment #1.2: configure.in --]
[-- Type: text/plain, Size: 1585 bytes --]

dnl Process this file with autoconf to produce a configure script.
AC_INIT(bbdate.cc)

AM_INIT_AUTOMAKE(bbdate, 0.2.4)

AC_PREFIX_DEFAULT(/usr/local)

AC_CHECK_PROGS(regex_cmd, sed)
if test x$regex_cmd = "x"; then
 AC_MSG_ERROR([error. sed is required to build the default bbtoolsrc file.])
fi


dnl get the debug values right
AC_DEFUN(AC_SET_DEBUG,
[
 test "$CFLAGS" = "" && CFLAGS="-g -Wall -DDEBUG"
 test "$CXXFLAGS" = "" && CXXFLAGS="-g -Wall -DDEBUG"
 test "$LDFLAGS" = "" && LDFLAGS=""
])

AC_DEFUN(AC_SET_NODEBUG,
[
 test "$CFLAGS" = "" && CFLAGS="-O2 -Wall"
 test "$CXXFLAGS" = "" && CXXFLAGS="-O2 -Wall"
 test "$LDFLAGS" = "" && LDFLAGS="-s"
])
      

AC_ARG_ENABLE(debug,[  --enable-debug	create debugging code [default=no]],
[
if test $enableval = "no";
 then AC_SET_NODEBUG
 else AC_SET_DEBUG
fi
],
AC_SET_NODEBUG)

dnl Checks for programs.
dnl needed by configure itsself
AC_PROG_CC
AC_PROG_CXX
AC_PROG_INSTALL

AC_PATH_X
AC_PATH_XTRA

CFLAGS="$CFLAGS $X_CFLAGS"
CXXFLAGS="$CXXFLAGS $X_CFLAGS"
LDFLAGS="$LDFLAGS $X_LIBS $X_PRE_LIBS"
dnl Checks for X libraries.
AC_CHECK_LIB(X11, XOpenDisplay, LDFLAGS="$LDFLAGS -lX11",
             AC_MSG_ERROR(XOpenDisplay not found in -lX11))

LDFLAGS="$LDFLAGS $X_EXTRA_LIBS"

dnl Checks for header files.
AC_HEADER_STDC
AC_CHECK_HEADERS(malloc.h)

dnl Checks for typedefs, structures, and compiler characteristics.
AC_C_CONST
AC_HEADER_TIME
AC_STRUCT_TM

dnl Checks for library functions.
AC_CHECK_FUNCS(select strstr)

AC_OUTPUT(version.h Makefile data/Makefile)

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 368 bytes --]

В сообщении от 13 мая 2006 13:56 Alexey I. Froloff написал(a):
> > Я потратил три часа на попытку починить и забил. У меня есть занятия
> > повеселее. Внутри там такая каша, что там надо просто переписывать все
> > Makefile.am и configure.ac.
>
> Ровно пять минут.  В спеке вызвать %__autoreconf перед
> %configure.

Блин... А патч я и не заметил...

-- 
ТТ34 - Амиго

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 489 bytes --]

* Igor Zubkov <icesik@> [060513 15:37]:
> > > Я потратил три часа на попытку починить и забил. У меня есть занятия
> > > повеселее. Внутри там такая каша, что там надо просто переписывать все
> > > Makefile.am и configure.ac.
> > Ровно пять минут.  В спеке вызвать %__autoreconf перед
> > %configure.
> Блин... А патч я и не заметил...
:-))))

Ну конечно же я проверил.  У меня уже был такой гадкий, противный
пакет.  Кажется даже это был aptitude.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 451 bytes --]

В сообщении от 13 мая 2006 13:56 Alexey I. Froloff написал(a):
> * Igor Zubkov <icesik@> [060513 14:13]:
> > Я потратил три часа на попытку починить и забил. У меня есть занятия
> > повеселее. Внутри там такая каша, что там надо просто переписывать все
> > Makefile.am и configure.ac.
>
> Ровно пять минут.  В спеке вызвать %__autoreconf перед
> %configure.

Блин, а оно таки собралось...

Усложним задачку... А пакет bbdate? :)

-- 
ТТ34 - Бельмондо

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 308 bytes --]

* Igor Zubkov <icesik@> [060513 15:47]:
> Усложним задачку... А пакет bbdate? :)
В %prep:

%__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,' configure.*

В %build вместо --no-as-needed:

%__autoreconf

Вот это наверно будет более общее решение.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 465 bytes --]

On Sat, May 13, 2006 at 04:08:45PM +0400, Alexey I. Froloff wrote:
> * Igor Zubkov <icesik@> [060513 15:47]:
> > Усложним задачку... А пакет bbdate? :)
> В %prep:
> %__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,' configure.*

Для нетривиальных подстановок лучше делать патчи.  Патчи имеют такое
преимущество, что если патч отваливается, то он отваливается явно,
как следует.  А что там подстановка может сделать в новой версии хз.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 655 bytes --]

* Alexey Tourbin <at@> [060513 16:30]:
> > > Усложним задачку... А пакет bbdate? :)
> > В %prep:
> > %__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,' configure.*
> Для нетривиальных подстановок лучше делать патчи.  Патчи имеют такое
> преимущество, что если патч отваливается, то он отваливается явно,
> как следует.  А что там подстановка может сделать в новой версии хз.
Мне в тот момент патчи делать было просто лень ;-)  Можно
починить configure этой командой, проверить и сделать патч.  А
ещё лучше вправить автору мозги на эту тему чтобы в следующей
версии этот патч уже был не нужен.

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 615 bytes --]

В сообщении от 13 мая 2006 15:29 Alexey Tourbin написал(a):
> On Sat, May 13, 2006 at 04:08:45PM +0400, Alexey I. Froloff wrote:
> > * Igor Zubkov <icesik@> [060513 15:47]:
> > > Усложним задачку... А пакет bbdate? :)
> >
> > В %prep:
> > %__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,'
> > configure.*
>
> Для нетривиальных подстановок лучше делать патчи.  Патчи имеют такое
> преимущество, что если патч отваливается, то он отваливается явно,

согласен.

> как следует.  А что там подстановка может сделать в новой версии хз.

у этих пакетов upstream is dead.

-- 
I.F.K. - Нам важно все

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

Re: [devel] Fwd: Re: as-needed sabotage

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 410 bytes --]

В сообщении от 13 мая 2006 15:08 Alexey I. Froloff написал(a):
> * Igor Zubkov <icesik@> [060513 15:47]:
> > Усложним задачку... А пакет bbdate? :)
>
> В %prep:
>
> %__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,'
> configure.*
>
> В %build вместо --no-as-needed:
>
> %__autoreconf
>
> Вот это наверно будет более общее решение.

Спасибо! Работает.

-- 
Сергей Маврин (Маврик) - Пророк

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]

[devel] [wiki] Re: Fwd: Re: as-needed bootleg

[Michael Shigorin]

On Sat, May 13, 2006 at 04:08:45PM +0400, Alexey I. Froloff wrote:
> > Усложним задачку... А пакет bbdate? :)
> В %prep:
> %__subst 's,LDFLAGS=\(.*\)\$LDFLAGS\(.*\(LIBS\|-l\).*\),LIBS=\1$LIBS\2,' configure.*
> В %build вместо --no-as-needed:
> %__autoreconf
> Вот это наверно будет более общее решение.

Добавил на http://wiki.sisyphus.ru/devel/UpStream/AsNeeded

PS: :)
http://lists.altlinux.ru/pipermail/devel/2005-December/027129.html

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/