Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open with Visual Studio Version 17.5.3 #36

Merged

Conversation

takano32
Copy link
Collaborator

@takano32 takano32 commented Apr 4, 2023

Update Packages and Update Projects

I’ll make this Ready for review after simple test in Linux and macOS.

  • GitHub Actions
  • UnitTest

closes #37

Update Packages and Update Projects
Same build directory with build-gui-win-x64.cmd
fix below warnings

```
/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/Pages/SinglePage.xaml.cs(324,24): warning CS0219: The variable 'format' is assigned but its value is never used [/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/PSXPackagerGUI_sd2buahs_wpftmp.csproj]
/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/Pages/SinglePage.xaml.cs(379,22): warning CS0169: The field 'SinglePage._total' is never used [/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/PSXPackagerGUI_sd2buahs_wpftmp.csproj]
/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/Pages/SinglePage.xaml.cs(324,24): warning CS0219: The variable 'format' is assigned but its value is never used [/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/PSXPackagerGUI.csproj]
/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/Pages/SinglePage.xaml.cs(379,22): warning CS0169: The field 'SinglePage._total' is never used [/home/takano32/GitHub/PSXPackager/PSXPackagerGUI/PSXPackagerGUI.csproj]
```
@RupertAvery RupertAvery marked this pull request as ready for review April 5, 2023 08:32
@RupertAvery
Copy link
Owner

Oops, sorry accidentally marked as ready for review

Git on Bash in Windows doesn't work well `/p` command line argument.
ref. https://stackoverflow.com/questions/3779701
@takano32
Copy link
Collaborator Author

takano32 commented Apr 8, 2023

@RupertAvery No, don't worry about it.

Sorry it took me a little while to get back to you.

I was able to run make in Git Bash(on Windows) to create a binary and confirm that it works.

I installed GNU make on Windows with choco

> choco install -y make

Invoking build-all.cmd via PowerShell also works well.

This is now ready for review, actually.

Thanks always.

Using SEVENZIP Package Update

```
ItemGroup Condition="$(DefineConstants.Contains('SEVENZIP'))"
```
@takano32 takano32 force-pushed the Open-with-Visual-Studio-Version-17.5.3 branch from 89f33ac to 4b2fe6d Compare April 12, 2023 16:07
@takano32
Copy link
Collaborator Author

takano32 commented Apr 13, 2023

I included #37 in this PR.

It may be difficult to review with large differences. Sorry.

Linux distributions running on Arm like Raspbian on Raspberry Pi Model 2+
Linux distributions running on 64-bit Arm like Ubuntu Server 64-bit on Raspberry Pi Model 3+
@takano32
Copy link
Collaborator Author

I confirmed gui-win-x64 and win-x64 binaries downloading from GitHub Actions Summary work well in my old LapTop installed Windows 10.

https://github.com/takano32/PSXPackager/actions/runs/4688277339

LGTM! 🙇

@RupertAvery RupertAvery merged commit b76c312 into RupertAvery:master Apr 14, 2023
@takano32
Copy link
Collaborator Author

Thank you for merging!

And GitHub Actions started generating binaries in this Repo! 🎉

https://github.com/RupertAvery/PSXPackager/actions

@takano32 takano32 deleted the Open-with-Visual-Studio-Version-17.5.3 branch April 14, 2023 18:24
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.

Build binaries for each commit using GitHub Actions
2 participants