Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: Doug Evans <dje@google.com>
Cc: Pedro Alves <palves@redhat.com>, gdb-patches@sourceware.org
Subject: Re: [patch] gdb.base/valgrind-infcall.exp compat. with Ubuntu 10.04.4
Date: Fri, 24 Aug 2012 14:28:00 -0000	[thread overview]
Message-ID: <20120824142733.GA5887@host2.jankratochvil.net> (raw)
In-Reply-To: <CADPb22Ry8_KvsE0o0XAMgpoA5DQ3VM3xr0xx+M_1WpvL=Mccrg@mail.gmail.com>

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

On Tue, 21 Aug 2012 22:00:44 +0200, Doug Evans wrote:
> I'd say keep the comment in some form.
> 
> The tcl/expect I've been using is
> $ expect -v
> expect version 5.44.1.14
> $ ldd /usr/bin/expect | grep tcl
>         libtcl8.5.so.0 => /usr/lib/libtcl8.5.so.0 (0x00007ff9bac15000)

After reverting 61ab2e32245eadc40c36c5d7d4c4962962df59bd on FSF GDB HEAD it
still works for me.

expect-5.44.1.14-0google.fc18.x86_64
tcl-8.5.12-1.fc18.x86_64

Built from Fedora 18 expect repository
	http://pkgs.fedoraproject.org/cgit/expect.git/tree/?h=f18&id=751b103fb1c95b705fd8a2eb9cf5c682f0aef373
by attached .spec patch with:
https://launchpad.net/ubuntu/+archive/primary/+files/expect_5.44.1.14.orig.tar.gz

Do you run some replaced valgrind on updated Ubuntu 10.04.4?  Stock Ubuntu
valgrind is too old for this testcase to run.


Regards,
Jan

[-- Attachment #2: expect.spec-google.patch --]
[-- Type: text/plain, Size: 2663 bytes --]

diff --git a/expect.spec b/expect.spec
index 60dcb5c..1ff01e4 100644
--- a/expect.spec
+++ b/expect.spec
@@ -1,16 +1,17 @@
+%global _unpackaged_files_terminate_build 0
 %{!?tcl_version: %define tcl_version %(echo 'puts $tcl_version' | tclsh)}
 %{!?tcl_sitearch: %define tcl_sitearch %{_libdir}/tcl%{tcl_version}}
-%global majorver 5.45
+%global majorver 5.44.1.14
 
 Summary: A program-script interaction and testing utility
 Name: expect
 Version: %{majorver}
-Release: 7%{?dist}
+Release: 0google%{?dist}
 License: Public Domain
 Group: Development/Languages
 # URL: probably more useful is http://sourceforge.net/projects/expect/
 URL: http://expect.nist.gov/
-Source: http://downloads.sourceforge.net/%{name}/%{name}%{version}.tar.gz
+Source: expect_5.44.1.14.orig.tar.gz
 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 Buildrequires: tcl-devel tk-devel autoconf automake libXft-devel chrpath
 # Patch0: fixes change log file permissions
@@ -71,24 +72,25 @@ Please use tclsh with package require Tk and Expect instead
 of expectk.
 
 %prep
-%setup -q -n expect%{version}
-%patch0 -p1 -b .log_file
-%patch1 -p1 -b .pkgpath
-%patch2 -p1 -b .man-page
-%patch3 -p1 -b .match-gt-numchars-segfault
+%setup -q
+#-n expect%{version}
+#patch0 -p1 -b .log_file
+#patch1 -p1 -b .pkgpath
+#patch2 -p1 -b .man-page
+#patch3 -p1 -b .match-gt-numchars-segfault
 # examples fixes
-%patch100 -p1 -b .random
-%patch101 -p1 -b .mkpasswd-dash
-%patch102 -p1 -b .check-telnet
-%patch103 -p1 -b .passmass-su-full-path
+#patch100 -p1 -b .random
+#patch101 -p1 -b .mkpasswd-dash
+#patch102 -p1 -b .check-telnet
+#patch103 -p1 -b .passmass-su-full-path
 # -pkgpath.patch touch configure.in
-aclocal
-autoconf
-( cd testsuite
-  autoconf -I.. )
+#aclocal
+#autoconf
+#( cd testsuite
+#  autoconf -I.. )
 
 %build
-%configure --with-tcl=%{_libdir} --with-tk=%{_libdir} --enable-shared \
+%configure --with-tcl=%{tcl_sitearch} --with-tk=%{_libdir} --enable-shared \
 	--with-tclinclude=%{_includedir}/tcl-private/generic
 make %{?_smp_mflags}
 
@@ -100,7 +102,10 @@ rm -rf "$RPM_BUILD_ROOT"
 make install DESTDIR="$RPM_BUILD_ROOT"
 
 # move
-mv "$RPM_BUILD_ROOT"%{tcl_sitearch}/expect%{version}/libexpect%{version}.so "$RPM_BUILD_ROOT"%{_libdir}
+mv "$RPM_BUILD_ROOT"%{_libdir}/expect%{version}/libexpect%{version}.so "$RPM_BUILD_ROOT"%{_libdir}
+mkdir "$RPM_BUILD_ROOT"%{tcl_sitearch}
+mkdir "$RPM_BUILD_ROOT"%{tcl_sitearch}/expect%{version}
+mv "$RPM_BUILD_ROOT"%{_libdir}/expect%{version}/pkgIndex.tcl "$RPM_BUILD_ROOT"%{tcl_sitearch}/expect%{version}/
 
 # for linking with -lexpect
 ln -s libexpect%{majorver}.so "$RPM_BUILD_ROOT"%{_libdir}/libexpect.so

  reply	other threads:[~2012-08-24 14:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-07  6:43 Jan Kratochvil
2012-08-07 11:39 ` [patchv2] " Jan Kratochvil
2012-08-07 18:00   ` dje
2012-08-07 18:06     ` [commit+7.5] " Jan Kratochvil
2012-08-21 18:20 ` [patch] " Pedro Alves
2012-08-21 18:27   ` Jan Kratochvil
2012-08-21 18:41     ` Pedro Alves
2012-08-21 19:04       ` Jan Kratochvil
2012-08-21 20:01         ` Doug Evans
2012-08-24 14:28           ` Jan Kratochvil [this message]
2012-09-13  5:36           ` [commit+7.5] " Jan Kratochvil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120824142733.GA5887@host2.jankratochvil.net \
    --to=jan.kratochvil@redhat.com \
    --cc=dje@google.com \
    --cc=gdb-patches@sourceware.org \
    --cc=palves@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox