[Bug gprof/2587] Failed to build gprof under gmake patched by Apple.

2019-11-14 Thread hp at sourceware dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=2587

Hans-Peter Nilsson  changed:

   What|Removed |Added

 CC||hp at sourceware dot org

--- Comment #12 from Hans-Peter Nilsson  ---
(In reply to Nick Clifton from comment #10)
> (In reply to parakleta from comment #8)
> > I think just changing the 'SUFFIXES = .m' to 'SUFFIXES = .c .m' and
> > rebuilding the "makefile.in" should do it.
> 
> OK, done.  Plus this time I actually rebuilt Makefile.in, which I forgot
> last time.  Doh.
> 
> I have also REOPENED this PR, so that it will remain active until you
> can confirm that the changes are working.
> 
> Cheers
>   Nick

My autotester for mmix-knuth-mmixware cris-axis-linux-gnu cris-axis-elf
running on Debian 9 x86_64 (GNU make 4.1) screams at 63442f6:
...
Setting warning flags = -W -Wall -Wstrict-prototypes -Wmissing-prototypes
-Wshadow -Wstack-usage=262144 -Werror
configure: updating cache ./config.cache
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating Makefile
config.status: creating po/Makefile.in
config.status: creating gconfig.h
config.status: executing depfiles commands
config.status: executing libtool commands
config.status: executing default-1 commands
config.status: creating po/POTFILES
config.status: creating po/Makefile
make[2]: *** No rule to make target 'flat_bl.c', needed by 'all'.  Stop.
make[2]: Entering directory '/tmp/hpautotest-binutils/cris-axis-elf/gprof'
make[2]: Leaving directory '/tmp/hpautotest-binutils/cris-axis-elf/gprof'
make[1]: *** [all-gprof] Error 2
Makefile:6185: recipe for target 'all-gprof' failed
make[1]: Leaving directory '/tmp/hpautotest-binutils/cris-axis-elf'
make: *** [all] Error 2
Makefile:850: recipe for target 'all' failed

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gprof/2587] Failed to build gprof under gmake patched by Apple.

2019-11-14 Thread parakleta at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=2587

--- Comment #11 from parakleta at gmail dot com ---
(In reply to cvs-com...@gcc.gnu.org from comment #4)
> The master branch has been updated by Nick Clifton :
> 
> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;
> h=415ed2a17588f855397cc2d03aa68a79e84f77f7
> 
> commit 415ed2a17588f855397cc2d03aa68a79e84f77f7
> Author: Nick Clifton 
> Date:   Wed Nov 13 11:21:39 2019 +
> 
> Fix the rule for building C files in the gprof makefile.
> 
>   PR 2587
>   * Makefile.am: Fix rule to build .c files from .m files.

You still need to revert this change.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug ld/22269] Undefined weak symbols isn't resolved to 0 in static PIE

2019-11-14 Thread patrick.oppenlander at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22269

patrick.oppenlander at gmail dot com changed:

   What|Removed |Added

 CC||patrick.oppenlander at gmail 
dot c
   ||om

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug ld/25029] Invalid PE file caused by discarded .rdata section

2019-11-14 Thread florin.saft...@soft-to-you.com
https://sourceware.org/bugzilla/show_bug.cgi?id=25029

--- Comment #13 from Florin Saftoiu  ---
Hi Nick,

I've tested with both patches applied and the executable is valid and works as
expected. SizeOfHeaders is 0x200 and SizeOfImage is 0x4000 as expected. So I
guess the patches are OK.

Weird about the order of the sections though, I'm guessing it could cause other
problems, since it's the offset of the last section in the ld script that
determines the size of the image.

Let me know if I can help further.

Best regards,
Florin

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug binutils/25191] New: binutils/strip-new: BFD (GNU Binutils) 2.33 internal error, aborting at elf.c:9101 in _bfd_elf_set_section_contents

2019-11-14 Thread fdgkhdkgh at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=25191

Bug ID: 25191
   Summary: binutils/strip-new: BFD (GNU Binutils) 2.33 internal
error, aborting at elf.c:9101 in
_bfd_elf_set_section_contents
   Product: binutils
   Version: 2.33
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: binutils
  Assignee: unassigned at sourceware dot org
  Reporter: fdgkhdkgh at gmail dot com
  Target Milestone: ---

Created attachment 12074
  --> https://sourceware.org/bugzilla/attachment.cgi?id=12074=edit
file that reproduces this problem

corrupted file can trigger the abort

Version : 2.33.1
(https://ftp.yzu.edu.tw/pub/gnu/binutils/binutils-2.33.1.tar.xz)

Steps to Reproduce :
download the sample from attachment

strip-new  -D -F elf64-x86-64 ./abort_sample
strip-new  -U -F elf64-x86-64 ./abort_sample
strip-new  --remove-relocations .text -F elf32-i386 ./abort_sample

OS : ubuntu 18.04.3
kernel : gnu/linux 5.0.0-32-generic
processor : Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz

Actual Results: the application abort

Expected Results: do not abort
(ex: strip-new -F pei-i386 ./abort_sample)

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug ld/25029] Invalid PE file caused by discarded .rdata section

2019-11-14 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=25029

--- Comment #12 from Nick Clifton  ---
Created attachment 12073
  --> https://sourceware.org/bugzilla/attachment.cgi?id=12073=edit
Proposed patch

Hi Florian,

  You are right - the sections are in a different order for me (.text, .data.
.rdata, .idata).  I do not know why this is happening, but it has given me an
idea for a potential patch.

  Please could you try out the patch attached to this update - along with the
original patch - and see if this solves the problem.  Thanks.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gprof/2587] Failed to build gprof under gmake patched by Apple.

2019-11-14 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=2587

Nick Clifton  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

--- Comment #10 from Nick Clifton  ---
(In reply to parakleta from comment #8)
> I think just changing the 'SUFFIXES = .m' to 'SUFFIXES = .c .m' and
> rebuilding the "makefile.in" should do it.

OK, done.  Plus this time I actually rebuilt Makefile.in, which I forgot
last time.  Doh.

I have also REOPENED this PR, so that it will remain active until you
can confirm that the changes are working.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug gprof/2587] Failed to build gprof under gmake patched by Apple.

2019-11-14 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=2587

--- Comment #9 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=63442f6a2eff9f4c5c67099565f59c0f135aff94

commit 63442f6a2eff9f4c5c67099565f59c0f135aff94
Author: Nick Clifton 
Date:   Thu Nov 14 12:11:43 2019 +

Another attempt at fixing building gprof with gmake.

PR 2587
* Makefile.am (SUFFIXES): Add .c.
* Makefile.in: Regenerate.

-- 
You are receiving this mail because:
You are on the CC list for the bug.