Commit Graph

1110 Commits

Author SHA1 Message Date
Andrew Burgess
5f67b86ed1 Remove gdb-lineno-makeup-test.patch
An equivalent test has been merged upstream in this commit:

  commit ef56b006501ba52b128d4e5f36657ddbf56d22bc
  Date:   Wed May 17 15:14:15 2023 +0100

      gdb/testsuite: test for a function with no line table

As this is only a test there seems little point in backporting this
commit to Fedora -- we'll pick it up next time we rebase.  For now I
suggest that we drop this patch.
2023-05-20 10:24:29 +01:00
Andrew Burgess
f71aced2fd Remove gdb-ccache-workaround.patch
This patch adds a work around for an issue with older versions of
ccache -- the line number for macros defined at the compiler command
line was '1' instead of '0'.

Not only is this ccache issue fixed with newer versions of ccache, but
upstream GDB now disables ccache during testing, see commit:

  commit 49b4de64242d4ae035e0e2197837278e33c187fc
  Date:   Thu Sep 15 14:04:20 2016 +0200

      testsuite: Disable ccache

As a result this work around is no longer needed, and this patch can
be dropped.
2023-05-16 16:38:15 +01:00
Andrew Burgess
2a935aa29b Remove gdb-ppc-power7-test.patch
The test added by this patch claims to be a power7 test, however, it
actually appears to test instructions from power7, power8, and power9.

All the instructions tested by this patch are already tested by
upstream tests:

  gdb.arch/powerpc-power7.exp
  gdb.arch/powerpc-power8.exp
  gdb.arch/powerpc-power9.exp

As such this patch is redundant and can be removed.
2023-05-16 15:23:14 +01:00
Alexandra Hájková
7046f4cf6f Remove gdb-rhel5.9-testcase-xlf-var-inside-mod.patch
The patch adds gdb.fortran/xlf-variable.exp test,
the test can only be run on PPC64 machines which
are not supported anymore.
2023-05-06 18:28:28 +02:00
Kevin Buettner
6c805458b5 Restore changes which fix C89-isms in gdb-6.6-buildid-locate-rpm.patch
These changes, originally added to Fedora GDB by Keith Seitz, and
described in the Nov 30 2022 Changelog eentry, were inadvertently lost
during the GDB 13.1 rebase.  Only downstream/local changes were lost -
the remainder of the changes were in the upstream GDB 13.1 sources.
2023-05-04 13:40:32 -07:00
Andrew Burgess
e19c0f8fb7 Rewrite (and rename) gdb-libexec-add-index.patch
It has been observed that the changes added by
gdb-libexec-add-index.patch will result in GDB testing hanging when
the tests are being run using an in-tree GDB; that is when using 'make
check'.  One test that is known to fail is gdb.base/with-mf.exp,
though any test that calls the gdb-add-index.sh script will also hang.

The problem is that when the gdb-add-index.sh script is run, the GDB
testsuite passes the GDB command to use within the GDB environment
variable.  For in-tree testing this will be something like:

  GDB="/path/to/gdb -data-directory /path/to/data-directory"

Notice that the environment variable contains both an executable and
an argument.

Our changes to gdb-add-index.sh add this:

  GDB2=/usr/libexec/gdb
  if test -x $GDB2 && ! which $GDB &>/dev/null; then
      GDB=$GDB2
  fi

The problem then is that '-data-directory' is treated as a set of
options to 'which'.  Many of these options are not known to 'which',
but the '-i' option is known.  The documentation of '-i' says:

    --read-alias, -i
        Read aliases from stdin, reporting matching ones on
        stdout. This is useful in combination with using an alias for
        which itself. For example
        alias which=´alias | which -i´.

And here's the problem; this option causes 'which' to read from
stdin.  As the GDB testsuite doesn't send any additional input on
stdin then the which command will never complete, and the test will
hang.

The solution I think is to avoid calling 'which' like this on a user
supplied GDB environment variable.

The changes in the gdb-libexec-add-index.patch were really about what
the _default_ GDB executable should be.  The upstream version of this
script does this:

  GDB=${GDB:=gdb}

That is, the default is just 'gdb'.  However, for RH this is not good
enough.  We want to handle two additional cases, first, when only the
gdb-minimal package is installed, in which case the default should be
/usr/bin/gdb.minimal.  Then we also want to handle the case where the
user doesn't have 'gdb' itself in their $PATH, but does have the 'gdb'
executable installed in /usr/libexec/gdb.

The code as it currently stands also has a problem where, if
gdb.minimal is installed on the machine this will _always_ be used in
preference to the user supplied GDB value (assuming the code worked at
all) this means that when doing in-tree testing we wouldn't actually
be using the in-tree GDB to build the index, which isn't ideal.

So in this commit I propose that we rework our gdb-add-index.sh
changes.  Now, we only use the RH special values in the case that
there is no GDB environment variable set.  I believe this handles all
the required use cases:

  1. When doing in-tree testing GDB environment variable will be set,
  and this will always be used as is, with no special processing,

  2. When gdb-add-index.sh is used and GDB environment variable is not
  set then we will use the first of the following as the default:

    (a) /usr/bin/gdb.minimal if this file exists and is executable,
    (b) The first gdb executable that can be found in the $PATH,
    (c) /usr/libexec/gdb if this file exists and is executable.

While I was changing this patch anyway I've removed the libexec part
of the patch name -- this no longer seemed relevant, I suspect this
related to an older version of this patch.
2023-05-04 15:11:19 +01:00
Kevin Buettner
f68e769d70 Backport upstream fix for RHBZ 2192105
Specifically, "Pass const frame_info_ptr reference for
skip_[language_]trampoline". (Mark Wielaard, RHBZ 2192105, build/30413)
2023-05-03 13:32:42 -07:00
Andrew Burgess
7cd90204c9 Remove gdb-opcodes-clflushopt-test.patch
Remove gdb-opcodes-clflushopt-test.patch.  This patch tests that GDB
can disassemble the clflushopt instruction correctly.  Such
disassembly is a feature of libopcode and is covered by the gas
tests i386/x86-64-clflushopt.s and i386/clflushopt.s.  Lets remove
this test from GDB and just rely on the gas tests instead.
2023-05-02 19:15:44 +01:00
Kevin Buettner
b1e9fbcc29 Remove gdb-6.7-charsign-test.patch
This patch originally contained some changes to GDB which were
rejected by upstream maintainers.  All that remained was a testcase
which had a number of failures due to the rest of the work not being
present in GDB.
2023-04-29 19:34:05 -07:00
Bruno Larsen
231ea6585a Remove gdb-6.5-bz109921-DW_AT_decl_file-test.patch
This patch only had a test for basic DWARF-2 support, and the test was
already covered by other tests, such as gdb.linespec/linespec.exp
2023-04-25 14:14:49 +02:00
Bruno Larsen
d18c2a5bd9 Remove gdb-6.5-last-address-space-byte-test.patch
This patch exercised an edge case for 32-bit systems in
target_xfer_memory, a function that has been removed in 2006.
The test that patch added has been irrelevant for some time.
2023-04-24 15:48:54 +02:00
Alexandra Hájková
9dbae0620d Remove gdb-6.3-bz140532-ppc-unwinding-test.patch.
It adds powerpc-bcl-prologue.exp test which seems
to be a subset of upstream powerpc-prologue.exp
2023-04-13 12:22:36 +02:00
Keith Seitz
ff60ea4e26 Backport fix for binutils/29988 2023-04-11 10:32:14 -07:00
Keith Seitz
33590be3ac Backport "Fix crash in inside_main_func"
Resolves: rhbz#2183595
2023-03-31 13:14:15 -07:00
Kevin Buettner
45618ff785 Update description and Summary in gdb.spec 2023-03-30 10:01:25 -07:00
Alexandra Hájková
5b461b9481 Update gdb-6.6-buildid-locate.patch to fix RHBZ 2181221. 2023-03-30 16:20:10 +02:00
Andrew Burgess
240ac8eb89 Set distro name using --with-pkgconfig configure option
Instead of placing the distro name (e.g. 'Fedora Linux') into the
version.in file, use the --with-pkgconfig configure option to set the
disto name.

This does change GDB's version output slightly.   Prior to this commit
we might expect to see something like this:

  $ gdb --version
  GNU gdb (GDB) Fedora Linux 13.1-1-fc38
  ...

Now we'll see this:

  $ gdb --version
  GNU gdb (Fedora Linux) 13.1-1-fc38
  ...

Notice that 'Fedora Linux' moved inside the parenthesis.

There are other benefits to this change, the Python gdb.VERSION
string will now contain '13.1-1-fc38' instead of 'Fedora Linux
13.1-1-fc38', this was reported as an issue in this bug:

  https://bugzilla.redhat.com/show_bug.cgi?id=2179554

I've also removed the change to gdb.gdb/selftest.exp as I don't
believe that this was ever being used -- when this test was run we
should have hit an earlier case.

I have added a brand new test that checks our version string.  I've
placed this new test into a separate file, so we should (hopefully)
not have any issues maintaining this as an out of tree test.

For implementation, by default we make use of the %dist_name macro,
however, this is not defined on RHEL 9 or earlier, so to support
reusing this commit on RHEL systems, I also detect the %rhel macro,
and in that case use a hard-coded string.
2023-03-29 16:30:04 +01:00
Kevin Buettner
7403afbaf9 Backport fix for RHBZ 2177655. (Luis Machado) 2023-03-24 19:22:09 -04:00
Bruno Larsen
50c864895c Remove gdb-rhbz1350436-type-printers-error.patch since it is upstreamed.
The test introduced by that patch is now available on upstream commit
83b755117d7a13e75877c3f166dfef1d8b69ec75, and will probably be
backported soon.
2023-03-20 10:18:06 +01:00
Kevin Buettner
92d162c371 Rebase to FSF GDB 13.1
Update gdb-6.3-rh-testversion-20041202.patch.
Update gdb-6.3-bz140532-ppc-unwinding-test.patch.
Update gdb-6.6-buildid-locate.patch.
Update gdb-6.6-buildid-locate-rpm.patch.
Remove 'Recommends: ' line for gcc-gdb-plugin for BZ2149246.
Add 'define _lto_cflags ' to avoid ODR violations.
Add -Wno-stringop-overflow to --enable-gdb-build-warnings to work around
  gcc problem.
2023-03-08 21:42:55 -07:00
Kevin Buettner
00fe7519d4 Update gdb-6.3-rh-testversion-20041202.patch
...so that $_gdb_major and $_gdb_minor will be obtained correctly.

Also, change define of 'use_guile' from empty (which rpmbuild
complained about) to a value of 1.  I encountered this problem
while attempting to build on F37.
2023-01-27 12:02:27 -07:00
Bruno Larsen
6de2f1fd32 Remove gdb-rhbz1398387-tab-crash-test.patch as that test didn't work anymore 2023-01-26 12:40:53 +01:00
Keith Seitz
06da7ee788 NVR bump for failed infra on -14 build. 2023-01-24 10:03:00 -08:00
Kevin Buettner
8d8fb49148 More tweaks to gdb-6.6-buildid-locate-rpm.patch
...in which rpmTag is replaced with rpmDbiTagVal.
2023-01-23 20:12:05 -07:00
Keith Seitz
e2df45df10 Backport of backtrace/29374
(Simon Marchi, sw backtrace/29374)
2023-01-23 04:26:24 -08:00
Kevin Buettner
b443fb3970 Fixes to get GDB to build again with GCC 13 and Python 3.12
- Backport fix for problems associated with GCC 13's self-move warning.
- Tweak gdb-6.6-buildid-locate-rpm.patch so that running GDB's configure
  script will not error out due to GCC 13's warnings.
- Backport replace deprecated distutils.sysconfig in python-config.
2023-01-20 17:14:37 -07:00
Alexandra Hájková
ed2e95dc63 Backport replace deprecated distutils.sysconfig in python-config.
(Lancelot SIX)
2023-01-19 19:33:30 +01:00
Fedora Release Engineering
8e9d3a4cf8 Rebuilt for https://fedoraproject.org/wiki/Fedora_38_Mass_Rebuild
Signed-off-by: Fedora Release Engineering <releng@fedoraproject.org>
2023-01-19 04:29:26 +00:00
Andrew Burgess
69f0c792d6 Back-port fix for RHBZ 2152431
Fixes RHBZ 2152431, an issue where reading a label symbol's value
would cause GDB to crash.  This pulls in two upstream commits
38665d717a3 and c3efaf0afd9 both of which should be in GDB 13 when it
is released.
2022-12-19 17:01:56 +00:00
Keith Seitz
0f5a6ebc3c Remove more unneeded tests and patches
Changes in this commit:
- Remove gdb-bz601887-dwarf4-rh-test.patch
  This adds a very simple test to check whether DWARF4
  is implemented. GDB has been using DWARF4 for several
  years, and the testsuite coverage supercedes this patch.
- Remove gdb-6.6-buildid-locate-rpm-scl.patch
  This patch is applicable only on RHEL6 where .gdb_index
  version 5 is used. In recent GDBs, i.e., all DTS releases,
  this version could only be read with the option
  "use-deprecated-index-sections". RHEL7+ use .gdb_index version 8.
  Since RHEL6 is no longer supported, this is unnecessary.
  Also remove GDB_INDEX_VERIFY_VENDOR references/code from
  gdb-6.6-buildid-locate.patch and gdb.spec
2022-12-16 09:14:19 -08:00
Andrew Burgess
b3008271fb remove gdb-fortran-frame-string.patch file
Remove the 'gdb-fortran-frame-string.patch' patch from Fedora GDB.  I
have pushed an upstream test that covers everything that this test
covered, see upstream GDB commit:

  commit 1c01b23603766fbca4ed4dd12fdd710860e6038e
  Date:   Fri Nov 4 14:51:11 2022 +0000

      gdb/fortran/testsuite: print values and types of string variables

As this is just a testsuite patch, I'm not going to backport this to
Fedora GDB, but when we next rebase, we will pick up this test.
2022-12-09 19:14:01 +00:00
Andrew Burgess
056260a609 remove gdb-bfd-aliasing.patch file
The patch 'gdb-bfd-aliasing.patch' is no longer applied as part of the
Fedora GDB build, since this commit to the Fedora GDB repository:

  commit c0cbb1aae4
  Date:   Sat Mar 19 21:26:41 2016 +0100

      .spec cleanup: Drop strict-aliasing GCC bug workaround (from RH BZ 1315191).

I propose that we remove this file from the repository.
2022-12-09 19:12:35 +00:00
Andrew Burgess
47989c5833 gdb-entryval-crash-*.patch files
The three patches:

  gdb-entryval-crash-1of3.patch
  gdb-entryval-crash-2of3.patch
  gdb-entryval-crash-3of3.patch

are not being applied to the Fedora build of GDB, since this commit to
the Fedora GDB repository:

  commit 902c8e5abc
  Date:   Thu Jul 24 20:04:47 2014 +0200

      Rebase to FSF GDB 7.7.91.20140724 (pre-7.8 snapshot).

The contents of the first two were reverting an upstream commit,
which, based on the dialog in the patch files, was done to mask a bug
in a different part of GDB, which I assume is now fixed.

The third patch adds a couple of tests, which are now all upstream,
and passing, I assume these tests covered the issue that was causing
the problem with the first two patches.

Given these patches are not being applied, and the relevant work seems
to be in upstream GDB, I propose we drop these three files from the
repository.
2022-12-09 19:12:35 +00:00
Keith Seitz
c09759de79 Disable Guile support on F38+
Resolves: rhbz#2151328
2022-12-07 12:03:23 -08:00
Keith Seitz
7e69ec9711 Remove several unnecessary test patches
- gdb-physname-pr11734-test.patch
- gdb-physname-pr12273-test.patch
These two tests were merged upstream under a different file name.

- gdb-runtest-pie-override.patch
This patch appears unnecessary today. I have tested on all supported
host architectures and detected no regressions. I presume this was
fixed in a different manner through the years.

- gdb-test-expr-cumulative-archer.patch
The patch contains two tests. Both are upstream already.
While one is a little more involved than what landed upstream,
both tests are equivalent. The other test, gdb.cp/namespace-no-imports.exp,
is identical to upstream's gdb.cp/nsnoimports.exp with whitespace
and other trivial changes.
2022-12-05 14:51:32 -08:00
Kevin Buettner
7ca61ebe91 Remove gdb-6.3-threaded-watchpoints2-20050225.patch
The test added by this patch from 2005 is a tweaked verion of the upstream
test gdb.threads/watchthreads.exp from 2004.  (The current version of
gdb.threads/watchthreads.exp has changed somewhat since 2004.)

The tweaks added to the circa 2004 upstream test consist of setting
a breakpoint on thread_function and then continuing to it.  At
that point, initial / runtime values of args[2] and args[3] are
fetched and saved in some TCL variables (which are renamed versions
of args_0 and args_1 in watchthreads.exp).  The rest of the changes
to the tweaked script are mostly related to the renaming related to
args_0 and args_1.  The other major difference is that watchthreads.exp
places hardware watchpoints on args[0] and args[1] whereas the
tweaked test instead watches args[2] and args[3].  However, all
of these initialized to 0.  In the tweaked test, fetching the
initial values when the thread function is first hit should result
in zero values being fetched for args[2] and args[3].

So, in conclusion, the tweaked version of the test doesn't actually
test anything new.  I took a look at old posts / threads from the
gdb-patches mailing list, but I couldn't find a post where Jeff
Johnston included this patch (and an explanation of what it does over
the other patch.)

Since this test doesn't offer anything new/better over what's already
upstream, I'm deleting it.
2022-12-01 19:20:12 -07:00
Bruno Larsen
46e82ad3b5 Remove gdb-rhbz1325795-framefilters-test.patch. This test doesn't
pass in the curret state, and the code that introduced the original
problem has been changed beyong recognition at this point.
2022-12-01 09:17:51 +01:00
Keith Seitz
910689b88d Updates for "ModernC" initiative
Resolves: rhbz#2143992
2022-11-29 14:19:51 -08:00
Kevin Buettner
3d614802ea Remove gdb-6.3-inheritancetest-20050726.patch. Upstream testcase
gdb.cp/impl-this.exp tests the printing of an instance variable from
an inherited class in the "print c" test.
2022-11-23 14:05:18 -07:00
Kevin Buettner
7de4cb0e83 Remove gdb-6.3-test-movedir-20050125.patch
Upstream test gdb.base/fullname.exp provides coverage for this case
and more.
2022-11-18 12:25:59 -07:00
Keith Seitz
73953d59d4 Constify target_desc and enable LTO
This patch backports the upstream commit to constify
target_desc. This eliminates the problems preventing us
from enabling LTO.
2022-11-03 06:40:16 -07:00
Bruno Larsen
eff447ffe1 backport fix to gdb.base/break-main-file-remove-fail 2022-10-18 12:10:01 +02:00
Bruno Larsen
89341dc865 remove gdb-6.3-test-dtor-fix, since it was upstreamed. 2022-10-18 10:46:33 +02:00
Alexandra Hájková
d6591abf53 Bump the release number. 2022-10-13 18:04:14 +02:00
Alexandra Hájková
e599c1296e gdb.spec: Fix the wrong date in a changelog. 2022-10-11 15:34:49 +02:00
Alexandra Hájková
c5ab349836 Backport upstream patch "Add support for readline 8.2". (Andreas Schwab) 2022-10-11 14:58:12 +02:00
Alexandra Hájková
267b55ff0e Update gdb-6.6-buildid-locate.patch to fix RHBZ 2122947. 2022-10-10 15:39:48 +02:00
Keith Seitz
a6e7b2b18f Disable -minimal RPM on RHEL9 GTS.
This patch disables the gdb-minimal package on GTS for RHEL9. Since
the -minimal package only exists to shrink the buildroot size and
we don't use GTS in the buildroot, there's no reason to build it.
2022-07-29 13:39:24 -07:00
Keith Seitz
fbba720a74 Fix s390x dependencies
Fedora doesn't build 32-bit s390x images.
There is no `fpc' package available for s390x, either.
2022-07-29 12:42:05 -07:00
Amit Shah
ddf8291871 Switch to using the dist_name macro for distribution
The new dist_vendor macro[1] in fedora-release helps simplify spec files
by removing the conditionals for populating the distro.

[1] https://src.fedoraproject.org/rpms/fedora-release/pull-request/223
2022-07-28 06:54:17 +00:00