patchGNU Octave - Patches: patch #9675, Using MSYS2 for mxe-octave

 
 

patch #9675: Using MSYS2 for mxe-octave

Submitter:  John Donoghue <lostbard>
Submitted:  Mon 13 Aug 2018 06:12:38 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  lostbard Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 20 Aug 2018 09:26:59 PM UTC, comment #8: 

Excellent. Re-compiled and ran the test suite. No un-expected errors:

Summary:

  PASS                            15266
  FAIL                                0
  XFAIL (reported bug)               36
  SKIP (missing feature)             52
  SKIP (run-time condition)           5


Closing again as done.

Markus Mützel <mmuetzel>
Group administrator
Mon 20 Aug 2018 07:55:14 PM UTC, comment #7: 

Pushed updates to mxe-octave

John Donoghue <lostbard>
Group Member
Mon 20 Aug 2018 06:33:12 PM UTC, comment #6: 

I have new patch that includes libcrypt, and another that updates some script paths to work with /mingw64 which gets around the issue with the paths you are seeing in comment #20.


I will also change the perl version from the original version in the patch to an earlier one to stop some other warnings that come up with the current version of texinfo.

Currently rebuilding everything to verify it works before pushing up.

John Donoghue <lostbard>
Group Member
Mon 20 Aug 2018 06:26:08 PM UTC, comment #5: 

I managed to get past that error message with the attached changeset (incomplete, x86 checksum missing). However, I now repeatedly see the following message:

Can't locate Texinfo/ModulePath.pm in @INC (you may need to install the Texinfo::ModulePath module) (@INC c
ontains: /usr/share/texinfo /usr/lib/perl5/site_perl /usr/share/perl5/site_perl /usr/lib/perl5/vendor_perl
/usr/share/perl5/vendor_perl /usr/lib/perl5/core_perl /usr/share/perl5/core_perl) at D:\Repositories\Octave
\test\octave-2018-08-20-19-55-w64_perl\octave-2018-08-20-19-55-w64\mingw64\bin\makeinfo line 95.
BEGIN failed--compilation aborted at D:\Repositories\Octave\test\octave-2018-08-20-19-55-w64_perl\octave-20
18-08-20-19-55-w64\mingw64\bin\makeinfo line 98.


I don't know how to continue from here. But sharing anyway in case that helps.

(file #44817)

Markus Mützel <mmuetzel>
Group administrator
Sun 19 Aug 2018 05:36:04 PM UTC, comment #4: 

Thanks for the quick fix.
I applied it locally and perl.exe is now included. However, running the test suite, I see a pop up window titled "perl.exe - Systemfehler" (system error) with the message "Die Ausführung des Codes kann nicht fortgesetzt werden, da msys-crypt-0.dll nicht gefunden wurde. [...]".

There is probably another package that includes "msys-crypt-0.dll" that we need to add.

Markus Mützel <mmuetzel>
Group administrator
Fri 17 Aug 2018 08:02:37 PM UTC, comment #3: 

Patch attached for perl. Building now, so as of yet, untested

(file #44802)

John Donoghue <lostbard>
Group Member
Fri 17 Aug 2018 07:46:30 PM UTC, comment #2: 

yeah somehow I missed that when going through the msys vs msys2 packages. I'll add it

John Donoghue <lostbard>
Group Member
Fri 17 Aug 2018 06:17:54 PM UTC, comment #1: 

I updated mxe-octave to hg id 68b3f1ccccac and ran the following in my existing mxe-octave directory:

make clean
autoconf
./configure --enable-devel-tools --enable-windows-64 --enable-octave=default --enable-binary-packages --with-pkg-dir=../mxe-octave-pkg --with-ccache --disable-system-opengl --enable-qt5
make all 7z-dist JOBS=3


When running the test suite on Windows 10, I get lots and lots of the following output:

Der Befehl "perl" ist entweder falsch geschrieben oder
konnte nicht gefunden werden.


That translates to something like: "The command "perl" is either spelled incorrectly or could not be found."
Indeed there is no "perl.exe" in the mingw64/bin directory.

We might need to also install perl from msys2.

Markus Mützel <mmuetzel>
Group administrator
Mon 13 Aug 2018 06:12:38 PM UTC, original submission:  


Patch that provides a --with-msys2 configure option and will use either msys2 64 bit or msys2 32 bit depending on whether compiling for windows 64

John Donoghue <lostbard>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #44817:  patch9675_msys2-perl_v2.patch added by mmuetzel (6KiB - application/octet-stream)
file #44802:  msys2-perl.patch added by lostbard (3KiB - application/octet-stream)
file #44776:  msys2.patch added by lostbard (94KiB - application/octet-stream)

 

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)
  •  

    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 logged-in users can vote.

     

    Follow 11 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-08-20 mmuetzel StatusReady For Test Done
        Open/ClosedOpen Closed
    2018-08-20 lostbard StatusIn Progress Ready For Test
    2018-08-20 mmuetzel Attached File- Added patch9675_msys2-perl_v2.patch, #44817
        StatusDone In Progress
        Open/ClosedClosed Open
    2018-08-17 lostbard Attached File- Added msys2-perl.patch, #44802
    2018-08-15 lostbard StatusNone Done
        Assigned toNone lostbard
        Open/ClosedOpen Closed
    2018-08-13 lostbard Attached File- Added msys2.patch, #44776

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code