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

Fix archivegen for windows installer #2221

Merged
merged 8 commits into from Mar 6, 2023
Merged

Fix archivegen for windows installer #2221

merged 8 commits into from Mar 6, 2023

Conversation

smk762
Copy link
Collaborator

@smk762 smk762 commented Mar 4, 2023

The latest scheduled dev CI build completed without issues, yet subsequent CI runs triggered by PRs are failing to build the windows installer properly.

CI logs in failing builds show the following error:
Cannot write entry header for "D:/a/atomicDEX-Desktop/atomicDEX-Desktop/b/bin/atomicdex-desktop.7z": Can't add archive to itself: No error.

Hopefully this PR will fix the error.

@smk762
Copy link
Collaborator Author

smk762 commented Mar 5, 2023

To test, download windows release installer from https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/4335737000
To test, make sure windows installer functions as expected.

Copy link
Contributor

@SirSevenG SirSevenG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Installs as it should

Copy link
Contributor

@Canialon Canialon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can install the app with the installer and run with no problems:)

@smk762 smk762 merged commit 360caec into dev Mar 6, 2023
@smk762 smk762 deleted the fix_installer branch March 6, 2023 10:45
@smk762 smk762 mentioned this pull request Mar 6, 2023
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.

None yet

3 participants