Skip to content

Conversation

@mmicko
Copy link
Member

@mmicko mmicko commented Sep 13, 2020

No description provided.

@cuavas
Copy link
Member

cuavas commented Sep 13, 2020

Why does this have a make binary in it? I’d like to avoid having anything that conflicts with regular MSYS64 tools.

What’s the purpose of the empty init.sh file? You could use bash --login /dev/null or bash --login <(true) to execute an empty script without needing a confusing empty file.

To minimise binary blobs, it would be better to add a dependency on p7zip package (from msys repo), not include the 7za.exe executable, and install the 7z.sfx file in /usr/lib/p7zip so the MSYS64 7za tool will find it.

Several scripts are missing the newline on the final line.

Is calling the directory where the miscellaneous MAME stuff goes /win32 really the best thing to do? Can we call it /opt/mamedev or something more conventional?

@cuavas
Copy link
Member

cuavas commented Sep 13, 2020

Given I have MSYS instances from a regular installation and git for Windows, how would I make an additional clean msys64 instance for testing purposes? I don’t want to run the installer again, as it creates start menu items etc. and would likely interfere with my existing msys64 instance. Is there a package that creates a self-contained MSYS instance without messing with start menu, registry, etc.?

@mmicko
Copy link
Member Author

mmicko commented Sep 13, 2020

Hi, I did not change any of scripts we used in previous builds so that is reason for location, missing newline at end of file and similar.
Reason why make.exe is there is that if you are using make.exe from msys2 then you are in posix environment, and with this modified make you can run in plain windows environment. Since we override path we are using this in non-posix environment.
For 7zip guess we can fix that and make old build scripts work without any change.

Regarding msys package that does not mess with start menu and registry, think there is no such thing. Best you can do is install it on VM and zip content of installed directory to place on new machine.

@mmicko
Copy link
Member Author

mmicko commented Sep 13, 2020

Note one thing. All binaries and scripts in package are ONLY for non-posix compilation, so out of msys2 environment (since that is what win32 users are get used to use)

@mmicko
Copy link
Member Author

mmicko commented Sep 13, 2020

All mentioned issues have been addressed.

@cuavas cuavas merged commit 600507f into mamedev:master Sep 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants