bugGNU Octave - Bugs: bug #57370, (mxe-octave) run issues after...

 
 

bug #57370: (mxe-octave) run issues after binutils update

Submitter:  John Donoghue <lostbard>
Submitted:  Fri 06 Dec 2019 01:19:22 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  None
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * dev Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sun 08 Dec 2019 04:35:59 PM UTC, comment #10: 

This fixed the issue for me too.

Closing report.

Markus Mützel <mmuetzel>
Group administrator
Fri 06 Dec 2019 10:14:23 PM UTC, comment #9: 

Pushed https://hg.octave.org/mxe-octave/rev/ad00dd601316
that seems to make everything work for me

John Donoghue <lostbard>
Group Member
Fri 06 Dec 2019 07:02:26 PM UTC, comment #8: 

Ok

well I think I'll go with the option of not stripping the msys-2.0.dll if that works for me




John Donoghue <lostbard>
Group Member
Fri 06 Dec 2019 06:18:38 PM UTC, comment #7: 

I configured with these switches for the native build that previously failed:

./configure --disable-devel-tools --enable-native-build --enable-octave=default --enable-qt5 --enable-lib64-directory --enable-pic-flag --disable-system-x11-libs --disable-system-fontconfig --disable-system-gcc --disable-system-opengl --with-pkg-dir=../mxe-octave-pkg --with-ccache gnu-linux


Running the test suite on Windows still crashes Octave for me (so nothing new on that front). But I don't see the message from comment #0.

I do see the error message when trying to execute "cmdshell.bat".


Markus Mützel <mmuetzel>
Group administrator
Fri 06 Dec 2019 03:51:22 PM UTC, comment #6: 

Looks like msys-2.0.dll is the only file (that Ive seen so far) that no longer works.

Using obj dump on it: (_w) is the one that works)


-msys-2.0.dll_n:     file format pei-x86-64
+msys-2.0.dll_w:     file format pei-x86-64

 Sections:
 Idx Name          Size      VMA               LMA               File off  Algn
@@ -27,7 +27,9 @@
                   CONTENTS, ALLOC, LOAD, DATA, SHARED
  11 .idata        00003640  0000000180312000  0000000180312000  00293400  2**2
                   CONTENTS, ALLOC, LOAD, DATA
- 12 .rsrc         00000408  0000000180317000  0000000180317000  00296c00  2**2
+ 12 .gnu_debuglink 00000014  0000000180316000  0000000180316000  00296c00  2**

+                  CONTENTS, READONLY, DEBUGGING
+ 13 .rsrc         00000408  0000000180317000  0000000180317000  00296e00  2**2
                   CONTENTS, ALLOC, LOAD, DATA
- 13 .cygheap      00308000  0000000180318000  0000000180318000  00000000  2**2
+ 14 .cygheap      00308000  0000000180318000  0000000180318000  00000000  2**2
                   ALLOC


So the easy answer may just be don't step the msys dll - we don't strip notepad dlls at the moment as they wont load otherwise, so its not a completely new thing.


John Donoghue <lostbard>
Group Member
Fri 06 Dec 2019 03:23:12 PM UTC, comment #5: 

Is this what we would need:
https://packages.msys2.org/package/mingw-w64-x86_64-gcc-libs?repo=mingw64

IIRC, I used the same options as the mxe-native-all buildbot. But I'll check which switches exactly I used when I'm back home.

Markus Mützel <mmuetzel>
Group administrator
Fri 06 Dec 2019 03:15:54 PM UTC, comment #4: 

hmmm ... if I copy over /usr from an older version to the new install, things appear to work

and if I diff the bin files from the old vs new, they are different, which I wouldnt have thought they would be, since they should the same file, but they do get stripped - and strip comes from the binutils, so I wonder if there is something in that?

John Donoghue <lostbard>
Group Member
Fri 06 Dec 2019 02:51:30 PM UTC, comment #3: 

We need our version binutils to build gcc - unless we used precompiled versions of that as well, but then that may potentially mess with anything natively installed vs cross compiled.

msys2 looks like they havent released a 9.20 version of gcc yet that I can see.

What build options were  you using natively ?

John Donoghue <lostbard>
Group Member
Fri 06 Dec 2019 02:16:32 PM UTC, comment #2: 

msys2 seems to ship the same binutils version 2.33.1 we are using now:
https://packages.msys2.org/package/mingw-w64-x86_64-binutils

Maybe it helps updating the msys2-gcc-libs to 9.2.0, too?
Or we should use their version of the binutils instead of building our own?

Markus Mützel <mmuetzel>
Group administrator
Fri 06 Dec 2019 01:57:36 PM UTC, comment #1: 

I updated to the newer version because the old version failed to work with gcc 9.2.0 for native builds.

If we revert to the older binutils, we should probably also revert to an older gcc that plays nicely together with that version.

I'll probably be able to check if I can reproduce later today or during the weekend.

Markus Mützel <mmuetzel>
Group administrator
Fri 06 Dec 2019 01:19:22 PM UTC, original submission:  


After the update to use binutils 2.33.1

https://hg.octave.org/mxe-octave/rev/36a76b7be686

I did a clean compile of mxe-octave and installed it.

If I go to run the test suite in octave, or open a shell window, or run pkg install, I get a error message window popup:


msys-2.0.dll is either not designed to run on windows or it contains and error. Try installing the program again using the original installation media or contact your system administrator or software vendor for support. Error status 0xc000007b.


If I install a version compiled from the commit just before the binutils update, all works ok.

We don't compile msys-2.0.dll, so it hasn't changed between commits, so something else has changed that invalidates loading of the dll.


Not sure if anyone else using a recent build of mxe sees the same issue on windows, or if its just me.


The easy fix is go back to the previous version, unless there was a specific reason that we needed the updated version ?

John Donoghue <lostbard>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by lostbard (Submitted the item)
  • -email is unavailable- added by lostbard
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-12-08 mmuetzel StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2019-12-06 lostbard StatusNone Ready For Test
    2019-12-06 lostbard Carbon-Copy- Added mmuetzel

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code